diff mbox series

[v20,08/19] dt-binding: memory: pl353-smc: Fix the reg property in the example

Message ID 20210519182636.1110080-9-miquel.raynal@bootlin.com
State New
Headers show
Series [v20,01/19] dt-binding: memory: pl353-smc: Rephrase the binding | expand

Commit Message

Miquel Raynal May 19, 2021, 6:26 p.m. UTC
The reg property should describe the entire accessible zone, not only
the one the driver wants to access. With this in mind, the example
should map two regions instead of one to be entirely accurate.

Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
 .../devicetree/bindings/memory-controllers/pl353-smc.txt       | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Rob Herring May 21, 2021, 1:55 a.m. UTC | #1
On Wed, May 19, 2021 at 08:26:25PM +0200, Miquel Raynal wrote:
> The reg property should describe the entire accessible zone, not only

> the one the driver wants to access. With this in mind, the example

> should map two regions instead of one to be entirely accurate.


No, not really. The address space is already covered by 'ranges'. Unless 
you need the range in the arm,pl353-smc-r2p1 driver.

> 

> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

> ---

>  .../devicetree/bindings/memory-controllers/pl353-smc.txt       | 3 ++-

>  1 file changed, 2 insertions(+), 1 deletion(-)

> 

> diff --git a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> index 1335b94291ad..9d220d4cb39d 100644

> --- a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> +++ b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> @@ -27,7 +27,8 @@ Example:

>  		compatible = "arm,pl353-smc-r2p1", "arm,primecell";

>  		clock-names = "memclk", "apb_pclk";

>  		clocks = <&clkc 11>, <&clkc 44>;

> -		reg = <0xe000e000 0x1000>;

> +		reg = <0xe000e000 0x0001000>,

> +		      <0xe1000000 0x5000000>;

>  		#address-cells = <2>;

>  		#size-cells = <1>;

>  		ranges = <0x0 0x0 0xe1000000 0x1000000 /* Nand CS region */

> -- 

> 2.27.0

>
Miquel Raynal May 27, 2021, 10:26 a.m. UTC | #2
Hi Rob,

Rob Herring <robh@kernel.org> wrote on Thu, 20 May 2021 20:55:35 -0500:

> On Wed, May 19, 2021 at 08:26:25PM +0200, Miquel Raynal wrote:

> > The reg property should describe the entire accessible zone, not only

> > the one the driver wants to access. With this in mind, the example

> > should map two regions instead of one to be entirely accurate.  

> 

> No, not really. The address space is already covered by 'ranges'. Unless 

> you need the range in the arm,pl353-smc-r2p1 driver.


Thanks for the clarification, I didn't understand the range property
correctly then.

This extra region should not be accessed from the SMC driver so I'll
drop the extra changes.

> 

> > 

> > Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

> > ---

> >  .../devicetree/bindings/memory-controllers/pl353-smc.txt       | 3 ++-

> >  1 file changed, 2 insertions(+), 1 deletion(-)

> > 

> > diff --git a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> > index 1335b94291ad..9d220d4cb39d 100644

> > --- a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> > +++ b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt

> > @@ -27,7 +27,8 @@ Example:

> >  		compatible = "arm,pl353-smc-r2p1", "arm,primecell";

> >  		clock-names = "memclk", "apb_pclk";

> >  		clocks = <&clkc 11>, <&clkc 44>;

> > -		reg = <0xe000e000 0x1000>;

> > +		reg = <0xe000e000 0x0001000>,

> > +		      <0xe1000000 0x5000000>;

> >  		#address-cells = <2>;

> >  		#size-cells = <1>;

> >  		ranges = <0x0 0x0 0xe1000000 0x1000000 /* Nand CS region */

> > -- 

> > 2.27.0

> >   


Thanks,
Miquèl
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt
index 1335b94291ad..9d220d4cb39d 100644
--- a/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt
+++ b/Documentation/devicetree/bindings/memory-controllers/pl353-smc.txt
@@ -27,7 +27,8 @@  Example:
 		compatible = "arm,pl353-smc-r2p1", "arm,primecell";
 		clock-names = "memclk", "apb_pclk";
 		clocks = <&clkc 11>, <&clkc 44>;
-		reg = <0xe000e000 0x1000>;
+		reg = <0xe000e000 0x0001000>,
+		      <0xe1000000 0x5000000>;
 		#address-cells = <2>;
 		#size-cells = <1>;
 		ranges = <0x0 0x0 0xe1000000 0x1000000 /* Nand CS region */