All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  5:44 ` Prabhakar Kushwaha
  0 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  5:44 UTC (permalink / raw)
  To: linux-mtd, devicetree
  Cc: mark.rutland, robh, boris.brezillon, poonam.aggrwal, leoyang.li,
	oss, Prabhakar Kushwaha, computersforpeace, shawnguo,
	linux-arm-kernel

Connection between flash and controller is not necessary to be always
of same type. It may varies from platform to platform.

Adding endianness (optional) property to provide connection type
information.

Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
Changes for v2: updated subject
Changes for v3: fixed typo for "big-endian"
Changes for v4: Moved binding definition in mtd-physmap.txt
as discussed at https://patchwork.ozlabs.org/patch/842543/
Changes for v5: Sending as it is
Changes for v6: Updated binding when endianness property is absent
Changes for v7: Updated binding details

 Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
index 4a0a48bf4ecb..232fa12e90ef 100644
--- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
+++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
@@ -41,6 +41,13 @@ additional (optional) property is defined:
 
  - erase-size : The chip's physical erase block size in bytes.
 
+ The device tree may optionally contain endianness property.
+ little-endian or big-endian : It Represents the endianness that should be used
+                               by the controller to  properly read/write data
+			       from/to the flash. If this property is missing,
+			       the endianness is chosen by the system
+			       (potentially based on extra configuration options).
+
 The device tree may optionally contain sub-nodes describing partitions of the
 address space. See partition.txt for more detail.
 
-- 
2.14.1

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

* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  5:44 ` Prabhakar Kushwaha
  0 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  5:44 UTC (permalink / raw)
  To: linux-mtd, devicetree
  Cc: robh, mark.rutland, shawnguo, boris.brezillon, computersforpeace,
	oss, linux-arm-kernel, leoyang.li, poonam.aggrwal,
	Prabhakar Kushwaha

Connection between flash and controller is not necessary to be always
of same type. It may varies from platform to platform.

Adding endianness (optional) property to provide connection type
information.

Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
Changes for v2: updated subject
Changes for v3: fixed typo for "big-endian"
Changes for v4: Moved binding definition in mtd-physmap.txt
as discussed at https://patchwork.ozlabs.org/patch/842543/
Changes for v5: Sending as it is
Changes for v6: Updated binding when endianness property is absent
Changes for v7: Updated binding details

 Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
index 4a0a48bf4ecb..232fa12e90ef 100644
--- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
+++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
@@ -41,6 +41,13 @@ additional (optional) property is defined:
 
  - erase-size : The chip's physical erase block size in bytes.
 
+ The device tree may optionally contain endianness property.
+ little-endian or big-endian : It Represents the endianness that should be used
+                               by the controller to  properly read/write data
+			       from/to the flash. If this property is missing,
+			       the endianness is chosen by the system
+			       (potentially based on extra configuration options).
+
 The device tree may optionally contain sub-nodes describing partitions of the
 address space. See partition.txt for more detail.
 
-- 
2.14.1

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

* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  5:44 ` Prabhakar Kushwaha
  0 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  5:44 UTC (permalink / raw)
  To: linux-arm-kernel

Connection between flash and controller is not necessary to be always
of same type. It may varies from platform to platform.

Adding endianness (optional) property to provide connection type
information.

Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
Reviewed-by: Rob Herring <robh@kernel.org>
---
Changes for v2: updated subject
Changes for v3: fixed typo for "big-endian"
Changes for v4: Moved binding definition in mtd-physmap.txt
as discussed at https://patchwork.ozlabs.org/patch/842543/
Changes for v5: Sending as it is
Changes for v6: Updated binding when endianness property is absent
Changes for v7: Updated binding details

 Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
index 4a0a48bf4ecb..232fa12e90ef 100644
--- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
+++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
@@ -41,6 +41,13 @@ additional (optional) property is defined:
 
  - erase-size : The chip's physical erase block size in bytes.
 
+ The device tree may optionally contain endianness property.
+ little-endian or big-endian : It Represents the endianness that should be used
+                               by the controller to  properly read/write data
+			       from/to the flash. If this property is missing,
+			       the endianness is chosen by the system
+			       (potentially based on extra configuration options).
+
 The device tree may optionally contain sub-nodes describing partitions of the
 address space. See partition.txt for more detail.
 
-- 
2.14.1

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

* Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
  2018-03-28  5:44 ` Prabhakar Kushwaha
  (?)
@ 2018-03-28  8:30   ` Boris Brezillon
  -1 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-28  8:30 UTC (permalink / raw)
  To: Prabhakar Kushwaha
  Cc: mark.rutland, devicetree, boris.brezillon, poonam.aggrwal,
	leoyang.li, oss, linux-mtd, computersforpeace, shawnguo, robh,
	linux-arm-kernel

On Wed, 28 Mar 2018 11:14:43 +0530
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Connection between flash and controller is not necessary to be always
> of same type. It may varies from platform to platform.
> 
> Adding endianness (optional) property to provide connection type
> information.

You forgot to adjust your commit message.

If you're okay, I'll replace your message by:

"
Provides a way to specify the endianness to use when accessing a
memory-mapped flash.
"

> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> Changes for v2: updated subject
> Changes for v3: fixed typo for "big-endian"
> Changes for v4: Moved binding definition in mtd-physmap.txt
> as discussed at https://patchwork.ozlabs.org/patch/842543/
> Changes for v5: Sending as it is
> Changes for v6: Updated binding when endianness property is absent
> Changes for v7: Updated binding details
> 
>  Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> index 4a0a48bf4ecb..232fa12e90ef 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> @@ -41,6 +41,13 @@ additional (optional) property is defined:
>  
>   - erase-size : The chip's physical erase block size in bytes.
>  
> + The device tree may optionally contain endianness property.
> + little-endian or big-endian : It Represents the endianness that should be used
> +                               by the controller to  properly read/write data
> +			       from/to the flash. If this property is missing,
> +			       the endianness is chosen by the system
> +			       (potentially based on extra configuration options).
> +
>  The device tree may optionally contain sub-nodes describing partitions of the
>  address space. See partition.txt for more detail.
>  



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  8:30   ` Boris Brezillon
  0 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-28  8:30 UTC (permalink / raw)
  To: Prabhakar Kushwaha
  Cc: linux-mtd, devicetree, mark.rutland, robh, boris.brezillon,
	poonam.aggrwal, leoyang.li, oss, computersforpeace, shawnguo,
	linux-arm-kernel

On Wed, 28 Mar 2018 11:14:43 +0530
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Connection between flash and controller is not necessary to be always
> of same type. It may varies from platform to platform.
> 
> Adding endianness (optional) property to provide connection type
> information.

You forgot to adjust your commit message.

If you're okay, I'll replace your message by:

"
Provides a way to specify the endianness to use when accessing a
memory-mapped flash.
"

> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> Changes for v2: updated subject
> Changes for v3: fixed typo for "big-endian"
> Changes for v4: Moved binding definition in mtd-physmap.txt
> as discussed at https://patchwork.ozlabs.org/patch/842543/
> Changes for v5: Sending as it is
> Changes for v6: Updated binding when endianness property is absent
> Changes for v7: Updated binding details
> 
>  Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> index 4a0a48bf4ecb..232fa12e90ef 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> @@ -41,6 +41,13 @@ additional (optional) property is defined:
>  
>   - erase-size : The chip's physical erase block size in bytes.
>  
> + The device tree may optionally contain endianness property.
> + little-endian or big-endian : It Represents the endianness that should be used
> +                               by the controller to  properly read/write data
> +			       from/to the flash. If this property is missing,
> +			       the endianness is chosen by the system
> +			       (potentially based on extra configuration options).
> +
>  The device tree may optionally contain sub-nodes describing partitions of the
>  address space. See partition.txt for more detail.
>  



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  8:30   ` Boris Brezillon
  0 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-28  8:30 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, 28 Mar 2018 11:14:43 +0530
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Connection between flash and controller is not necessary to be always
> of same type. It may varies from platform to platform.
> 
> Adding endianness (optional) property to provide connection type
> information.

You forgot to adjust your commit message.

If you're okay, I'll replace your message by:

"
Provides a way to specify the endianness to use when accessing a
memory-mapped flash.
"

> 
> Signed-off-by: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Reviewed-by: Rob Herring <robh@kernel.org>
> ---
> Changes for v2: updated subject
> Changes for v3: fixed typo for "big-endian"
> Changes for v4: Moved binding definition in mtd-physmap.txt
> as discussed at https://patchwork.ozlabs.org/patch/842543/
> Changes for v5: Sending as it is
> Changes for v6: Updated binding when endianness property is absent
> Changes for v7: Updated binding details
> 
>  Documentation/devicetree/bindings/mtd/mtd-physmap.txt | 7 +++++++
>  1 file changed, 7 insertions(+)
> 
> diff --git a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> index 4a0a48bf4ecb..232fa12e90ef 100644
> --- a/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> +++ b/Documentation/devicetree/bindings/mtd/mtd-physmap.txt
> @@ -41,6 +41,13 @@ additional (optional) property is defined:
>  
>   - erase-size : The chip's physical erase block size in bytes.
>  
> + The device tree may optionally contain endianness property.
> + little-endian or big-endian : It Represents the endianness that should be used
> +                               by the controller to  properly read/write data
> +			       from/to the flash. If this property is missing,
> +			       the endianness is chosen by the system
> +			       (potentially based on extra configuration options).
> +
>  The device tree may optionally contain sub-nodes describing partitions of the
>  address space. See partition.txt for more detail.
>  



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* RE: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
  2018-03-28  8:30   ` Boris Brezillon
  (?)
@ 2018-03-28  9:48     ` Prabhakar Kushwaha
  -1 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  9:48 UTC (permalink / raw)
  To: Boris Brezillon
  Cc: mark.rutland, devicetree, boris.brezillon, Poonam Aggrwal,
	Leo Li, oss, linux-mtd, computersforpeace, shawnguo, robh,
	linux-arm-kernel

Hi Boris,

> -----Original Message-----
> From: Boris Brezillon [mailto:boris.brezillon@bootlin.com]
> Sent: Wednesday, March 28, 2018 2:00 PM
> To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Cc: linux-mtd@lists.infradead.org; devicetree@vger.kernel.org;
> mark.rutland@arm.com; robh@kernel.org; boris.brezillon@free-electrons.com;
> Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> oss@buserror.net; computersforpeace@gmail.com; shawnguo@kernel.org;
> linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> supports
> 
> On Wed, 28 Mar 2018 11:14:43 +0530
> Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> 
> > Connection between flash and controller is not necessary to be always
> > of same type. It may varies from platform to platform.
> >
> > Adding endianness (optional) property to provide connection type
> > information.
> 
> You forgot to adjust your commit message.
> 
> If you're okay, I'll replace your message by:
> 
> "
> Provides a way to specify the endianness to use when accessing a memory-
> mapped flash.
> "
> 

Thanks for catching this. Please adjust as you suggested.

--prabhakar

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

* RE: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  9:48     ` Prabhakar Kushwaha
  0 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  9:48 UTC (permalink / raw)
  To: Boris Brezillon
  Cc: linux-mtd, devicetree, mark.rutland, robh, boris.brezillon,
	Poonam Aggrwal, Leo Li, oss, computersforpeace, shawnguo,
	linux-arm-kernel

Hi Boris,

> -----Original Message-----
> From: Boris Brezillon [mailto:boris.brezillon@bootlin.com]
> Sent: Wednesday, March 28, 2018 2:00 PM
> To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Cc: linux-mtd@lists.infradead.org; devicetree@vger.kernel.org;
> mark.rutland@arm.com; robh@kernel.org; boris.brezillon@free-electrons.com;
> Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> oss@buserror.net; computersforpeace@gmail.com; shawnguo@kernel.org;
> linux-arm-kernel@lists.infradead.org
> Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> supports
> 
> On Wed, 28 Mar 2018 11:14:43 +0530
> Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> 
> > Connection between flash and controller is not necessary to be always
> > of same type. It may varies from platform to platform.
> >
> > Adding endianness (optional) property to provide connection type
> > information.
> 
> You forgot to adjust your commit message.
> 
> If you're okay, I'll replace your message by:
> 
> "
> Provides a way to specify the endianness to use when accessing a memory-
> mapped flash.
> "
> 

Thanks for catching this. Please adjust as you suggested.

--prabhakar

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

* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-28  9:48     ` Prabhakar Kushwaha
  0 siblings, 0 replies; 12+ messages in thread
From: Prabhakar Kushwaha @ 2018-03-28  9:48 UTC (permalink / raw)
  To: linux-arm-kernel

Hi Boris,

> -----Original Message-----
> From: Boris Brezillon [mailto:boris.brezillon at bootlin.com]
> Sent: Wednesday, March 28, 2018 2:00 PM
> To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> Cc: linux-mtd at lists.infradead.org; devicetree at vger.kernel.org;
> mark.rutland at arm.com; robh at kernel.org; boris.brezillon at free-electrons.com;
> Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> oss at buserror.net; computersforpeace at gmail.com; shawnguo at kernel.org;
> linux-arm-kernel at lists.infradead.org
> Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> supports
> 
> On Wed, 28 Mar 2018 11:14:43 +0530
> Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> 
> > Connection between flash and controller is not necessary to be always
> > of same type. It may varies from platform to platform.
> >
> > Adding endianness (optional) property to provide connection type
> > information.
> 
> You forgot to adjust your commit message.
> 
> If you're okay, I'll replace your message by:
> 
> "
> Provides a way to specify the endianness to use when accessing a memory-
> mapped flash.
> "
> 

Thanks for catching this. Please adjust as you suggested.

--prabhakar

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

* Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
  2018-03-28  9:48     ` Prabhakar Kushwaha
  (?)
@ 2018-03-29 20:22       ` Boris Brezillon
  -1 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-29 20:22 UTC (permalink / raw)
  To: Prabhakar Kushwaha
  Cc: mark.rutland, devicetree, boris.brezillon, Poonam Aggrwal,
	Leo Li, oss, linux-mtd, computersforpeace, shawnguo, robh,
	linux-arm-kernel

On Wed, 28 Mar 2018 09:48:08 +0000
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Hi Boris,
> 
> > -----Original Message-----
> > From: Boris Brezillon [mailto:boris.brezillon@bootlin.com]
> > Sent: Wednesday, March 28, 2018 2:00 PM
> > To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> > Cc: linux-mtd@lists.infradead.org; devicetree@vger.kernel.org;
> > mark.rutland@arm.com; robh@kernel.org; boris.brezillon@free-electrons.com;
> > Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> > oss@buserror.net; computersforpeace@gmail.com; shawnguo@kernel.org;
> > linux-arm-kernel@lists.infradead.org
> > Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> > supports
> > 
> > On Wed, 28 Mar 2018 11:14:43 +0530
> > Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> >   
> > > Connection between flash and controller is not necessary to be always
> > > of same type. It may varies from platform to platform.
> > >
> > > Adding endianness (optional) property to provide connection type
> > > information.  
> > 
> > You forgot to adjust your commit message.
> > 
> > If you're okay, I'll replace your message by:
> > 
> > "
> > Provides a way to specify the endianness to use when accessing a memory-
> > mapped flash.
> > "
> >   
> 
> Thanks for catching this. Please adjust as you suggested.

Applied.

Thanks,

Boris

> 
> --prabhakar
> 
> 



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-29 20:22       ` Boris Brezillon
  0 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-29 20:22 UTC (permalink / raw)
  To: Prabhakar Kushwaha
  Cc: linux-mtd, devicetree, mark.rutland, robh, boris.brezillon,
	Poonam Aggrwal, Leo Li, oss, computersforpeace, shawnguo,
	linux-arm-kernel

On Wed, 28 Mar 2018 09:48:08 +0000
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Hi Boris,
> 
> > -----Original Message-----
> > From: Boris Brezillon [mailto:boris.brezillon@bootlin.com]
> > Sent: Wednesday, March 28, 2018 2:00 PM
> > To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> > Cc: linux-mtd@lists.infradead.org; devicetree@vger.kernel.org;
> > mark.rutland@arm.com; robh@kernel.org; boris.brezillon@free-electrons.com;
> > Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> > oss@buserror.net; computersforpeace@gmail.com; shawnguo@kernel.org;
> > linux-arm-kernel@lists.infradead.org
> > Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> > supports
> > 
> > On Wed, 28 Mar 2018 11:14:43 +0530
> > Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> >   
> > > Connection between flash and controller is not necessary to be always
> > > of same type. It may varies from platform to platform.
> > >
> > > Adding endianness (optional) property to provide connection type
> > > information.  
> > 
> > You forgot to adjust your commit message.
> > 
> > If you're okay, I'll replace your message by:
> > 
> > "
> > Provides a way to specify the endianness to use when accessing a memory-
> > mapped flash.
> > "
> >   
> 
> Thanks for catching this. Please adjust as you suggested.

Applied.

Thanks,

Boris

> 
> --prabhakar
> 
> 



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

* [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports
@ 2018-03-29 20:22       ` Boris Brezillon
  0 siblings, 0 replies; 12+ messages in thread
From: Boris Brezillon @ 2018-03-29 20:22 UTC (permalink / raw)
  To: linux-arm-kernel

On Wed, 28 Mar 2018 09:48:08 +0000
Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:

> Hi Boris,
> 
> > -----Original Message-----
> > From: Boris Brezillon [mailto:boris.brezillon at bootlin.com]
> > Sent: Wednesday, March 28, 2018 2:00 PM
> > To: Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com>
> > Cc: linux-mtd at lists.infradead.org; devicetree at vger.kernel.org;
> > mark.rutland at arm.com; robh at kernel.org; boris.brezillon at free-electrons.com;
> > Poonam Aggrwal <poonam.aggrwal@nxp.com>; Leo Li <leoyang.li@nxp.com>;
> > oss at buserror.net; computersforpeace at gmail.com; shawnguo at kernel.org;
> > linux-arm-kernel at lists.infradead.org
> > Subject: Re: [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness
> > supports
> > 
> > On Wed, 28 Mar 2018 11:14:43 +0530
> > Prabhakar Kushwaha <prabhakar.kushwaha@nxp.com> wrote:
> >   
> > > Connection between flash and controller is not necessary to be always
> > > of same type. It may varies from platform to platform.
> > >
> > > Adding endianness (optional) property to provide connection type
> > > information.  
> > 
> > You forgot to adjust your commit message.
> > 
> > If you're okay, I'll replace your message by:
> > 
> > "
> > Provides a way to specify the endianness to use when accessing a memory-
> > mapped flash.
> > "
> >   
> 
> Thanks for catching this. Please adjust as you suggested.

Applied.

Thanks,

Boris

> 
> --prabhakar
> 
> 



-- 
Boris Brezillon, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com

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

end of thread, other threads:[~2018-03-29 20:22 UTC | newest]

Thread overview: 12+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-03-28  5:44 [PATCH 1/2][v7] dt-bindings: mtd-physmap: Add endianness supports Prabhakar Kushwaha
2018-03-28  5:44 ` Prabhakar Kushwaha
2018-03-28  5:44 ` Prabhakar Kushwaha
2018-03-28  8:30 ` Boris Brezillon
2018-03-28  8:30   ` Boris Brezillon
2018-03-28  8:30   ` Boris Brezillon
2018-03-28  9:48   ` Prabhakar Kushwaha
2018-03-28  9:48     ` Prabhakar Kushwaha
2018-03-28  9:48     ` Prabhakar Kushwaha
2018-03-29 20:22     ` Boris Brezillon
2018-03-29 20:22       ` Boris Brezillon
2018-03-29 20:22       ` Boris Brezillon

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.