[U-Boot] [PATCH 1/4] sunxi_nand_spl: Rename SPL_NAND_SUNXIto NAND_SUNXI
Olliver Schinagl
oliver+list at schinagl.nl
Fri Sep 18 13:51:24 CEST 2015
Bah gmane!
Adding Hans and Ian to the CC list.
On 18-09-15 13:43, Olliver Schinagl wrote:
> Hans de Goede <hdegoede <at> redhat.com> writes:
>
> Hey Hans,
>> We eventually want to add full nand support, since it makes no sense
>> to build SPL with nand support and u-boot without, or the other way
>> around, a single option will suffice.
>>
>> Renaming the Kconfig option now makes things easier when we add full
>> nand support in the future.
>>
>> The "obj-$(CONFIG_NAND_SUNXI) += sunxi_nand_spl.o" is moved to an
>> "ifdef CONFIG_SPL_BUILD" block in the Makefile.
>>
>> Signed-off-by: Hans de Goede <hdegoede <at> redhat.com>
>> ---
>> board/sunxi/board.c | 2 +-
>> drivers/mtd/nand/Kconfig | 18 +++++++++---------
>> drivers/mtd/nand/Makefile | 2 +-
>> include/configs/sunxi-common.h | 2 +-
>> 4 files changed, 12 insertions(+), 12 deletions(-)
>>
>> diff --git a/board/sunxi/board.c b/board/sunxi/board.c
>> index d411e96..9c855f6 100644
>> --- a/board/sunxi/board.c
>> +++ b/board/sunxi/board.c
>> <at> <at> -108,7 +108,7 <at> <at> int dram_init(void)
>> return 0;
>> }
>>
>> -#if defined(CONFIG_SPL_NAND_SUNXI) && defined(CONFIG_SPL_BUILD)
>> +#if defined(CONFIG_NAND_SUNXI) && defined(CONFIG_SPL_BUILD)
> While I agree this being a good way forward, the latest (and probably this
> change included) break nand support entirely and cause link errors when
> enabeling sunxi nand.
>
> sunxi-bsp/u-boot-sunxi/drivers/mtd/nand/nand.c:104: undefined reference to
> `board_nand_init' (which is from board/sunxi/board.c:144)
>
> I think it's wise (for now) to remove the && defined(CONFIG_SPL_BUILD) and
> re-add this guard later.
>
> By removing we atleast can still build u-boot with nand support (thus
> opening the option of testing and patches for users) and don't break
> previously working systems.
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> http://lists.denx.de/mailman/listinfo/u-boot
More information about the U-Boot
mailing list