[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <D848JET5TDL6.2L4ZQR0YHYQU6@cknow.org>
Date: Fri, 28 Feb 2025 17:55:47 +0100
From: "Diederik de Haas" <didi.debian@...ow.org>
To: "Yao Zi" <ziyao@...root.org>, "Rob Herring" <robh@...nel.org>,
"Krzysztof Kozlowski" <krzk+dt@...nel.org>, "Conor Dooley"
<conor+dt@...nel.org>, "Heiko Stuebner" <heiko@...ech.de>, "Dragan Simic"
<dsimic@...jaro.org>, "Johan Jonker" <jbx6244@...il.com>, "Wenhao Cui"
<lasstp5011@...il.com>, "Yuteng Zhong" <zonyitoo@...il.com>
Cc: <devicetree@...r.kernel.org>, <linux-arm-kernel@...ts.infradead.org>,
<linux-rockchip@...ts.infradead.org>, <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] arm64: dts: rockchip: Remove undocumented sdmmc
property from lubancat-1
On Fri Feb 28, 2025 at 5:31 PM CET, Yao Zi wrote:
> Property "supports-cd" isn't documented anywhere and is unnecessary for
s/supports-cd/supports-sd/ ?
Cheers,
Diederik
> mainline driver to function. It seems a property used by downstream
> kernel was brought into mainline.
>
> This should be reported by dtbs_check, but mmc-controller-common.yaml
> defaults additionalProperties to true thus allows it. Remove the
> property to clean the devicetree up and avoid possible confusion.
>
> Fixes: 8d94da58de53 ("arm64: dts: rockchip: Add EmbedFire LubanCat 1")
> Signed-off-by: Yao Zi <ziyao@...root.org>
> ---
> arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts b/arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts
> index 61dd71c259aa..ddf84c2a19cf 100644
> --- a/arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts
> +++ b/arch/arm64/boot/dts/rockchip/rk3566-lubancat-1.dts
> @@ -512,7 +512,6 @@ &sdhci {
>
> &sdmmc0 {
> max-frequency = <150000000>;
> - supports-sd;
> bus-width = <4>;
> cap-mmc-highspeed;
> cap-sd-highspeed;
Download attachment "signature.asc" of type "application/pgp-signature" (229 bytes)
Powered by blists - more mailing lists