Re: [PATCH] Make nand block functions use provided byte/word helpers.



On Tue, Feb 28, 2006 at 08:59:03PM +0000, Jonathan McDowell wrote:
Hi.

I've been writing a NAND driver for the flash on the Amstrad E3. One of
the peculiarities of this device is that the write & read enable lines
are on a latch, rather than strobed by the act of reading/writing from
the data latch. As such I've got custom read_byte/write_byte functions
defined. However the nand_*_buf functions in drivers/mtd/nand/nand_base.c
are all appropriate, except for the fact they call readb/writeb
themselves, instead of using this->read_byte or this->write_byte. The
patch below changes them to use these functions, meaning a driver just
needs to define read_byte and write_byte functions and gains all the
nand_*_buf functions free.

Why not make life easier on everyone else by over-riding the
functions for read/write buffer (etc) in the nand driver... less
intrusive into the core code!

--
Ben (ben@xxxxxxxxx, http://www.fluff.org/)

'a smiley only costs 4 bytes'
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • Re: SYNC beteen kernel and driver
    ... conflict with my NAND driver. ... function in NAND driver and OAL.lib will link my NAND driver. ... critical sections in the driver and the OAL, and they won't protect ... and use CriticalSection to ...
    (microsoft.public.windowsce.platbuilder)
  • [PATCH] Make nand block functions use provided byte/word helpers.
    ... I've been writing a NAND driver for the flash on the Amstrad E3. ... the peculiarities of this device is that the write & read enable lines ... the data latch. ...
    (Linux-Kernel)