diff mbox series

[1/1] cmd: CONFIG_CMD_MMC depends on CONFIG_MMC

Message ID 20210327104354.37837-1-xypron.glpk@gmx.de
State Accepted
Commit ec611871f3eb366f7efd557167ec2284e5f5069b
Delegated to: Tom Rini
Headers show
Series [1/1] cmd: CONFIG_CMD_MMC depends on CONFIG_MMC | expand

Commit Message

Heinrich Schuchardt March 27, 2021, 10:43 a.m. UTC
Trying to compile with CONFIG_CMD_MMC=y and CONFIG_MMC=n leads to errors:

riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmcops':
cmd/mmc.c:984: undefined reference to `get_mmc_num'
riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmc_setdsr':
cmd/mmc.c:873: undefined reference to `find_mmc_device'

Add missing dependency.

Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
---
 cmd/Kconfig | 1 +
 1 file changed, 1 insertion(+)

--
2.29.2

Comments

Bin Meng March 27, 2021, 12:04 p.m. UTC | #1
On Sat, Mar 27, 2021 at 6:44 PM Heinrich Schuchardt <xypron.glpk@gmx.de> wrote:
>
> Trying to compile with CONFIG_CMD_MMC=y and CONFIG_MMC=n leads to errors:
>
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmcops':
> cmd/mmc.c:984: undefined reference to `get_mmc_num'
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmc_setdsr':
> cmd/mmc.c:873: undefined reference to `find_mmc_device'
>
> Add missing dependency.
>
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> ---
>  cmd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
>

Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Jaehoon Chung March 28, 2021, 11:02 p.m. UTC | #2
On 3/27/21 7:43 PM, Heinrich Schuchardt wrote:
> Trying to compile with CONFIG_CMD_MMC=y and CONFIG_MMC=n leads to errors:
> 
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmcops':
> cmd/mmc.c:984: undefined reference to `get_mmc_num'
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmc_setdsr':
> cmd/mmc.c:873: undefined reference to `find_mmc_device'
> 
> Add missing dependency.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>

Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

Best Regards,
Jaehoon Chung

> ---
>  cmd/Kconfig | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/cmd/Kconfig b/cmd/Kconfig
> index 0107c0fa3e..3ce223bed0 100644
> --- a/cmd/Kconfig
> +++ b/cmd/Kconfig
> @@ -1052,6 +1052,7 @@ config CMD_MISC
> 
>  config CMD_MMC
>  	bool "mmc"
> +	depends on MMC
>  	help
>  	  MMC memory mapped support.
> 
> --
> 2.29.2
> 
>
Tom Rini April 20, 2021, 2:20 p.m. UTC | #3
On Sat, Mar 27, 2021 at 11:43:54AM +0100, Heinrich Schuchardt wrote:

> Trying to compile with CONFIG_CMD_MMC=y and CONFIG_MMC=n leads to errors:
> 
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmcops':
> cmd/mmc.c:984: undefined reference to `get_mmc_num'
> riscv64-linux-gnu-ld.bfd: cmd/built-in.o: in function `do_mmc_setdsr':
> cmd/mmc.c:873: undefined reference to `find_mmc_device'
> 
> Add missing dependency.
> 
> Signed-off-by: Heinrich Schuchardt <xypron.glpk@gmx.de>
> Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
> Reviewed-by: Jaehoon Chung <jh80.chung@samsung.com>

Applied to u-boot/master, thanks!
diff mbox series

Patch

diff --git a/cmd/Kconfig b/cmd/Kconfig
index 0107c0fa3e..3ce223bed0 100644
--- a/cmd/Kconfig
+++ b/cmd/Kconfig
@@ -1052,6 +1052,7 @@  config CMD_MISC

 config CMD_MMC
 	bool "mmc"
+	depends on MMC
 	help
 	  MMC memory mapped support.