Re: [U-Boot] [PATCH] riscv: qemu: Fix kconfig build warning
On Wed, 2019-08-07 at 23:04 -0700, Bin Meng wrote: > When 'make qemu-riscv64_defconfig', there is a build warning: > > board/emulation/qemu-riscv/Kconfig:24: > warning: config symbol defined without type > > Fix it by specifying the config symbol type to 'hex'. > > Signed-off-by: Bin Meng > --- > > board/emulation/qemu-riscv/Kconfig | 1 + > 1 file changed, 1 insertion(+) > Ah, I missed this. Thanks! Reviewed-by: Lukas Auer ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] riscv: qemu: Fix kconfig build warning
Hi Rick, On Thu, Aug 8, 2019 at 2:49 PM Rick Chen wrote: > > Hi Bin > > > > From: Bin Meng [mailto:bmeng...@gmail.com] > > > Sent: Thursday, August 08, 2019 2:05 PM > > > To: Lukas Auer; Rick Jian-Zhi Chen(陳建志); U-Boot Mailing List > > > Subject: [PATCH] riscv: qemu: Fix kconfig build warning > > > > > > When 'make qemu-riscv64_defconfig', there is a build warning: > > > > > > board/emulation/qemu-riscv/Kconfig:24: > > > warning: config symbol defined without type > > > > > > Fix it by specifying the config symbol type to 'hex'. > > > > > > Signed-off-by: Bin Meng > > > --- > > > > > > board/emulation/qemu-riscv/Kconfig | 1 + > > > 1 file changed, 1 insertion(+) > > > > > > diff --git a/board/emulation/qemu-riscv/Kconfig > > > b/board/emulation/qemu-riscv/Kconfig > > > index 1928d6d..7ce1201 100644 > > > --- a/board/emulation/qemu-riscv/Kconfig > > > +++ b/board/emulation/qemu-riscv/Kconfig > > > @@ -22,6 +22,7 @@ config SPL_TEXT_BASE > > > default 0x8000 > > > > > > config SPL_OPENSBI_LOAD_ADDR > > > + hex > > > default 0x8100 > > > > > > config BOARD_SPECIFIC_OPTIONS # dummy > > > -- > > > 2.7.4 > > > > Good catch! > > But I am wondering why SPL_TEXT_BASE will not have the same warning message? > They are declared the same way. > Do you know why ? > Because SPL_TEXT_BASE has a type defined in common/spl/Kconfig hence Kconfig knows its type. > Reviewed-by: Rick Chen Regards, Bin ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
Re: [U-Boot] [PATCH] riscv: qemu: Fix kconfig build warning
Hi Bin > > From: Bin Meng [mailto:bmeng...@gmail.com] > > Sent: Thursday, August 08, 2019 2:05 PM > > To: Lukas Auer; Rick Jian-Zhi Chen(陳建志); U-Boot Mailing List > > Subject: [PATCH] riscv: qemu: Fix kconfig build warning > > > > When 'make qemu-riscv64_defconfig', there is a build warning: > > > > board/emulation/qemu-riscv/Kconfig:24: > > warning: config symbol defined without type > > > > Fix it by specifying the config symbol type to 'hex'. > > > > Signed-off-by: Bin Meng > > --- > > > > board/emulation/qemu-riscv/Kconfig | 1 + > > 1 file changed, 1 insertion(+) > > > > diff --git a/board/emulation/qemu-riscv/Kconfig > > b/board/emulation/qemu-riscv/Kconfig > > index 1928d6d..7ce1201 100644 > > --- a/board/emulation/qemu-riscv/Kconfig > > +++ b/board/emulation/qemu-riscv/Kconfig > > @@ -22,6 +22,7 @@ config SPL_TEXT_BASE > > default 0x8000 > > > > config SPL_OPENSBI_LOAD_ADDR > > + hex > > default 0x8100 > > > > config BOARD_SPECIFIC_OPTIONS # dummy > > -- > > 2.7.4 > Good catch! But I am wondering why SPL_TEXT_BASE will not have the same warning message? They are declared the same way. Do you know why ? Reviewed-by: Rick Chen ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
[U-Boot] [PATCH] riscv: qemu: Fix kconfig build warning
When 'make qemu-riscv64_defconfig', there is a build warning: board/emulation/qemu-riscv/Kconfig:24: warning: config symbol defined without type Fix it by specifying the config symbol type to 'hex'. Signed-off-by: Bin Meng --- board/emulation/qemu-riscv/Kconfig | 1 + 1 file changed, 1 insertion(+) diff --git a/board/emulation/qemu-riscv/Kconfig b/board/emulation/qemu-riscv/Kconfig index 1928d6d..7ce1201 100644 --- a/board/emulation/qemu-riscv/Kconfig +++ b/board/emulation/qemu-riscv/Kconfig @@ -22,6 +22,7 @@ config SPL_TEXT_BASE default 0x8000 config SPL_OPENSBI_LOAD_ADDR + hex default 0x8100 config BOARD_SPECIFIC_OPTIONS # dummy -- 2.7.4 ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot