On Sun, Oct 30, 2022 at 10:37:01PM +0000, Lad, Prabhakar wrote: > Hi Conor, > > On Sun, Oct 30, 2022 at 6:24 PM Conor Dooley <conor@xxxxxxxxxx> wrote: > > > > On Fri, Oct 28, 2022 at 05:59:14PM +0100, Prabhakar wrote: > > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@xxxxxxxxxxxxxx> > > > > > > Hi All, > > > > > > The RZ/Five microprocessor includes a RISC-V CPU Core (AX45MP Single) > > > 1.0 GHz, 16-bit DDR3L/DDR4 interface. And it also has many interfaces such > > > as Gbit-Ether, CAN, and USB 2.0, making it ideal for applications such as > > > entry-class social infrastructure gateway control and industrial gateway > > > control. > > > > > > This patch series adds initial SoC DTSi support for Renesas RZ/Five > > > (R9A07G043) SoC. Below is the list of IP blocks enabled in the initial > > > board DTS which can be used to boot via initramfs on RZ/Five SMARC EVK: > > > - AX45MP CPU > > > - CPG > > > - PINCTRL > > > > Hey, > > Looks like you've got a pair of warnings here from dtbs_check. I tested > > this on top of 20221028's next, with the three branches below merged in, > > hopefully my merges aren't the source of them: > > > > linux/arch/riscv/boot/dts/renesas/r9a07g043f01-smarc.dtb: pinctrl@11030000: 'interrupt-controller' is a required property > > From schema: Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml > > linux/arch/riscv/boot/dts/renesas/r9a07g043f01-smarc.dtb: pinctrl@11030000: '#interrupt-cells' is a required property > > From schema: linux/Documentation/devicetree/bindings/pinctrl/renesas,rzg2l-pinctrl.yaml > > > Thanks for the review and test. The warnings above are coming from [0] > as support for IRQC is missing, once that is added the warnings should > go away. Right. I merged in the stuff in Geert's trees & I don't think I saw any pending patches in the cover that I missed. Is there something else that adds the support that's not going to make v6.2? I got rid of all the dtbs_check warnings for v6.1 and I'd really like to keep things that way! > > With this sorted, whatever wasn't already is now: > > Reviewed-by: Conor Dooley <conor.dooley@xxxxxxxxxxxxx> > > Thanks for putting up with my messing around re: kconfig symbols and I > > am glad that we ended up being able to share the dts across archs in the > > end, so thanks to everyone involved in that :) > > > :-) > > [0] https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/tree/arch/arm64/boot/dts/renesas/r9a07g043.dtsi > > Cheers, > Prabhakar
