cfi_endian.h

cfi_endian.h

[Alice Hennessy]

Hi,

What do you think of the idea of making the choice
of  CFI_LITTLE_ENDIAN, CFI_HOST_ENDIAN,
CFI_BIG_ENDIAN and CFI_LART_ENDIAN configurable so
that different boards can setup the driver via configuring physmap
and not have to touch  the code at all (if they don't need to set vpp
and
just want a single device)?

Alice



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

Re: cfi_endian.h

[David Woodhouse]

ahennessy@mvista.com said:
> What do you think of the idea of making the choice of
> CFI_LITTLE_ENDIAN, CFI_HOST_ENDIAN, CFI_BIG_ENDIAN and CFI_LART_ENDIAN
> configurable so that different boards can setup the driver via
> configuring physmap and not have to touch  the code at all (if they
> don't need to set vpp and just want a single device)?

Has anyone actually needed to use anything other than HOST_ENDIAN yet?

--
dwmw2




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

RE: cfi_endian.h

[Kári Davíðsson]

Why is it not always CFI_HOST_ENDIAN?

K.D.

> -----Original Message-----
> From: Alice Hennessy [mailto:ahennessy@mvista.com]
> Sent: 14. mars 2001 01:50
> To: mtd@infradead.org
> Cc: ahennessy@mvista.com
> Subject: cfi_endian.h
> 
> 
> Hi,
> 
> What do you think of the idea of making the choice
> of  CFI_LITTLE_ENDIAN, CFI_HOST_ENDIAN,
> CFI_BIG_ENDIAN and CFI_LART_ENDIAN configurable so
> that different boards can setup the driver via configuring physmap
> and not have to touch  the code at all (if they don't need to set vpp
> and
> just want a single device)?
> 
> Alice
> 
> 
> 
> To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org
> 


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

Re: cfi_endian.h

[David Woodhouse]

kd@flaga.is said:
> Why is it not always CFI_HOST_ENDIAN? 

The default is CFI_HOST_ENDIAN now. That's probably good enough for almost 
everyone, so it shouldn't be necessary to make it configurable.

--
dwmw2




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

Re: cfi_endian.h

[Alice Hennessy]

David Woodhouse wrote:

> kd@flaga.is said:
> > Why is it not always CFI_HOST_ENDIAN?
>
> The default is CFI_HOST_ENDIAN now. That's probably good enough for almost
> everyone, so it shouldn't be necessary to make it configurable.
>
> --
> dwmw2
>
> To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org

I just tested a board today (SH proc)  that if put in big endian mode
needs CFI_BIG_ENDIAN enabled.   The hardware guys didn't swap the
bytes for us this time :-)

Alice



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

Re: cfi_endian.h

[David Woodhouse]

ahennessy@mvista.com said:
>  I just tested a board today (SH proc)  that if put in big endian mode
> needs CFI_BIG_ENDIAN enabled.   The hardware guys didn't swap the
> bytes for us this time :-) 

heh, ok. But to keep the complexity down for new users, let's hide them all 
behind 'CFI Advanced configuration options' choice, and probably stick the 
interleave/buswidth optimisation options behind it too.

--
dwmw2




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

Re: cfi_endian.h

[Nicolas Pitre]

On Thu, 15 Mar 2001, David Woodhouse wrote:

>
> ahennessy@mvista.com said:
> >  I just tested a board today (SH proc)  that if put in big endian mode
> > needs CFI_BIG_ENDIAN enabled.   The hardware guys didn't swap the
> > bytes for us this time :-)
>
> heh, ok. But to keep the complexity down for new users, let's hide them all
> behind 'CFI Advanced configuration options' choice, and probably stick the
> interleave/buswidth optimisation options behind it too.

Agreed.


Nicolas



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

Re: cfi_endian.h

[Alice Hennessy]

David Woodhouse wrote:

> ahennessy@mvista.com said:
> >  I just tested a board today (SH proc)  that if put in big endian mode
> > needs CFI_BIG_ENDIAN enabled.   The hardware guys didn't swap the
> > bytes for us this time :-)
>
> heh, ok. But to keep the complexity down for new users, let's hide them all
> behind 'CFI Advanced configuration options' choice, and probably stick the
> interleave/buswidth optimisation options behind it too.
>
> --
> dwmw2
>
> To unsubscribe, send "unsubscribe mtd" to majordomo@infradead.org

Done.   I created "CFI Advanced configuration options" under CFI, moved the
geometry
selection  under it and added the endianness selection under it as well.

So, a new Config.in and cfi_endian.h

Alice



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