linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used
       [not found] <20190107092823.11188-1-frieder.schrempf@kontron.de>
@ 2019-01-07  9:29 ` Schrempf Frieder
  2019-01-13  3:31   ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
  2019-01-07  9:29 ` [PATCH v8 6/7] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip " Schrempf Frieder
  1 sibling, 1 reply; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-07  9:29 UTC (permalink / raw)
  To: linux-mtd, boris.brezillon, linux-spi, broonie, Shawn Guo,
	Li Yang, Rob Herring, Mark Rutland
  Cc: devicetree, yogeshnarayan.gaur, richard, linux-kernel,
	Schrempf Frieder, marek.vasut, han.xu, prabhakar.kushwaha,
	miquel.raynal, fabio.estevam, david.wolfe, computersforpeace,
	dwmw2, linux-arm-kernel

From: Frieder Schrempf <frieder.schrempf@kontron.de>

After switching to the new FSL QSPI driver the property
'fsl,qspi-has-second-chip' is not needed anymore.

The driver now uses the 'reg' property to determine the bus and
the chipselect.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts b/arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts
index 6a83f30029ea..d3a1a73ae97e 100644
--- a/arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts
+++ b/arch/arm/boot/dts/ls1021a-moxa-uc-8410a.dts
@@ -203,7 +203,6 @@
 };
 
 &qspi {
-	fsl,qspi-has-second-chip;
 	status = "okay";
 
 	flash: flash@0 {
-- 
2.17.1

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* [PATCH v8 6/7] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip as it is not used
       [not found] <20190107092823.11188-1-frieder.schrempf@kontron.de>
  2019-01-07  9:29 ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used Schrempf Frieder
@ 2019-01-07  9:29 ` Schrempf Frieder
  2019-01-13  3:32   ` Shawn Guo
  1 sibling, 1 reply; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-07  9:29 UTC (permalink / raw)
  To: linux-mtd, boris.brezillon, linux-spi, broonie, Shawn Guo, Li Yang
  Cc: Mark Rutland, devicetree, yogeshnarayan.gaur, Rob Herring,
	richard, linux-kernel, Schrempf Frieder, marek.vasut, han.xu,
	prabhakar.kushwaha, miquel.raynal, fabio.estevam, david.wolfe,
	computersforpeace, dwmw2, linux-arm-kernel

From: Frieder Schrempf <frieder.schrempf@kontron.de>

After switching to the new FSL QSPI driver the property
'fsl,qspi-has-second-chip' is not needed anymore.

The driver now uses the 'reg' property to determine the bus and
the chipselect.

Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
---
 arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
index 9a2106e60e19..b2996db5903c 100644
--- a/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
+++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi
@@ -215,7 +215,6 @@
 			clock-names = "qspi_en", "qspi";
 			clocks = <&clockgen 4 1>, <&clockgen 4 1>;
 			big-endian;
-			fsl,qspi-has-second-chip;
 			status = "disabled";
 		};
 
-- 
2.17.1

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply related	[flat|nested] 7+ messages in thread

* Re: [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip as it is not used
  2019-01-07  9:29 ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used Schrempf Frieder
@ 2019-01-13  3:31   ` Shawn Guo
  2019-01-14  8:18     ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Schrempf Frieder
  0 siblings, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2019-01-13  3:31 UTC (permalink / raw)
  To: Schrempf Frieder
  Cc: Mark Rutland, devicetree, computersforpeace, marek.vasut,
	richard, prabhakar.kushwaha, han.xu, linux-kernel, Rob Herring,
	Li Yang, boris.brezillon, yogeshnarayan.gaur, broonie, linux-mtd,
	miquel.raynal, fabio.estevam, david.wolfe, linux-spi, dwmw2,
	linux-arm-kernel

On Mon, Jan 07, 2019 at 09:29:54AM +0000, Schrempf Frieder wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> After switching to the new FSL QSPI driver the property
> 'fsl,qspi-has-second-chip' is not needed anymore.
> 
> The driver now uses the 'reg' property to determine the bus and
> the chipselect.
> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>

Applied, thanks.

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v8 6/7] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip as it is not used
  2019-01-07  9:29 ` [PATCH v8 6/7] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip " Schrempf Frieder
@ 2019-01-13  3:32   ` Shawn Guo
  0 siblings, 0 replies; 7+ messages in thread
From: Shawn Guo @ 2019-01-13  3:32 UTC (permalink / raw)
  To: Schrempf Frieder
  Cc: Mark Rutland, devicetree, computersforpeace, marek.vasut,
	richard, prabhakar.kushwaha, han.xu, linux-kernel, Rob Herring,
	Li Yang, boris.brezillon, yogeshnarayan.gaur, broonie, linux-mtd,
	miquel.raynal, fabio.estevam, david.wolfe, linux-spi, dwmw2,
	linux-arm-kernel

On Mon, Jan 07, 2019 at 09:29:55AM +0000, Schrempf Frieder wrote:
> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> After switching to the new FSL QSPI driver the property
> 'fsl,qspi-has-second-chip' is not needed anymore.
> 
> The driver now uses the 'reg' property to determine the bus and
> the chipselect.
> 
> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>

Different from arm dts files, we use prefix 'arm64: dts: ...'.
I fixed it up and applied the patch.

Shawn

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used
  2019-01-13  3:31   ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
@ 2019-01-14  8:18     ` Schrempf Frieder
  2019-01-15 15:18       ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
  0 siblings, 1 reply; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-14  8:18 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Mark Rutland, devicetree, computersforpeace, marek.vasut,
	richard, prabhakar.kushwaha, han.xu, linux-kernel, Rob Herring,
	Li Yang, boris.brezillon, yogeshnarayan.gaur, broonie, linux-mtd,
	miquel.raynal, fabio.estevam, david.wolfe, linux-spi, dwmw2,
	linux-arm-kernel

Hi Shawn,

On 13.01.19 04:31, Shawn Guo wrote:
> On Mon, Jan 07, 2019 at 09:29:54AM +0000, Schrempf Frieder wrote:
>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>
>> After switching to the new FSL QSPI driver the property
>> 'fsl,qspi-has-second-chip' is not needed anymore.
>>
>> The driver now uses the 'reg' property to determine the bus and
>> the chipselect.
>>
>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> 
> Applied, thanks.

I just noticed, that this and the arm64 patch depend on the driver 
changes for 5.1 that go through the SPI tree. So I guess we can't be 
sure that this will be merged in the right order and we need to delay 
this cleanup to 5.2, or am I wrong?

Thanks,
Frieder
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip as it is not used
  2019-01-14  8:18     ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Schrempf Frieder
@ 2019-01-15 15:18       ` Shawn Guo
  2019-01-15 15:24         ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Schrempf Frieder
  0 siblings, 1 reply; 7+ messages in thread
From: Shawn Guo @ 2019-01-15 15:18 UTC (permalink / raw)
  To: Schrempf Frieder
  Cc: Mark Rutland, devicetree, computersforpeace, marek.vasut,
	richard, prabhakar.kushwaha, han.xu, linux-kernel, Rob Herring,
	Li Yang, boris.brezillon, yogeshnarayan.gaur, broonie, linux-mtd,
	miquel.raynal, fabio.estevam, david.wolfe, linux-spi, dwmw2,
	linux-arm-kernel

On Mon, Jan 14, 2019 at 08:18:49AM +0000, Schrempf Frieder wrote:
> Hi Shawn,
> 
> On 13.01.19 04:31, Shawn Guo wrote:
> > On Mon, Jan 07, 2019 at 09:29:54AM +0000, Schrempf Frieder wrote:
> >> From: Frieder Schrempf <frieder.schrempf@kontron.de>
> >>
> >> After switching to the new FSL QSPI driver the property
> >> 'fsl,qspi-has-second-chip' is not needed anymore.
> >>
> >> The driver now uses the 'reg' property to determine the bus and
> >> the chipselect.
> >>
> >> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
> > 
> > Applied, thanks.
> 
> I just noticed, that this and the arm64 patch depend on the driver 
> changes for 5.1 that go through the SPI tree. So I guess we can't be 
> sure that this will be merged in the right order and we need to delay 
> this cleanup to 5.2, or am I wrong?

You are right.  I just dropped the both patches.  Ping me or resend when
they are ready to go.

Shawn

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

* Re: [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used
  2019-01-15 15:18       ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
@ 2019-01-15 15:24         ` Schrempf Frieder
  0 siblings, 0 replies; 7+ messages in thread
From: Schrempf Frieder @ 2019-01-15 15:24 UTC (permalink / raw)
  To: Shawn Guo
  Cc: Mark Rutland, devicetree, computersforpeace, marek.vasut,
	richard, prabhakar.kushwaha, han.xu, linux-kernel, Rob Herring,
	Li Yang, boris.brezillon, yogeshnarayan.gaur, broonie, linux-mtd,
	miquel.raynal, fabio.estevam, david.wolfe, linux-spi, dwmw2,
	linux-arm-kernel

On 15.01.19 16:18, Shawn Guo wrote:
> On Mon, Jan 14, 2019 at 08:18:49AM +0000, Schrempf Frieder wrote:
>> Hi Shawn,
>>
>> On 13.01.19 04:31, Shawn Guo wrote:
>>> On Mon, Jan 07, 2019 at 09:29:54AM +0000, Schrempf Frieder wrote:
>>>> From: Frieder Schrempf <frieder.schrempf@kontron.de>
>>>>
>>>> After switching to the new FSL QSPI driver the property
>>>> 'fsl,qspi-has-second-chip' is not needed anymore.
>>>>
>>>> The driver now uses the 'reg' property to determine the bus and
>>>> the chipselect.
>>>>
>>>> Signed-off-by: Frieder Schrempf <frieder.schrempf@kontron.de>
>>>
>>> Applied, thanks.
>>
>> I just noticed, that this and the arm64 patch depend on the driver
>> changes for 5.1 that go through the SPI tree. So I guess we can't be
>> sure that this will be merged in the right order and we need to delay
>> this cleanup to 5.2, or am I wrong?
> 
> You are right.  I just dropped the both patches.  Ping me or resend when
> they are ready to go.

Thanks, will do so.

Frieder
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-01-15 15:24 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <20190107092823.11188-1-frieder.schrempf@kontron.de>
2019-01-07  9:29 ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip as it is not used Schrempf Frieder
2019-01-13  3:31   ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
2019-01-14  8:18     ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Schrempf Frieder
2019-01-15 15:18       ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl,qspi-has-second-chip " Shawn Guo
2019-01-15 15:24         ` [PATCH v8 5/7] ARM: dts: ls1021a: Remove fsl, qspi-has-second-chip " Schrempf Frieder
2019-01-07  9:29 ` [PATCH v8 6/7] ARM64: dts: ls1046a: Remove fsl,qspi-has-second-chip " Schrempf Frieder
2019-01-13  3:32   ` Shawn Guo

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).