CVS updates

[Abraham vd Merwe]

Hi Erik!

> > I've added a standalone driver for the 28F160xx chips on LART. I've also
> > removed the LART definitions in the sa1100 maps driver (it's wasn't used in
> > anycase and it's obsolete now anyway). Same for the LART endian definitions
> > in cfi_endian.h and the cfi chip driver Config file.
> 
> Why? If I understood the MTD stuff properly it is layered with the chip
> drivers at the lowest level, the maps driver on that, and the device
> drivers on top of the maps driver. As far as I can see (and what my
> plan was before you started) you only need a new chip driver for LART
> and reuse the other parts. Because the LART is an sa1100 design we
> should avoid special casing and just use the sa1100 maps driver.

The driver I wrote for LART is a standalone driver, so it doesn't need aby
maps driver.

Originally I wanted to change the maps driver and just write a chip driver,
but the sa1100 maps driver is dependant on CFI being configured. Since the
28F160xx chips isn't CFI compliant, it makes no sense to put the map
definitions in there (It's only about 20 lines of code anyway: read/write
routines and the partition definition), so I decided to just make a
standalone driver.

Anyway, even if I decided to make/patch an existing maps driver, the endian
definitions for LART doesn't belong in cfi_endian.h.

-- 

Regards
 Abraham

Vail's Second Axiom:
	The amount of work to be done increases in proportion to the
	amount of work already completed.

__________________________________________________________
 Abraham vd Merwe - 2d3D, Inc.

 Device Driver Development, Outsourcing, Embedded Systems

  Cell: +27 82 565 4451         Snailmail:
   Tel: +27 21 761 7549            Block C, Antree Park
   Fax: +27 21 761 7648            Doncaster Road
 Email: abraham at 2d3d.co.za         Kenilworth, 7700
  Http: http://www.2d3d.com        South Africa

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 232 bytes
Desc: not available
Url : http://lists.infradead.org/pipermail/linux-mtd/attachments/20010618/11a69af1/attachment.bin