All of lore.kernel.org
 help / color / mirror / Atom feed
From: Andrew Jeffery <andrew@aj.id.au>
To: Rob Herring <robh@kernel.org>
Cc: Linus Walleij <linus.walleij@linaro.org>,
	Mark Rutland <mark.rutland@arm.com>,
	Alexandre Courbot <gnurou@gmail.com>,
	devicetree@vger.kernel.org,
	Benjamin Herrenschmidt <benh@kernel.crashing.org>,
	Russell King <linux@armlinux.org.uk>,
	linux-kernel@vger.kernel.org, linux-gpio@vger.kernel.org,
	Joel Stanley <joel@jms.id.au>,
	linux-arm-kernel@lists.infradead.org, Jeremy Kerr <jk@ozlabs.org>
Subject: Re: [PATCH 07/12] syscon: dt-bindings: Add documentation for Aspeed system control units
Date: Thu, 21 Jul 2016 10:34:40 +0930	[thread overview]
Message-ID: <1469063080.954.109.camel@aj.id.au> (raw)
In-Reply-To: <20160720190901.GA11147@rob-hp-laptop>

[-- Attachment #1: Type: text/plain, Size: 1248 bytes --]

On Wed, 2016-07-20 at 14:09 -0500, Rob Herring wrote:
> On Wed, Jul 20, 2016 at 03:28:28PM +0930, Andrew Jeffery wrote:
> > 
> > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > ---
> >  Documentation/devicetree/bindings/mfd/aspeed-scu.txt | 16 ++++++++++++++++
> >  1 file changed, 16 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > 
> > diff --git a/Documentation/devicetree/bindings/mfd/aspeed-scu.txt b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > new file mode 100644
> > index 000000000000..4df798799101
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > @@ -0,0 +1,16 @@
> > +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,g4-scu", "syscon", "simple-mfd"
> > +		"aspeed,g5-scu", "syscon", "simple-mfd"
> These (and the pinctrl probably) really need SoC specific compatible 
> strings. You can keep these as fallbacks though. These are the parts of 
> SoCs that really vary chip to chip.

I'll add the SoC-specific compatible strings.

Cheers,

Andrew

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: andrew@aj.id.au (Andrew Jeffery)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH 07/12] syscon: dt-bindings: Add documentation for Aspeed system control units
Date: Thu, 21 Jul 2016 10:34:40 +0930	[thread overview]
Message-ID: <1469063080.954.109.camel@aj.id.au> (raw)
In-Reply-To: <20160720190901.GA11147@rob-hp-laptop>

On Wed, 2016-07-20 at 14:09 -0500, Rob Herring wrote:
> On Wed, Jul 20, 2016 at 03:28:28PM +0930, Andrew Jeffery wrote:
> > 
> > Signed-off-by: Andrew Jeffery <andrew@aj.id.au>
> > ---
> > ?Documentation/devicetree/bindings/mfd/aspeed-scu.txt | 16 ++++++++++++++++
> > ?1 file changed, 16 insertions(+)
> > ?create mode 100644 Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > 
> > diff --git a/Documentation/devicetree/bindings/mfd/aspeed-scu.txt b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > new file mode 100644
> > index 000000000000..4df798799101
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/mfd/aspeed-scu.txt
> > @@ -0,0 +1,16 @@
> > +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,g4-scu", "syscon", "simple-mfd"
> > +		"aspeed,g5-scu", "syscon", "simple-mfd"
> These (and the pinctrl probably) really need SoC specific compatible?
> strings. You can keep these as fallbacks though. These are the parts of?
> SoCs that really vary chip to chip.

I'll add the SoC-specific compatible strings.

Cheers,

Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160721/23cf50ff/attachment-0001.sig>

  reply	other threads:[~2016-07-21  1:04 UTC|newest]

Thread overview: 79+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-20  5:58 [PATCH 00/12] aspeed: Add pinctrl and gpio drivers Andrew Jeffery
2016-07-20  5:58 ` Andrew Jeffery
2016-07-20  5:58 ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 01/12] pinctrl: dt-bindings: Add documentation for Aspeed pin controllers Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20 19:05   ` Rob Herring
2016-07-20 19:05     ` Rob Herring
2016-07-20 23:13     ` Andrew Jeffery
2016-07-20 23:13       ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 02/12] pinctrl: Add core pinctrl support for Aspeed SoCs Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-22  6:29   ` Joel Stanley
2016-07-22  6:29     ` Joel Stanley
2016-08-11  8:41   ` Linus Walleij
2016-08-11  8:41     ` Linus Walleij
2016-08-11  8:41     ` Linus Walleij
2016-08-12  0:33     ` Andrew Jeffery
2016-08-12  0:33       ` Andrew Jeffery
2016-08-12  0:33       ` Andrew Jeffery
2016-08-12 13:18       ` Linus Walleij
2016-08-12 13:18         ` Linus Walleij
2016-08-12 13:18         ` Linus Walleij
2016-08-13  0:58         ` Benjamin Herrenschmidt
2016-08-13  0:58           ` Benjamin Herrenschmidt
2016-08-13  0:58           ` Benjamin Herrenschmidt
     [not found]           ` <1471049894.12231.41.camel-XVmvHMARGAS8U2dJNN8I7kB+6BGkLq7r@public.gmane.org>
2016-08-15  0:36             ` Andrew Jeffery
2016-08-15  0:36               ` Andrew Jeffery
2016-08-15  0:36               ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 03/12] pinctrl: Add pinctrl-aspeed-g4 driver Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 04/12] pinctrl: Add pinctrl-aspeed-g5 driver Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 05/12] gpio: dt-bindings: Add documentation for Aspeed GPIO controllers Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
     [not found]   ` <1468994313-13538-6-git-send-email-andrew-zrmu5oMJ5Fs@public.gmane.org>
2016-07-20 19:06     ` Rob Herring
2016-07-20 19:06       ` Rob Herring
2016-07-20 19:06       ` Rob Herring
2016-07-20  5:58 ` [PATCH 06/12] gpio: Add Aspeed driver Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-21 20:12   ` Paul Gortmaker
2016-07-21 20:12     ` Paul Gortmaker
2016-07-21 20:12     ` Paul Gortmaker
2016-07-22  0:49     ` Andrew Jeffery
2016-07-22  0:49       ` Andrew Jeffery
2016-07-22  0:49       ` Andrew Jeffery
     [not found]   ` <1468994313-13538-7-git-send-email-andrew-zrmu5oMJ5Fs@public.gmane.org>
2016-08-11  9:20     ` Linus Walleij
2016-08-11  9:20       ` Linus Walleij
2016-08-11  9:20       ` Linus Walleij
     [not found]       ` <CACRpkdYvZdTa7dpSs5=vnK77wpRGFT2W3e3=AS3GJGY7CXdwRA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2016-08-12  0:54         ` Andrew Jeffery
2016-08-12  0:54           ` Andrew Jeffery
2016-08-12  0:54           ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 07/12] syscon: dt-bindings: Add documentation for Aspeed system control units Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20 19:09   ` Rob Herring
2016-07-20 19:09     ` Rob Herring
2016-07-21  1:04     ` Andrew Jeffery [this message]
2016-07-21  1:04       ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 08/12] aspeed-g4: Add syscon and pin controller nodes Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 09/12] palmetto: Request relevant mux functions in devicetree Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 10/12] aspeed-g4: Add gpio controller to devicetree Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 11/12] aspeed-g5: Add syscon and pin controller nodes Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58 ` [PATCH 12/12] aspeed-g5: Add gpio controller to devicetree Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery
2016-07-20  5:58   ` Andrew Jeffery

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=1469063080.954.109.camel@aj.id.au \
    --to=andrew@aj.id.au \
    --cc=benh@kernel.crashing.org \
    --cc=devicetree@vger.kernel.org \
    --cc=gnurou@gmail.com \
    --cc=jk@ozlabs.org \
    --cc=joel@jms.id.au \
    --cc=linus.walleij@linaro.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-gpio@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=robh@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 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.