|
From: EXTERNAL B. M. (P. ST-FIR/Eng) <ext...@de...> - 2007-08-16 08:26:09
|
Hi,
I didn't have access to the hardware yesterday.
> Markus, could you please fix this by hand and check if the patch is
ok?
The patch needed another fix. It was my old friend "!" =3D)
After that it worked.
- if (!primary) {
+ if (primary) {
/*
* only secondary priority should access registers, so
* set _INTC_FN(h) =3D REG_FN_ERR for intc_set_priority()
*/
I don't have problems with "!" in conditions, because it's a logical
operator in a logical condition and it's not a not not in a numerical
calculation ;)
I think the way you check handle against REG_FN_ERR is problematic. You
set it with a binary or,
+ hp->handle &=3D ~_INTC_MK(0x0f, 0, 0, 0, 0, 0);
+ hp->handle |=3D _INTC_MK(REG_FN_ERR, 0, 0, 0, 0,
0);
but you check it directly.
if (_INTC_FN(ihp->handle) !=3D REG_FN_ERR)
I'm already hunting the next bug.=20
It seems to be without influence on the functionality. The value of
"handle" for the sense is also wrong.
So I better find the bug before it finds us ;)
Markus
|