Hi Tom, > The spi-nor-core.c contains optional logics like SFDP and manufcaturer > specific fixups. The spi-nor-ids.c has single large ID database. There > are many #ifdef directives in those files and that negatively affects > readability and maintainability. > > This series, as the first step of cleanup, move SFDP logic, IDs, and > fixups to newly source files. Reworking, renaming, and reformating > existing code is not a scope of this series and further cleanup will be > taken place in follow-up patches. > > Here is the code size impact when applying this series. > Measured by my AMD Zynq-7000 (ARM32) environment with SFDP and all > manufcaturers are enabled. > > SPI-NOR: > text data bss dec hex filename > BEFORE 731133 14696 0 745829 b6165 u-boot > AFTER 732301 14696 0 746997 b65f5 u-boot > DELTA +1168 0 0 > > TINY: > text data bss dec hex filename > BEFORE 80492 3744 248 84484 14a04 spl/u-boot-spl > AFTER 80748 3744 248 84740 14b04 spl/u-boot-spl > DELTA +256 > > --- > Changes in v2: > - Rebase on top of next > - Link to v1: > https://lore.kernel.org/u-boot/20260612-split-id-table-into-vendor-files-v1-0-dfaaa82cc...@infineon.com/
I forgot to update drivers/mtd/spi/sandbox.c. Will address that in the next revision. Thanks, Takahiro
