Skip to content

cfi_flash: A bit of cleanup and code simplification

Stefan Roese u-boot 1 version New

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest4 patches 0 repliesNew

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/4cfi_flash: Simplify flash_get_info()drivers/mtd/cfi_flash.c+2 -20 replies
2/4cfi_flash: Add weak default for cfi_flash_bank_addr()drivers/mtd/cfi_flash.c+9 -41 reply
3/4cfi_flash: Simplify dynamic flash bank number detectionboard/esd/apc405/apc405.c board/tqc/tqm834x/tqm834x.c common/cmd_bootm.c +7 more+34 -341 reply
4/4cfi_flash: Remove uneccessary #ifdef CONFIG_SYS_MAX_FLASH_BANKS_DETECTcommon/cmd_flash.c drivers/mtd/cfi_mtd.c+3 -170 replies

Reviews trailers per patch

1 of 4 patches has a Reviewed-by or Acked-by.

PatchHeiko Schocher
1/4
2/4
3/4A-b
4/4

Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes.

Read the v1 thread

Cover letter [U-Boot] [PATCH 0/4] cfi_flash: A bit of cleanup and code simplification

Hi,

following are a few patches with minor code clean-up and simplification for the cfi_flash driver. Especially the usage of CONFIG_SYS_MAX_FLASH_BANKS_DETECT to support dynamic flash bank number detection is reworked a bit. Additionally, now the flash base address list can be dynamically changed if needed as well.

Here the shortlog:

Stefan Roese (4):
cfi_flash: Simplify flash_get_info()
cfi_flash: Add weak default for cfi_flash_bank_addr()
cfi_flash: Simplify dynamic flash bank number detection
cfi_flash: Remove uneccessary #ifdef CONFIG_SYS_MAX_FLASH_BANKS_DETECT

 board/esd/apc405/apc405.c   |    6 ++----
 board/tqc/tqm834x/tqm834x.c |   12 +++++-------
 common/cmd_bootm.c          |    1 +
 common/cmd_flash.c          |   10 ++--------
 common/flash.c              |    1 +
 drivers/mtd/cfi_flash.c     |   32 +++++++++++++++-----------------
 drivers/mtd/cfi_mtd.c       |   10 +---------
 include/configs/APC405.h    |    6 ------
 include/configs/IDS8247.h   |    3 +--
 include/configs/TQM834x.h   |    4 ----
 include/flash.h             |    2 ++
 include/mtd/cfi_flash.h     |   18 ++++++++++++++++++
 12 files changed, 48 insertions(+), 57 deletions(-)

As you can see, these patches touch non-cfi related files too. If nobody objects, I'll collect these patches in my cfi-flash repository for the next merge window.

Cheers,
Stefan

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office at denx.de