diff mbox

[v3,2/8] syscon: dt-bindings: Add documentation for Aspeed system control units

Message ID 20160830075427.11493-3-andrew@aj.id.au
State New
Headers show

Commit Message

Andrew Jeffery Aug. 30, 2016, 7:54 a.m. UTC
Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
Acked-by: Rob Herring <robh@kernel.org>
Acked-by: Joel Stanley <joel@jms.id.au>
---
 Documentation/devicetree/bindings/mfd/aspeed-scu.txt | 18 ++++++++++++++++++
 1 file changed, 18 insertions(+)
 create mode 100644 Documentation/devicetree/bindings/mfd/aspeed-scu.txt

Comments

Linus Walleij Sept. 7, 2016, 2:38 p.m. UTC | #1
On Tue, Aug 30, 2016 at 9:54 AM, Andrew Jeffery <andrew@aj.id.au> wrote:

> Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> Acked-by: Rob Herring <robh@kernel.org>
> Acked-by: Joel Stanley <joel@jms.id.au>

Acked-by: Linus Walleij <linus.walleij@linaro.org>

I guess the MFD maintainer (Lee Jones) should merge this?

Why is he not on CC?

Yours,
Linus Walleij
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Lee Jones Sept. 7, 2016, 4:30 p.m. UTC | #2
On Wed, 07 Sep 2016, Linus Walleij wrote:

> On Tue, Aug 30, 2016 at 9:54 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
> 
> > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > Acked-by: Rob Herring <robh@kernel.org>
> > Acked-by: Joel Stanley <joel@jms.id.au>
> 
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> 
> I guess the MFD maintainer (Lee Jones) should merge this?
> 
> Why is he not on CC?

Thanks Linus.

Yes, please resend, so I can take it in.
Andrew Jeffery Sept. 8, 2016, 1:01 a.m. UTC | #3
On Wed, 2016-09-07 at 16:38 +0200, Linus Walleij wrote:
> On Tue, Aug 30, 2016 at 9:54 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
> 
> > 
> > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > Acked-by: Rob Herring <robh@kernel.org>
> > Acked-by: Joel Stanley <joel@jms.id.au>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>
> 
> I guess the MFD maintainer (Lee Jones) should merge this?
> 
> Why is he not on CC?

I used get_maintainer.pl on the series to construct the CC, but Lee
wasn't listed in the output. I'll poke further at MAINTAINERs in the
future to make sure I have coverage for affected subsystems.

Andrew

> 
> Yours,
> Linus Walleij
Andrew Jeffery Sept. 8, 2016, 1:03 a.m. UTC | #4
On Wed, 2016-09-07 at 17:30 +0100, Lee Jones wrote:
> On Wed, 07 Sep 2016, Linus Walleij wrote:
> 
> > 
> > On Tue, Aug 30, 2016 at 9:54 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
> > 
> > > 
> > > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > > Acked-by: Rob Herring <robh@kernel.org>
> > > Acked-by: Joel Stanley <joel@jms.id.au>
> > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> > 
> > I guess the MFD maintainer (Lee Jones) should merge this?
> > 
> > Why is he not on CC?
> Thanks Linus.
> 
> Yes, please resend, so I can take it in.
> 

Will do.

Cheers,

Andrew
Lee Jones Sept. 8, 2016, 7:01 a.m. UTC | #5
On Thu, 08 Sep 2016, Andrew Jeffery wrote:
> On Wed, 2016-09-07 at 16:38 +0200, Linus Walleij wrote:
> > On Tue, Aug 30, 2016 at 9:54 AM, Andrew Jeffery <andrew@aj.id.au> wrote:
> > 
> > > 
> > > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > > Acked-by: Rob Herring <robh@kernel.org>
> > > Acked-by: Joel Stanley <joel@jms.id.au>
> > Acked-by: Linus Walleij <linus.walleij@linaro.org>
> > 
> > I guess the MFD maintainer (Lee Jones) should merge this?
> > 
> > Why is he not on CC?
> 
> I used get_maintainer.pl on the series to construct the CC, but Lee
> wasn't listed in the output. I'll poke further at MAINTAINERs in the
> future to make sure I have coverage for affected subsystems.

Good point.  Will fix.
diff mbox

Patch

diff --git a/Documentation/devicetree/bindings/mfd/aspeed-scu.txt b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
new file mode 100644
index 000000000000..4fc5b83726d6
--- /dev/null
+++ b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
@@ -0,0 +1,18 @@ 
+The Aspeed System Control Unit manages the global behaviour of the SoC,
+configuring elements such as clocks, pinmux, and reset.
+
+Required properties:
+- compatible:	One of:
+		"aspeed,ast2400-scu", "syscon", "simple-mfd"
+		"aspeed,g4-scu", "syscon", "simple-mfd"
+		"aspeed,ast2500-scu", "syscon", "simple-mfd"
+		"aspeed,g5-scu", "syscon", "simple-mfd"
+
+- reg:		contains the offset and length of the SCU memory region
+
+Example:
+
+syscon: syscon@1e6e2000 {
+	compatible = "aspeed,ast2400-scu", "syscon", "simple-mfd";
+	reg = <0x1e6e2000 0x1a8>;
+};