Hi, On Tue, Mar 8, 2022 at 8:55 AM Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> wrote: > > use max clock during probe/bind sequence from the opp table. > The clock will be scaled down when framework sends an update. > > Signed-off-by: Vinod Polimera <quic_vpolimer@xxxxxxxxxxx> > --- > drivers/gpu/drm/msm/disp/dpu1/dpu_kms.c | 3 +++ > 1 file changed, 3 insertions(+) In addition to Dmitry's requests, can you also make this patch #1 in the series since the DTS stuff really ought to come _after_ this one. ...and actually, thinking about it further: 1. If we land this fix, we actually don't _need_ the dts patches, right? Sure, the clock rate will get assigned before probe but then we'll change it right away in probe, right? 2. If we land the dts patches _before_ the driver patch then it will be a regression, right? 3. The dts patches and driver patch will probably land through separate trees. The driver patch will go through the MSM DRM tree and the device tree patches through the Qualcomm armsoc tree, right? Assuming that the above is right, we should: 1. Put the driver patch first. 2. Remove the "Fixes" tag on the dts patches. I guess in theory we could have a FIxes tag on this patch? 3. Note in the dts patches commit messages that they depend on the driver patch. 4. Delay the dts patches until the driver change has made it to mainline. Does that sound reasonable?
