Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-25 Thread Bjorn Andersson
On Fri 25 Jun 10:55 CDT 2021, khs...@codeaurora.org wrote: > On 2021-06-22 19:52, Bjorn Andersson wrote: > > On Tue 22 Jun 15:23 CDT 2021, Stephen Boyd wrote: > > > > > Quoting Bjorn Andersson (2021-06-18 14:41:50) > > > > On Fri 18 Jun 15:49 CDT 2021, Stephen Boyd wrote: > > > > > > > > >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-25 Thread khsieh
On 2021-06-22 19:52, Bjorn Andersson wrote: On Tue 22 Jun 15:23 CDT 2021, Stephen Boyd wrote: Quoting Bjorn Andersson (2021-06-18 14:41:50) > On Fri 18 Jun 15:49 CDT 2021, Stephen Boyd wrote: > > > Quoting khs...@codeaurora.org (2021-06-10 09:54:05) > > > On 2021-06-08 16:10, Bjorn Andersson

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-22 Thread Bjorn Andersson
On Tue 22 Jun 15:23 CDT 2021, Stephen Boyd wrote: > Quoting Bjorn Andersson (2021-06-18 14:41:50) > > On Fri 18 Jun 15:49 CDT 2021, Stephen Boyd wrote: > > > > > Quoting khs...@codeaurora.org (2021-06-10 09:54:05) > > > > On 2021-06-08 16:10, Bjorn Andersson wrote: > > > > > On Tue 08 Jun 17:44

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-22 Thread Stephen Boyd
Quoting Bjorn Andersson (2021-06-18 14:41:50) > On Fri 18 Jun 15:49 CDT 2021, Stephen Boyd wrote: > > > Quoting khs...@codeaurora.org (2021-06-10 09:54:05) > > > On 2021-06-08 16:10, Bjorn Andersson wrote: > > > > On Tue 08 Jun 17:44 CDT 2021, Stephen Boyd wrote: > > > > > > > >> Honestly I

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-18 Thread Bjorn Andersson
On Fri 18 Jun 15:49 CDT 2021, Stephen Boyd wrote: > Quoting khs...@codeaurora.org (2021-06-10 09:54:05) > > On 2021-06-08 16:10, Bjorn Andersson wrote: > > > On Tue 08 Jun 17:44 CDT 2021, Stephen Boyd wrote: > > > > > >> Honestly I suspect the DP PHY is _not_ in the CX domain as CX is for > > >>

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-18 Thread Stephen Boyd
Quoting khs...@codeaurora.org (2021-06-10 09:54:05) > On 2021-06-08 16:10, Bjorn Andersson wrote: > > On Tue 08 Jun 17:44 CDT 2021, Stephen Boyd wrote: > > > >> Honestly I suspect the DP PHY is _not_ in the CX domain as CX is for > >> digital logic. Probably the PLL is the hardware that has some

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-10 Thread khsieh
On 2021-06-08 16:10, Bjorn Andersson wrote: On Tue 08 Jun 17:44 CDT 2021, Stephen Boyd wrote: Quoting Bjorn Andersson (2021-06-08 15:34:01) > On Tue 08 Jun 17:29 CDT 2021, Stephen Boyd wrote: > > > Quoting Bjorn Andersson (2021-06-08 15:26:23) > > > On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Bjorn Andersson
On Tue 08 Jun 17:44 CDT 2021, Stephen Boyd wrote: > Quoting Bjorn Andersson (2021-06-08 15:34:01) > > On Tue 08 Jun 17:29 CDT 2021, Stephen Boyd wrote: > > > > > Quoting Bjorn Andersson (2021-06-08 15:26:23) > > > > On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd wrote: > > > > > > > > > Quoting Bjorn

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Stephen Boyd
Quoting Bjorn Andersson (2021-06-08 15:34:01) > On Tue 08 Jun 17:29 CDT 2021, Stephen Boyd wrote: > > > Quoting Bjorn Andersson (2021-06-08 15:26:23) > > > On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd wrote: > > > > > > > Quoting Bjorn Andersson (2021-06-07 16:31:47) > > > > > On Mon 07 Jun 12:48

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Bjorn Andersson
On Tue 08 Jun 17:29 CDT 2021, Stephen Boyd wrote: > Quoting Bjorn Andersson (2021-06-08 15:26:23) > > On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd wrote: > > > > > Quoting Bjorn Andersson (2021-06-07 16:31:47) > > > > On Mon 07 Jun 12:48 CDT 2021, khs...@codeaurora.org wrote: > > > > > > > > >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Stephen Boyd
Quoting Bjorn Andersson (2021-06-08 15:26:23) > On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd wrote: > > > Quoting Bjorn Andersson (2021-06-07 16:31:47) > > > On Mon 07 Jun 12:48 CDT 2021, khs...@codeaurora.org wrote: > > > > > > > Sorry about the confusion. What I meant is that even though DP > > >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Bjorn Andersson
On Tue 08 Jun 17:15 CDT 2021, Stephen Boyd wrote: > Quoting Bjorn Andersson (2021-06-07 16:31:47) > > On Mon 07 Jun 12:48 CDT 2021, khs...@codeaurora.org wrote: > > > > > On 2021-06-05 22:07, Bjorn Andersson wrote: > > > > On Thu 03 Jun 16:56 CDT 2021, khs...@codeaurora.org wrote: > > > > > > > >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-08 Thread Stephen Boyd
Quoting Bjorn Andersson (2021-06-07 16:31:47) > On Mon 07 Jun 12:48 CDT 2021, khs...@codeaurora.org wrote: > > > On 2021-06-05 22:07, Bjorn Andersson wrote: > > > On Thu 03 Jun 16:56 CDT 2021, khs...@codeaurora.org wrote: > > > > > > > On 2021-06-03 09:53, Bjorn Andersson wrote: > > > > > On Thu

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-07 Thread Bjorn Andersson
On Mon 07 Jun 12:48 CDT 2021, khs...@codeaurora.org wrote: > On 2021-06-05 22:07, Bjorn Andersson wrote: > > On Thu 03 Jun 16:56 CDT 2021, khs...@codeaurora.org wrote: > > > > > On 2021-06-03 09:53, Bjorn Andersson wrote: > > > > On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: > > [..] > > > >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-07 Thread khsieh
On 2021-06-05 22:07, Bjorn Andersson wrote: On Thu 03 Jun 16:56 CDT 2021, khs...@codeaurora.org wrote: On 2021-06-03 09:53, Bjorn Andersson wrote: > On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: [..] > > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi [..] > > +

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-05 Thread Bjorn Andersson
On Thu 03 Jun 16:56 CDT 2021, khs...@codeaurora.org wrote: > On 2021-06-03 09:53, Bjorn Andersson wrote: > > On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: [..] > > > diff --git a/arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi [..] > > > + power-domains = < SC7180_CX>; >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-03 Thread khsieh
On 2021-06-03 09:53, Bjorn Andersson wrote: On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: Add DP device node on sc7180. Changes in v2: -- replace msm_dp with dp -- replace dp_opp_table with opp_table I'm sorry for those suggestions, I don't like either one of them. And for everything

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-03 Thread Stephen Boyd
Quoting khs...@codeaurora.org (2021-06-03 14:28:37) > On 2021-06-03 09:53, Bjorn Andersson wrote: > > On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: > > > >> Add DP device node on sc7180. > >> > >> Changes in v2: > >> -- replace msm_dp with dp > >> -- replace dp_opp_table with opp_table > >> >

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-03 Thread khsieh
On 2021-06-03 09:53, Bjorn Andersson wrote: On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: Add DP device node on sc7180. Changes in v2: -- replace msm_dp with dp -- replace dp_opp_table with opp_table I'm sorry for those suggestions, I don't like either one of them. And for everything

Re: [Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-03 Thread Bjorn Andersson
On Thu 03 Jun 11:09 CDT 2021, Kuogee Hsieh wrote: > Add DP device node on sc7180. > > Changes in v2: > -- replace msm_dp with dp > -- replace dp_opp_table with opp_table > I'm sorry for those suggestions, I don't like either one of them. And for everything but changes to the DRM code the

[Freedreno] [PATCH v2] arm64/dts/qcom/sc7180: Add Display Port dt node

2021-06-03 Thread Kuogee Hsieh
Add DP device node on sc7180. Changes in v2: -- replace msm_dp with dp -- replace dp_opp_table with opp_table Signed-off-by: Kuogee Hsieh --- arch/arm64/boot/dts/qcom/sc7180-trogdor.dtsi | 9 arch/arm64/boot/dts/qcom/sc7180.dtsi | 78 2 files changed,

Re: [Freedreno] [PATCH v2] arm64: dts: qcom: sc7180: Add Display Port dt node

2020-06-19 Thread tanmay
Thanks Stephen for reviews. I will post new change addressing your all comments. On 2020-06-18 18:12, Stephen Boyd wrote: Quoting Tanmay Shah (2020-06-18 16:21:13) Enable DP driver for sc7180. Add DP device node on sc7180? This isn't a driver. Done. This change depends-on following

Re: [Freedreno] [PATCH v2] arm64: dts: qcom: sc7180: Add Display Port dt node

2020-06-18 Thread Stephen Boyd
Quoting Tanmay Shah (2020-06-18 16:21:13) > Enable DP driver for sc7180. Add DP device node on sc7180? This isn't a driver. > > This change depends-on following series: > https://patchwork.freedesktop.org/series/78583/ > and https://patchwork.freedesktop.org/patch/351990/ > > Changes in v2: >

[Freedreno] [PATCH v2] arm64: dts: qcom: sc7180: Add Display Port dt node

2020-06-18 Thread Tanmay Shah
Enable DP driver for sc7180. This change depends-on following series: https://patchwork.freedesktop.org/series/78583/ and https://patchwork.freedesktop.org/patch/351990/ Changes in v2: - Add assigned-clocks and assigned-clock-parents - Remove cell-index and pixel_rcg - Change compatible to