[U-Boot-Users] [patch] add support for "eeprom info"

[J. William Campbell <jwilliamcampbell@comcast.net>]

Mike Frysinger wrote:
> On Friday 25 January 2008, Wolfgang Denk wrote:
>   
>> Well, there is a big difference between NOR flash  and  your  device:
>> NOR  flash is memory, but your's is a storage device attached to some
>> bus interface (SPI here). The problem is that we don't  have  a  good
>> solution to interface with such devices yet.
>>
>> Using the eeprom command for such devices is misleading, IMO. If  you
>> can  live  with the spiflash command, then this is not perfect, but I
>> guess it will have to do for now as we don't have a  better  solution
>> yet.
>>     
>
> so we're clear, the proposed interface is:
> spiflash info
> spiflash <read|write> address offset count
> spiflash erase offset count
>
>   
This sounds fine to me. The address is obviously a ram 
source/destination address. I vote for an offset in bytes and a count in 
bytes. If the offset is not an appropriate sector boundary, it is an 
error. The count can be a partial sector without problems. Having the 
offset and count in different units is confusing IMHO, so bytes for 
both. Also, Mike, I assume you will also add a module similar to 
env_eeprom.c to support environment in spiflash?

Best Regards,
Bill Campbell
> and once this comes together (i dont think it'll take long), we can consider a 
> generic "flash" command.  i'm still not sure what it'll gain us (if 
> anything) ... but i guess once we have more things to look at and compare, we 
> can make a better decision
> -mike
>   
> ------------------------------------------------------------------------
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Microsoft
> Defy all challenges. Microsoft(R) Visual Studio 2008.
> http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/
> ------------------------------------------------------------------------
>
> _______________________________________________
> U-Boot-Users mailing list
> U-Boot-Users at lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/u-boot-users
>