From: maxime.ripard@bootlin.com (Maxime Ripard)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v4 01/19] dt-bindings: sram: sunxi: Add A13, A20 and A33 SRAM controller bindings
Date: Mon, 18 Jun 2018 18:02:50 +0200 [thread overview]
Message-ID: <20180618160250.ik44c3gcbdxylmyd@flea> (raw)
In-Reply-To: <20180618145843.14631-2-paul.kocialkowski@bootlin.com>
On Mon, Jun 18, 2018 at 04:58:25PM +0200, Paul Kocialkowski wrote:
> This introduces dedicated bindings for the SRAM controllers found on the
> A13, A20 and A33 sunxi platforms.
>
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Explaining why you need to add these new compatibles would be great.
> diff --git a/Documentation/devicetree/bindings/sram/sunxi-sram.txt b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> index d087f04a4d7f..19cc0b892672 100644
> --- a/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> +++ b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> @@ -11,6 +11,9 @@ Controller Node
> Required properties:
> - compatible : should be:
> - "allwinner,sun4i-a10-sram-controller"
> + - "allwinner,sun5i-a13-sram-controller"
> + - "allwinner,sun7i-a20-sram-controller"
> + - "allwinner,sun8i-a33-sram-controller"
And I think Chen-Yu asked you to rename this compatible to
-system-controller for the previous iteration?
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180618/5f55c0f0/attachment-0001.sig>
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>
To: Paul Kocialkowski
<paul.kocialkowski-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>
Cc: linux-media-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org,
linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Mauro Carvalho Chehab
<mchehab-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Rob Herring <robh+dt-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>,
Mark Rutland <mark.rutland-5wv7dgnIgG8@public.gmane.org>,
Chen-Yu Tsai <wens-jdAy2FN1RRM@public.gmane.org>,
Marco Franchi <marco.franchi-3arQi8VN3Tc@public.gmane.org>,
Icenowy Zheng <icenowy-h8G6r0blFSE@public.gmane.org>,
Hans Verkuil <hverkuil-qWit8jRvyhVmR6Xm/wNWPw@public.gmane.org>,
Sylwester Nawrocki
<s.nawrocki-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Tom Saeger <tom.saeger-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org>,
Smitha T Murthy
<smitha.t-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Sakari Ailus
<sakari.ailus-VuQAYsv1563Yd54FQh9/CA@public.gmane.org>,
Andrzej Hajda <a.hajda-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>,
Jonathan Corbet <corbet-T1hC0tSOHrs@public.gmane.org>,
"David S . Miller"
<davem-fT/PcQaiUtIeIZ0/mPfg9Q@public.gmane.org>,
Andrew Morton
<akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org>,
Greg Kroah-Hartman
<gregkh-hQyY1W1yCW8ekmWlsbkhG0B+6BGkLq7r@public.gmane.org>,
Linus Walleij
<linus.walleij-QSEj5FYQhm4dnm+yROfE0A@public.gmane.org>,
Randy Dunlap <rdunlap-wEGCiKHe2LqWVfeAwA7xHQ@public.gmane.org>,
Arnd Bergmann <arnd-r2nGTMty4D4@public.gmane.org>
Subject: Re: [PATCH v4 01/19] dt-bindings: sram: sunxi: Add A13, A20 and A33 SRAM controller bindings
Date: Mon, 18 Jun 2018 18:02:50 +0200 [thread overview]
Message-ID: <20180618160250.ik44c3gcbdxylmyd@flea> (raw)
In-Reply-To: <20180618145843.14631-2-paul.kocialkowski-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>
[-- Attachment #1: Type: text/plain, Size: 1141 bytes --]
On Mon, Jun 18, 2018 at 04:58:25PM +0200, Paul Kocialkowski wrote:
> This introduces dedicated bindings for the SRAM controllers found on the
> A13, A20 and A33 sunxi platforms.
>
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski-LDxbnhwyfcJBDgjK7y7TUQ@public.gmane.org>
Explaining why you need to add these new compatibles would be great.
> diff --git a/Documentation/devicetree/bindings/sram/sunxi-sram.txt b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> index d087f04a4d7f..19cc0b892672 100644
> --- a/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> +++ b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> @@ -11,6 +11,9 @@ Controller Node
> Required properties:
> - compatible : should be:
> - "allwinner,sun4i-a10-sram-controller"
> + - "allwinner,sun5i-a13-sram-controller"
> + - "allwinner,sun7i-a20-sram-controller"
> + - "allwinner,sun8i-a33-sram-controller"
And I think Chen-Yu asked you to rename this compatible to
-system-controller for the previous iteration?
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
WARNING: multiple messages have this Message-ID (diff)
From: Maxime Ripard <maxime.ripard@bootlin.com>
To: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org,
linux-arm-kernel@lists.infradead.org,
linux-kernel@vger.kernel.org,
Mauro Carvalho Chehab <mchehab@kernel.org>,
Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>, Chen-Yu Tsai <wens@csie.org>,
Marco Franchi <marco.franchi@nxp.com>,
Icenowy Zheng <icenowy@aosc.io>,
Hans Verkuil <hverkuil@xs4all.nl>,
Sylwester Nawrocki <s.nawrocki@samsung.com>,
Tom Saeger <tom.saeger@oracle.com>,
Smitha T Murthy <smitha.t@samsung.com>,
Sakari Ailus <sakari.ailus@linux.intel.com>,
Andrzej Hajda <a.hajda@samsung.com>,
Jonathan Corbet <corbet@lwn.net>,
"David S . Miller" <davem@davemloft.net>,
Andrew Morton <akpm@linux-foundation.org>,
Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
Linus Walleij <linus.walleij@linaro.org>,
Randy Dunlap <rdunlap@infradead.org>,
Arnd Bergmann <arnd@arndb.de>,
Stanimir Varbanov <stanimir.varbanov@linaro.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Ramesh Shanmugasundaram <ramesh.shanmugasundaram@bp.renesas.com>,
Jacob Chen <jacob-chen@iotwrt.com>,
Steve Longerbeam <slongerbeam@gmail.com>,
Todor Tomov <todor.tomov@linaro.org>,
Jacopo Mondi <jacopo+renesas@jmondi.org>,
Alexandre Courbot <acourbot@chromium.org>,
Marek Szyprowski <m.szyprowski@samsung.com>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Tomasz Figa <tfiga@chromium.org>,
Ricardo Ribalda Delgado <ricardo.ribalda@gmail.com>,
Hans de Goede <hdegoede@redhat.com>,
Sami Tolvanen <samitolvanen@google.com>,
Thomas Petazzoni <thomas.petazzoni@bootlin.com>,
linux-sunxi@googlegroups.com,
Hugues Fruchet <hugues.fruchet@st.com>,
Randy Li <ayaka@soulik.info>
Subject: Re: [PATCH v4 01/19] dt-bindings: sram: sunxi: Add A13, A20 and A33 SRAM controller bindings
Date: Mon, 18 Jun 2018 18:02:50 +0200 [thread overview]
Message-ID: <20180618160250.ik44c3gcbdxylmyd@flea> (raw)
In-Reply-To: <20180618145843.14631-2-paul.kocialkowski@bootlin.com>
[-- Attachment #1: Type: text/plain, Size: 1143 bytes --]
On Mon, Jun 18, 2018 at 04:58:25PM +0200, Paul Kocialkowski wrote:
> This introduces dedicated bindings for the SRAM controllers found on the
> A13, A20 and A33 sunxi platforms.
>
> Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com>
Explaining why you need to add these new compatibles would be great.
> diff --git a/Documentation/devicetree/bindings/sram/sunxi-sram.txt b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> index d087f04a4d7f..19cc0b892672 100644
> --- a/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> +++ b/Documentation/devicetree/bindings/sram/sunxi-sram.txt
> @@ -11,6 +11,9 @@ Controller Node
> Required properties:
> - compatible : should be:
> - "allwinner,sun4i-a10-sram-controller"
> + - "allwinner,sun5i-a13-sram-controller"
> + - "allwinner,sun7i-a20-sram-controller"
> + - "allwinner,sun8i-a33-sram-controller"
And I think Chen-Yu asked you to rename this compatible to
-system-controller for the previous iteration?
Maxime
--
Maxime Ripard, Bootlin (formerly Free Electrons)
Embedded Linux and Kernel engineering
https://bootlin.com
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2018-06-18 16:02 UTC|newest]
Thread overview: 97+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-06-18 14:58 [PATCH v4 00/19] Sunxi-Cedrus driver for the Allwinner Video Engine, using media requests Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 01/19] dt-bindings: sram: sunxi: Add A13, A20 and A33 SRAM controller bindings Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 16:02 ` Maxime Ripard [this message]
2018-06-18 16:02 ` Maxime Ripard
2018-06-18 16:02 ` Maxime Ripard
2018-06-18 16:09 ` Chen-Yu Tsai
2018-06-18 16:09 ` Chen-Yu Tsai
2018-06-18 14:58 ` [PATCH v4 02/19] dt-bindings: sram: sunxi: Add A10 binding for the C1 SRAM region Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-20 15:59 ` Rob Herring
2018-06-20 15:59 ` Rob Herring
2018-06-20 15:59 ` Rob Herring
2018-06-18 14:58 ` [PATCH v4 03/19] dt-bindings: sram: sunxi: Add A13 " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 16:01 ` Maxime Ripard
2018-06-18 16:01 ` Maxime Ripard
2018-06-18 16:01 ` Maxime Ripard
2018-06-18 14:58 ` [PATCH v4 04/19] dt-bindings: sram: sunxi: Add A20 " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 16:04 ` Maxime Ripard
2018-06-18 16:04 ` Maxime Ripard
2018-06-18 16:04 ` Maxime Ripard
2018-06-18 14:58 ` [PATCH v4 05/19] dt-bindings: sram: sunxi: Add A33 " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-20 16:00 ` Rob Herring
2018-06-20 16:00 ` Rob Herring
2018-06-20 16:00 ` Rob Herring
2018-06-22 14:19 ` Maxime Ripard
2018-06-22 14:19 ` Maxime Ripard
2018-06-22 14:19 ` Maxime Ripard
2018-06-18 14:58 ` [PATCH v4 06/19] drivers: soc: sunxi: Add support " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 07/19] drivers: soc: sunxi: Add dedicated compatibles for the A13 and A33 Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 08/19] ARM: dts: sun5i: Use dedicated SRAM controller compatible Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 09/19] ARM: dts: sun7i-a20: Also use " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 10/19] ARM: sun5i: Add support for the C1 SRAM region with the SRAM controller Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 11/19] ARM: sun7i-a20: " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 12/19] ARM: sun8i-a33: Add SRAM controller node and C1 SRAM region Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-22 14:20 ` Maxime Ripard
2018-06-22 14:20 ` Maxime Ripard
2018-06-22 14:20 ` Maxime Ripard
2018-06-18 14:58 ` [PATCH v4 13/19] media: v4l: Add definitions for MPEG2 slice format and header metadata Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 14/19] media: v4l: Add definition for Allwinner's MB32-tiled NV12 format Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 15/19] dt-bindings: media: Document bindings for the Sunxi-Cedrus VPU driver Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-20 15:40 ` Rob Herring
2018-06-20 15:40 ` Rob Herring
2018-06-20 15:40 ` Rob Herring
2018-06-20 15:42 ` Paul Kocialkowski
2018-06-20 15:42 ` Paul Kocialkowski
2018-06-20 15:42 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 16/19] media: platform: Add Sunxi-Cedrus VPU decoder driver Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 19:37 ` kbuild test robot
2018-06-18 19:37 ` kbuild test robot
2018-07-10 1:41 ` [linux-sunxi] " Ezequiel Garcia
2018-07-10 1:41 ` Ezequiel Garcia
2018-07-10 1:41 ` Ezequiel Garcia
2018-07-24 14:28 ` Paul Kocialkowski
2018-07-24 14:28 ` Paul Kocialkowski
2018-07-24 14:28 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 17/19] ARM: dts: sun5i: Add Video Engine and reserved memory nodes Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 18/19] ARM: dts: sun7i-a20: " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` [PATCH v4 19/19] ARM: dts: sun8i-a33: " Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
2018-06-18 14:58 ` Paul Kocialkowski
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=20180618160250.ik44c3gcbdxylmyd@flea \
--to=maxime.ripard@bootlin.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 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.