[U-Boot] [PATCH] Noisily disable the legacy NAND subsystem.
Becky Bruce
beckyb at kernel.crashing.org
Wed Apr 1 23:42:41 CEST 2009
On Apr 1, 2009, at 4:32 PM, Scott Wood wrote:
> Becky Bruce wrote:
>> On Apr 1, 2009, at 4:15 PM, Scott Wood wrote:
>>> Legacy NAND is marked for feature removal after April 2009 (i.e.
>>> this
>>> upcoming release). There are still several boards that reference it
>>> (though many do so only for disk-on-chip support which has been
>>> silently
>>> disabled for a while now). These boards will now fail to build with
>> Drum roll....? You're missing the rest of your sentence, or you
>> have too much sentence, or something.
>
> D'oh. The next line started with "#error", so git treated it as a
> comment.
>
> Commit amended to say "These boards will now fail to build with
> #error, though the code is still there if the user removes #error."
Thanks ;)
>
>
>>> +#error This code is broken and will be removed outright in the
>>> next release.
>>> +#error If you need diskonchip support, please update
>>> +#error drivers/mtd/nand/diskonchip.c to work with u-boot.
>> "to work with u-boot"? As opposed to what, exactly?
>
> Linux, where the code was taken from as part of the NAND subsystem
> importation.
>
>> How about "please update blah to be functional"
>
> "be functional" == "work". I was just clarifying the nature of the
> fixing that needed to be done.
it wasn't the "work" I had a problem with, it was the "with u-boot"
part, and getting rid of the "with u-boot" made "work" sound silly :)
But if you're actually talking about u-boot vs linux, the comment is
fine as is.
Thanks,
B
More information about the U-Boot
mailing list