Re: [PATCH 2/2] ARM: allow for ATAGS to be configured out when DT support is selected

[Nicolas Pitre <nicolas.pitre-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>]

On Sat, 1 Sep 2012, Arnd Bergmann wrote:

> On Thursday 30 August 2012, Nicolas Pitre wrote:
> > Now that ATAGS support is well contained, we can easily remove it from
> > the kernel build if so desired.  It has to explicitly be disabled, and
> > only when DT support is selected.
> > 
> > Note: disabling kernel ATAGS support does not prevent the usage of
> > CONFIG_ARM_ATAG_DTB_COMPAT.
> 
> > ...
> 
> > +config ATAGS
> > +	bool "Support for the traditional ATAGS boot data passing" if USE_OF
> > +	default y
> > +	help
> > +	  This is the traditional way of passing data to the kernel at boot
> > +	  time. If you are solely relying on the flattened device tree (or
> > +	  the ARM_ATAG_DTB_COMPAT option) then you may unselect this option
> > +	  to remove ATAGS support from your kernel binary.  If unsure,
> > +	  leave this to y.
> 
> That's a clever way of implementing it, I had not though of making it an
> optional default-y symbol.
> 
> Acked-by: Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>

Thanks.

> Should we do patches to the individual platforms bsaed on this to disable
> the ATAG based board files when CONFIG_ATAG is disabled?

Well... that could be a good thing to make them depend on ATAGS indeed.  
But that can be done incrementally in later patches.


Nicolas