From: Shawn Guo <shawnguo@kernel.org>
To: Fabio Estevam <festevam@gmail.com>
Cc: robh+dt@kernel.org, devicetree@vger.kernel.org,
Fabio Estevam <fabio.estevam@nxp.com>
Subject: Re: [PATCH 1/4] Revert "arm64: dts: fsl: fix ifc simple-bus unit address format warnings"
Date: Mon, 26 Mar 2018 22:44:46 +0800 [thread overview]
Message-ID: <20180326144445.GE8857@dragon> (raw)
In-Reply-To: <1521382651-10256-1-git-send-email-festevam@gmail.com>
On Sun, Mar 18, 2018 at 11:17:28AM -0300, Fabio Estevam wrote:
> From: Fabio Estevam <fabio.estevam@nxp.com>
>
> This reverts commit f81d7af7957539b7808961f929f945381530acb9.
>
> As explained by Rob Herring:
>
> "This "fix" is wrong. Memory controllers with chip selects should have
> the chip select in the unit-address. The correct fix here is you should
> drop "simple-bus"."
Right, Rob talked to me about this last week during Linaro Connect.
>
> Signed-off-by: Fabio Estevam <fabio.estevam@nxp.com>
The series should be copied to LAKML as well. Also, I suspect that we
will have regressions if we drop "simple-bus" from IFC node right now,
because the child devices will likely not get populated. I would
suggest that you resend two reverting patches firstly, and send the
other two separately later with someone being able to test the changes
on copy, to make sure we do not break things.
Shawn
> ---
> arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts | 6 +++---
> arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts | 6 +++---
> arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts | 6 +++---
> arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts | 4 ++--
> arch/arm64/boot/dts/freescale/fsl-ls1088a-qds.dts | 6 +++---
> arch/arm64/boot/dts/freescale/fsl-ls1088a-rdb.dts | 4 ++--
> arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi | 6 +++---
> 7 files changed, 19 insertions(+), 19 deletions(-)
>
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> index 3a09297..6341281 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-qds.dts
> @@ -84,19 +84,19 @@
> 0x2 0x0 0x0 0x7fb00000 0x00000100>;
> status = "okay";
>
> - nor@0 {
> + nor@0,0 {
> compatible = "cfi-flash";
> reg = <0x0 0x0 0x8000000>;
> bank-width = <2>;
> device-width = <1>;
> };
>
> - nand@100000000 {
> + nand@1,0 {
> compatible = "fsl,ifc-nand";
> reg = <0x1 0x0 0x10000>;
> };
>
> - fpga: board-control@200000000 {
> + fpga: board-control@2,0 {
> compatible = "fsl,ls1043aqds-fpga", "fsl,fpga-qixis";
> reg = <0x2 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> index 0a82bf9..3dc0c8e 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1043a-rdb.dts
> @@ -97,7 +97,7 @@
> 0x1 0x0 0x0 0x7e800000 0x00010000
> 0x2 0x0 0x0 0x7fb00000 0x00000100>;
>
> - nor@0 {
> + nor@0,0 {
> compatible = "cfi-flash";
> #address-cells = <1>;
> #size-cells = <1>;
> @@ -106,14 +106,14 @@
> device-width = <1>;
> };
>
> - nand@100000000 {
> + nand@1,0 {
> compatible = "fsl,ifc-nand";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x1 0x0 0x10000>;
> };
>
> - cpld: board-control@200000000 {
> + cpld: board-control@2,0 {
> compatible = "fsl,ls1043ardb-cpld";
> reg = <0x2 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> index 410dc61..434383b 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-qds.dts
> @@ -175,19 +175,19 @@
> 0x2 0x0 0x0 0x7fb00000 0x00000100>;
> status = "okay";
>
> - nor@0 {
> + nor@0,0 {
> compatible = "cfi-flash";
> reg = <0x0 0x0 0x8000000>;
> bank-width = <2>;
> device-width = <1>;
> };
>
> - nand@100000000 {
> + nand@1,0 {
> compatible = "fsl,ifc-nand";
> reg = <0x1 0x0 0x10000>;
> };
>
> - fpga: board-control@200000000 {
> + fpga: board-control@2,0 {
> compatible = "fsl,ls1046aqds-fpga", "fsl,fpga-qixis";
> reg = <0x2 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> index 18d3c5b..5dc2782 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1046a-rdb.dts
> @@ -122,14 +122,14 @@
> 0x2 0x0 0x0 0x7fb00000 0x00000100>;
> status = "okay";
>
> - nand@0 {
> + nand@0,0 {
> compatible = "fsl,ifc-nand";
> #address-cells = <1>;
> #size-cells = <1>;
> reg = <0x0 0x0 0x10000>;
> };
>
> - cpld: board-control@200000000 {
> + cpld: board-control@2,0 {
> compatible = "fsl,ls1046ardb-cpld";
> reg = <0x2 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-qds.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-qds.dts
> index 0b96d6d..3012805 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-qds.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-qds.dts
> @@ -116,19 +116,19 @@
> 3 0 0x5 0x20000000 0x00010000>;
> status = "okay";
>
> - nor@0 {
> + nor@0,0 {
> compatible = "cfi-flash";
> reg = <0x0 0x0 0x8000000>;
> bank-width = <2>;
> device-width = <1>;
> };
>
> - nand@200000000 {
> + nand@2,0 {
> compatible = "fsl,ifc-nand";
> reg = <0x2 0x0 0x10000>;
> };
>
> - fpga: board-control@300000000 {
> + fpga: board-control@3,0 {
> compatible = "fsl,ls1088aqds-fpga", "fsl,fpga-qixis";
> reg = <0x3 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls1088a-rdb.dts b/arch/arm64/boot/dts/freescale/fsl-ls1088a-rdb.dts
> index 3d57008e..4f17601 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls1088a-rdb.dts
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls1088a-rdb.dts
> @@ -99,12 +99,12 @@
> 2 0 0x5 0x20000000 0x00010000>;
> status = "okay";
>
> - nand@0 {
> + nand@0,0 {
> compatible = "fsl,ifc-nand";
> reg = <0x0 0x0 0x10000>;
> };
>
> - fpga: board-control@200000000 {
> + fpga: board-control@2,0 {
> compatible = "fsl,ls1088ardb-fpga", "fsl,fpga-qixis";
> reg = <0x2 0x0 0x0000100>;
> };
> diff --git a/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi b/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
> index 5ac3412..1de6188 100644
> --- a/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
> +++ b/arch/arm64/boot/dts/freescale/fsl-ls208xa-qds.dtsi
> @@ -58,7 +58,7 @@
> 0x2 0x0 0x5 0x30000000 0x00010000
> 0x3 0x0 0x5 0x20000000 0x00010000>;
>
> - nor@0 {
> + nor@0,0 {
> #address-cells = <1>;
> #size-cells = <1>;
> compatible = "cfi-flash";
> @@ -67,12 +67,12 @@
> device-width = <1>;
> };
>
> - nand@200000000 {
> + nand@2,0 {
> compatible = "fsl,ifc-nand";
> reg = <0x2 0x0 0x10000>;
> };
>
> - cpld@300000000 {
> + cpld@3,0 {
> reg = <0x3 0x0 0x10000>;
> compatible = "fsl,ls2080aqds-fpga", "fsl,fpga-qixis";
> };
> --
> 2.7.4
>
prev parent reply other threads:[~2018-03-26 14:44 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-03-18 14:17 [PATCH 1/4] Revert "arm64: dts: fsl: fix ifc simple-bus unit address format warnings" Fabio Estevam
2018-03-18 14:17 ` [PATCH 2/4] arm64: dts: ls: Remove "simple-bus" from ifc compatible Fabio Estevam
2018-03-26 22:23 ` Rob Herring
2018-03-18 14:17 ` [PATCH 3/4] Revert "dt-bindings: ifc: Fix the unit address format in the examples" Fabio Estevam
2018-03-26 22:23 ` Rob Herring
2018-03-18 14:17 ` [PATCH 4/4] dt-bindings: ifc: Remove "simple-bus" from ifc compatible string Fabio Estevam
2018-03-26 22:23 ` Rob Herring
2018-03-26 14:44 ` Shawn Guo [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180326144445.GE8857@dragon \
--to=shawnguo@kernel.org \
--cc=devicetree@vger.kernel.org \
--cc=fabio.estevam@nxp.com \
--cc=festevam@gmail.com \
--cc=robh+dt@kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).