JEDEC support broken?

[David Woodhouse]

On Tue, 14 Nov 2000, Alice Hennessy wrote:

> Do you mean to use map->size instead of bank_size (to spread the
> individual start offsets over if there are gaps between chips)

Nope, discard the concept of bank_size altogether. Each chip needs its own
information structure, much like the 'struct flchip' in
include/mtd/flashchip.h, which contains the start offset of the chip in
the map.

It's reasonable to assume that all chips in the array will be the same
size. You have an array of struct flchip (or your equivalent if you don't
want to use struct flchip) and you index into that by dividing by the
individual chip size - and then just address the chip you've selected by
using the start offset listed for it.

> Also,  is spin_lock support going to be added in jedec.c?

It certainly _needs_ doing. Feel free :)

> I have a board that needs support for both cfi and jedec flashes.
> So, the approach I have taken is to use physmap.c with an array of
> "physmap_info" structures that describe the window address, window
> size, buswidth and number of partitions for each flash.
 < ... deletia ... >

> Is this in keeping with the intent of the map file?

Sounds like it. Show us the code though.

-- 
dwmw2




To unsubscribe, send "unsubscribe mtd" to majordomo at infradead.org