adding pxa3xx_nand_flash to platform data

Eric Miao eric.y.miao at gmail.com
Wed May 5 04:56:01 EDT 2010


On Wed, May 5, 2010 at 3:58 PM, Siddarth Gore <gores at marvell.com> wrote:
> Hello Eric,
>
> Currently I need to enable the deprecated MTD_NAND_PXA3xx_BUILTIN
> support for Aspenite DB board (pxa168). I have a patch ready to add
> pxa3xx_nand_flash in aspenite.c but I was wondering about the cmdset
> field in it.
> Looks like it is the same for all boards (depending upon the page size).
> So should I add it to the board specific setup file or remove it from
> pxa3xx_nand_platform_data (and put it in say, pxa3xx_nand_info)?
>
> If there are boards which require a different cmdset then we have to go
> with first option, if not, we can use option two and save some code
> duplication.
>

There are normally two sets of commands, yet the code derived from some
previous work that there were several subtle different command sets.

Haojian,

Could you help check if there are other possibilities? Otherwsie, we can
just make this field internal and decide by page_size?

- eric



More information about the linux-arm-kernel mailing list