Linux-Renesas-SoC Archive mirror
 help / color / mirror / Atom feed
From: Geert Uytterhoeven <geert@linux-m68k.org>
To: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>
Cc: geert+renesas@glider.be, magnus.damm@gmail.com,
	 linux-renesas-soc@vger.kernel.org
Subject: Re: [PATCH 1/2] arm64: dts: renesas: r8a779g0: Add PCIe Host and Endpoint nodes
Date: Thu, 14 Mar 2024 14:56:20 +0100	[thread overview]
Message-ID: <CAMuHMdXWsnukKDjfQ4oHsXOVPiF+18odkURGP+9BWp4XN1DU0A@mail.gmail.com> (raw)
In-Reply-To: <20240229120741.2553702-2-yoshihiro.shimoda.uh@renesas.com>

Hi Shimoda-san,

On Thu, Feb 29, 2024 at 1:07 PM Yoshihiro Shimoda
<yoshihiro.shimoda.uh@renesas.com> wrote:
> Add PCIe Host and Endpoint nodes for R-Car V4H (R8A779G0).
>
> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@renesas.com>

Thanks for your patch!

> --- a/arch/arm64/boot/dts/renesas/r8a779g0.dtsi
> +++ b/arch/arm64/boot/dts/renesas/r8a779g0.dtsi
> @@ -723,6 +737,126 @@ hscif3: serial@e66a0000 {
>                         status = "disabled";
>                 };
>
> +               pciec0: pcie@e65d0000 {
> +                       compatible = "renesas,r8a779g0-pcie",
> +                                    "renesas,rcar-gen4-pcie";
> +                       reg = <0 0xe65d0000 0 0x1000>, <0 0xe65d2000 0 0x0800>,
> +                             <0 0xe65d3000 0 0x2000>, <0 0xe65d5000 0 0x1200>,
> +                             <0 0xe65d6200 0 0x0e00>, <0 0xe65d7000 0 0x0400>,
> +                             <0 0xfe000000 0 0x400000>;
> +                       reg-names = "dbi", "dbi2", "atu", "dma", "app", "phy", "config";
> +                       interrupts = <GIC_SPI 449 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 450 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 451 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 455 IRQ_TYPE_LEVEL_HIGH>;
> +                       interrupt-names = "msi", "dma", "sft_ce", "app";
> +                       clocks = <&cpg CPG_MOD 624>, <&pcie0_clkref>;
> +                       clock-names = "core", "ref";
> +                       power-domains = <&sysc R8A779G0_PD_ALWAYS_ON>;
> +                       resets = <&cpg 624>;
> +                       reset-names = "pwr";
> +                       max-link-speed = <4>;
> +                       num-lanes = <2>;
> +                       #address-cells = <3>;
> +                       #size-cells = <2>;
> +                       bus-range = <0x00 0xff>;
> +                       device_type = "pci";
> +                       ranges = <0x01000000 0 0x00000000 0 0xfe000000 0 0x00400000>,
> +                                <0x02000000 0 0x30000000 0 0x30000000 0 0x10000000>;
> +                       dma-ranges = <0x42000000 0 0x00000000 0 0x00000000 1 0x00000000>;
> +                       #interrupt-cells = <1>;
> +                       interrupt-map-mask = <0 0 0 7>;
> +                       interrupt-map = <0 0 0 1 &gic GIC_SPI 416 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 2 &gic GIC_SPI 416 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 3 &gic GIC_SPI 416 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 4 &gic GIC_SPI 416 IRQ_TYPE_LEVEL_HIGH>;

s/416/449 for all four lines.

> +                       snps,enable-cdm-check;
> +                       status = "disabled";
> +               };
> +
> +               pciec1: pcie@e65d8000 {
> +                       compatible = "renesas,r8a779g0-pcie",
> +                                    "renesas,rcar-gen4-pcie";
> +                       reg = <0 0xe65d8000 0 0x1000>, <0 0xe65da000 0 0x0800>,
> +                             <0 0xe65db000 0 0x2000>, <0 0xe65dd000 0 0x1200>,
> +                             <0 0xe65de200 0 0x0e00>, <0 0xe65df000 0 0x0400>,
> +                             <0 0xee900000 0 0x400000>;
> +                       reg-names = "dbi", "dbi2", "atu", "dma", "app", "phy", "config";
> +                       interrupts = <GIC_SPI 456 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 457 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 458 IRQ_TYPE_LEVEL_HIGH>,
> +                                    <GIC_SPI 462 IRQ_TYPE_LEVEL_HIGH>;
> +                       interrupt-names = "msi", "dma", "sft_ce", "app";
> +                       clocks = <&cpg CPG_MOD 625>, <&pcie1_clkref>;
> +                       clock-names = "core", "ref";
> +                       power-domains = <&sysc R8A779G0_PD_ALWAYS_ON>;
> +                       resets = <&cpg 625>;
> +                       reset-names = "pwr";
> +                       max-link-speed = <4>;
> +                       num-lanes = <2>;
> +                       #address-cells = <3>;
> +                       #size-cells = <2>;
> +                       bus-range = <0x00 0xff>;
> +                       device_type = "pci";
> +                       ranges = <0x01000000 0 0x00000000 0 0xee900000 0 0x00400000>,
> +                                <0x02000000 0 0xc0000000 0 0xc0000000 0 0x10000000>;
> +                       dma-ranges = <0x42000000 0 0x00000000 0 0x00000000 1 0x00000000>;
> +                       #interrupt-cells = <1>;
> +                       interrupt-map-mask = <0 0 0 7>;
> +                       interrupt-map = <0 0 0 1 &gic GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 2 &gic GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 3 &gic GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>,
> +                                       <0 0 0 4 &gic GIC_SPI 423 IRQ_TYPE_LEVEL_HIGH>;

s/423/456 for all four lines.

> +                       snps,enable-cdm-check;
> +                       status = "disabled";
> +               };

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

  reply	other threads:[~2024-03-14 13:56 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-29 12:07 [PATCH 0/2] arm64: dts: renesas: r8a779g0: add PCIe support Yoshihiro Shimoda
2024-02-29 12:07 ` [PATCH 1/2] arm64: dts: renesas: r8a779g0: Add PCIe Host and Endpoint nodes Yoshihiro Shimoda
2024-03-14 13:56   ` Geert Uytterhoeven [this message]
2024-04-25  7:20     ` Geert Uytterhoeven
2024-04-25  7:34       ` Yoshihiro Shimoda
2024-04-25  7:56         ` Geert Uytterhoeven
2024-02-29 12:07 ` [PATCH 2/2] arm64: dts: renesas: white-hawk-cpu-common: Enable PCIe Host ch0 Yoshihiro Shimoda
2024-03-14 14:08   ` Geert Uytterhoeven

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=CAMuHMdXWsnukKDjfQ4oHsXOVPiF+18odkURGP+9BWp4XN1DU0A@mail.gmail.com \
    --to=geert@linux-m68k.org \
    --cc=geert+renesas@glider.be \
    --cc=linux-renesas-soc@vger.kernel.org \
    --cc=magnus.damm@gmail.com \
    --cc=yoshihiro.shimoda.uh@renesas.com \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).