From: Geert Uytterhoeven <geert@linux-m68k.org>
To: "Niklas Söderlund" <niklas.soderlund+renesas@ragnatech.se>
Cc: Linux-Renesas <linux-renesas-soc@vger.kernel.org>
Subject: Re: [PATCH v2] arm64: dts: renesas: r8a779a0: Add and connect all CSI-2, ISP and VIN nodes
Date: Tue, 12 Oct 2021 09:58:00 +0200 [thread overview]
Message-ID: <CAMuHMdVY+f22BAT7XBvNX3Nironhu8+vat=_ZOEJZHbSUeQpCQ@mail.gmail.com> (raw)
In-Reply-To: <20211010175227.3247895-1-niklas.soderlund+renesas@ragnatech.se>
Hi Niklas,
On Sun, Oct 10, 2021 at 7:52 PM Niklas Söderlund
<niklas.soderlund+renesas@ragnatech.se> wrote:
> The V3U have 32 VIN, 4 CSI-2 and 4 ISP nodes that interact with each
> other for video capture. Add all nodes and record how they are
> interconnected.
>
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
> ---
> * Changes since v1
> - Fix endpoint for vin00 to vin07 that where missing @0 and
> {address,size}-cells and reg properties.
> - Fix endpoint for all isp nodes that where missing @0 and
> {address,size}-cells and reg properties.
> - Update compatible for ISP from renesas,isp-r8a779a0 to
> renesas,r8a779a0-isp.
Thanks for the update!
> The last binding document for the CSI-2 nodes are now merged in the
> media-tree and this completes the coverage for the nodes in this patch.
>
> If you are happy with the corrections I made from v1 I think this is
> ready to be picked up.
The new version looks good to me, but I still get one complaint from
"make dtbs_check", cfr. below.
> --- a/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a779a0.dtsi
> @@ -1268,6 +2164,426 @@ vspd1: vsp@fea28000 {
> renesas,fcp = <&fcpvd1>;
> };
>
> + csi40: csi2@feaa0000 {
> + compatible = "renesas,r8a779a0-csi2";
> + reg = <0 0xfeaa0000 0 0x10000>;
> + interrupts = <GIC_SPI 192 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 331>;
> + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
> + resets = <&cpg 331>;
> + status = "disabled";
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@1 {
> + reg = <1>;
> + csi40isp0: endpoint {
> + remote-endpoint = <&isp0csi40>;
> + };
> + };
> + };
> + };
> +
> + csi41: csi2@feab0000 {
> + compatible = "renesas,r8a779a0-csi2";
> + reg = <0 0xfeab0000 0 0x10000>;
> + interrupts = <GIC_SPI 193 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 400>;
> + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
> + resets = <&cpg 400>;
> + status = "disabled";
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@1 {
> + reg = <1>;
> + csi41isp1: endpoint {
> + remote-endpoint = <&isp1csi41>;
> + };
> + };
arch/arm64/boot/dts/renesas/r8a779a0-falcon.dt.yaml: csi2@feab0000:
ports: 'port@0' is a required property
From schema: Documentation/devicetree/bindings/media/renesas,csi2.yaml
(only for this CSI instance).
> + };
> + };
> +
> + csi42: csi2@fed60000 {
> + compatible = "renesas,r8a779a0-csi2";
> + reg = <0 0xfed60000 0 0x10000>;
> + interrupts = <GIC_SPI 194 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 401>;
> + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
> + resets = <&cpg 401>;
> + status = "disabled";
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@1 {
> + reg = <1>;
> + csi42isp2: endpoint {
> + remote-endpoint = <&isp2csi42>;
> + };
> + };
> + };
> + };
> +
> + csi43: csi2@fed70000 {
> + compatible = "renesas,r8a779a0-csi2";
> + reg = <0 0xfed70000 0 0x10000>;
> + interrupts = <GIC_SPI 195 IRQ_TYPE_LEVEL_HIGH>;
> + clocks = <&cpg CPG_MOD 402>;
> + power-domains = <&sysc R8A779A0_PD_ALWAYS_ON>;
> + resets = <&cpg 402>;
> + status = "disabled";
> +
> + ports {
> + #address-cells = <1>;
> + #size-cells = <0>;
> +
> + port@1 {
> + reg = <1>;
> + csi43isp3: endpoint {
> + remote-endpoint = <&isp3csi43>;
> + };
> + };
> + };
> + };
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
next prev parent reply other threads:[~2021-10-12 7:58 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-10-10 17:52 [PATCH v2] arm64: dts: renesas: r8a779a0: Add and connect all CSI-2, ISP and VIN nodes Niklas Söderlund
2021-10-12 7:58 ` Geert Uytterhoeven [this message]
2021-10-12 9:22 ` Geert Uytterhoeven
2021-10-12 9:33 ` Niklas Söderlund
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='CAMuHMdVY+f22BAT7XBvNX3Nironhu8+vat=_ZOEJZHbSUeQpCQ@mail.gmail.com' \
--to=geert@linux-m68k.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=niklas.soderlund+renesas@ragnatech.se \
/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.