u-boot.git/common/cmd_eeprom.c, branch master

Move all command code into its own directory

2016-01-25T15:39:43+00:00

There are a lot of unrelated files in common, including all of the commands.
Moving them into their own directory makes them easier to find and is more
logical.

Some commands include non-command code, such as cmd_scsi.c. This should be
sorted out at some point so that the function can be enabled with or without
the associated command.

Unfortunately, with m68k I get this error:

m68k:  +   M5329AFEE
+arch/m68k/cpu/mcf532x/start.o: In function `_start':
+arch/m68k/cpu/mcf532x/start.S:159:(.text+0x452): relocation truncated to fit: R_68K_PC16 against symbol `board_init_f' defined in .text.board_init_f section in common/built-in.o

I hope someone can shed some light on what this means. I hope it isn't
depending on the position of code in the image.

Signed-off-by: Simon Glass 
Reviewed-by: Bin Meng 
Reviewed-by: Heiko Schocher 
Acked-by: Stefan Roese 
Acked-by: Przemyslaw Marczak

eeprom: fix eeprom write procedure

2015-12-16T15:31:31+00:00

This fixes commit 1a37889b0ad084a740b4f785031d7ae9955d947b:
----------------------->8--------------------
eeprom: Pull out the RW loop

Unify the code for doing read/write into single function, since the
code for both the read and write is almost identical. This again
trims down the code duplication.
----------------------->8--------------------

where the same one routine is utilized for both EEPROM writing and
reading. The only difference was supposed to be a "read" flag which
in both cases was set with 1 somehow.

That lead to a missing delay in case of writing which lead to write
failure (in my case no data was written).

Signed-off-by: Alexey Brodkin 
Acked-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher

eeprom: Clean up checkpatch issues

2015-11-22T02:50:27+00:00

Cosmetic fixes to the file, make it checkpatch clean.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Add support for selecting i2c bus

2015-11-22T02:50:26+00:00

Add additional parameter into the eeprom command to select
the I2C bus on which the eeprom resides.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Add bus argument to eeprom_init()

2015-11-22T02:50:26+00:00

Add bus argument to eeprom_init(), so that it can select
the I2C bus number on which the eeprom resides. Any negative
value of the $bus argument will preserve the old behavior.
This is in place so that old code does not randomly break.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher 
[trini: Wrap i2c_set_bus_num() call with CONFIG_SYS_I2C test]
Signed-off-by: Tom Rini

eeprom: Pull out the RW loop

2015-11-22T02:50:25+00:00

Unify the code for doing read/write into single function, since the
code for both the read and write is almost identical. This again
trims down the code duplication.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Pull out transfer length computation

2015-11-22T02:50:25+00:00

Pull out the code which computes the length of the transfer
into separate code and clean it up a little. This again trims
down the code duplication.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Pull out CONFIG_SYS_EEPROM_PAGE_WRITE_BITS

2015-11-22T02:50:24+00:00

Implement default value of 8 for this macro and pull out all of
this macro out of the code. The default value of 8 actually does
implement exactly the same behavior as the previous code which
was in the #else clause of the ifdef.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Suck the ifdef into eeprom_init()

2015-11-22T02:50:24+00:00

Just suck the ugly ifdef around eeprom_init() call into eeprom_init()
function itself. This puts all of the ifdef mess into one place.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher

eeprom: Pull out CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS

2015-11-22T02:50:24+00:00

Pull this macro to the beginning of the cmd_eeprom.c and remove
another nasty ifdef from the code. Note that this is legal, since
udelay(0) changes the behavior only such that it pings the WDT if
WDT is enabled and otherwise does not wait.

Signed-off-by: Marek Vasut 
Cc: Simon Glass 
Cc: Tom Rini 
Cc: Heiko Schocher 
Reviewed-by: Heiko Schocher