[Freedreno] [PATCH v1] arm64: dts: qcom: sc7180: Add Display Port dt node
Enable DP driver for sc7180. This change depends-on following series: https://patchwork.freedesktop.org/series/78247/ and https://patchwork.freedesktop.org/patch/351990/ Signed-off-by: Tanmay Shah --- arch/arm64/boot/dts/qcom/sc7180.dtsi | 58 ++-- 1 file changed, 54 insertions(+), 4 deletions(-) diff --git a/arch/arm64/boot/dts/qcom/sc7180.dtsi b/arch/arm64/boot/dts/qcom/sc7180.dtsi index 998f101ad623..989960d2200b 100644 --- a/arch/arm64/boot/dts/qcom/sc7180.dtsi +++ b/arch/arm64/boot/dts/qcom/sc7180.dtsi @@ -1557,10 +1557,19 @@ ports { #address-cells = <1>; #size-cells = <0>; + port@1 { + reg = <1>; + dpu_intf1_out: endpoint { + remote-endpoint = +<_in>; + }; + }; + port@0 { reg = <0>; - dpu_intf1_out: endpoint { - remote-endpoint = <_in>; + dpu_intf0_out: endpoint { + remote-endpoint = +<_in>; }; }; }; @@ -1634,6 +1643,47 @@ dsi_phy: dsi-phy@ae94400 { }; }; + msm_dp: displayport-controller@ae9{ + status = "ok"; + cell-index = <0>; + compatible = "qcom,dp-display"; + + reg = <0 0xae9 0 0x1400>; + reg-names = "dp_controller"; + + interrupt-parent = <>; + interrupts = <12 0>; + + clocks = < DISP_CC_MDSS_DP_AUX_CLK>, +< DISP_CC_MDSS_DP_LINK_CLK>, +< DISP_CC_MDSS_DP_LINK_INTF_CLK>, +< DISP_CC_MDSS_DP_PIXEL_CLK>, +< DISP_CC_MDSS_DP_PIXEL_CLK_SRC>; + clock-names = "core_aux", "ctrl_link", + "ctrl_link_iface", "stream_pixel", + "pixel_rcg"; + #clock-cells = <1>; + + data-lanes = <0 1>; + + ports { + #address-cells = <1>; + #size-cells = <0>; + port@0 { + reg = <0>; + dp_in: endpoint { + remote-endpoint = +<_intf0_out>; + }; + }; + + port@1 { + reg = <1>; + dp_out: endpoint { }; + }; + }; + }; + dispcc: clock-controller@af0 { compatible = "qcom,sc7180-dispcc"; reg = <0 0x0af0 0 0x20>; @@ -1641,8 +1691,8 @@ dispcc: clock-controller@af0 { < GCC_DISP_GPLL0_CLK_SRC>, <_phy 0>, <_phy 1>, -<0>, -<0>; +<_dp 0>, +<_dp 1>; clock-names = "bi_tcxo", "gcc_disp_gpll0_clk_src", "dsi0_phy_pll_out_byteclk", -- The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH v6 0/5] Add support for DisplayPort driver on
On 2020-06-15 16:04, Jeffrey Hugo wrote: On Mon, Jun 15, 2020 at 4:51 PM wrote: On 2020-06-12 16:26, Stephen Boyd wrote: Thanks for reviews Stephen. > Quoting Tanmay Shah (2020-06-11 18:50:25) >> These patches add support for Display-Port driver on SnapDragon >> hardware. It adds >> DP driver and DP PLL driver files along with the needed device-tree >> bindings. >> >> The block diagram of DP driver is shown below: >> >> >> +-+ >> |DRM FRAMEWORK| >> +--+--+ >> | >>+v+ >>| DP DRM | >>+++ >> | >>+v+ >> ++| DP+--++--+ >> ++---+| DISPLAY |+---+ | | >> |++-+-+-+| | | >> || | | | | | >> || | | | | | >> || | | | | | >> vv v v v v v >> +--+ +--+ +---+ ++ ++ +---+ +-+ >> | DP | | DP | |DP | | DP | | DP | |DP | | DP | >> |PARSER| | HPD | |AUX| |LINK| |CTRL| |PHY| |POWER| >> +--+---+ +---+--+ +---+ ++ +--+-+ +-+-+ +-+ >> | | | >> +--v---+ +v-v+ >> |DEVICE| | DP | >> | TREE | |CATALOG| >> +--+ +---+---+ >> | >> +---v+ >> |CTRL/PHY| >> | HW | >> ++ >> > > I've never seen a block diagram for a driver before... > It is here for v5. https://patchwork.freedesktop.org/series/74312/ I think Stephen is nitpicking your wording, and you seem to not be understanding his comment. I'm sorry if I am mistaken. The "DP driver" would seem to refer to the linux software driver you are proposing patches for, however this diagram looks like a hardware diagram of the various hardware blocks that the Linux driver code (the "DP driver") is expected to interact with. I believe you should re-word "The block diagram of DP driver is shown below:" to be more specific of what you are describing with your figure. IE your words say this is a block diagram of the software, when it looks like it is a block diagram of the hardware. Thanks for reviews. I am not sure what Stephen meant, but this diagram was available before. Just for clarification this is not hardware diagram at all. This is modeling of DP driver for msm. Each box name above except "DRM framework", is file name in driver i.e. software module. Each line and arrow shows how modules interact with each other. For example, "DP PARSER" Box is pointing towards "DEVICE TREE" Box, that means dp_parser.c file contains functions which are parsing device tree properties and so on... ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH v6 0/5] Add support for DisplayPort driver on
On Mon, Jun 15, 2020 at 4:51 PM wrote: > > On 2020-06-12 16:26, Stephen Boyd wrote: > > Thanks for reviews Stephen. > > > Quoting Tanmay Shah (2020-06-11 18:50:25) > >> These patches add support for Display-Port driver on SnapDragon > >> hardware. It adds > >> DP driver and DP PLL driver files along with the needed device-tree > >> bindings. > >> > >> The block diagram of DP driver is shown below: > >> > >> > >> +-+ > >> |DRM FRAMEWORK| > >> +--+--+ > >> | > >>+v+ > >>| DP DRM | > >>+++ > >> | > >>+v+ > >> ++| DP+--++--+ > >> ++---+| DISPLAY |+---+ | | > >> |++-+-+-+| | | > >> || | | | | | > >> || | | | | | > >> || | | | | | > >> vv v v v v v > >> +--+ +--+ +---+ ++ ++ +---+ +-+ > >> | DP | | DP | |DP | | DP | | DP | |DP | | DP | > >> |PARSER| | HPD | |AUX| |LINK| |CTRL| |PHY| |POWER| > >> +--+---+ +---+--+ +---+ ++ +--+-+ +-+-+ +-+ > >> | | | > >> +--v---+ +v-v+ > >> |DEVICE| | DP | > >> | TREE | |CATALOG| > >> +--+ +---+---+ > >> | > >> +---v+ > >> |CTRL/PHY| > >> | HW | > >> ++ > >> > > > > I've never seen a block diagram for a driver before... > > > It is here for v5. https://patchwork.freedesktop.org/series/74312/ I think Stephen is nitpicking your wording, and you seem to not be understanding his comment. I'm sorry if I am mistaken. The "DP driver" would seem to refer to the linux software driver you are proposing patches for, however this diagram looks like a hardware diagram of the various hardware blocks that the Linux driver code (the "DP driver") is expected to interact with. I believe you should re-word "The block diagram of DP driver is shown below:" to be more specific of what you are describing with your figure. IE your words say this is a block diagram of the software, when it looks like it is a block diagram of the hardware. ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH v6 0/5] Add support for DisplayPort driver on
On 2020-06-12 16:26, Stephen Boyd wrote: Thanks for reviews Stephen. Quoting Tanmay Shah (2020-06-11 18:50:25) These patches add support for Display-Port driver on SnapDragon hardware. It adds DP driver and DP PLL driver files along with the needed device-tree bindings. The block diagram of DP driver is shown below: +-+ |DRM FRAMEWORK| +--+--+ | +v+ | DP DRM | +++ | +v+ ++| DP+--++--+ ++---+| DISPLAY |+---+ | | |++-+-+-+| | | || | | | | | || | | | | | || | | | | | vv v v v v v +--+ +--+ +---+ ++ ++ +---+ +-+ | DP | | DP | |DP | | DP | | DP | |DP | | DP | |PARSER| | HPD | |AUX| |LINK| |CTRL| |PHY| |POWER| +--+---+ +---+--+ +---+ ++ +--+-+ +-+-+ +-+ | | | +--v---+ +v-v+ |DEVICE| | DP | | TREE | |CATALOG| +--+ +---+---+ | +---v+ |CTRL/PHY| | HW | ++ I've never seen a block diagram for a driver before... It is here for v5. https://patchwork.freedesktop.org/series/74312/ These patches have dependency on clock driver changes mentioned below: https://patchwork.kernel.org/patch/11245895/ https://patchwork.kernel.org/cover/11069083/ These are merged right? Don't need to include this if it's already merged. Ok Thanks. Can you include a changelog in the cover letter too so we know what has changed between versions of the patchset? Sure. Chandan Uddaraju (4): dt-bindings: msm/dp: add bindings of DP/DP-PLL driver for Snapdragon drm: add constant N value in helper file drm/msm/dp: add displayPort driver support drm/msm/dp: add support for DP PLL driver Jeykumar Sankaran (1): drm/msm/dpu: add display port support in DPU [...] base-commit: 48f99181fc118d82dc8bf6c7221ad1c654cb8bc2 What is this commit? I don't see it in linux-next. Here: https://git.kernel.org/pub/scm/linux/kernel/git/next/linux-next.git/commit/?h=next-20200603=48f99181fc118d82dc8bf6c7221ad1c654cb8bc2 ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno
Re: [Freedreno] [PATCH v3 0/6] Add support for GPU DDR BW scaling
On 06-06-20, 09:55, Sharat Masetty wrote: > This is a respin of [1]. Incorported review feedback and fixed issues observed > during testing. Picked up the Georgi's series from opp/linux-next [2], and > this > series is also dependent on a helper function needed to set and clear ddr > bandwidth vote [3]. Patch number 4 in the series adds support for SDM845 as > well > but its not tested yet(WIP), but the SC7180 patches are well tested now. > > [1] https://patchwork.freedesktop.org/series/75291/ > [2] > https://kernel.googlesource.com/pub/scm/linux/kernel/git/vireshk/pm/+log/opp/linux-next/ > [3] https://patchwork.kernel.org/patch/11590563/ Had a quick look of the series and looked mostly fine to me. -- viresh ___ Freedreno mailing list Freedreno@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/freedreno