From: Paul Kocialkowski <paul.kocialkowski@bootlin.com> To: Priit Laes <plaes@plaes.org> Cc: linux-media@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Icenowy Zheng <icenowy@aosc.xyz>, Florent Revest <revestflo@gmail.com>, Alexandre Courbot <acourbot@chromium.org>, Hans Verkuil <hans.verkuil@cisco.com>, Laurent Pinchart <laurent.pinchart@ideasonboard.com>, Sakari Ailus <sakari.ailus@linux.intel.com>, Maxime Ripard <maxime.ripard@bootlin.com>, Thomas van Kleef <thomas@vitsch.nl>, "Signed-off-by : Bob Ham" <rah@settrans.net>, Thomas Petazzoni <thomas.petazzoni@bootlin.com>, Chen-Yu Tsai <wens@csie.org> Subject: Re: [linux-sunxi] [PATCH 6/9] sunxi-cedrus: Add device tree binding document Date: Fri, 09 Mar 2018 14:45:46 +0100 [thread overview] Message-ID: <1520603146.15946.4.camel@bootlin.com> (raw) In-Reply-To: <20180309133857.GA20392@solar> [-- Attachment #1: Type: text/plain, Size: 2932 bytes --] Hi, Thanks for the review! On Fri, 2018-03-09 at 15:38 +0200, Priit Laes wrote: > On Fri, Mar 09, 2018 at 11:14:42AM +0100, Paul Kocialkowski wrote: > > From: Florent Revest <florent.revest@free-electrons.com> > > > > Device Tree bindings for the Allwinner's video engine > > > > Signed-off-by: Florent Revest <florent.revest@free-electrons.com> > > --- > > .../devicetree/bindings/media/sunxi-cedrus.txt | 44 > > ++++++++++++++++++++++ > > 1 file changed, 44 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt > > > > diff --git a/Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt b/Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt > > new file mode 100644 > > index 000000000000..138581113c49 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/sunxi-cedrus.txt > > @@ -0,0 +1,44 @@ > > +Device-Tree bindings for SUNXI video engine found in sunXi SoC > > family > > + > > +Required properties: > > +- compatible : "allwinner,sun4i-a10-video-engine"; > > +- memory-region : DMA pool for buffers allocation; > > +- clocks : list of clock specifiers, corresponding to > > + entries in clock-names property; > > +- clock-names : should contain "ahb", "mod" and "ram" > > entries; > > +- resets : phandle for reset; > > +- interrupts : should contain VE interrupt number; > > +- reg : should contain register base and length > > of VE. > > + > > +Example: > > + > > +reserved-memory { > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges; > > + > > + ve_reserved: cma { > > + compatible = "shared-dma-pool"; > > + reg = <0x43d00000 0x9000000>; > > + no-map; > > + linux,cma-default; > > + }; > > +}; > > + > > +video-engine { > > + compatible = "allwinner,sun4i-a10-video-engine"; > > + memory-region = <&ve_reserved>; > > + > > + clocks = <&ahb_gates 32>, <&ccu CLK_VE>, > > + <&dram_gates 0>; > > This should be updated to sunxi-ng clocks: > > clocks = <&ccu CLK_BUS_VE>, <&ccu CLK_VE>, <&ccu CLK_DRAM_VE>; I will definitely keep that in mind and make the change for the next revision, thanks! > > + clock-names = "ahb", "mod", "ram"; > > + > > + assigned-clocks = <&ccu CLK_VE>; > > + assigned-clock-rates = <320000000>; > > + > > + resets = <&ccu RST_VE>; > > + > > + interrupts = <53>; > > + > > + reg = <0x01c0e000 4096>; > > +}; > > -- > > 2.16.2 > > > > -- > > You received this message because you are subscribed to the Google > > Groups "linux-sunxi" group. > > To unsubscribe from this group and stop receiving emails from it, > > send an email to linux-sunxi+unsubscribe@googlegroups.com. > > For more options, visit https://groups.google.com/d/optout. -- Paul Kocialkowski, Bootlin (formerly Free Electrons) Embedded Linux and kernel engineering https://bootlin.com [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 488 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: paul.kocialkowski@bootlin.com (Paul Kocialkowski) To: linux-arm-kernel@lists.infradead.org Subject: [linux-sunxi] [PATCH 6/9] sunxi-cedrus: Add device tree binding document Date: Fri, 09 Mar 2018 14:45:46 +0100 [thread overview] Message-ID: <1520603146.15946.4.camel@bootlin.com> (raw) In-Reply-To: <20180309133857.GA20392@solar> Hi, Thanks for the review! On Fri, 2018-03-09 at 15:38 +0200, Priit Laes wrote: > On Fri, Mar 09, 2018 at 11:14:42AM +0100, Paul Kocialkowski wrote: > > From: Florent Revest <florent.revest@free-electrons.com> > > > > Device Tree bindings for the Allwinner's video engine > > > > Signed-off-by: Florent Revest <florent.revest@free-electrons.com> > > --- > > .../devicetree/bindings/media/sunxi-cedrus.txt | 44 > > ++++++++++++++++++++++ > > 1 file changed, 44 insertions(+) > > create mode 100644 Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt > > > > diff --git a/Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt b/Documentation/devicetree/bindings/media/sunxi- > > cedrus.txt > > new file mode 100644 > > index 000000000000..138581113c49 > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/media/sunxi-cedrus.txt > > @@ -0,0 +1,44 @@ > > +Device-Tree bindings for SUNXI video engine found in sunXi SoC > > family > > + > > +Required properties: > > +- compatible : "allwinner,sun4i-a10-video-engine"; > > +- memory-region : DMA pool for buffers allocation; > > +- clocks : list of clock specifiers, corresponding to > > + entries in clock-names property; > > +- clock-names : should contain "ahb", "mod" and "ram" > > entries; > > +- resets : phandle for reset; > > +- interrupts : should contain VE interrupt number; > > +- reg : should contain register base and length > > of VE. > > + > > +Example: > > + > > +reserved-memory { > > + #address-cells = <1>; > > + #size-cells = <1>; > > + ranges; > > + > > + ve_reserved: cma { > > + compatible = "shared-dma-pool"; > > + reg = <0x43d00000 0x9000000>; > > + no-map; > > + linux,cma-default; > > + }; > > +}; > > + > > +video-engine { > > + compatible = "allwinner,sun4i-a10-video-engine"; > > + memory-region = <&ve_reserved>; > > + > > + clocks = <&ahb_gates 32>, <&ccu CLK_VE>, > > + <&dram_gates 0>; > > This should be updated to sunxi-ng clocks: > > clocks = <&ccu CLK_BUS_VE>, <&ccu CLK_VE>, <&ccu CLK_DRAM_VE>; I will definitely keep that in mind and make the change for the next revision, thanks! > > + clock-names = "ahb", "mod", "ram"; > > + > > + assigned-clocks = <&ccu CLK_VE>; > > + assigned-clock-rates = <320000000>; > > + > > + resets = <&ccu RST_VE>; > > + > > + interrupts = <53>; > > + > > + reg = <0x01c0e000 4096>; > > +}; > > -- > > 2.16.2 > > > > -- > > You received this message because you are subscribed to the Google > > Groups "linux-sunxi" group. > > To unsubscribe from this group and stop receiving emails from it, > > send an email to linux-sunxi+unsubscribe at googlegroups.com. > > For more options, visit https://groups.google.com/d/optout. -- Paul Kocialkowski, 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: 488 bytes Desc: This is a digitally signed message part URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20180309/aa35cdbb/attachment-0001.sig>
next prev parent reply other threads:[~2018-03-09 13:46 UTC|newest] Thread overview: 58+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-03-09 10:09 [PATCH 0/9] Sunxi-Cedrus driver for the Allwinner Video Engine, using the V4L2 request API Paul Kocialkowski 2018-03-09 10:09 ` Paul Kocialkowski 2018-03-09 10:09 ` [PATCH 1/9] media: vim2m: Try to schedule a m2m device run on request submission Paul Kocialkowski 2018-03-09 10:09 ` Paul Kocialkowski 2018-03-09 10:09 ` Paul Kocialkowski 2018-03-09 10:09 ` [PATCH 2/9] media: videobuf2-v4l2: Copy planes when needed in request qbuf Paul Kocialkowski 2018-03-09 10:09 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 3/9] v4l: Add sunxi Video Engine pixel format Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 4/9] v4l: Add MPEG2 low-level decoder API control Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 5/9] media: platform: Add Sunxi Cedrus decoder driver Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 13:57 ` Maxime Ripard 2018-03-09 13:57 ` Maxime Ripard 2018-03-09 14:25 ` Paul Kocialkowski 2018-03-09 14:25 ` Paul Kocialkowski 2018-04-19 14:58 ` Paul Kocialkowski 2018-04-19 14:58 ` Paul Kocialkowski 2018-04-19 14:58 ` Paul Kocialkowski 2018-03-12 17:15 ` [linux-sunxi] " Joonas Kylmälä 2018-03-12 17:15 ` Joonas Kylmälä 2018-03-12 17:15 ` Joonas Kylmälä 2018-04-19 14:56 ` [linux-sunxi] " Paul Kocialkowski 2018-04-19 14:56 ` Paul Kocialkowski 2018-04-19 14:56 ` Paul Kocialkowski 2018-03-12 20:29 ` [linux-sunxi] " Joonas Kylmälä 2018-03-12 20:29 ` Joonas Kylmälä 2018-03-12 20:29 ` Joonas Kylmälä 2018-04-19 14:56 ` [linux-sunxi] " Paul Kocialkowski 2018-04-19 14:56 ` Paul Kocialkowski 2018-04-19 14:56 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 6/9] sunxi-cedrus: Add device tree binding document Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 13:38 ` [linux-sunxi] " Priit Laes 2018-03-09 13:38 ` Priit Laes 2018-03-09 13:38 ` Priit Laes 2018-03-09 13:45 ` Paul Kocialkowski [this message] 2018-03-09 13:45 ` [linux-sunxi] " Paul Kocialkowski 2018-03-18 12:48 ` Rob Herring 2018-03-18 12:48 ` Rob Herring 2018-03-18 12:48 ` Rob Herring 2018-04-19 14:55 ` Paul Kocialkowski 2018-04-19 14:55 ` Paul Kocialkowski 2018-04-19 14:55 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 7/9] ARM: dts: sun5i: Use video-engine node Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 8/9] ARM: dts: sun8i: add video engine support for A33 Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-09 10:14 ` [PATCH 9/9] ARM: dts: sun7i: Add video engine support for the A20 Paul Kocialkowski 2018-03-09 10:14 ` Paul Kocialkowski 2018-03-12 18:18 ` [linux-sunxi] [PATCH 2/9] media: videobuf2-v4l2: Copy planes when needed in request qbuf Joonas Kylmälä 2018-03-12 18:18 ` Joonas Kylmälä 2018-03-12 18:18 ` Joonas Kylmälä 2018-03-09 10:18 ` [PATCH 0/9] Sunxi-Cedrus driver for the Allwinner Video Engine, using the V4L2 request API Paul Kocialkowski 2018-03-09 10:18 ` 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=1520603146.15946.4.camel@bootlin.com \ --to=paul.kocialkowski@bootlin.com \ --cc=acourbot@chromium.org \ --cc=devicetree@vger.kernel.org \ --cc=hans.verkuil@cisco.com \ --cc=icenowy@aosc.xyz \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linux-sunxi@googlegroups.com \ --cc=maxime.ripard@bootlin.com \ --cc=plaes@plaes.org \ --cc=rah@settrans.net \ --cc=revestflo@gmail.com \ --cc=sakari.ailus@linux.intel.com \ --cc=thomas.petazzoni@bootlin.com \ --cc=thomas@vitsch.nl \ --cc=wens@csie.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: linkBe 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.