From mboxrd@z Thu Jan 1 00:00:00 1970 From: eric.y.miao@gmail.com (Eric Miao) Date: Wed, 5 May 2010 16:56:01 +0800 Subject: adding pxa3xx_nand_flash to platform data In-Reply-To: <1273046285.29563.311.camel@pe-dt434> References: <1273046285.29563.311.camel@pe-dt434> Message-ID: To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Wed, May 5, 2010 at 3:58 PM, Siddarth Gore 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