linux-kernel - Re: [v5 2/2] arm64: dts: lx2160a: add sata node support

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 for Android: free password hash cracker in your pocket

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

Message-ID: <20190211053920.GM22487@dragon>
Date:   Mon, 11 Feb 2019 13:39:22 +0800
From:   Shawn Guo <shawnguo@...nel.org>
To:     Peng Ma <peng.ma@....com>
Cc:     "axboe@...nel.dk" <axboe@...nel.dk>,
        "mark.rutland@....com" <mark.rutland@....com>,
        Andy Tang <andy.tang@....com>,
        "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        Leo Li <leoyang.li@....com>,
        "linux-ide@...r.kernel.org" <linux-ide@...r.kernel.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [v5 2/2] arm64: dts: lx2160a: add sata node support

On Mon, Feb 11, 2019 at 02:53:18AM +0000, Peng Ma wrote:
> 
> 
> >-----Original Message-----
> >From: Shawn Guo <shawnguo@...nel.org>
> >Sent: 2019年2月1日 14:30
> >To: Peng Ma <peng.ma@....com>
> >Cc: axboe@...nel.dk; robh+dt@...nel.org; mark.rutland@....com; Leo Li
> ><leoyang.li@....com>; linux-ide@...r.kernel.org; devicetree@...r.kernel.org;
> >linux-kernel@...r.kernel.org; linux-arm-kernel@...ts.infradead.org; Andy Tang
> ><andy.tang@....com>
> >Subject: Re: [v5 2/2] arm64: dts: lx2160a: add sata node support
> >
> >On Fri, Jan 25, 2019 at 08:10:13AM +0000, Peng Ma wrote:
> >> Add SATA device nodes for fsl-lx2160a and enable support for QDS and
> >> RDB boards.
> >>
> >> Signed-off-by: Peng Ma <peng.ma@....com>
> >> ---
> >> changed for V5:
> >> 	- no change
> >>
> >>  arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts |   16 +++++++
> >>  arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts |   16 +++++++
> >>  arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi    |   44
> >+++++++++++++++++++++
> >>  3 files changed, 76 insertions(+), 0 deletions(-)
> >>
> >> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> >> b/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> >> index 99a22ab..1a5acf6 100644
> >> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> >> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-qds.dts
> >> @@ -95,6 +95,22 @@
> >>  	};
> >>  };
> >>
> >> +&sata0 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata1 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata2 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata3 {
> >> +	status = "okay";
> >> +};
> >> +
> >>  &uart0 {
> >>  	status = "okay";
> >>  };
> >> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> >> b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> >> index 6481e5f..5b6799e 100644
> >> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> >> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a-rdb.dts
> >> @@ -102,6 +102,22 @@
> >>  	};
> >>  };
> >>
> >> +&sata0 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata1 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata2 {
> >> +	status = "okay";
> >> +};
> >> +
> >> +&sata3 {
> >> +	status = "okay";
> >> +};
> >> +
> >>  &uart0 {
> >>  	status = "okay";
> >>  };
> >> diff --git a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> >> b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> >> index a79f5c1..592034b 100644
> >> --- a/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> >> +++ b/arch/arm64/boot/dts/freescale/fsl-lx2160a.dtsi
> >> @@ -671,6 +671,50 @@
> >>  			status = "disabled";
> >>  		};
> >>
> >> +		sata0: sata@...0000 {
> >> +			compatible = "fsl,lx2160a-ahci";
> >
> >Has the kernel driver been patched to probe the compatible?
> >
> >Shawn
> Yes, the driver patch is http://patchwork.ozlabs.org/patch/1034083/

Ping me when the driver patch is accepted.  Ideally, we only accept dts
changes after the driver counterpart lands on mainline.

Shawn