linux-kernel - Re: [PATCH] arm64: dts: marvell: espressobin: De-duplicate eMMC definitions

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Fri, 2 Oct 2020 14:13:52 +0200
From:   Pali Rohár <pali@...nel.org>
To:     Andre Heider <a.heider@...il.com>,
        Gregory Clement <gregory.clement@...tlin.com>
Cc:     Jason Cooper <jason@...edaemon.net>, Andrew Lunn <andrew@...n.ch>,
        Sebastian Hesselbarth <sebastian.hesselbarth@...il.com>,
        Rob Herring <robh+dt@...nel.org>, devicetree@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH] arm64: dts: marvell: espressobin: De-duplicate eMMC
 definitions

On Sunday 27 September 2020 08:40:41 Andre Heider wrote:
> On 25/09/2020 10:50, Pali Rohár wrote:
> > eMMC definitions in files armada-3720-espressobin-emmc.dts and
> > armada-3720-espressobin-v7-emmc.dts is same. So move it into common
> > armada-3720-espressobin.dtsi file with status "disabled".
> > 
> > This change simplifies eMMC variants of DTS files for Espressobin.
> > 
> > Signed-off-by: Pali Rohár <pali@...nel.org>
> 
> Reviewed-by: Andre Heider <a.heider@...il.com>
> Tested-by: Andre Heider <a.heider@...il.com>
> 
> > ---
> > Compiled DTB files armada-3720-espressobin-emmc.dtb and
> > armada-3720-espressobin-v7-emmc.dtb are identical as without applying
> > this patch.
> > 
> > Files armada-3720-espressobin.dtb and armada-3720-espressobin-v7.dtb
> > are slightly different compared to version without this patch.
> > 
> > Main change is that numering in all "phandle" nodes is shifted and
> > "sdhci0" node contains more attributes, but node is disabled.
> > 
> > 
> > Andre, could you test this change on Espressobin (without eMMC) if
> > everything is OK and there is no issue?
> 
> Look good to me, the node appears with status=disabled and everything seems
> to work as before.

Great!

Gregory, would do you think about including this patch into next queue too?

> > 
> > ---
> >   .../marvell/armada-3720-espressobin-emmc.dts  | 18 --------------
> >   .../armada-3720-espressobin-v7-emmc.dts       | 18 --------------
> >   .../dts/marvell/armada-3720-espressobin.dtsi  | 24 +++++++++++++++++++
> >   3 files changed, 24 insertions(+), 36 deletions(-)
> > 
> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts
> > index ec72a11ed80f..5c4d8f379704 100644
> > --- a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts
> > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-emmc.dts
> > @@ -21,24 +21,6 @@
> >   		     "marvell,armada3720", "marvell,armada3710";
> >   };
> > -/* U11 */
> >   &sdhci0 {
> > -	non-removable;
> > -	bus-width = <8>;
> > -	mmc-ddr-1_8v;
> > -	mmc-hs400-1_8v;
> > -	marvell,xenon-emmc;
> > -	marvell,xenon-tun-count = <9>;
> > -	marvell,pad-type = "fixed-1-8v";
> > -
> > -	pinctrl-names = "default";
> > -	pinctrl-0 = <&mmc_pins>;
> >   	status = "okay";
> > -
> > -	#address-cells = <1>;
> > -	#size-cells = <0>;
> > -	mmccard: mmccard@0 {
> > -		compatible = "mmc-card";
> > -		reg = <0>;
> > -	};
> >   };
> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts
> > index 6062a7df7342..4775a7eda481 100644
> > --- a/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts
> > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin-v7-emmc.dts
> > @@ -36,24 +36,6 @@
> >   	label = "wan";
> >   };
> > -/* U11 */
> >   &sdhci0 {
> > -	non-removable;
> > -	bus-width = <8>;
> > -	mmc-ddr-1_8v;
> > -	mmc-hs400-1_8v;
> > -	marvell,xenon-emmc;
> > -	marvell,xenon-tun-count = <9>;
> > -	marvell,pad-type = "fixed-1-8v";
> > -
> > -	pinctrl-names = "default";
> > -	pinctrl-0 = <&mmc_pins>;
> >   	status = "okay";
> > -
> > -	#address-cells = <1>;
> > -	#size-cells = <0>;
> > -	mmccard: mmccard@0 {
> > -		compatible = "mmc-card";
> > -		reg = <0>;
> > -	};
> >   };
> > diff --git a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi
> > index 3169a820558f..8a1c678bea5f 100644
> > --- a/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi
> > +++ b/arch/arm64/boot/dts/marvell/armada-3720-espressobin.dtsi
> > @@ -58,6 +58,30 @@
> >   	phy-names = "sata-phy";
> >   };
> > +/* U11 */
> > +&sdhci0 {
> > +	/* Main DTS file for Espressobin is without eMMC */
> > +	status = "disabled";
> > +
> > +	non-removable;
> > +	bus-width = <8>;
> > +	mmc-ddr-1_8v;
> > +	mmc-hs400-1_8v;
> > +	marvell,xenon-emmc;
> > +	marvell,xenon-tun-count = <9>;
> > +	marvell,pad-type = "fixed-1-8v";
> > +
> > +	pinctrl-names = "default";
> > +	pinctrl-0 = <&mmc_pins>;
> > +
> > +	#address-cells = <1>;
> > +	#size-cells = <0>;
> > +	mmccard: mmccard@0 {
> > +		compatible = "mmc-card";
> > +		reg = <0>;
> > +	};
> > +};
> > +
> >   /* J1 */
> >   &sdhci1 {
> >   	wp-inverted;
> > 
>