From: Laurent Pinchart <laurent.pinchart@ideasonboard.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-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Chen-Yu Tsai <wens@csie.org>, Jernej Skrabec <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org>, Sakari Ailus <sakari.ailus@linux.intel.com>, Hans Verkuil <hans.verkuil@cisco.com>, Maxime Ripard <mripard@kernel.org>, Thomas Petazzoni <thomas.petazzoni@bootlin.com>, Rob Herring <robh@kernel.org> Subject: Re: [PATCH v6 1/6] dt-bindings: media: Add Allwinner A31 ISP bindings documentation Date: Sat, 27 Aug 2022 00:12:59 +0300 [thread overview] Message-ID: <Ywk3W6pTOOlzLYVn@pendragon.ideasonboard.com> (raw) In-Reply-To: <20220826184144.605605-2-paul.kocialkowski@bootlin.com> Hi Paul, Thank you for the patch. On Fri, Aug 26, 2022 at 08:41:39PM +0200, Paul Kocialkowski wrote: > This introduces YAML bindings documentation for the Allwinner A31 Image > Signal Processor (ISP). > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > .../media/allwinner,sun6i-a31-isp.yaml | 97 +++++++++++++++++++ > 1 file changed, 97 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > > diff --git a/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml b/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > new file mode 100644 > index 000000000000..2fda6e05e16c > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > @@ -0,0 +1,97 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/allwinner,sun6i-a31-isp.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Allwinner A31 Image Signal Processor Driver (ISP) Device Tree Bindings > + > +maintainers: > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > + > +properties: > + compatible: > + enum: > + - allwinner,sun6i-a31-isp > + - allwinner,sun8i-v3s-isp > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + items: > + - description: Bus Clock > + - description: Module Clock > + - description: DRAM Clock > + > + clock-names: > + items: > + - const: bus > + - const: mod > + - const: ram > + > + resets: > + maxItems: 1 > + > + ports: > + $ref: /schemas/graph.yaml#/properties/ports > + > + properties: > + port@0: > + $ref: /schemas/graph.yaml#/properties/port > + description: CSI0 input port > + > + port@1: > + $ref: /schemas/graph.yaml#/properties/port > + description: CSI1 input port > + > + anyOf: > + - required: > + - port@0 > + - required: > + - port@1 I'd still like to see all ports that exist in the hardware being mandatory. I assume at least one of the A31 and V3s has two connected ports in the SoC or you wouldn't declare them both here :-) Apart from that, this looks good. > + > +required: > + - compatible > + - reg > + - interrupts > + - clocks > + - clock-names > + - resets > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/clock/sun8i-v3s-ccu.h> > + #include <dt-bindings/reset/sun8i-v3s-ccu.h> > + > + isp: isp@1cb8000 { > + compatible = "allwinner,sun8i-v3s-isp"; > + reg = <0x01cb8000 0x1000>; > + interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&ccu CLK_BUS_CSI>, > + <&ccu CLK_CSI1_SCLK>, > + <&ccu CLK_DRAM_CSI>; > + clock-names = "bus", "mod", "ram"; > + resets = <&ccu RST_BUS_CSI>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + isp_in_csi0: endpoint { > + remote-endpoint = <&csi0_out_isp>; > + }; > + }; > + }; > + }; > + > +... -- Regards, Laurent Pinchart
WARNING: multiple messages have this Message-ID (diff)
From: Laurent Pinchart <laurent.pinchart@ideasonboard.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-sunxi@lists.linux.dev, linux-kernel@vger.kernel.org, linux-staging@lists.linux.dev, Mauro Carvalho Chehab <mchehab@kernel.org>, Rob Herring <robh+dt@kernel.org>, Krzysztof Kozlowski <krzk+dt@kernel.org>, Chen-Yu Tsai <wens@csie.org>, Jernej Skrabec <jernej.skrabec@gmail.com>, Samuel Holland <samuel@sholland.org>, Sakari Ailus <sakari.ailus@linux.intel.com>, Hans Verkuil <hans.verkuil@cisco.com>, Maxime Ripard <mripard@kernel.org>, Thomas Petazzoni <thomas.petazzoni@bootlin.com>, Rob Herring <robh@kernel.org> Subject: Re: [PATCH v6 1/6] dt-bindings: media: Add Allwinner A31 ISP bindings documentation Date: Sat, 27 Aug 2022 00:12:59 +0300 [thread overview] Message-ID: <Ywk3W6pTOOlzLYVn@pendragon.ideasonboard.com> (raw) In-Reply-To: <20220826184144.605605-2-paul.kocialkowski@bootlin.com> Hi Paul, Thank you for the patch. On Fri, Aug 26, 2022 at 08:41:39PM +0200, Paul Kocialkowski wrote: > This introduces YAML bindings documentation for the Allwinner A31 Image > Signal Processor (ISP). > > Signed-off-by: Paul Kocialkowski <paul.kocialkowski@bootlin.com> > Reviewed-by: Rob Herring <robh@kernel.org> > --- > .../media/allwinner,sun6i-a31-isp.yaml | 97 +++++++++++++++++++ > 1 file changed, 97 insertions(+) > create mode 100644 Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > > diff --git a/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml b/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > new file mode 100644 > index 000000000000..2fda6e05e16c > --- /dev/null > +++ b/Documentation/devicetree/bindings/media/allwinner,sun6i-a31-isp.yaml > @@ -0,0 +1,97 @@ > +# SPDX-License-Identifier: (GPL-2.0 OR BSD-2-Clause) > +%YAML 1.2 > +--- > +$id: http://devicetree.org/schemas/media/allwinner,sun6i-a31-isp.yaml# > +$schema: http://devicetree.org/meta-schemas/core.yaml# > + > +title: Allwinner A31 Image Signal Processor Driver (ISP) Device Tree Bindings > + > +maintainers: > + - Paul Kocialkowski <paul.kocialkowski@bootlin.com> > + > +properties: > + compatible: > + enum: > + - allwinner,sun6i-a31-isp > + - allwinner,sun8i-v3s-isp > + > + reg: > + maxItems: 1 > + > + interrupts: > + maxItems: 1 > + > + clocks: > + items: > + - description: Bus Clock > + - description: Module Clock > + - description: DRAM Clock > + > + clock-names: > + items: > + - const: bus > + - const: mod > + - const: ram > + > + resets: > + maxItems: 1 > + > + ports: > + $ref: /schemas/graph.yaml#/properties/ports > + > + properties: > + port@0: > + $ref: /schemas/graph.yaml#/properties/port > + description: CSI0 input port > + > + port@1: > + $ref: /schemas/graph.yaml#/properties/port > + description: CSI1 input port > + > + anyOf: > + - required: > + - port@0 > + - required: > + - port@1 I'd still like to see all ports that exist in the hardware being mandatory. I assume at least one of the A31 and V3s has two connected ports in the SoC or you wouldn't declare them both here :-) Apart from that, this looks good. > + > +required: > + - compatible > + - reg > + - interrupts > + - clocks > + - clock-names > + - resets > + > +additionalProperties: false > + > +examples: > + - | > + #include <dt-bindings/interrupt-controller/arm-gic.h> > + #include <dt-bindings/clock/sun8i-v3s-ccu.h> > + #include <dt-bindings/reset/sun8i-v3s-ccu.h> > + > + isp: isp@1cb8000 { > + compatible = "allwinner,sun8i-v3s-isp"; > + reg = <0x01cb8000 0x1000>; > + interrupts = <GIC_SPI 83 IRQ_TYPE_LEVEL_HIGH>; > + clocks = <&ccu CLK_BUS_CSI>, > + <&ccu CLK_CSI1_SCLK>, > + <&ccu CLK_DRAM_CSI>; > + clock-names = "bus", "mod", "ram"; > + resets = <&ccu RST_BUS_CSI>; > + > + ports { > + #address-cells = <1>; > + #size-cells = <0>; > + > + port@0 { > + reg = <0>; > + > + isp_in_csi0: endpoint { > + remote-endpoint = <&csi0_out_isp>; > + }; > + }; > + }; > + }; > + > +... -- Regards, Laurent Pinchart _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2022-08-26 21:13 UTC|newest] Thread overview: 38+ messages / expand[flat|nested] mbox.gz Atom feed top 2022-08-26 18:41 [PATCH v6 0/6] Allwinner A31/A83T MIPI CSI-2 and A31 ISP / ISP Driver Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 18:41 ` [PATCH v6 1/6] dt-bindings: media: Add Allwinner A31 ISP bindings documentation Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 21:12 ` Laurent Pinchart [this message] 2022-08-26 21:12 ` Laurent Pinchart 2022-09-01 15:03 ` Paul Kocialkowski 2022-09-01 15:03 ` Paul Kocialkowski 2022-09-01 18:12 ` Laurent Pinchart 2022-09-01 18:12 ` Laurent Pinchart 2022-08-26 18:41 ` [PATCH v6 2/6] dt-bindings: media: sun6i-a31-csi: Add ISP output port Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 22:29 ` Laurent Pinchart 2022-08-26 22:29 ` Laurent Pinchart 2022-09-01 14:57 ` Paul Kocialkowski 2022-09-01 14:57 ` Paul Kocialkowski 2022-08-26 18:41 ` [PATCH v6 3/6] staging: media: Add support for the Allwinner A31 ISP Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 18:41 ` [PATCH v6 4/6] MAINTAINERS: Add entry for the Allwinner A31 ISP driver Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 22:30 ` Laurent Pinchart 2022-08-26 22:30 ` Laurent Pinchart 2022-08-26 18:41 ` [PATCH v6 5/6] media: sun6i-csi: Detect the availability of the ISP Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 22:39 ` Laurent Pinchart 2022-08-26 22:39 ` Laurent Pinchart 2022-09-01 15:11 ` Paul Kocialkowski 2022-09-01 15:11 ` Paul Kocialkowski 2022-09-01 18:14 ` Laurent Pinchart 2022-09-01 18:14 ` Laurent Pinchart 2022-08-26 18:41 ` [PATCH v6 6/6] media: sun6i-csi: Add support for hooking to the isp devices Paul Kocialkowski 2022-08-26 18:41 ` Paul Kocialkowski 2022-08-26 22:44 ` Laurent Pinchart 2022-08-26 22:44 ` Laurent Pinchart 2022-09-01 14:55 ` Paul Kocialkowski 2022-09-01 14:55 ` Paul Kocialkowski 2022-09-01 18:17 ` Laurent Pinchart 2022-09-01 18:17 ` Laurent Pinchart
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=Ywk3W6pTOOlzLYVn@pendragon.ideasonboard.com \ --to=laurent.pinchart@ideasonboard.com \ --cc=devicetree@vger.kernel.org \ --cc=hans.verkuil@cisco.com \ --cc=jernej.skrabec@gmail.com \ --cc=krzk+dt@kernel.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=linux-staging@lists.linux.dev \ --cc=linux-sunxi@lists.linux.dev \ --cc=mchehab@kernel.org \ --cc=mripard@kernel.org \ --cc=paul.kocialkowski@bootlin.com \ --cc=robh+dt@kernel.org \ --cc=robh@kernel.org \ --cc=sakari.ailus@linux.intel.com \ --cc=samuel@sholland.org \ --cc=thomas.petazzoni@bootlin.com \ --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.