Hi Maxime, On 2023-03-21 at 15:57:39 +0100, Maxime Ripard <maxime@xxxxxxxxxx> wrote: > Hi, > > On Sun, Mar 19, 2023 at 05:07:04PM +0100, Frank Oltmanns wrote: >> Set the required PLL rate by adjusting the dotclock rate when calling >> clk_set_rate() when using DSI. >> >> According to the Allwinners A64’s BSP code, a TCON divider of 4 has to >> be used and the PLL rate needs to be set to the following frequency when >> using DSI: >> PLL rate = DCLK * bpp / lanes >> >> After this change the common mode set function would only contain >> setting the resolution. Therefore, dissolve the function and transfer >> the functionality to the individual mode set functions. >> >> Signed-off-by: Frank Oltmanns <frank@xxxxxxxxxxxx> > > This is similar to: > <https://lore.kernel.org/all/20230320161636.24411-1-romanberanek@xxxxxxxxxx/> > > What’s the story there? Sorry, as Roman wrote in the other thread, I submitted the patch after not hearing back from him for a week. My apologies, I wasn’t patient enough. So now there are two submissions to consider. FWIW, I think this patch is a bit more straightforward than the other one. Best regards, Frank > Maxime >