lkml.org 
[lkml]   [2009]   [Feb]   [6]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [git pull -tip] headers_check fixes for other architectures
On Fri, Feb 06, 2009 at 08:30:07PM +0530, Jaswinder Singh Rajput wrote:
> On Fri, 2009-02-06 at 14:51 +0000, Russell King - ARM Linux wrote:
> > > >
> > > > So no point including this file in assembly with this patch - the ifndef
> > > > disables the entire file.
> > >
> > > Truth Table of linux/types.h :
> > >
> > > If Assembly then N
> > > otherwise Y
> > >
> > > what your table says.
> >
> > If the entire file is not suitable for assembly, don't include the file
> > in assembly files. Nice and simple, and no need to add additional ifdefs.
>
> Ahh, so your truth table says:
>
> If Assembly then make different header files
> If C then make another header files

No. My point is exactly as I stated above, please don't twist my
statement into something that it isn't.

Taking this further, if you're including linux/types.h into another
header file, you're including it because you want some C type from
that or an included file. Use of that type is also not ASM friendly,
so the use is going to have to be excluded by ifndef in that header.

So why not do as we *already* do and ensure that the linux/types.h
inclusion happens within that section.

Why change the rules?


\
 
 \ /
  Last update: 2009-02-06 16:15    [W:0.181 / U:0.028 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site