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.czechow...@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

Reply via email to