mbox series

[0/3] RZ/Five: Enable ADC/CANFD/I2C/OPP/Thermal Zones/TSU

Message ID 20221115105135.1180490-1-prabhakar.mahadev-lad.rj@bp.renesas.com
Headers
Series RZ/Five: Enable ADC/CANFD/I2C/OPP/Thermal Zones/TSU |

Message

Lad, Prabhakar Nov. 15, 2022, 10:51 a.m. UTC 
  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

 arch/riscv/Kconfig                            |  2 ++
 arch/riscv/boot/dts/renesas/r9a07g043f.dtsi   |  2 ++
 .../boot/dts/renesas/rzfive-smarc-som.dtsi    | 11 --------
 arch/riscv/boot/dts/renesas/rzfive-smarc.dtsi | 27 -------------------
 4 files changed, 4 insertions(+), 38 deletions(-)

Comments

Conor Dooley Nov. 15, 2022, 2:20 p.m. UTC | #1 
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?

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,
Conor.

> 
>  arch/riscv/Kconfig                            |  2 ++
>  arch/riscv/boot/dts/renesas/r9a07g043f.dtsi   |  2 ++
>  .../boot/dts/renesas/rzfive-smarc-som.dtsi    | 11 --------
>  arch/riscv/boot/dts/renesas/rzfive-smarc.dtsi | 27 -------------------
>  4 files changed, 4 insertions(+), 38 deletions(-)
> 
> -- 
> 2.25.1
>
Lad, Prabhakar Nov. 15, 2022, 6:21 p.m. UTC | #2 
Hi Conor,

Thank you for the review.

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?

Cheers,
Prabhakar
Geert Uytterhoeven Nov. 16, 2022, 9:26 a.m. UTC | #3 
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
patchwork-bot+linux-riscv@kernel.org Nov. 17, 2022, 10 p.m. UTC | #4 
Hello:

This series was applied to riscv/linux.git (for-next)
by Palmer Dabbelt <palmer@rivosinc.com>:

On Tue, 15 Nov 2022 10:51:32 +0000 you 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
> 
> [...]

Here is the summary with links:
  - [1/3] riscv: Kconfig: Enable cpufreq kconfig menu
    https://git.kernel.org/riscv/c/effae0e3d9e1
  - [2/3] riscv: dts: renesas: r9a07g043f/rzfive-smarc-som: Enable ADC/OPP/Thermal Zones/TSU
    (no matching commit)
  - [3/3] riscv: dts: renesas: rzfive-smarc: Enable CANFD/I2C
    (no matching commit)

You are awesome, thank you!