Re: [PATCH v2 3/4] media: platform: Add Renesas RZ/G2L MIPI CSI-2 receiver driver

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Prabhakar,

On Thu, Sep 22, 2022 at 01:08:33PM +0100, Lad, Prabhakar wrote:
> > > * Switched to manually turn ON/OFF the clocks instead of pm_runtime so that
> > >   the mipi/dhpy initialization happens as per the HW manual
> >
> > That doesn't look right. The driver doesn't use runtime PM anymore, so
> > power domains may not be handled properly. What was the problem with
> > clock handling using runtime PM ?
> >
> If we use the runtime PM all the clocks will be turned ON when we call
> pm_runtime_resume_and_get() which I dont want to. As per the "Starting
> reception for MIPI CSI-2 Input" section 35.3.1 for example we first
> need to turn ON all the clocks and later further down the line we need
> to just turn OFF VCLK -> Enable Link -> turn ON VCLK. Due to such
> cases I have switched to individual clock handling.

If that is the case, then you should control just that clock directly,
outside runtime PM callbacks.

Runtime PM may be needed e.g. for resuming a parent device.

-- 
Regards,

Sakari Ailus



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux