Hi Dave, On Tue, Jun 14, 2022 at 05:47:28PM +0100, Dave Stevenson wrote: > Hi Maxime. > > On Fri, 10 Jun 2022 at 10:30, Maxime Ripard <maxime@xxxxxxxxxx> wrote: > > > > Adding a device-managed action will make the error path easier, so let's > > create one to disable our clock. > > The DPI block has two clocks (core and pixel), and this is only > affecting one of them (the core clock). Thanks for the suggestion, I've amended the commit message. > (I'm actually puzzling over what it's wanting to do with the core > clock here as it's only enabling it rather than setting a rate. I > think it may be redundant). Could it be that it a "bus" clock that we need it to access the registers? Maxime
Attachment:
signature.asc
Description: PGP signature