Re: [PATCH 3/3] clk: renesas: r8a779h0: Add CSI-2 clocks

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Geert,

Thanks for your review.

On 2024-05-29 17:09:09 +0200, Geert Uytterhoeven wrote:
> Hi Niklas,
> 
> On Mon, May 27, 2024 at 3:16 PM Niklas Söderlund
> <niklas.soderlund+renesas@xxxxxxxxxxxx> wrote:
> > Add the CSI40 and CSI41 module clocks, which are used by the CSI-2
> > interfaces on the Renesas R-Car V4M (R8A779H0) SoC.
> >
> > Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@xxxxxxxxxxxx>
> 
> Thanks for your patch!
> 
> > --- a/drivers/clk/renesas/r8a779h0-cpg-mssr.c
> > +++ b/drivers/clk/renesas/r8a779h0-cpg-mssr.c
> > @@ -176,6 +176,8 @@ static const struct mssr_mod_clk r8a779h0_mod_clks[] = {
> >         DEF_MOD("avb0:rgmii0",  211,    R8A779H0_CLK_S0D8_HSC),
> >         DEF_MOD("avb1:rgmii1",  212,    R8A779H0_CLK_S0D8_HSC),
> >         DEF_MOD("avb2:rgmii2",  213,    R8A779H0_CLK_S0D8_HSC),
> > +       DEF_MOD("csi40",        331,    R8A779H0_CLK_CSI),
> > +       DEF_MOD("csi41",        400,    R8A779H0_CLK_CSI),
> 
> According to the documentation for the Module Stop Control Registers
> 3 and 4, these are called "csitop[01]".

I noticed that too, the issue is how they are named on V4H and I thought 
aligning the Gen4 board was a good idea. I don't feel strongly about 
this, but before I send a v2.

On V4H the bits are named SRT31 and SRT0, while the "Target Module for 
Software Reset" lists them as "CSI40 (CSI-2-RX0)" and "CSI40 
(CSI-2-RX1)". The later is the same for V4H and V4M, while the bit name 
differs.

Should we rename the V4H modules as well, keep the names for V4M, or do 
we not really care the same modules have different names on V4H and V4M?

> 
> >         DEF_MOD("hscif0",       514,    R8A779H0_CLK_SASYNCPERD1),
> >         DEF_MOD("hscif1",       515,    R8A779H0_CLK_SASYNCPERD1),
> >         DEF_MOD("hscif2",       516,    R8A779H0_CLK_SASYNCPERD1),
> 
> Gr{oetje,eeting}s,
> 
>                         Geert
> 
> -- 
> Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx
> 
> 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

-- 
Kind Regards,
Niklas Söderlund




[Index of Archives]     [Linux Samsung SOC]     [Linux Wireless]     [Linux Kernel]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]

  Powered by Linux