Hello Sergey, > > + /* Report required division ratio setting for the MIPI clock > > +dividers > > /* should be on a line of its own (unless you do a networking patch)...
OK. > > + * pllclk/2 = vclk * DSI divider > > Why no spaces around / while there are spaces around *? I missed that one. Thank you. Chris -----Original Message----- From: Sergey Shtylyov <[email protected]> Sent: Thursday, July 10, 2025 4:45 AM To: Chris Brandt <[email protected]>; Geert Uytterhoeven <[email protected]>; Michael Turquette <[email protected]>; Stephen Boyd <[email protected]>; Biju Das <[email protected]>; Maarten Lankhorst <[email protected]>; Maxime Ripard <[email protected]>; Thomas Zimmermann <[email protected]>; David Airlie <[email protected]>; Simona Vetter <[email protected]>; Hien Huynh <[email protected]>; Nghia Vo <[email protected]>; Hugo Villeneuve <[email protected]> Cc: [email protected]; [email protected]; [email protected] Subject: Re: [PATCH 2/2] drm: renesas: rz-du: Set DSI divider based on target MIPI device On 7/9/25 11:55 PM, Chris Brandt wrote: > Before the MIPI DSI clock source can be configured, the target divide > ratio needs to be known. > > Signed-off-by: Chris Brandt <[email protected]> > --- > drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c | 17 +++++++++++++++++ > 1 file changed, 17 insertions(+) > > diff --git a/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c > b/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c > index 4ad0be03a5b4..a8796df43f75 100644 > --- a/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c > +++ b/drivers/gpu/drm/renesas/rz-du/rzg2l_mipi_dsi.c [...] > @@ -666,6 +667,22 @@ static int rzg2l_mipi_dsi_host_attach(struct > mipi_dsi_host *host, > > drm_bridge_add(&dsi->bridge); > > + /* Report required division ratio setting for the MIPI clock > +dividers /* should be on a line of its own (unless you do a networking patch)... > + * Assume the default clock source is FOUTPOSTDIV (PLL/2) being fed to > the DSI-PHY, but also > + * the DSI-PHY must be 16x the MIPI-DSI HS clock. > + * > + * pllclk/2 = vclk * DSI divider Why no spaces around / while there are spaces around *? [...] MBR, Sergey
