Re: [PATCH v2 3/5] ram: rockchip: Add separate RAM_ROCKCHIP_DEBUG config for TPL/SPL

2024-09-10 Thread Quentin Schulz
Hi Kever, Lukasz, On 9/10/24 5:17 PM, Łukasz Czechowski wrote: Hi Kever, Thanks for the review. This patch is indeed intended to disallow enabling RAM_ROCKCHIP_DEBUG when SILENT_CONSOLE is enabled and control what is printed on the debug console more independently. Patch was created in response

Re: [PATCH v2 3/5] ram: rockchip: Add separate RAM_ROCKCHIP_DEBUG config for TPL/SPL

2024-09-10 Thread Łukasz Czechowski
Hi Kever, Thanks for the review. This patch is indeed intended to disallow enabling RAM_ROCKCHIP_DEBUG when SILENT_CONSOLE is enabled and control what is printed on the debug console more independently. Patch was created in response to review of patchset v1, and implements change suggested by Quent

Re: [PATCH v2 3/5] ram: rockchip: Add separate RAM_ROCKCHIP_DEBUG config for TPL/SPL

2024-09-10 Thread Kever Yang
Hi Lukasz, On 2024/9/4 00:38, Lukasz Czechowski wrote: Introduce new config symbols TPL_RAM_ROCKCHIP_DEBUG and SPL_RAM_ROCKCHIP_DEBUG to allow for better dependencies control of RAM driver debugging configuration. The RAM_ROCKCHIP_DEBUG should enough because this only happen when ram driver

[PATCH v2 3/5] ram: rockchip: Add separate RAM_ROCKCHIP_DEBUG config for TPL/SPL

2024-09-03 Thread Lukasz Czechowski
Introduce new config symbols TPL_RAM_ROCKCHIP_DEBUG and SPL_RAM_ROCKCHIP_DEBUG to allow for better dependencies control of RAM driver debugging configuration. Add negative dependencies to TPL_SILENT_CONSOLE and SPL_SILENT_CONSOLE, respectively. Replace IS_ENABLED(CONFIG_RAM_ROCKCHIP_DEBUG) directi