Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
MIME-Version: 1.0
References: <20221115105135.1180490-1-prabhakar.mahadev-lad.rj@bp.renesas.com>
 <Y3OgSAnst0qt5WSk@wendy> <CA+V-a8vL5r-xMeCiYRax-dM2Uo0ghgzRiyY99nNn089CjJwoBg@mail.gmail.com>
In-Reply-To: <CA+V-a8vL5r-xMeCiYRax-dM2Uo0ghgzRiyY99nNn089CjJwoBg@mail.gmail.com>
From:   Geert Uytterhoeven <geert@linux-m68k.org>
Date:   Wed, 16 Nov 2022 10:26:37 +0100
Message-ID: <CAMuHMdXquU632vu37WkewmashjSxV8f8J6WGawAD4uL3B=6pHA@mail.gmail.com>
Subject: Re: [PATCH 0/3] RZ/Five: Enable ADC/CANFD/I2C/OPP/Thermal Zones/TSU
To:     "Lad, Prabhakar" <prabhakar.csengg@gmail.com>
Cc:     Conor Dooley <conor.dooley@microchip.com>,
        Paul Walmsley <paul.walmsley@sifive.com>,
        Palmer Dabbelt <palmer@dabbelt.com>,
        Albert Ou <aou@eecs.berkeley.edu>,
        Magnus Damm <magnus.damm@gmail.com>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        Rob Herring <robh+dt@kernel.org>,
        linux-riscv@lists.infradead.org, linux-kernel@vger.kernel.org,
        linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org,
        Biju Das <biju.das.jz@bp.renesas.com>,
        Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
Content-Type: text/plain; charset="UTF-8"
Precedence: bulk

On Tue, Nov 15, 2022 at 7:23 PM Lad, Prabhakar
<prabhakar.csengg@gmail.com> wrote:
> On Tue, Nov 15, 2022 at 2:21 PM Conor Dooley <conor.dooley@microchip.com> wrote:
> > On Tue, Nov 15, 2022 at 10:51:32AM +0000, Prabhakar wrote:
> > > From: Lad Prabhakar <prabhakar.mahadev-lad.rj@bp.renesas.com>
> > >
> > > Hi All,
> > >
> > > This patch series aims to enable support for below blocks
> > > on RZ/Five SoC/SMARC EVK:
> > > - ADC
> > > - CANFD
> > > - I2C
> > > - OPP
> > > - Thermal Zones
> > > - TSU
> > >
> > > Note, patches apply on top of [0].
> > >
> > > [0] https://git.kernel.org/pub/scm/linux/kernel/git/geert/renesas-devel.git/log/?h=renesas-riscv-dt-for-v6.2
> > >
> > > Cheers,
> > > Prabhakar
> > >
> > > Lad Prabhakar (3):
> > >   riscv: Kconfig: Enable cpufreq kconfig menu
> > >   riscv: dts: renesas: r9a07g043f/rzfive-smarc-som: Enable
> > >     ADC/OPP/Thermal Zones/TSU
> > >   riscv: dts: renesas: rzfive-smarc: Enable CANFD/I2C
> >
> > I know ~nothing about your SoC so idk if the values are correct, but I
> > did give it a go earlier to see if it did anything warning wise. Seeing
> > that it didn't cause any I am curious - how come these didn't land in
> > the original dts? Just waiting for driver stuff to land to support it?
> >
> I wanted bare minimal stuff in the initial patchset.
>
> > Anyway, no new warnings which is what I care about - I suppose that
> > makes it an acked-by?
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> > Can include that tag if you like. Either way, nice to see some of the
> > /delete-node/s etc being removed.
> >
> Thanks, Geert should pick the tag while picking the DTS/I patches.
>
> For the Kconfig patch it needs to go via the RISCV tree or can that be
> picked by Geert too with the rest of the other patches?

IMHO the Kconfig patch (or Conor's earlier version) should go through
the RISCV tree, as there is no hard dependency.

Yes, I can take it with an Acked-by, if all else fails, but the soc deadline
is near.

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