On Wed, Nov 4, 2020 at 11:21 PM Simon Glass sjg@chromium.org wrote:
This uclass is intended to provide a way to obtain information about a U-Boot board. But the concept of a U-Boot 'board' is the whole system, not just one circuit board, meaning that 'board' is something of a misnomer for this uclass.
In addition, the name 'board' is a bit overused in U-Boot and we want to use the same uclass to provide SMBIOS information.
The obvious name is 'system' but that is so vague as to be meaningless. Use 'sysinfo' instead, since this uclass is aimed at providing information on the system.
Rename everything accordingly.
Note: Due to the patch delta caused by the symbol renames, this patch shows some renamed files as being deleted in one place and created in another.
Signed-off-by: Simon Glass sjg@chromium.org
Changes in v5:
- Fix "sysinfo_gazerbeam" compatible string to use a hyphen
- Add a note as to why the patch does not just rename some files
arch/powerpc/dts/gdsys/gazerbeam-uboot.dtsi | 2 +- arch/sandbox/dts/test.dts | 8 +- board/gdsys/common/cmd_ioloop.c | 12 +-- board/gdsys/mpc8308/gazerbeam.c | 47 ++++++----- board/google/chromebook_coral/coral.c | 2 +- common/spl/spl_fit.c | 15 ++-- configs/chromebook_coral_defconfig | 2 +- configs/gazerbeam_defconfig | 4 +- configs/sandbox64_defconfig | 4 +- configs/sandbox_defconfig | 4 +- configs/sandbox_flattree_defconfig | 4 +- configs/sandbox_spl_defconfig | 4 +- .../gdsys,sysinfo_gazerbeam.txt} | 8 +- drivers/Kconfig | 4 +- drivers/Makefile | 2 +- drivers/board/board-uclass.c | 71 ---------------- drivers/{board => sysinfo}/Kconfig | 16 ++-- drivers/{board => sysinfo}/Makefile | 6 +- drivers/{board => sysinfo}/gazerbeam.c | 74 ++++++++--------- drivers/{board => sysinfo}/gazerbeam.h | 0 drivers/{board => sysinfo}/sandbox.c | 50 ++++++------ drivers/{board => sysinfo}/sandbox.h | 0 drivers/sysinfo/sysinfo-uclass.c | 71 ++++++++++++++++ drivers/timer/mpc83xx_timer.c | 10 +-- include/dm/uclass-id.h | 2 +- include/{board.h => sysinfo.h} | 80 +++++++++---------- test/dm/Makefile | 2 +- test/dm/board.c | 59 -------------- test/dm/sysinfo.c | 59 ++++++++++++++ 29 files changed, 314 insertions(+), 308 deletions(-) rename doc/device-tree-bindings/{board/gdsys,board_gazerbeam.txt => sysinfo/gdsys,sysinfo_gazerbeam.txt} (90%) delete mode 100644 drivers/board/board-uclass.c rename drivers/{board => sysinfo}/Kconfig (65%) rename drivers/{board => sysinfo}/Makefile (50%) rename drivers/{board => sysinfo}/gazerbeam.c (69%) rename drivers/{board => sysinfo}/gazerbeam.h (100%) rename drivers/{board => sysinfo}/sandbox.c (50%) rename drivers/{board => sysinfo}/sandbox.h (100%) create mode 100644 drivers/sysinfo/sysinfo-uclass.c rename include/{board.h => sysinfo.h} (65%) delete mode 100644 test/dm/board.c create mode 100644 test/dm/sysinfo.c
Reviewed-by: Bin Meng bmeng.cn@gmail.com