[U-Boot] [PATCH v3 15/16] RFC: dm: tegra: Enable driver model for serial
Simon Glass
sjg at chromium.org
Fri Aug 1 00:11:04 CEST 2014
Hi Stephen,
On 31 July 2014 21:18, Stephen Warren <swarren at wwwdotorg.org> wrote:
> On 07/30/2014 03:49 AM, Simon Glass wrote:
>>
>> Use driver model for serial ports (for test and comment only).
>
>
> A couple of nits:
>
>> diff --git a/include/configs/tegra-common.h
>> b/include/configs/tegra-common.h
>
>
>> #define CONFIG_SYS_MALLOC_LEN (4 << 20) /* 4MB */
>> +#define CONFIG_SYS_MALLOC_F_LEN (1 << 10)
>
>
> README says:
>
> Pre-relocation malloc() is only supported on sandbox
> at present but is fairly easy to enable for other archs.
>
> I assume that should be deleted now.
There's another series ahead of this patch which changes this message.
>
>
>>
>> /*
>> * NS16550 Configuration
>> */
>> -#define CONFIG_SYS_NS16550
>> +#ifdef CONFIG_SPL_BUILD
>> #define CONFIG_SYS_NS16550_SERIAL
>> #define CONFIG_SYS_NS16550_REG_SIZE (-4)
>> #define CONFIG_SYS_NS16550_CLK V_NS16550_CLK
>> +#endif
>> +#define CONFIG_SYS_NS16550
>
>
> Leaving that define in the same place would simplify the diff.
OK, will do.
Regards,
Simon
More information about the U-Boot
mailing list