linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH v5] dt-bindings: mtd: add new compatible strings and improve description
@ 2017-08-04  8:10 Guochun Mao
  2017-08-04  8:10 ` [PATCH v5 1/1] " Guochun Mao
  0 siblings, 1 reply; 5+ messages in thread
From: Guochun Mao @ 2017-08-04  8:10 UTC (permalink / raw)
  To: Rob Herring, Matthias Brugger, Cyrille Pitchen
  Cc: Boris Brezillon, Marek Vasut, David Woodhouse,
	Richard Weinberger, Mark Rutland, Russell King, linux-mtd,
	devicetree, linux-arm-kernel, linux-mediatek, linux-kernel

Correct compatibles description and explicate fallback compatible.

Guochun Mao (1):
  dt-bindings: mtd: add new compatible strings and improve description

 Documentation/devicetree/bindings/mtd/mtk-quadspi.txt | 15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

-- 
1.9.1

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

* [PATCH v5 1/1] dt-bindings: mtd: add new compatible strings and improve description
  2017-08-04  8:10 [PATCH v5] dt-bindings: mtd: add new compatible strings and improve description Guochun Mao
@ 2017-08-04  8:10 ` Guochun Mao
  2017-08-10 18:21   ` Rob Herring
  0 siblings, 1 reply; 5+ messages in thread
From: Guochun Mao @ 2017-08-04  8:10 UTC (permalink / raw)
  To: Rob Herring, Matthias Brugger, Cyrille Pitchen
  Cc: Boris Brezillon, Marek Vasut, David Woodhouse,
	Richard Weinberger, Mark Rutland, Russell King, linux-mtd,
	devicetree, linux-arm-kernel, linux-mediatek, linux-kernel,
	Guochun Mao

Add "mediatek,mt2712-nor" and "mediatek,mt7622-nor"
for nor flash node's compatible strings.
Explicate the fallback compatible.

Signed-off-by: Guochun Mao <guochun.mao@mediatek.com>
---
 .../devicetree/bindings/mtd/mtk-quadspi.txt        |   15 +++++++++------
 1 file changed, 9 insertions(+), 6 deletions(-)

diff --git a/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt b/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
index 5ded66a..f97d601 100644
--- a/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
+++ b/Documentation/devicetree/bindings/mtd/mtk-quadspi.txt
@@ -1,13 +1,16 @@
 * Serial NOR flash controller for MTK MT81xx (and similar)
 
 Required properties:
-- compatible: 	  The possible values are:
-		  "mediatek,mt2701-nor"
-		  "mediatek,mt7623-nor"
+- compatible: 	  For mt8173, compatible should be "mediatek,mt8173-nor",
+		  and it's the fallback compatible for other Soc.
+		  For every other SoC, should contain both the SoC-specific compatible
+		  string and "mediatek,mt8173-nor".
+		  The possible values are:
+		  "mediatek,mt2701-nor", "mediatek,mt8173-nor"
+		  "mediatek,mt2712-nor", "mediatek,mt8173-nor"
+		  "mediatek,mt7622-nor", "mediatek,mt8173-nor"
+		  "mediatek,mt7623-nor", "mediatek,mt8173-nor"
 		  "mediatek,mt8173-nor"
-		  For mt8173, compatible should be "mediatek,mt8173-nor".
-		  For every other SoC, should contain both the SoC-specific compatible string
-		  and "mediatek,mt8173-nor".
 - reg: 		  physical base address and length of the controller's register
 - clocks: 	  the phandle of the clocks needed by the nor controller
 - clock-names: 	  the names of the clocks
-- 
1.7.9.5

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

* Re: [PATCH v5 1/1] dt-bindings: mtd: add new compatible strings and improve description
  2017-08-04  8:10 ` [PATCH v5 1/1] " Guochun Mao
@ 2017-08-10 18:21   ` Rob Herring
  2017-08-17  1:51     ` Guochun Mao
  0 siblings, 1 reply; 5+ messages in thread
From: Rob Herring @ 2017-08-10 18:21 UTC (permalink / raw)
  To: Guochun Mao
  Cc: Matthias Brugger, Cyrille Pitchen, Boris Brezillon, Marek Vasut,
	David Woodhouse, Richard Weinberger, Mark Rutland, Russell King,
	linux-mtd, devicetree, linux-arm-kernel, linux-mediatek,
	linux-kernel

On Fri, Aug 04, 2017 at 04:10:39PM +0800, Guochun Mao wrote:
> Add "mediatek,mt2712-nor" and "mediatek,mt7622-nor"
> for nor flash node's compatible strings.
> Explicate the fallback compatible.
> 
> Signed-off-by: Guochun Mao <guochun.mao@mediatek.com>
> ---
>  .../devicetree/bindings/mtd/mtk-quadspi.txt        |   15 +++++++++------
>  1 file changed, 9 insertions(+), 6 deletions(-)

Acked-by: Rob Herring <robh@kernel.org>

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

* Re: [PATCH v5 1/1] dt-bindings: mtd: add new compatible strings and improve description
  2017-08-10 18:21   ` Rob Herring
@ 2017-08-17  1:51     ` Guochun Mao
  2017-08-28  8:47       ` Guochun Mao
  0 siblings, 1 reply; 5+ messages in thread
From: Guochun Mao @ 2017-08-17  1:51 UTC (permalink / raw)
  To: Cyrille Pitchen, Matthias Brugger, Rob Herring
  Cc: Matthias Brugger, Cyrille Pitchen, Boris Brezillon, Marek Vasut,
	David Woodhouse, Richard Weinberger, Mark Rutland, Russell King,
	linux-mtd, devicetree, linux-arm-kernel, linux-mediatek,
	linux-kernel

On Thu, 2017-08-10 at 13:21 -0500, Rob Herring wrote:
> On Fri, Aug 04, 2017 at 04:10:39PM +0800, Guochun Mao wrote:
> > Add "mediatek,mt2712-nor" and "mediatek,mt7622-nor"
> > for nor flash node's compatible strings.
> > Explicate the fallback compatible.
> > 
> > Signed-off-by: Guochun Mao <guochun.mao@mediatek.com>
> > ---
> >  .../devicetree/bindings/mtd/mtk-quadspi.txt        |   15 +++++++++------
> >  1 file changed, 9 insertions(+), 6 deletions(-)
> 
> Acked-by: Rob Herring <robh@kernel.org>

Hi Rob,
Thank you very much!

Hi Matthias, Cyrille,
Thanks a lot for your efforts in reviewing the patch.

Is there any comment about the patch this version?
I appreciate your suggestion.

BR,
Guochun

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

* Re: [PATCH v5 1/1] dt-bindings: mtd: add new compatible strings and improve description
  2017-08-17  1:51     ` Guochun Mao
@ 2017-08-28  8:47       ` Guochun Mao
  0 siblings, 0 replies; 5+ messages in thread
From: Guochun Mao @ 2017-08-28  8:47 UTC (permalink / raw)
  To: Cyrille Pitchen
  Cc: Matthias Brugger, Rob Herring, Boris Brezillon, Marek Vasut,
	David Woodhouse, Richard Weinberger, Mark Rutland, Russell King,
	linux-mtd, devicetree, linux-arm-kernel, linux-mediatek,
	linux-kernel

Hi Cyrille,

It seems that I sent mail to wrong email-address ;)
Please help to review this patch.

Thanks a lot.

Guochun

On Thu, 2017-08-17 at 09:51 +0800, Guochun Mao wrote:
> On Thu, 2017-08-10 at 13:21 -0500, Rob Herring wrote:
> > On Fri, Aug 04, 2017 at 04:10:39PM +0800, Guochun Mao wrote:
> > > Add "mediatek,mt2712-nor" and "mediatek,mt7622-nor"
> > > for nor flash node's compatible strings.
> > > Explicate the fallback compatible.
> > > 
> > > Signed-off-by: Guochun Mao <guochun.mao@mediatek.com>
> > > ---
> > >  .../devicetree/bindings/mtd/mtk-quadspi.txt        |   15 +++++++++------
> > >  1 file changed, 9 insertions(+), 6 deletions(-)
> > 
> > Acked-by: Rob Herring <robh@kernel.org>
> 
> Hi Rob,
> Thank you very much!
> 
> Hi Matthias, Cyrille,
> Thanks a lot for your efforts in reviewing the patch.
> 
> Is there any comment about the patch this version?
> I appreciate your suggestion.
> 
> BR,
> Guochun

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

end of thread, other threads:[~2017-08-28  8:48 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-08-04  8:10 [PATCH v5] dt-bindings: mtd: add new compatible strings and improve description Guochun Mao
2017-08-04  8:10 ` [PATCH v5 1/1] " Guochun Mao
2017-08-10 18:21   ` Rob Herring
2017-08-17  1:51     ` Guochun Mao
2017-08-28  8:47       ` Guochun Mao

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