[PATCH v3 5/6] docs: platform: Add documentation for Renesas RZ/Five SoC
Biju Das
biju.das.jz at bp.renesas.com
Thu Dec 1 01:13:22 PST 2022
Hi Prabhakar,
> Subject: Re: [PATCH v3 5/6] docs: platform: Add documentation for
> Renesas RZ/Five SoC
>
> Hi Biju,
>
> Thank you for the review.
>
> On Thu, Dec 1, 2022 at 8:33 AM Biju Das <biju.das.jz at bp.renesas.com>
> wrote:
> >
> > Hi Prabhakar,
> >
> > Prabhakar Mahadev Lad
> > > <prabhakar.mahadev-lad.rj at bp.renesas.com>
> > > Subject: [PATCH v3 5/6] docs: platform: Add documentation for
> > > Renesas RZ/Five SoC
> > >
> > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj at bp.renesas.com>
> > >
> > > This patch adds documentation to build Renesas RZ/Five (R9A07G043F)
> SoC.
> > >
> > > Signed-off-by: Lad Prabhakar
> > > <prabhakar.mahadev-lad.rj at bp.renesas.com>
> > > ---
> > > v2->v3
> > > * Updated doc with new build command
> > >
> > > RFC->v2
> > > * Updated doc
> > > ---
> > > docs/platform/platform.md | 5 ++
> > > docs/platform/renesas-rzfive.md | 145
> > > ++++++++++++++++++++++++++++++++
> > > 2 files changed, 150 insertions(+)
> > > create mode 100644 docs/platform/renesas-rzfive.md
> > >
> > > diff --git a/docs/platform/platform.md b/docs/platform/platform.md
> > > index
> > > f291931..4504d87 100644
> > > --- a/docs/platform/platform.md
> > > +++ b/docs/platform/platform.md
> > > @@ -39,6 +39,10 @@ OpenSBI currently supports the following virtual
> > > and hardware platforms:
> > > processor based SOCs. More details on this platform can be found
> > > in the
> > > file *[shakti_cclass.md]*.
> > >
> > > +* **Renesas RZ/Five SoC**: Platform support for Renesas RZ/Five
> > > +(R9A07G043F) SoC
> > > + used on the Renesas RZ/Five SMARC EVK board. More details on this
> > > +platform can
> > > + be found in the file *[renesas-rzfive.md]*.
> > > +
> > > The code for these supported platforms can be used as example to
> > > implement support for other platforms. The *platform/template*
> > > directory also provides template files for implementing support for
> > > a new platform. The *objects.mk*, @@ -54,3 +58,4 @@ comments to
> > > facilitate the implementation.
> > > [spike.md]: spike.md
> > > [fpga-openpiton.md]: fpga-openpiton.md
> > > [shakti_cclass.md]: shakti_cclass.md
> > > +[renesas-rzfive.md]: renesas-rzfive.md
> > > diff --git a/docs/platform/renesas-rzfive.md
> > > b/docs/platform/renesas- rzfive.md new file mode 100644 index
> > > 0000000..a900cc7
> > > --- /dev/null
> > > +++ b/docs/platform/renesas-rzfive.md
> > > @@ -0,0 +1,145 @@
> > > +Renesas RZ/Five SoC (R9A07G043F) Platform
> > > +=========================================
> > > +The RZ/Five microprocessor includes a single RISC-V CPU Core (Andes
> > > +AX45MP)
> > > +1.0 GHz, 16-bit DDR3L/DDR4 interface. Supported interfaces include:
> > > +- Gigabit Ethernet 2ch
> > > +- CAN interface (CAN-FD) 2ch
> > > +- USB 2.0 interface 2ch
> > > +- SD interface 2ch
> > > +- AD converter 2ch
> > > +making it ideal for applications such as entry-class social
> > > +infrastructure gateway control and industrial gateway control.
> > > +
> > > +To build platform specific library and firmwares, provide the
> > > +*PLATFORM=generic* parameter to the top level make command.
> > > +
> > > +Platform Options
> > > +----------------
> > > +
> > > +The Renesas RZ/Five platform does not have any platform-specific
> > > options.
> > > +
> > > +Building Renesas RZ/Five Platform
> > > +---------------------------------
> > > +
> > > +```
> > > +make PLATFORM=generic
> > > +```
> > > +
> > > +DTS Example: (RZ/Five AX45MP)
> > > +-----------------------------
> > > +
> > > +```
> > > + compatible = "renesas,r9a07g043";
> > > +
> > > + cpus {
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > + timebase-frequency = <12000000>;
> > > +
> > > + cpu0: cpu at 0 {
> > > + compatible = "andestech,ax45mp", "riscv";
> > > + device_type = "cpu";
> > > + reg = <0x0>;
> > > + status = "okay";
> > > + riscv,isa = "rv64imafdc";
> > > + mmu-type = "riscv,sv39";
> > > + i-cache-size = <0x8000>;
> > > + i-cache-line-size = <0x40>;
> > > + d-cache-size = <0x8000>;
> > > + d-cache-line-size = <0x40>;
> > > + clocks = <&cpg CPG_CORE
> R9A07G043_AX45MP_CORE0_CLK>,
> > > + <&cpg CPG_CORE
> > > + R9A07G043_AX45MP_ACLK>;
> >
> > Missing clock names.
> >
> > > + cpu0_intc: interrupt-controller {
> > > + #interrupt-cells = <1>;
> > > + compatible = "riscv,cpu-intc";
> > > + interrupt-controller;
> > > + };
> > > + };
> > > + };
> > > +
> > > + soc {
> > > + compatible = "simple-bus";
> > > + #address-cells = <1>;
> > > + #size-cells = <0>;
> > > + ranges;
> > > +
> > > + scif0: serial at 1004b800 {
> > > + compatible = "renesas,scif-r9a07g043",
> > > + "renesas,scif-r9a07g044";
> > > + reg = <0 0x1004b800 0 0x400>;
> > > + interrupts = <412 IRQ_TYPE_LEVEL_HIGH>,
> > > + <414 IRQ_TYPE_LEVEL_HIGH>,
> > > + <415 IRQ_TYPE_LEVEL_HIGH>,
> > > + <413 IRQ_TYPE_LEVEL_HIGH>,
> > > + <416 IRQ_TYPE_LEVEL_HIGH>,
> > > + <416 IRQ_TYPE_LEVEL_HIGH>;
> > > + interrupt-names = "eri", "rxi", "txi",
> > > + "bri", "dri", "tei";
> > > + clocks = <&cpg CPG_MOD
> R9A07G043_SCIF0_CLK_PCK>;
> > > + clock-names = "fck";
> > > + power-domains = <&cpg>;
> > > + resets = <&cpg R9A07G043_SCIF0_RST_SYSTEM_N>;
> > > + status = "disabled";
> > > + };
> > > +
> > > + cpg: clock-controller at 11010000 {
> > > + compatible = "renesas,r9a07g043-cpg";
> > > + reg = <0 0x11010000 0 0x10000>;
> > > + clocks = <&extal_clk>;
> > > + clock-names = "extal";
> > > + #clock-cells = <2>;
> > > + #reset-cells = <1>;
> > > + #power-domain-cells = <0>;
> > > + };
> > > +
> > > + sysc: system-controller at 11020000 {
> > > + compatible = "renesas,r9a07g043-sysc";
> > > + reg = <0 0x11020000 0 0x10000>;
> > > + status = "disabled";
> > > + };
> > > +
> > > + pinctrl: pinctrl at 11030000 {
> > > + compatible = "renesas,r9a07g043-pinctrl";
> > > + reg = <0 0x11030000 0 0x10000>;
> > > + gpio-controller;
> > > + #gpio-cells = <2>;
> > > + #interrupt-cells = <2>;
> > > + interrupt-controller;
> > > + gpio-ranges = <&pinctrl 0 0 152>;
> > > + clocks = <&cpg CPG_MOD R9A07G043_GPIO_HCLK>;
> > > + power-domains = <&cpg>;
> > > + resets = <&cpg R9A07G043_GPIO_RSTN>,
> > > + <&cpg R9A07G043_GPIO_PORT_RESETN>,
> > > + <&cpg R9A07G043_GPIO_SPARE_RESETN>;
> > Missing reset names.
> >
> Binding doesn't have reset names.
Maybe add here and upstream the same patch to Linux.
Cheers,
Biju
More information about the opensbi
mailing list