[PATCH 2/4] ram: rockchip: Kconfig: Fix dependency of RAM_ROCKCHIP_DEBUG
Quentin Schulz
quentin.schulz at cherry.de
Thu Aug 29 12:19:14 CEST 2024
Hi Lukasz,
On 8/27/24 4:20 PM, Lukasz Czechowski wrote:
> The RAM_ROCKCHIP_DEBUG can be used only if DEBUG_UART is
> available, otherwise it won't have any effect.
> Add negative dependency to TPL_SILENT_CONSOLE.
>
> Signed-off-by: Lukasz Czechowski <lukasz.czechowski at thaumatec.com>
> ---
> drivers/ram/rockchip/Kconfig | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/ram/rockchip/Kconfig b/drivers/ram/rockchip/Kconfig
> index 67c63ecba04..4a6a6328676 100644
> --- a/drivers/ram/rockchip/Kconfig
> +++ b/drivers/ram/rockchip/Kconfig
> @@ -15,6 +15,8 @@ if RAM_ROCKCHIP
>
> config RAM_ROCKCHIP_DEBUG
> bool "Rockchip ram drivers debugging"
> + depends on DEBUG_UART
> + depends on !TPL_SILENT_CONSOLE
I am wondering if we should also do this for TPL-less systems? i.e. the
ones for which the dram is configured in SPL.
Something like:
depends on !SPL_SILENT_CONSOLE if SPL_RAM
depends on !TPL_SILENT_CONSOLE if TPL_RAM
maybe?
Cheers,
Quentin
More information about the U-Boot
mailing list