Hello,
Here is a patch for the 2 bugs in irq.h.
This is the first time, I generate a patch.
I don't know if this is the right method.
Best regards
Laurent
>
> In the ./linux/include/asm-ppc/irq.h file, I encountered a problem with
the
> timer3 interrupt definition :
> #define SIU_INT_TIMER3 ((uint)0x0e+CPM_IRQ_OFFSET)
>
> This definition generates a compilation error because is interpreted as
Note that this is not the only such definition. At least this one is
critical, too:
#define SIU_INT_PC1 ((uint)0x3e+CPM_IRQ_OFFSET)
Best regards,
Wolfgang Denk
> -----Message d'origine-----
> De : Kumar Gala [mailto:galak at kernel.crashing.org]
> Envoye : mar. 28 fevrier 2006 15:38
> A : Laurent Lagrange
> Cc : linuxppc-embedded at ozlabs.org
> Objet : Re: Minor bug in file irq.h
>
>
>
> On Feb 28, 2006, at 2:50 AM, Laurent Lagrange wrote:
>
> >
> > Hello,
> >
> > I use a Linux 2.6.9 on MPC85xx.
> >
> > In the ./linux/include/asm-ppc/irq.h file, I encountered a problem
> > with the
> > timer3 interrupt definition :
> > #define SIU_INT_TIMER3 ((uint)0x0e+CPM_IRQ_OFFSET)
> >
> > This definition generates a compilation error because is
> > interpreted as
> > something like
> > ((uint)0x0(e+CPM_IRQ_OFFSET)) ie 0x0 exponent + CPM_IRQ_OFFSET.
> >
> > To work around just parenthesis or add a space character between
> > 'e' and '+'
> > #define SIU_INT_TIMER3 ((uint)0x0e +CPM_IRQ_OFFSET)
>
> Do you mind providing this in patch form with a signed-off-by. I'll
> include it in the queue of 85xx related patches.
>
> - kumar
>
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: irq.patch
Type: application/octet-stream
Size: 1155 bytes
Desc: not available
Url :
http://ozlabs.org/pipermail/linuxppc-embedded/attachments/20060301/dae2c63c/attachment.obj
