mbox series

[00/11] 'eeprom' command improvements

Message ID 20240521071335.4193-1-kabel@kernel.org
Headers show
Series 'eeprom' command improvements | expand

Message

Marek Behún May 21, 2024, 7:13 a.m. UTC 
Hi Tom,

this series contains improvements for the 'eeprom' command:
- refactors
- fixes
- improvements
- ability to use driver model EEPROMs (uclass UCLASS_I2C_EEPROM)
- more flexible EEPROM layout support

It should not cause any behavior change for any existing board.

This series is a dependency for some DDR issue fixes for Turris Omnia.
I will be sending that one separately.

github PR link (with CI):
  https://github.com/u-boot/u-boot/pull/540
- there is a failure for
    test.py for sandbox sandbox_clang
  but it seems unrelated to these changes

Marek Behún (11):
  common: eeprom_layout: Assign default layout methods and parameters
    before specific ones
  common: eeprom_layout: Split field finding code from the field update
    function
  common: eeprom_field: Fix updating binary field
  common: eeprom_field: Drop unnecessary comparison
  cmd: eeprom: Fix usage help for the eeprom command
  cmd: eeprom: Hide eeprom layout versioning behind a Kconfig option
  cmd: eeprom: Deduplicate parse_i2c_bus_addr() calls
  cmd: eeprom: Refactor eeprom device specifier parsing
  cmd: eeprom: Refactor command execution into function by action
  cmd: eeprom: Don't read/write whole EEPROM if not necessary
  cmd: eeprom: Extend to EEPROMs probed via driver model

 cmd/Kconfig                   |   9 +-
 cmd/eeprom.c                  | 305 +++++++++++++++++++++++++---------
 common/eeprom/eeprom_field.c  |   4 +-
 common/eeprom/eeprom_layout.c |  54 +++---
 configs/cm_fx6_defconfig      |   1 +
 configs/cm_t43_defconfig      |   1 +
 include/eeprom_layout.h       |   4 +
 7 files changed, 273 insertions(+), 105 deletions(-)

Comments

Tom Rini June 7, 2024, 6:57 p.m. UTC | #1 
On Tue, 21 May 2024 09:13:24 +0200, Marek Behún wrote:

> this series contains improvements for the 'eeprom' command:
> - refactors
> - fixes
> - improvements
> - ability to use driver model EEPROMs (uclass UCLASS_I2C_EEPROM)
> - more flexible EEPROM layout support
> 
> [...]

Applied to u-boot/next, thanks!