[PATCH v4 3/5] ram: rockchip: Fix dependency of RAM_ROCKCHIP_DEBUG
Quentin Schulz
quentin.schulz at cherry.de
Fri May 16 17:31:39 CEST 2025
Hi Lukasz,
On 5/15/25 5:55 PM, Lukasz Czechowski wrote:
> The RAM_ROCKCHIP_DEBUG can be used only if DEBUG_UART is
> available, otherwise it won't have any effect.
>
I think it used to have an effect before patch 2/5 no?
If that is the case, I would recommend to reorder the patches so that
3/5 is before 2/5 and in the commit log you explain that the next commit
is going to require DEBUG_UART to be defined in order to be able to use
printch/printascii/... That way, expected behavior won't change (i.e. if
I checkout at patch 2/5, I don't get anything on the serial anymore, but
I do if I checkout 1/5 or 3/5).
Looks good to me otherwise:
Reviewed-by: Quentin Schulz <quentin.schulz at cherry.de>
Thanks!
Quentin
More information about the U-Boot
mailing list