Re: [PowerPC] - Define error in arch/arch/powerpc/boot/io.h ??

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ok, I will ask for a embedded powerpc board to test this. 

Thanks


--
Lucas Tanure 
+55 (19) 988176559


On Thu, Jul 24, 2014 at 12:43 PM, <Valdis.Kletnieks@xxxxxx> wrote:
On Thu, 24 Jul 2014 14:19:37 +0200, Kristof Provost said:
> On 2014-07-24 09:00:22 (-0300), Lucas Tanure <tanure@xxxxxxxxx> wrote:
> > Line 2 of arch/arch/powerpc/boot/io.h :
> >
> >  1  #ifndef _IO_H
> >  2  #define __IO_H
> >  3
> >  4  #include "types.h"
> >
> > Should be _IO_H, not __IO_H. I'm right ?
> >
> That certainly looks wrong. It's also been wrong since October 2006, so
> it's probably not actually causing problems.

On the other hand, it would probably be a *really* good idea to test compile
it on a powerpc box, see if it shakes out any latent bugs.  All that fixing
this will do is stop us from over-including io.h (and thus types.h) - but if
there's any code that breaks because we don't include types.h and they don't
pick it up otherwise.....

(In other words, it's probably not causing a problem, but it *could* potentially
be papering over a problem elsewhere...)

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies

[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux