linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] arm64: zynqmp: Remove not documented is-dual property
@ 2021-08-06  8:42 Michal Simek
  2021-08-09  1:52 ` Laurent Pinchart
  2021-08-25  6:21 ` Michal Simek
  0 siblings, 2 replies; 3+ messages in thread
From: Michal Simek @ 2021-08-06  8:42 UTC (permalink / raw)
  To: linux-kernel, monstr, michal.simek, git
  Cc: Quanyang Wang, Krzysztof Kozlowski, Laurent Pinchart,
	Rob Herring, devicetree, linux-arm-kernel

Remove is-dual not documented property and also update comment about QSPI
sizes to reflect dual configuration as 16MB + 16MB.
Only single configuration is supported now.

Reported-by: Quanyang Wang <quanyang.wang@windriver.com>
Signed-off-by: Michal Simek <michal.simek@xilinx.com>
---

 arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts | 3 +--
 arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 3 +--
 arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts | 3 +--
 3 files changed, 3 insertions(+), 6 deletions(-)

diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
index becfc23a5610..3d8d14ef1ede 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
@@ -937,9 +937,8 @@ &psgtr {
 
 &qspi {
 	status = "okay";
-	is-dual = <1>;
 	flash@0 {
-		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
+		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
 		#address-cells = <1>;
 		#size-cells = <1>;
 		reg = <0x0>;
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
index d2219373580a..057c04352591 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
@@ -931,9 +931,8 @@ &psgtr {
 
 &qspi {
 	status = "okay";
-	is-dual = <1>;
 	flash@0 {
-		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
+		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
 		#address-cells = <1>;
 		#size-cells = <1>;
 		reg = <0x0>;
diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
index dac5ba67a160..e1fff62a4cd5 100644
--- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
+++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
@@ -773,9 +773,8 @@ &psgtr {
 
 &qspi {
 	status = "okay";
-	is-dual = <1>;
 	flash@0 {
-		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
+		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
 		#address-cells = <1>;
 		#size-cells = <1>;
 		reg = <0x0>;
-- 
2.32.0


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

* Re: [PATCH] arm64: zynqmp: Remove not documented is-dual property
  2021-08-06  8:42 [PATCH] arm64: zynqmp: Remove not documented is-dual property Michal Simek
@ 2021-08-09  1:52 ` Laurent Pinchart
  2021-08-25  6:21 ` Michal Simek
  1 sibling, 0 replies; 3+ messages in thread
From: Laurent Pinchart @ 2021-08-09  1:52 UTC (permalink / raw)
  To: Michal Simek
  Cc: linux-kernel, monstr, git, Quanyang Wang, Krzysztof Kozlowski,
	Rob Herring, devicetree, linux-arm-kernel

Hi Michal,

Thank you for the patch.

On Fri, Aug 06, 2021 at 10:42:29AM +0200, Michal Simek wrote:
> Remove is-dual not documented property and also update comment about QSPI
> sizes to reflect dual configuration as 16MB + 16MB.
> Only single configuration is supported now.
> 
> Reported-by: Quanyang Wang <quanyang.wang@windriver.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>

Reviewed-by: Laurent Pinchart <laurent.pinchart@ideasonboard.com>

> ---
> 
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts | 3 +--
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 3 +--
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts | 3 +--
>  3 files changed, 3 insertions(+), 6 deletions(-)
> 
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> index becfc23a5610..3d8d14ef1ede 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> @@ -937,9 +937,8 @@ &psgtr {
>  
>  &qspi {
>  	status = "okay";
> -	is-dual = <1>;
>  	flash@0 {
> -		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>  		#address-cells = <1>;
>  		#size-cells = <1>;
>  		reg = <0x0>;
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> index d2219373580a..057c04352591 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> @@ -931,9 +931,8 @@ &psgtr {
>  
>  &qspi {
>  	status = "okay";
> -	is-dual = <1>;
>  	flash@0 {
> -		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>  		#address-cells = <1>;
>  		#size-cells = <1>;
>  		reg = <0x0>;
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> index dac5ba67a160..e1fff62a4cd5 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> @@ -773,9 +773,8 @@ &psgtr {
>  
>  &qspi {
>  	status = "okay";
> -	is-dual = <1>;
>  	flash@0 {
> -		compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +		compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>  		#address-cells = <1>;
>  		#size-cells = <1>;
>  		reg = <0x0>;

-- 
Regards,

Laurent Pinchart

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

* Re: [PATCH] arm64: zynqmp: Remove not documented is-dual property
  2021-08-06  8:42 [PATCH] arm64: zynqmp: Remove not documented is-dual property Michal Simek
  2021-08-09  1:52 ` Laurent Pinchart
@ 2021-08-25  6:21 ` Michal Simek
  1 sibling, 0 replies; 3+ messages in thread
From: Michal Simek @ 2021-08-25  6:21 UTC (permalink / raw)
  To: LKML, Michal Simek, git
  Cc: Quanyang Wang, Krzysztof Kozlowski, Laurent Pinchart,
	Rob Herring, DTML, linux-arm

pá 6. 8. 2021 v 10:42 odesílatel Michal Simek <michal.simek@xilinx.com> napsal:
>
> Remove is-dual not documented property and also update comment about QSPI
> sizes to reflect dual configuration as 16MB + 16MB.
> Only single configuration is supported now.
>
> Reported-by: Quanyang Wang <quanyang.wang@windriver.com>
> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
> ---
>
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts | 3 +--
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts | 3 +--
>  arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts | 3 +--
>  3 files changed, 3 insertions(+), 6 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> index becfc23a5610..3d8d14ef1ede 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu102-revA.dts
> @@ -937,9 +937,8 @@ &psgtr {
>
>  &qspi {
>         status = "okay";
> -       is-dual = <1>;
>         flash@0 {
> -               compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +               compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>                 #address-cells = <1>;
>                 #size-cells = <1>;
>                 reg = <0x0>;
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> index d2219373580a..057c04352591 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu106-revA.dts
> @@ -931,9 +931,8 @@ &psgtr {
>
>  &qspi {
>         status = "okay";
> -       is-dual = <1>;
>         flash@0 {
> -               compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +               compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>                 #address-cells = <1>;
>                 #size-cells = <1>;
>                 reg = <0x0>;
> diff --git a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> index dac5ba67a160..e1fff62a4cd5 100644
> --- a/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> +++ b/arch/arm64/boot/dts/xilinx/zynqmp-zcu111-revA.dts
> @@ -773,9 +773,8 @@ &psgtr {
>
>  &qspi {
>         status = "okay";
> -       is-dual = <1>;
>         flash@0 {
> -               compatible = "m25p80", "jedec,spi-nor"; /* 32MB */
> +               compatible = "m25p80", "jedec,spi-nor"; /* 16MB + 16MB */
>                 #address-cells = <1>;
>                 #size-cells = <1>;
>                 reg = <0x0>;
> --
> 2.32.0
>

Applied.
M

-- 
Michal Simek, Ing. (M.Eng), OpenPGP -> KeyID: FE3D1F91
w: www.monstr.eu p: +42-0-721842854
Maintainer of Linux kernel - Xilinx Microblaze
Maintainer of Linux kernel - Xilinx Zynq ARM and ZynqMP ARM64 SoCs
U-Boot custodian - Xilinx Microblaze/Zynq/ZynqMP/Versal SoCs

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

end of thread, other threads:[~2021-08-25  6:21 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-06  8:42 [PATCH] arm64: zynqmp: Remove not documented is-dual property Michal Simek
2021-08-09  1:52 ` Laurent Pinchart
2021-08-25  6:21 ` Michal Simek

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).