Re: [[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-07-10 Thread ryadav
A brief update on this topic: The DSI clock warnings are addressed after adding runtime_pm support to DPU driver [1]. MDSS GDSC is used as genpd w/ above series and is requested by parent MDSS device on behalf of all child devices (like DPU, DSI etc). Before adding the runtime_pm support, DSI

Re: [[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-06-27 Thread Rob Clark
On Wed, Jun 27, 2018 at 2:48 PM, Doug Anderson wrote: > Hi, > > On Tue, Jun 26, 2018 at 10:27 AM, Rob Clark wrote: >> On Tue, Jun 26, 2018 at 11:55 AM, Doug Anderson >> wrote: >>> Hi, >>> >>> On Mon, Jun 25, 2018 at 9:45 PM, Sandeep Panda >>> wrote: From: Abhinav Kumar

Re: [[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-06-27 Thread Doug Anderson
Hi, On Tue, Jun 26, 2018 at 10:27 AM, Rob Clark wrote: > On Tue, Jun 26, 2018 at 11:55 AM, Doug Anderson wrote: >> Hi, >> >> On Mon, Jun 25, 2018 at 9:45 PM, Sandeep Panda wrote: >>> From: Abhinav Kumar >>> >>> Setting the DSI PLL src in probe doesn't provide the clock >>> driver sufficient

Re: [[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-06-26 Thread Rob Clark
On Tue, Jun 26, 2018 at 11:55 AM, Doug Anderson wrote: > Hi, > > On Mon, Jun 25, 2018 at 9:45 PM, Sandeep Panda wrote: >> From: Abhinav Kumar >> >> Setting the DSI PLL src in probe doesn't provide the clock >> driver sufficient time to reclaim unused clock resources >> from coreboot resulting

Re: [[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-06-26 Thread Doug Anderson
Hi, On Mon, Jun 25, 2018 at 9:45 PM, Sandeep Panda wrote: > From: Abhinav Kumar > > Setting the DSI PLL src in probe doesn't provide the clock > driver sufficient time to reclaim unused clock resources > from coreboot resulting in warnings from clock driver. > > Move the DSI PLL src setting to

[[DPU]PATCH] drm/msm/dsi: move the API setting PLL src to modeset_init()

2018-06-26 Thread Sandeep Panda
From: Abhinav Kumar Setting the DSI PLL src in probe doesn't provide the clock driver sufficient time to reclaim unused clock resources from coreboot resulting in warnings from clock driver. Move the DSI PLL src setting to modeset_init() so that the clock driver can claim unused display clock