linux-kernel - Re: [PATCH v3 05/10] arm: dts: mt7623: update pio, usb and crypto nodes

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]

Message-ID: <1508464539.13396.6.camel@mtkswgap22>
Date:   Fri, 20 Oct 2017 09:55:39 +0800
From:   Ryder Lee <ryder.lee@...iatek.com>
To:     Matthias Brugger <matthias.bgg@...il.com>
CC:     <linux-mediatek@...ts.infradead.org>,
        <linux-kernel@...r.kernel.org>, <devicetree@...r.kernel.org>,
        <linux-arm-kernel@...ts.infradead.org>,
        Erin Lo <erin.lo@...iatek.com>, YT Shen <yt.shen@...iatek.com>
Subject: Re: [PATCH v3 05/10] arm: dts: mt7623: update pio, usb and crypto
 nodes

Hi

On Thu, 2017-10-19 at 18:29 +0200, Matthias Brugger wrote:
> 
> On 10/02/2017 03:54 AM, Ryder Lee wrote:
> > This patch updates pio, usb and crypto nodes to make them be consistent
> > with the binding documents.
> > 
> > Signed-off-by: Ryder Lee <ryder.lee@...iatek.com>
> > ---
> >  arch/arm/boot/dts/mt7623.dtsi | 26 ++++++++++++++------------
> >  1 file changed, 14 insertions(+), 12 deletions(-)
> > 
> > diff --git a/arch/arm/boot/dts/mt7623.dtsi b/arch/arm/boot/dts/mt7623.dtsi
> > index 381843e..0640fb7 100644
> > --- a/arch/arm/boot/dts/mt7623.dtsi
> > +++ b/arch/arm/boot/dts/mt7623.dtsi
> > @@ -227,8 +227,7 @@
> >  	};
> >  
> >  	pio: pinctrl@...05000 {
> > -		compatible = "mediatek,mt7623-pinctrl",
> > -			     "mediatek,mt2701-pinctrl";
> > +		compatible = "mediatek,mt7623-pinctrl";
> 
> Looks good.
> 
> >  		reg = <0 0x1000b000 0 0x1000>;
> >  		mediatek,pctl-regmap = <&syscfg_pctl_a>;
> >  		pins-are-numbered;
> > @@ -680,7 +679,7 @@
> >  		interrupts = <GIC_SPI 196 IRQ_TYPE_LEVEL_LOW>;
> >  		clocks = <&hifsys CLK_HIFSYS_USB0PHY>,
> >  			 <&topckgen CLK_TOP_ETHIF_SEL>;
> > -		clock-names = "sys_ck", "free_ck";
> > +		clock-names = "sys_ck", "ref_ck";
> 
> This is already merged. Please rebase.

I didn't see any changes in mt7623.dtsi.

> >  		power-domains = <&scpsys MT2701_POWER_DOMAIN_HIF>;
> >  		phys = <&u2port0 PHY_TYPE_USB2>, <&u3port0 PHY_TYPE_USB3>;
> >  		status = "disabled";
> > @@ -690,8 +689,6 @@
> >  		compatible = "mediatek,mt7623-u3phy",
> >  			     "mediatek,mt2701-u3phy";
> >  		reg = <0 0x1a1c4000 0 0x0700>;
> > -		clocks = <&clk26m>;
> > -		clock-names = "u3phya_ref";
> 
> Same here.
> 
> >  		#address-cells = <2>;
> >  		#size-cells = <2>;
> >  		ranges;
> > @@ -699,12 +696,16 @@
> >  
> >  		u2port0: usb-phy@...c4800 {
> >  			reg = <0 0x1a1c4800 0 0x0100>;
> > +			clocks = <&topckgen CLK_TOP_USB_PHY48M>;
> > +			clock-names = "ref";
> 
> Same here.
> 
> >  			#phy-cells = <1>;
> >  			status = "okay";
> >  		};
> >  
> >  		u3port0: usb-phy@...c4900 {
> >  			reg = <0 0x1a1c4900 0 0x0700>;
> > +			clocks = <&clk26m>;
> > +			clock-names = "ref";
> 
> Same here.
> 
> >  			#phy-cells = <1>;
> >  			status = "okay";
> >  		};
> > @@ -719,7 +720,7 @@
> >  		interrupts = <GIC_SPI 197 IRQ_TYPE_LEVEL_LOW>;
> >  		clocks = <&hifsys CLK_HIFSYS_USB1PHY>,
> >  			 <&topckgen CLK_TOP_ETHIF_SEL>;
> > -		clock-names = "sys_ck", "free_ck";
> > +		clock-names = "sys_ck", "ref_ck";
> 
> Same here.
> 
> >  		power-domains = <&scpsys MT2701_POWER_DOMAIN_HIF>;
> >  		phys = <&u2port1 PHY_TYPE_USB2>, <&u3port1 PHY_TYPE_USB3>;
> >  		status = "disabled";
> > @@ -729,8 +730,6 @@
> >  		compatible = "mediatek,mt7623-u3phy",
> >  			     "mediatek,mt2701-u3phy";
> >  		reg = <0 0x1a244000 0 0x0700>;
> > -		clocks = <&clk26m>;
> > -		clock-names = "u3phya_ref";
> 
> Same here.
> 
> >  		#address-cells = <2>;
> >  		#size-cells = <2>;
> >  		ranges;
> > @@ -738,12 +737,16 @@
> >  
> >  		u2port1: usb-phy@...44800 {
> >  			reg = <0 0x1a244800 0 0x0100>;
> > +			clocks = <&topckgen CLK_TOP_USB_PHY48M>;
> > +			clock-names = "ref";
> 
> Same here.
> 
> >  			#phy-cells = <1>;
> >  			status = "okay";
> >  		};
> >  
> >  		u3port1: usb-phy@...44900 {
> >  			reg = <0 0x1a244900 0 0x0700>;
> > +			clocks = <&clk26m>;
> > +			clock-names = "ref";
> 
> Same here.
> 
> >  			#phy-cells = <1>;
> >  			status = "okay";
> >  		};
> > @@ -784,16 +787,15 @@
> >  	};
> >  
> >  	crypto: crypto@...40000 {
> > -		compatible = "mediatek,mt7623-crypto";
> > +		compatible = "mediatek,eip97-crypto";
> 
> Crypto node is not present in upstream.

Crypto driver is already merged. 

See Documentation/devicetree/bindings/crypto/mediatek-crypto.txt

Thanks