Hi Jerome,

Thank you for the patch.

On mar., sept. 10, 2024 at 15:00, Jerome Forissier 
<[email protected]> wrote:

> Make SPL_RAM_SUPPORT a hidden Kconfig symbol, automatically selected
> by SPL_RAM_DEVICE or SPL_DFU. Avoids the situation where SPL_RAM_SUPPORT
> may be enabled without the other two being enabled, which results in the
> following build warning:
>
> common/spl/spl_ram.c:19:14: warning: ‘spl_ram_load_read’ defined but not used 
> [-Wunused-function]
>    19 | static ulong spl_ram_load_read(struct spl_load_info *load, ulong 
> sector,
>       |              ^~~~~~~~~~~~~~~~~
>
> Signed-off-by: Jerome Forissier <[email protected]>

Reviewed-by: Mattijs Korpershoek <[email protected]>

> ---
>  common/spl/Kconfig         | 8 ++------
>  drivers/usb/gadget/Kconfig | 2 +-
>  2 files changed, 3 insertions(+), 7 deletions(-)
>
> diff --git a/common/spl/Kconfig b/common/spl/Kconfig
> index c08ff064493..885a012461c 100644
> --- a/common/spl/Kconfig
> +++ b/common/spl/Kconfig
> @@ -1238,15 +1238,11 @@ config SPL_POWER_DOMAIN
>         the drivers in drivers/power/domain as part of a SPL build.
>  
>  config SPL_RAM_SUPPORT
> -     bool "Support booting from RAM"
> -     default y if MICROBLAZE || ARCH_SOCFPGA || ARCH_TEGRA || ARCH_ZYNQ
> -     help
> -       Enable booting of an image in RAM. The image can be preloaded or
> -       it can be loaded by SPL directly into RAM (e.g. using USB).
> +     bool
>  
>  config SPL_RAM_DEVICE
>       bool "Support booting from preloaded image in RAM"
> -     depends on SPL_RAM_SUPPORT
> +     select SPL_RAM_SUPPORT
>       default y if MICROBLAZE || ARCH_SOCFPGA || ARCH_TEGRA || ARCH_ZYNQ
>       help
>         Enable booting of an image already loaded in RAM. The image has to
> diff --git a/drivers/usb/gadget/Kconfig b/drivers/usb/gadget/Kconfig
> index 4621a6fd5e6..b1247a9eeae 100644
> --- a/drivers/usb/gadget/Kconfig
> +++ b/drivers/usb/gadget/Kconfig
> @@ -323,7 +323,7 @@ config SPL_DFU
>       bool "Support DFU (Device Firmware Upgrade) in SPL"
>       select SPL_HASH
>       select SPL_DFU_NO_RESET
> -     depends on SPL_RAM_SUPPORT
> +     select SPL_RAM_SUPPORT
>       help
>         This feature enables the DFU (Device Firmware Upgrade) in SPL with
>         RAM memory device support. The ROM code will load and execute
> -- 
> 2.40.1

Reply via email to