From mboxrd@z Thu Jan 1 00:00:00 1970 From: Boris Brezillon Subject: Re: [PATCH 1/2][v2][RESEND] ARM: dts: Add big-endian for IFC on LS1021A Date: Thu, 22 Feb 2018 11:31:47 +0100 Message-ID: <20180222113147.2cce0f92@bbrezillon> References: <20180222101511.29631-1-prabhakar.kushwaha@nxp.com> <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=m.gmane.org@lists.infradead.org To: Prabhakar Kushwaha Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, robh@kernel.org, boris.brezillon@free-electrons.com, Alison Wang , leoyang.li@nxp.com, oss@buserror.net, linux-mtd@lists.infradead.org, Jagdish Gediya , cyrille.pitchen@wedev4u.fr, computersforpeace@gmail.com, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org List-Id: devicetree@vger.kernel.org On Thu, 22 Feb 2018 15:45:10 +0530 Prabhakar Kushwaha wrote: > For the patch to update struct map_info's swap field based on device > characteristics defined in device tree, big-endian parameter is added > for LS1021A. > > Signed-off-by: Alison Wang > Signed-off-by: Jagdish Gediya > Signed-off-by: Prabhakar Kushwaha > --- > Due to patch series http://patchwork.ozlabs.org/project/linux-mtd/list/?series=28735 > this patch is now independent of flash propoerty. So resending. > > Resending as all patches of patchset never reaches to patchwork. Again, that's not the case, I just marked them as "Not applicable" (see my previous reply). Also, according to the cover letter this patch should not be part of "dt-bindings: mtd-physmap: Add endianness supports", so why are you sending it here? > > arch/arm/boot/dts/ls1021a.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi > index 9319e1f0f1d8..babb086ad94f 100644 > --- a/arch/arm/boot/dts/ls1021a.dtsi > +++ b/arch/arm/boot/dts/ls1021a.dtsi > @@ -145,6 +145,7 @@ > ifc: ifc@1530000 { > compatible = "fsl,ifc", "simple-bus"; > reg = <0x0 0x1530000 0x0 0x10000>; > + big-endian; > interrupts = ; > }; > -- Boris Brezillon, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com From mboxrd@z Thu Jan 1 00:00:00 1970 Date: Thu, 22 Feb 2018 11:31:47 +0100 From: Boris Brezillon To: Prabhakar Kushwaha Cc: linux-mtd@lists.infradead.org, devicetree@vger.kernel.org, robh@kernel.org, mark.rutland@arm.com, shawnguo@kernel.org, linux-arm-kernel@lists.infradead.org, boris.brezillon@free-electrons.com, cyrille.pitchen@wedev4u.fr, computersforpeace@gmail.com, oss@buserror.net, leoyang.li@nxp.com, Alison Wang , Jagdish Gediya Subject: Re: [PATCH 1/2][v2][RESEND] ARM: dts: Add big-endian for IFC on LS1021A Message-ID: <20180222113147.2cce0f92@bbrezillon> In-Reply-To: <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> References: <20180222101511.29631-1-prabhakar.kushwaha@nxp.com> <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, 22 Feb 2018 15:45:10 +0530 Prabhakar Kushwaha wrote: > For the patch to update struct map_info's swap field based on device > characteristics defined in device tree, big-endian parameter is added > for LS1021A. > > Signed-off-by: Alison Wang > Signed-off-by: Jagdish Gediya > Signed-off-by: Prabhakar Kushwaha > --- > Due to patch series http://patchwork.ozlabs.org/project/linux-mtd/list/?series=28735 > this patch is now independent of flash propoerty. So resending. > > Resending as all patches of patchset never reaches to patchwork. Again, that's not the case, I just marked them as "Not applicable" (see my previous reply). Also, according to the cover letter this patch should not be part of "dt-bindings: mtd-physmap: Add endianness supports", so why are you sending it here? > > arch/arm/boot/dts/ls1021a.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi > index 9319e1f0f1d8..babb086ad94f 100644 > --- a/arch/arm/boot/dts/ls1021a.dtsi > +++ b/arch/arm/boot/dts/ls1021a.dtsi > @@ -145,6 +145,7 @@ > ifc: ifc@1530000 { > compatible = "fsl,ifc", "simple-bus"; > reg = <0x0 0x1530000 0x0 0x10000>; > + big-endian; > interrupts = ; > }; > -- Boris Brezillon, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com From mboxrd@z Thu Jan 1 00:00:00 1970 From: boris.brezillon@bootlin.com (Boris Brezillon) Date: Thu, 22 Feb 2018 11:31:47 +0100 Subject: [PATCH 1/2][v2][RESEND] ARM: dts: Add big-endian for IFC on LS1021A In-Reply-To: <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> References: <20180222101511.29631-1-prabhakar.kushwaha@nxp.com> <20180222101511.29631-6-prabhakar.kushwaha@nxp.com> Message-ID: <20180222113147.2cce0f92@bbrezillon> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Thu, 22 Feb 2018 15:45:10 +0530 Prabhakar Kushwaha wrote: > For the patch to update struct map_info's swap field based on device > characteristics defined in device tree, big-endian parameter is added > for LS1021A. > > Signed-off-by: Alison Wang > Signed-off-by: Jagdish Gediya > Signed-off-by: Prabhakar Kushwaha > --- > Due to patch series http://patchwork.ozlabs.org/project/linux-mtd/list/?series=28735 > this patch is now independent of flash propoerty. So resending. > > Resending as all patches of patchset never reaches to patchwork. Again, that's not the case, I just marked them as "Not applicable" (see my previous reply). Also, according to the cover letter this patch should not be part of "dt-bindings: mtd-physmap: Add endianness supports", so why are you sending it here? > > arch/arm/boot/dts/ls1021a.dtsi | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/arch/arm/boot/dts/ls1021a.dtsi b/arch/arm/boot/dts/ls1021a.dtsi > index 9319e1f0f1d8..babb086ad94f 100644 > --- a/arch/arm/boot/dts/ls1021a.dtsi > +++ b/arch/arm/boot/dts/ls1021a.dtsi > @@ -145,6 +145,7 @@ > ifc: ifc at 1530000 { > compatible = "fsl,ifc", "simple-bus"; > reg = <0x0 0x1530000 0x0 0x10000>; > + big-endian; > interrupts = ; > }; > -- Boris Brezillon, Bootlin (formerly Free Electrons) Embedded Linux and Kernel engineering https://bootlin.com