diff mbox series

[02/22] sunxi: Remove unnecessary Kconfig selections

Message ID 20221101050835.2421-3-samuel@sholland.org
State Deferred
Delegated to: Tom Rini
Headers show
Series sunxi: Prepare platform Kconfig to support multiple architectures | expand

Commit Message

Samuel Holland Nov. 1, 2022, 5:08 a.m. UTC
Two of these selections are redundant and have no effect:
 - DM_KEYBOARD is selected by USB_KEYBOARD
 - DM_MMC is selected by MMC

This selection has no effect by default and is unnecessarily strong:
 - USB_STORAGE is implied by DISTRO_DEFAULTS

Signed-off-by: Samuel Holland <samuel@sholland.org>
---

 arch/arm/Kconfig | 3 ---
 1 file changed, 3 deletions(-)

Comments

Andre Przywara Nov. 11, 2022, 1:09 a.m. UTC | #1
On Tue,  1 Nov 2022 00:08:14 -0500
Samuel Holland <samuel@sholland.org> wrote:

> Two of these selections are redundant and have no effect:
>  - DM_KEYBOARD is selected by USB_KEYBOARD
>  - DM_MMC is selected by MMC
> 
> This selection has no effect by default and is unnecessarily strong:
>  - USB_STORAGE is implied by DISTRO_DEFAULTS

With patch 01/22 removed (for now), this disables DM_KEYBOARD on 15
boards, it looks like for those without USB. I wouldn't be aware of any
other keyboard type supported on Allwinner boards, so the change is
fine:

> Signed-off-by: Samuel Holland <samuel@sholland.org>

Reviewed-by: Andre Przywara <andre.przywara@arm.com>

Cheers,
Andre

> ---
> 
>  arch/arm/Kconfig | 3 ---
>  1 file changed, 3 deletions(-)
> 
> diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
> index d3a1f03b36..939f76867e 100644
> --- a/arch/arm/Kconfig
> +++ b/arch/arm/Kconfig
> @@ -1144,8 +1144,6 @@ config ARCH_SUNXI
>  	select DM_I2C if I2C
>  	select DM_SPI if SPI
>  	select DM_SPI_FLASH if SPI
> -	select DM_KEYBOARD
> -	select DM_MMC if MMC
>  	select DM_SCSI if SCSI
>  	select DM_SERIAL
>  	select GPIO_EXTRA_HEADER
> @@ -1163,7 +1161,6 @@ config ARCH_SUNXI
>  	select SYS_THUMB_BUILD if !ARM64
>  	select USB if DISTRO_DEFAULTS
>  	select USB_KEYBOARD if DISTRO_DEFAULTS && USB_HOST
> -	select USB_STORAGE if DISTRO_DEFAULTS && USB_HOST
>  	select SPL_USE_TINY_PRINTF
>  	select USE_PREBOOT
>  	select SYS_RELOC_GD_ENV_ADDR
diff mbox series

Patch

diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig
index d3a1f03b36..939f76867e 100644
--- a/arch/arm/Kconfig
+++ b/arch/arm/Kconfig
@@ -1144,8 +1144,6 @@  config ARCH_SUNXI
 	select DM_I2C if I2C
 	select DM_SPI if SPI
 	select DM_SPI_FLASH if SPI
-	select DM_KEYBOARD
-	select DM_MMC if MMC
 	select DM_SCSI if SCSI
 	select DM_SERIAL
 	select GPIO_EXTRA_HEADER
@@ -1163,7 +1161,6 @@  config ARCH_SUNXI
 	select SYS_THUMB_BUILD if !ARM64
 	select USB if DISTRO_DEFAULTS
 	select USB_KEYBOARD if DISTRO_DEFAULTS && USB_HOST
-	select USB_STORAGE if DISTRO_DEFAULTS && USB_HOST
 	select SPL_USE_TINY_PRINTF
 	select USE_PREBOOT
 	select SYS_RELOC_GD_ENV_ADDR