[U-Boot] [PATCH 1/1] am33xx: add CONFIG_SYS_NAND_DEVICE_WIDTH to determine NAND device bus-width

[Scott Wood]

On Wed, 2013-09-25 at 22:08 +0000, Woodruff, Richard wrote:
> Short comment.  Apologies for top posting:
> 
> The first incarnations of SPL and loader mainly cared about the boot flash device.

Is there an SPL that cares about something other than the boot device
now?

> OMAPs require a resistor strap to specify the width of the boot device.  The values is latched for SW to read.
> 
> As such always a run time check of width was sufficient for boot device.
> 
> On dev board we would have many devices and use DIP switch to select which one was in use.  A run time not compile time was way to support this.
> 
> If information is dynamically available it is better to use this if focus is boot device management.
> 
> Other tricks writing patterns and reading results is also possible but simplicity of compile vs. that is debatable.
> 
> Regards,
> Richard W.

I agree.  Outside of SPL (where hardcoding can be useful due to size
constraints, and you only care about the boot device), it's best to let
drivers determine the best way to learn about such configuration.

> -----Original Message-----
> From: Gupta, Pekon 
> Sent: Wednesday, September 25, 2013 12:18 AM
> To: scottwood at freescale.com; Rini, Tom
> Cc: u-boot at lists.denx.de; Balbi, Felipe; Kipisz, Steven; sbabic at denx.de; notasas at gmail.com; luca.ceresoli at comelit.it; Woodruff, Richard; weber at corscience.de; peter.barada at logicpd.com; frederik at kriewitz.eu; Tom.Rix at windriver.com; Menon, Nishanth; srinath at mistralsolutions.com; Hiremath, Vaibhav; Gupta, Pekon
> Subject: [PATCH 1/1] am33xx: add CONFIG_SYS_NAND_DEVICE_WIDTH to determine NAND device bus-width
> 
> NAND driver needs to know bus-width of the connected NAND device, in order to perform proper I/O and initialize itself. Currently there is no CONFIG option to provide this information to NAND driver.
> - SPL NAND driver does not have framework to parse ONFI parameter page.

Is this about SPL?  It looks like a more general change.

> - NAND drivers which cannot self initialize !defined(CONFIG_SYS_NAND_SELF_INIT)
>   do not have any information about device bus-width during board_nand_init(),
>   So, any device-width specific configurations are not possible there.

Convert to self-init.  That's what it's for.

-Scott