Re: [PATCH v2] Re: Adding support for device tree and command line

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

 



On Sat, 2016-05-28 at 12:05 -0700, Daniel Gimpelevich wrote:
> On Sat, 2016-05-28 at 13:31 +0300, Antony Pavlov wrote:
> >   Can we use 'if' instead of preprocessor's '#if' here?
> > 
> >   If we use regular C 'if' operator with IS_ENABLED() instead of
> > '#if/#ifdef'
> >   then the compiler can check all the code.
> > 
> >   E.g. please see this barebox patch:
> > 
> > 
> > http://lists.infradead.org/pipermail/barebox/2014-February/017834.html
> 
> Sigh. I guess I will resubmit again…

Upon further review, no, we cannot use 'if' instead of '#if' here. The
reference to the appended DTB would throw a linker error if the option
to put it there is not enabled. Sorry.





[Index of Archives]     [Linux MIPS Home]     [LKML Archive]     [Linux ARM Kernel]     [Linux ARM]     [Linux]     [Git]     [Yosemite News]     [Linux SCSI]     [Linux Hams]

  Powered by Linux