Re: [PATCH 0/2] staging: mtd: Support for GigaDevice SPI NAND flash

[Ezequiel Garcia <ezequiel.garcia@imgtec.com>]

On 11/25/2014 04:02 AM, bpqw wrote:
>> Yeah, I've been thinking about this for some time. Right now, I think that
>> SPI NAND is just that: a NAND over SPI, so I'm doing some experiments
>> around this design:
>>
>>    Userspace
>>  ------------------
>>    MTD
>>  ------------------
>>    NAND core
>>  ------------------
>>    SPI NAND core
>>  ------------------
>>    SPI NAND device
>>  ------------------
>>    SPI core
>>  ------------------
>>    SPI master
>>  ------------------
>>    Hardware
> 
> I have finished a initial version of SPI-NAND framework based on drivers/staging/
> mt29f_spinand/mt29f_spinand.c . The structure is the same as yours, just treate SPI
> NAND as a NAND over SPI, put SPI NAND core under NAND core. Do you think it is a
> good structure to start with? Because it is a quick product, I even do not finish
> the unregisger function. So just care about the structure. Any suggestion about the
> patch is welcomed.
> 

Hm.. interesting. Ionela and I have started a similar work. However,
we've splitted the code in two: spi-nand-base.c and spi-nand-device.c.
The former handles the NAND logic, and the latter implements the
specific SPI commands.

The idea behind this separation is to allow for better evolution of the
code, given we are not sure the direction the SPI NAND devices will take.

I'm not sure how to handle a parallel effort like this and I don't want
to stop you from working. I'll have some code submitted by this week.

Maybe we can discuss it as well?
-- 
Ezequiel