adding pxa3xx_nand_flash to platform data

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


On Wed, May 5, 2010 at 4:56 PM, Eric Miao <eric.y.miao at gmail.com> wrote:
> 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?
>

Haojian CC'ed.



More information about the linux-arm-kernel mailing list