public inbox for linux-arm-kernel@lists.infradead.org
 help / color / mirror / Atom feed
From: clabbe.montjoie@gmail.com (Corentin Labbe)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 04/18] dt-bindings: syscon: Add DT bindings documentation for Allwinner syscon
Date: Thu, 20 Apr 2017 08:50:53 +0200	[thread overview]
Message-ID: <20170420065053.GA28021@Red> (raw)
In-Reply-To: <9382f9ae-895a-397b-3d8c-846cb0bc4486@arm.com>

On Thu, Apr 20, 2017 at 12:38:50AM +0100, Andr? Przywara wrote:
> On 12/04/17 12:13, Corentin Labbe wrote:
> > This patch adds documentation for Device-Tree bindings for the
> > syscon present in allwinner devices.
> > 
> > Signed-off-by: Corentin Labbe <clabbe.montjoie@gmail.com>
> > ---
> >  .../devicetree/bindings/misc/allwinner,syscon.txt     | 19 +++++++++++++++++++
> >  1 file changed, 19 insertions(+)
> >  create mode 100644 Documentation/devicetree/bindings/misc/allwinner,syscon.txt
> > 
> > diff --git a/Documentation/devicetree/bindings/misc/allwinner,syscon.txt b/Documentation/devicetree/bindings/misc/allwinner,syscon.txt
> > new file mode 100644
> > index 0000000..c056c5b
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/misc/allwinner,syscon.txt
> > @@ -0,0 +1,19 @@
> > +* Allwinner sun8i system controller
> > +
> > +This file describes the bindings for the system controller present in
> > +Allwinner SoC H3, A83T and A64.
> > +The principal function of this syscon is to control EMAC PHY choice and
> > +config.
> > +
> > +Required properties for the system controller:
> > +- reg: address and length of the register for the device.
> > +- compatible: should be "syscon" and one of the following string:
> > +		"allwinner,sun8i-h3-system-controller"
> > +		"allwinner,sun8i-a64-system-controller"
> 
> While sun8i might make some sense technically, all 64-bit sunxi
> compatible strings use the sun50i prefix to follow the Allwinner naming.
> So this should read:
> 		"allwinner,sun50i-a64-system-controller"
> 
> Also I am wondering if we should add a compatible string for the H5
> (support for that SoC is in -next already):
> 		"allwinner,sun50i-h5-system-controller"
> 
> Cheers,
> Andre.
> 

I agree with sun50i-a64-system-controller, the sun8i-a64 is a copy/paste error.
I will add sun50i-h5-system-controller since all xxx-system-controller are just hypothetic compatible.

Thanks
Regards

  reply	other threads:[~2017-04-20  6:50 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-12 11:13 [PATCH v4 00/18] net-next: stmmac: add dwmac-sun8i ethernet driver Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 01/18] net: stmmac: export stmmac_set_mac_addr/stmmac_get_mac_addr Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 02/18] net: stmmac: add optional setup function Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 03/18] dt-bindings: net: Add DT bindings documentation for Allwinner dwmac-sun8i Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 04/18] dt-bindings: syscon: Add DT bindings documentation for Allwinner syscon Corentin Labbe
2017-04-19 21:06   ` Rob Herring
2017-04-19 23:38   ` André Przywara
2017-04-20  6:50     ` Corentin Labbe [this message]
2017-04-12 11:13 ` [PATCH v4 05/18] net-next: stmmac: Add dwmac-sun8i Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 06/18] arm: sun8i: sunxi-h3-h5: Add dt node for the syscon control module Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 07/18] arm: sun8i: sunxi-h3-h5: add dwmac-sun8i ethernet driver Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 08/18] arm: sun8i: orangepi-pc: Enable dwmac-sun8i Corentin Labbe
2017-04-12 12:39   ` Maxime Ripard
2017-04-12 11:13 ` [PATCH v4 09/18] arm: sun8i: orangepi-2: " Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 10/18] arm: sun8i: orangepi-one: " Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 11/18] arm: sun8i: orangepi-pc-plus: Set EMAC activity LEDs to active high Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 12/18] arm64: allwinner: sun50i-a64: Add dt node for the syscon control module Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 13/18] arm64: allwinner: sun50i-a64: add dwmac-sun8i Ethernet driver Corentin Labbe
2017-04-12 12:41   ` Maxime Ripard
2017-04-24 12:24     ` Corentin Labbe
2017-04-24 12:58       ` Chen-Yu Tsai
2017-04-12 11:13 ` [PATCH v4 14/18] arm: allwinner: pine64: Enable dwmac-sun8i Corentin Labbe
2017-04-12 12:42   ` Maxime Ripard
2017-04-12 11:13 ` [PATCH v4 15/18] arm: allwinner: pine64-plus: " Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 16/18] arm64: allwinner: bananapi-m64: " Corentin Labbe
2017-04-12 11:13 ` [PATCH v4 17/18] arm: sunxi: Enable dwmac-sun8i driver on sunxi_defconfig Corentin Labbe
2017-04-12 11:14 ` [PATCH v4 18/18] arm: multi_v7: Enable dwmac-sun8i driver on multi_v7_defconfig Corentin Labbe
2017-04-12 12:42   ` Maxime Ripard

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=20170420065053.GA28021@Red \
    --to=clabbe.montjoie@gmail.com \
    --cc=linux-arm-kernel@lists.infradead.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox