Re: [PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-24 Thread Dmitry Baryshkov
On Fri, 21 Jan 2022 at 23:44, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-01-20 23:37:45) > > On Fri, 21 Jan 2022 at 07:30, Stephen Boyd wrote: > > > > > > Quoting Dmitry Baryshkov (2022-01-19 14:16:15) > > > > diff --git a/drivers/gpu/drm/msm/msm_io_utils.c > > > >

Re: [PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-21 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-01-20 23:37:45) > On Fri, 21 Jan 2022 at 07:30, Stephen Boyd wrote: > > > > Quoting Dmitry Baryshkov (2022-01-19 14:16:15) > > > diff --git a/drivers/gpu/drm/msm/msm_io_utils.c > > > b/drivers/gpu/drm/msm/msm_io_utils.c > > > index 7b504617833a..5533c87c7158 100644

Re: [PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-20 Thread Dmitry Baryshkov
On Fri, 21 Jan 2022 at 07:30, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-01-19 14:16:15) > > diff --git a/drivers/gpu/drm/msm/msm_io_utils.c > > b/drivers/gpu/drm/msm/msm_io_utils.c > > index 7b504617833a..5533c87c7158 100644 > > --- a/drivers/gpu/drm/msm/msm_io_utils.c > > +++

Re: [PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-20 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-01-19 14:16:15) > diff --git a/drivers/gpu/drm/msm/msm_io_utils.c > b/drivers/gpu/drm/msm/msm_io_utils.c > index 7b504617833a..5533c87c7158 100644 > --- a/drivers/gpu/drm/msm/msm_io_utils.c > +++ b/drivers/gpu/drm/msm/msm_io_utils.c > @@ -5,6 +5,8 @@ > * Author:

Re: [PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-19 Thread Jessica Zhang
On 1/19/2022 2:16 PM, Dmitry Baryshkov wrote: DPU driver contains code to parse clock items from device tree into special data struct and then enable/disable/set rate for the clocks using that data struct. However the DPU driver itself uses only parsing and enabling/disabling part (the rate

[PATCH v3 2/3] drm/msm/dpu: simplify clocks handling

2022-01-19 Thread Dmitry Baryshkov
DPU driver contains code to parse clock items from device tree into special data struct and then enable/disable/set rate for the clocks using that data struct. However the DPU driver itself uses only parsing and enabling/disabling part (the rate setting is used by DP driver). Move this