Re: [PATCH] arm64: dts: qcom: hamoa: Fix OPP tables for all DisplayPort controllers

[Dmitry Baryshkov]

On Wed, Mar 18, 2026 at 04:22:59PM +0200, Abel Vesa wrote:
> On 26-03-09 21:52:01, Dmitry Baryshkov wrote:
> > On Mon, Mar 09, 2026 at 04:44:45PM +0200, Abel Vesa wrote:
> > > According to internal documentation, the corners specific for each rate
> > > from the DP link clock are:
> > >  - LOWSVS_D1 -> 19.2 MHz
> > >  - LOWSVS    -> 270 MHz
> > >  - SVS       -> 540 MHz (594 MHz in case of DP3)
> > >  - SVS_L1    -> 594 MHz
> > >  - NOM       -> 810 MHz
> > >  - NOM_L1    -> 810 MHz
> > >  - TURBO     -> 810 MHz
> > > 
> > > So fix all tables for each of the four controllers according to the
> > > documentation.
> > > 
> > > The 19.2 @ LOWSVS_D1 isn't needed as the controller will select 162 MHz
> > > for RBR, which falls under the 270 MHz and it will vote for that LOWSVS
> > > in that case.
> > 
> > The list of issues isn't limited to Hamoa. As we started to look at it,
> > could you please also fix Lemans (drop 160, 270, use 594 instead of
> > 540, use single OPP table), Monaco (the same), SAR2130P (leave just 270
> > and 810), sc7180 (270 at low_svs, drop 160), etc.
> 
> For now, I'll just do Hamoa as this is the only one out of the ones you
> mentioned here for which I have access to documentation (yet).
> 
> I have prepared patches for all the other ones, but I need to double
> check the documentation, after I get access.

Sure, thanks!

Please ping me if there are any questions or delays, we can check the
docs together.

-- 
With best wishes
Dmitry