[PATCH] Kconfigs: Correct default of "0" on hex type entries
Tom Rini
trini at konsulko.com
Thu Aug 3 14:02:10 CEST 2023
On Wed, Aug 02, 2023 at 03:31:22PM -0600, Simon Glass wrote:
> Hi Tom,
>
> On Wed, 2 Aug 2023 at 09:10, Tom Rini <trini at konsulko.com> wrote:
> >
> > It is not a parse error to have a default value of "0" for a "hex" type
> > entry, instead of "0x0". However, "0" and "0x0" are not treated the
> > same even by the tools themselves. Correct this by changing the default
> > value from "0" to "0x0" for all hex type questions that had the
> > incorrect default. Fix one instance (in two configs) of a default of "0"
> > being used on a hex question to be "0x0". Remove the cases where a
> > defconfig had set a value of "0x0" to be used as the default had been
> > "0".
> >
> > Signed-off-by: Tom Rini <trini at konsulko.com>
> > ---
> > arch/arm/cpu/armv7/ls102xa/Kconfig | 1 -
> > arch/arm/cpu/armv8/fsl-layerscape/Kconfig | 2 +-
> > arch/arm/mach-imx/Kconfig | 2 +-
> > arch/arm/mach-imx/imx8/Kconfig | 4 ++--
> > arch/arm/mach-rockchip/Kconfig | 2 +-
> > arch/arm/mach-sunxi/Kconfig | 4 ++--
> > arch/powerpc/cpu/mpc8xx/Kconfig | 2 +-
> > boot/Kconfig | 2 +-
> > cmd/Kconfig | 8 ++++----
> > common/Kconfig | 2 +-
> > common/spl/Kconfig | 2 +-
> > common/spl/Kconfig.tpl | 2 +-
> > configs/am335x_guardian_defconfig | 1 -
> > configs/efi-x86_app32_defconfig | 2 +-
> > configs/efi-x86_app64_defconfig | 2 +-
> > configs/evb-rk3229_defconfig | 1 -
> > configs/evb-rk3308_defconfig | 1 -
> > configs/imx6q_logic_defconfig | 1 -
> > configs/imx8mm_data_modul_edm_sbc_defconfig | 1 -
> > configs/imx8mn_beacon_2g_defconfig | 1 -
> > configs/imx8mn_beacon_defconfig | 1 -
> > configs/imx8mn_beacon_fspi_defconfig | 1 -
> > configs/imx8mp_beacon_defconfig | 1 -
> > configs/imx8mp_data_modul_edm_sbc_defconfig | 1 -
> > configs/imx8mp_dhcom_pdk2_defconfig | 1 -
> > configs/imx8mp_dhcom_pdk3_defconfig | 1 -
> > configs/kontron_sl28_defconfig | 2 --
> > configs/lion-rk3368_defconfig | 1 -
> > configs/roc-cc-rk3308_defconfig | 1 -
> > configs/rock-pi-s-rk3308_defconfig | 1 -
> > configs/sandbox64_defconfig | 1 -
> > configs/sandbox_defconfig | 1 -
> > configs/sandbox_flattree_defconfig | 1 -
> > configs/sandbox_noinst_defconfig | 1 -
> > configs/sandbox_spl_defconfig | 1 -
> > configs/sandbox_vpl_defconfig | 1 -
> > configs/xilinx_versal_net_virt_defconfig | 1 -
> > configs/xilinx_zynqmp_virt_defconfig | 1 -
> > drivers/block/Kconfig | 2 +-
> > drivers/fastboot/Kconfig | 2 +-
> > drivers/i2c/Kconfig | 16 ++++++++--------
> > drivers/misc/Kconfig | 2 +-
> > drivers/serial/Kconfig | 6 +++---
> > drivers/usb/gadget/Kconfig | 2 +-
> > drivers/video/Kconfig | 4 ++--
> > env/Kconfig | 4 ++--
> > lib/Kconfig | 2 +-
> > 47 files changed, 38 insertions(+), 64 deletions(-)
> >
>
> I'm not sure how you found all those, but it looks good to me from a spot check.
Manually. It's thankfully not many cases of "default 0".
> Can we have a checkpatch check?
Sadly, no. But I think if we spent some time fixing other issues too,
kconfiglib could be used as a tool to catch these.
--
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20230803/ce51dfc8/attachment.sig>
More information about the U-Boot
mailing list