Adding a new platform: renesas rz

[Robert Marko]

On Tue, 31 Oct 2023 at 09:56, Michele Bisogno <micbis.openwrt at gmail.com> wrote:
>
> Hi Robert,
>
> thanks for the encouragement.
> I'll go for plain "renesas" then.
>
> I am a bit hesitating on when / how to submit patches.
> At the moment the image generated is for the sd card (similar to
> Raspberry Pi) and for a single device (RZ/G2L)
> However I am planning to support more boot memories (definitely QSPI
> NOR) and more device from the same "family", i.e. very similar and
> from OpenWRT point of view probably it's just a device tree change.
>
> Shall I focus on one device first and later on submit the rest?
> I am still a newbie, I might discover that things should have been
> done in a specific way only after the first review, that I know it
> will be a disaster... :)

To me, the most sense would be to provide one or 2 boards as part of the initial
target but with multiple boot options supported so later boards can be
easily added.

Personally, GH PR-s are rather good for adding new stuff as it makes
commenting on
things much easier and cleaner, and we can let CI do its thing as well.

Regards,
Robert
>
> Regards,
> Michele
>
> On Mon, 30 Oct 2023 at 21:40, Robert Marko <robimarko at gmail.com> wrote:
> >
> > On Mon, 30 Oct 2023 at 10:40, Michele Bisogno <micbis.openwrt at gmail.com> wrote:
> > >
> > > Hi,
> > >
> > > I've been a happy OpenWRT user for many years now, always buying
> > > routers that could allow me to run it easily.
> >
> > Hi, nice to hear this.
> >
> > >
> > > I've been working (actually only in my free time as a hobby) on
> > > porting OpenWRT onto this Renesas board:
> > > https://www.renesas.com/eu/en/products/microcontrollers-microprocessors/rz-mpus/rzg2l-evkit-rzg2l-evaluation-board-kit
> > > which features this Arm Cortex-A55 based device:
> > > https://www.renesas.com/eu/en/products/microcontrollers-microprocessors/rz-mpus/rzg2l-general-purpose-microprocessors-dual-core-arm-cortex-a55-12-ghz-cpus-and-single-core-arm-cortex-m33
> > >
> > > I guess this is a new platform, since it does not really fit in any of
> > > the already supported ones.
> > >
> > > I got it working and I am polishing the patch before submitting it.
> > > The support of this device is already mainlined in the Linux kernel,
> > > so I am using 6.1 as is with the appropriate config.
> > > Arm Trusted Firmware and u-boot are instead taken from the respective
> > > Renesas forks, available on github.
> >
> > Great, these Reneas SoC-s are rather well supported in respective
> > upstream projects.
> >
> > >
> > > However this is the first time I do that, and I am still reading and
> > > learning, so I would like to get opinions, hints, why not constructive
> > > criticism and hopefully encouragement.
> >
> > I definitively encourage people to contribute towards more HW support,
> > especially HW which is mostly upstream.
> >
> > >
> > > For example, naming (renesas? renesas-rz? rz?) and structure of the
> > > subfolders. There are other variants I would like to add: RZ/G2LC,
> > > RZ/G2UL and maybe others.
> >
> > To me, just "renesas" sounds good and then you can introduce the individual
> > SoC families as subtargets.
> >
> > Regards,
> > Robert
> > >
> > > System
> > >
> > > HostnameOpenWrt
> > > ModelRenesas SMARC EVK based on r9a07g044l2
> > > ArchitectureARMv8 Processor rev 0
> > > Target Platformrz/g2l
> > > Firmware VersionOpenWrt SNAPSHOT r24187-bb8fd41f9a / LuCI Master
> > > git-23.294.26440-30a8a0d
> > > Kernel Version6.1.59
> > > Local Time2023-10-29 07:44:17
> > > Uptime0h 9m 11s
> > > Load Average0.13, 0.08, 0.05
> > >
> > > _______________________________________________
> > > openwrt-devel mailing list
> > > openwrt-devel at lists.openwrt.org
> > > https://lists.openwrt.org/mailman/listinfo/openwrt-devel