Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-04-29 Thread Dmitry Baryshkov
On Fri, 29 Apr 2022 at 04:20, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > > The only clock for which we set the rate is the "stream_pixel". Rather > > than storing the rate and then setting it by looping over all the > > clocks, set the clock rate directly. > > > >

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-04-28 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > The only clock for which we set the rate is the "stream_pixel". Rather > than storing the rate and then setting it by looping over all the > clocks, set the clock rate directly. > > Signed-off-by: Dmitry Baryshkov > --- Reviewed-by: Stephen Boyd

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-04-28 Thread Dmitry Baryshkov
On Fri, 29 Apr 2022 at 00:49, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-04-19 09:34:18) > > On 08/03/2022 23:46, Stephen Boyd wrote: > > > > > > Yes that's my concern. The qmp phy driver has a couple clk_set_rate() > > > calls in the .configure_dp_phy callback. That is called from >

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-04-28 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-04-19 09:34:18) > On 08/03/2022 23:46, Stephen Boyd wrote: > > > > Yes that's my concern. The qmp phy driver has a couple clk_set_rate() > > calls in the .configure_dp_phy callback. That is called from > > phy_power_on() (see qcom_qmp_phy_power_on() and

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-04-19 Thread Dmitry Baryshkov
On 08/03/2022 23:46, Stephen Boyd wrote: Quoting Dmitry Baryshkov (2022-03-03 23:58:58) On Fri, 4 Mar 2022 at 07:31, Stephen Boyd wrote: Quoting Dmitry Baryshkov (2022-03-03 20:23:06) On Fri, 4 Mar 2022 at 01:32, Stephen Boyd wrote: Quoting Dmitry Baryshkov (2022-02-16 21:55:27) The

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-03-08 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-03 23:58:58) > On Fri, 4 Mar 2022 at 07:31, Stephen Boyd wrote: > > > > Quoting Dmitry Baryshkov (2022-03-03 20:23:06) > > > On Fri, 4 Mar 2022 at 01:32, Stephen Boyd wrote: > > > > > > > > Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > > > > > The only clock

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-03-03 Thread Dmitry Baryshkov
On Fri, 4 Mar 2022 at 07:31, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-03-03 20:23:06) > > On Fri, 4 Mar 2022 at 01:32, Stephen Boyd wrote: > > > > > > Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > > > > The only clock for which we set the rate is the "stream_pixel". Rather > > >

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-03-03 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-03-03 20:23:06) > On Fri, 4 Mar 2022 at 01:32, Stephen Boyd wrote: > > > > Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > > > The only clock for which we set the rate is the "stream_pixel". Rather > > > than storing the rate and then setting it by looping over all

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-03-03 Thread Dmitry Baryshkov
On Fri, 4 Mar 2022 at 01:32, Stephen Boyd wrote: > > Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > > The only clock for which we set the rate is the "stream_pixel". Rather > > than storing the rate and then setting it by looping over all the > > clocks, set the clock rate directly. > > > >

Re: [Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-03-03 Thread Stephen Boyd
Quoting Dmitry Baryshkov (2022-02-16 21:55:27) > The only clock for which we set the rate is the "stream_pixel". Rather > than storing the rate and then setting it by looping over all the > clocks, set the clock rate directly. > > Signed-off-by: Dmitry Baryshkov [...] > diff --git

[Freedreno] [PATCH v5 3/5] drm/msm/dp: set stream_pixel rate directly

2022-02-16 Thread Dmitry Baryshkov
The only clock for which we set the rate is the "stream_pixel". Rather than storing the rate and then setting it by looping over all the clocks, set the clock rate directly. Signed-off-by: Dmitry Baryshkov --- drivers/gpu/drm/msm/dp/dp_clk_util.c | 33