[PATCH] basic pxa3 nand controller support

eric miao eric.y.miao at gmail.com
Fri Jan 11 06:15:10 EST 2008


On Jan 7, 2008 8:23 AM, Marcel Ziswiler <marcel at ziswiler.com> wrote:
> eric miao <eric.y.miao <at> gmail.com> writes:
>
> > Early feedback is welcome, thanks!
>
> Hi Eric
>
> Overall the patch looks good, just a few comments.
>
> The patch does not apply cleanly unless I remove the spurious newline before
> line 1284 resulting in the following error:
> patch: **** malformed patch at line 1284: pm_message_t state)
>

I might have modified the patch directly, but not sure. Git will
complain but plain
"patch" should work.

> The DRCMR97 register is not defined anywhere, at lest not in 2.6.24-rc6:
> drivers/mtd/nand/pxa3xx_nand.c:542: error: 'DRCMR97' undeclared (first use in
> this function)
>
> The registers and bit definitions should be moved to pxa-regs.h, not?
>

right, but the trend is not to introduce too many DRCMRxx register definitions
into pxa-regs.h, use DRCMR(xx) is more preferred, so you can modify that
to DRCMR(97) please.

> I had my troubles getting it to work on a custom PXA320 board. I kinda figured
> out what platform data it needs, but still can't seem to get it to work properly:
> pxa3xx-nand pxa3xx-nand: failed to detect flash
>

pxa320 may be connected to external NAND flash, which the driver does not
yet support (only those __builtin_xxx are supported for now), sigh, the reason
is a long story

You may let me know the NAND flash type, so I can check with the driver.

> What kernel does your patch apply to? What CPUs/boards did you try it on? Can
> you provide an example platform integration required to make it work?
>

zylonite/pxa{300,310}, both work. Yes, I'll post the remaining patch next week,
hopefully.

> Cheers
>
> Marcel
>
>
>
> ______________________________________________________
> Linux MTD discussion mailing list
> http://lists.infradead.org/mailman/listinfo/linux-mtd/
>



-- 
Cheers
- eric



More information about the linux-mtd mailing list