[U-Boot] [PATCH V2 1/9] spl: replace CONFIG_SPL_SPI_* with CONFIG_SF_DEFAULT_*

Nikita Kiryanov nikita at compulab.co.il
Sun Aug 10 17:19:38 CEST 2014



On 07/08/14 16:49, Marek Vasut wrote:
> On Thursday, August 07, 2014 at 03:05:28 PM, Nikita Kiryanov wrote:
>> Currently, CONFIG_SPL_SPI_* #defines are used for controlling SPI boot in
>> SPL. These #defines do not allow the user to select SPI mode for the SPI
>> flash (there's no CONFIG_SPL_SPI_MODE, so the SPI mode is hardcoded in
>> spi_spl_load.c), and duplicate information already provided by
>> CONFIG_SF_DEFAULT_* #defines.
>>
>> Kill CONFIG_SPL_SPI_*, and use CONFIG_SF_DEFAULT_* instead.
>>
>> Cc: Jagannadha Sutradharudu Teki <jagannadh.teki at gmail.com>
>> Cc: Tom Rini <trini at ti.com>
>> Cc: Sudhakar Rajashekhara <sudhakar.raj at ti.com>
>> Cc: Lokesh Vutla <lokeshvutla at ti.com>
>> Cc: Vitaly Andrianov <vitalya at ti.com>
>> Cc: Lars Poeschel <poeschel at lemonage.de>
>> Cc: Bo Shen <voice.shen at atmel.com>
>> Cc: Hannes Petermaier <hannes.petermaier at br-automation.com>
>> Cc: Michal Simek <monstr at monstr.eu>
>> Cc: Marek Vasut <marex at denx.de>
>> Signed-off-by: Nikita Kiryanov <nikita at compulab.co.il>
>
> Oh, right. This doesn't collide with the Kconfig work, does it ?

It does not.
(No "config CONFIG_SPL_SPI_*", nor "config CONFIG_SF_DEFAULT_*" in any 
Kconfig files)

>
> Otherwise, I see this as OK,
>
> Reviewed-by: Marek Vasut <marex at denx.de>
>
> Best regards,
> Marek Vasut
>


More information about the U-Boot mailing list