Re: [PATCH v2] arm64: dts: rockchip: fix emmc reset polarity on px30-cobra

[Quentin Schulz]

Hi Jakob,

On 6/9/26 10:17 AM, Jakob Unterwurzacher wrote:
> From: Jakob Unterwurzacher <jakob.unterwurzacher@xxxxxxxxx>
>
> Technically, the reset signal is active low - it's called RST_n after all.
>
> But it is ignored completely unless RST_n_FUNCTION=1 (byte 162 in extcsd)
> is set in the emmc. It is 0 per default.
>
> For emmcs that have RST_n_FUNCTION=1 we failed like this:
>
> 	[    3.074480] mmc1: Failed to initialize a non-removable card
>
> With this change they work normally.
>
> Cc: stable@xxxxxxxxxxxxxxx
> Fixes: bb510ddc9d3e ("arm64: dts: rockchip: add px30-cobra base dtsi and board variants")
> Signed-off-by: Jakob Unterwurzacher <jakob.unterwurzacher@xxxxxxxxx>

Please try to not forget to add tags (Reviewed-by, Acked-by, Tested-by, 
...) given in earlier versions of a newer version of the patch (except 
if substantial changes were made, which isn't the case here).

Reviewed-by: Quentin Schulz <quentin.schulz@xxxxxxxxx>

Thanks!
Quentin