diff mbox series

[v4,4/7] imx: imx6sx: Remove unused 'num-cs' property

Message ID 1578900390-12119-5-git-send-email-kuldeep.singh@nxp.com
State Not Applicable
Delegated to: Priyanka Jain
Headers show
Series Transition of fsl qspi driver to spi-mem framework | expand

Commit Message

Kuldeep Singh Jan. 13, 2020, 7:26 a.m. UTC
From: Frieder Schrempf <frieder.schrempf@kontron.de>

This property is not used by the driver anymore so let's remove it.
Other dts still have 'num-cs' set, but they need a resync with the
Linux kernel anyway, so let's only do the U-Boot-specific files for
now.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com>
---
Depends on https://patchwork.ozlabs.org/patch/1219462/
Depends on https://patchwork.ozlabs.org/patch/1208299/

v4: No change
v3: No change
v2: No change

 arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi | 2 --
 arch/arm/dts/imx6sx-sdb-u-boot.dtsi       | 2 --
 2 files changed, 4 deletions(-)

Comments

Priyanka Jain (OSS) Feb. 19, 2020, 10:13 a.m. UTC | #1
>-----Original Message-----
>From: U-Boot <u-boot-bounces@lists.denx.de> On Behalf Of Kuldeep Singh
>Sent: Monday, January 13, 2020 12:56 PM
>To: u-boot@lists.denx.de
>Cc: sr@denx.de
>Subject: [Patch v4 4/7] imx: imx6sx: Remove unused 'num-cs' property
>
>From: Frieder Schrempf <frieder.schrempf@kontron.de>
>
>This property is not used by the driver anymore so let's remove it.
>Other dts still have 'num-cs' set, but they need a resync with the Linux kernel
>anyway, so let's only do the U-Boot-specific files for now.
>
>Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>Signed-off-by: Kuldeep Singh <kuldeep.singh@nxp.com>
>---
>Depends on https://patchwork.ozlabs.org/patch/1219462/
>Depends on https://patchwork.ozlabs.org/patch/1208299/
>
>v4: No change
>v3: No change
>v2: No change
>
> arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi | 2 --
> arch/arm/dts/imx6sx-sdb-u-boot.dtsi       | 2 --
> 2 files changed, 4 deletions(-)
>
>diff --git a/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
>b/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
>index 549461d..5200448 100644
>--- a/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
>+++ b/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
>@@ -4,8 +4,6 @@
>  */
>
> &qspi1 {
>-	num-cs = <2>;
>-
> 	flash0: n25q256a@0 {
> 		compatible = "jedec,spi-nor";
> 	};
>diff --git a/arch/arm/dts/imx6sx-sdb-u-boot.dtsi b/arch/arm/dts/imx6sx-sdb-
>u-boot.dtsi
>index 8f9236d..3c0fd87 100644
>--- a/arch/arm/dts/imx6sx-sdb-u-boot.dtsi
>+++ b/arch/arm/dts/imx6sx-sdb-u-boot.dtsi
>@@ -4,8 +4,6 @@
>  */
>
> &qspi2 {
>-	num-cs = <2>;
>-
> 	flash0: n25q256a@0 {
> 		compatible = "jedec,spi-nor";
> 	};
>--
>2.7.4
Stefano/Fabio,

Please review/ack this patch.

Thanks
Priyanka
Fabio Estevam Feb. 19, 2020, 1 p.m. UTC | #2
Hi Priyanka,

On Wed, Feb 19, 2020 at 7:13 AM Priyanka Jain (OSS)
<priyanka.jain@oss.nxp.com> wrote:

> Stefano/Fabio,
>
> Please review/ack this patch.

Last month I did a patch series to sync the imx6sx device tree files
with upstream kernel, but thanks to your email I have just realized
that I missed to send it to the list.

Just sent it to the list:
https://lists.denx.de/pipermail/u-boot/2020-February/400727.html

After my series gets applied this patch will no longer be needed.

Regards,

Fabio Estevam
Kuldeep Singh Feb. 20, 2020, 9:12 a.m. UTC | #3
Hi Fabio,

> -----Original Message-----
> From: Fabio Estevam <festevam@gmail.com>
> Sent: Wednesday, February 19, 2020 6:31 PM
> To: Priyanka Jain (OSS) <priyanka.jain@oss.nxp.com>
> Cc: Kuldeep Singh <kuldeep.singh@nxp.com>; u-boot@lists.denx.de;
> Stefano Babic <sbabic@denx.de>; sr@denx.de
> Subject: [EXT] Re: [Patch v4 4/7] imx: imx6sx: Remove unused 'num-cs'
> property
> 
> Caution: EXT Email
> 
> Hi Priyanka,
> 
> On Wed, Feb 19, 2020 at 7:13 AM Priyanka Jain (OSS)
> <priyanka.jain@oss.nxp.com> wrote:
> 
> > Stefano/Fabio,
> >
> > Please review/ack this patch.
> 
> Last month I did a patch series to sync the imx6sx device tree files with
> upstream kernel, but thanks to your email I have just realized that I missed
> to send it to the list.
> 
> Just sent it to the list:
> https://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Flists.
> denx.de%2Fpipermail%2Fu-boot%2F2020-
> February%2F400727.html&amp;data=02%7C01%7Ckuldeep.singh%40nxp.c
> om%7C051c54e8e136406ca45d08d7b53bc31f%7C686ea1d3bc2b4c6fa92cd
> 99c5c301635%7C0%7C0%7C637177140593582319&amp;sdata=OhuEC000t
> b6rbd5LXYhM0jxfbaVbVC8%2BepA5cNx%2BdxI%3D&amp;reserved=0
> 
> After my series gets applied this patch will no longer be needed.

Ok. I will drop this patch from series.

Thanks
Kuldeep

> 
> Regards,
> 
> Fabio Estevam
diff mbox series

Patch

diff --git a/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi b/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
index 549461d..5200448 100644
--- a/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
+++ b/arch/arm/dts/imx6sx-sabreauto-u-boot.dtsi
@@ -4,8 +4,6 @@ 
  */
 
 &qspi1 {
-	num-cs = <2>;
-
 	flash0: n25q256a@0 {
 		compatible = "jedec,spi-nor";
 	};
diff --git a/arch/arm/dts/imx6sx-sdb-u-boot.dtsi b/arch/arm/dts/imx6sx-sdb-u-boot.dtsi
index 8f9236d..3c0fd87 100644
--- a/arch/arm/dts/imx6sx-sdb-u-boot.dtsi
+++ b/arch/arm/dts/imx6sx-sdb-u-boot.dtsi
@@ -4,8 +4,6 @@ 
  */
 
 &qspi2 {
-	num-cs = <2>;
-
 	flash0: n25q256a@0 {
 		compatible = "jedec,spi-nor";
 	};