From mboxrd@z Thu Jan 1 00:00:00 1970 From: Icenowy Zheng Subject: Re: [PATCH v2 00/27] Add support for R40 HDMI pipeline Date: Tue, 19 Jun 2018 02:49:31 +0800 Message-ID: <6deccceaf1d44ce7f20a9571965c4f6fbf146e10.camel@aosc.io> References: <20180612200036.21483-1-jernej.skrabec@siol.net> <5084905.C41JrL457O@jernej-laptop> <15787507.kzqBcLUy0z@jernej-laptop> Reply-To: icenowy-h8G6r0blFSE@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: <15787507.kzqBcLUy0z@jernej-laptop> List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: jernej.skrabec-gGgVlfcn5nU@public.gmane.org, Jagan Teki Cc: Maxime Ripard , Chen-Yu Tsai , Rob Herring , David Airlie , gustavo-THi1TnShQwVAfugRpC6u6w@public.gmane.org, maarten.lankhorst-VuQAYsv1563Yd54FQh9/CA@public.gmane.org, Sean Paul , Mark Rutland , dri-devel , devicetree , linux-arm-kernel , linux-kernel , linux-clk , linux-sunxi List-Id: devicetree@vger.kernel.org =E5=9C=A8 2018-06-18=E4=B8=80=E7=9A=84 16:43 +0200=EF=BC=8CJernej =C5=A0kra= bec=E5=86=99=E9=81=93=EF=BC=9A > Dne ponedeljek, 18. junij 2018 ob 14:58:02 CEST je Jagan Teki > napisal(a): > > On Thu, Jun 14, 2018 at 10:59 PM, Jernej =C5=A0krabec > >=20 > > wrote: > > > Dne =C4=8Detrtek, 14. junij 2018 ob 19:16:46 CEST je Jagan Teki > > > napisal(a): > > > > On Thu, Jun 14, 2018 at 8:04 PM, Jernej =C5=A0krabec > > > @siol.net> > > >=20 > > > wrote: > > > > > Dne =C4=8Detrtek, 14. junij 2018 ob 09:12:41 CEST je Jagan Teki > > > > > napisal(a): > > > > > > On Wed, Jun 13, 2018 at 1:30 AM, Jernej Skrabec > > > > > > > > > > >=20 > > > > > wrote: > > > > > > > This series adds support for R40 HDMI pipeline. It is a > > > > > > > bit special > > > > > > > than other already supported pipelines because it has > > > > > > > additional > > > > > > > unit > > > > > > > called TCON TOP responsible for relationship > > > > > > > configuration between > > > > > > > mixers, TCONs and HDMI. Additionally, it has additional > > > > > > > gates for > > > > > > > DSI > > > > > > > and TV TCONs, TV encoder clock settings and pin muxing > > > > > > > between LCD > > > > > > > and TV encoders. > > > > > > >=20 > > > > > > > However, it seems that TCON TOP will become a norm, since > > > > > > > newer > > > > > > > Allwinner SoCs like H6 also have this unit. > > > > > > >=20 > > > > > > > I tested different possible configurations: > > > > > > > - mixer0 <> TCON-TV0 <> HDMI > > > > > > > - mixer0 <> TCON-TV1 <> HDMI > > > > > > > - mixer1 <> TCON-TV0 <> HDMI > > > > > > > - mixer1 <> TCON-TV1 <> HDMI > > > > > > >=20 > > > > > > > Please review. > > > > > > >=20 > > > > > > > Best regards, > > > > > > > Jernej > > > > > > >=20 > > > > > > > Changes from v1: > > > > > > > - Split DT bindings patch and updated description > > > > > > > - Split HDMI PHY patch > > > > > > > - Move header file from TCON TOP patch to dt bindings > > > > > > > patch > > > > > > > - Added Rob reviewed-by tag > > > > > > > - Used clk_hw_register_gate() instead of custom gate > > > > > > > registration > > > > > > > code > > > > > > > - Reworked TCON TOP to be part of of-graph. Because of > > > > > > > that, a lot > > > > > > > of > > > > > > >=20 > > > > > > > new patches were added. > > > > > > >=20 > > > > > > > - Droped mixer index quirk patch > > > > > > > - Reworked TCON support for TCON TOP > > > > > > > - Updated commit messages > > > > > > >=20 > > > > > > > Jernej Skrabec (27): > > > > > > > clk: sunxi-ng: r40: Add minimal rate for video PLLs > > > > > > > clk: sunxi-ng: r40: Allow setting parent rate to > > > > > > > display related > > > > > > > =20 > > > > > > > clocks > > > > > > > =20 > > > > > > > clk: sunxi-ng: r40: Export video PLLs > > > > > > > dt-bindings: display: sunxi-drm: Add TCON TOP > > > > > > > description > > > > > > > drm/sun4i: Add TCON TOP driver > > > > > > > drm/sun4i: Fix releasing node when enumerating enpoints > > > > > > > drm/sun4i: Split out code for enumerating endpoints in > > > > > > > output port > > > > > > > drm/sun4i: Add support for traversing graph with TCON > > > > > > > TOP > > > > > > > drm/sun4i: Don't skip TCONs if they don't have channel > > > > > > > 0 > > > > > > > dt-bindings: display: sun4i-drm: Add R40 TV TCON > > > > > > > description > > > > > > > drm/sun4i: tcon: Add support for tcon-top gate > > > > > > > drm/sun4i: tcon: Generalize engine search algorithm > > > > > > > drm/sun4i: Don't check for LVDS and RGB when TCON has > > > > > > > only ch1 > > > > > > > drm/sun4i: Don't check for panel or bridge on TV TCONs > > > > > > > drm/sun4i: Add support for R40 TV TCON > > > > > > > dt-bindings: display: sun4i-drm: Add R40 mixer > > > > > > > compatibles > > > > > > > drm/sun4i: Add support for R40 mixers > > > > > > > dt-bindings: display: sun4i-drm: Add description of A64 > > > > > > > HDMI PHY > > > > > > > drm/sun4i: Enable DW HDMI PHY clock > > > > > > > drm/sun4i: Don't change clock bits in DW HDMI PHY > > > > > > > driver > > > > > > > drm/sun4i: DW HDMI PHY: Add support for second PLL > > > > > > > drm/sun4i: Add support for second clock parent to DW > > > > > > > HDMI PHY clk > > > > > > > =20 > > > > > > > driver > > > > > > > =20 > > > > > > > drm/sun4i: Add support for A64 HDMI PHY > > > > > > > drm: of: Export drm_crtc_port_mask() > > > > > > > drm/sun4i: DW HDMI: Expand algorithm for possible crtcs > > > > > > > ARM: dts: sun8i: r40: Add HDMI pipeline > > > > > > > ARM: dts: sun8i: r40: Enable HDMI output on BananaPi M2 > > > > > > > Ultra > > > > > >=20 > > > > > > Tested whole series on top of linux-next. > > > > > >=20 > > > > > > Tested-by: Jagan Teki > > > > >=20 > > > > > Thanks! > > > >=20 > > > > I've V40 board, which is same as R40. I'm able to detect the > > > > HDMI but > > > > seems edid not detecting properly. > > > >=20 > > > > [ 0.983007] sun4i-drm display-engine: bound 1100000.mixer > > > > (ops > > > > 0xc074a80c) [ 0.999043] sun4i-drm display-engine: bound > > > > 1200000.mixer > > > > (ops 0xc074a80c) [ 1.006229] sun4i-drm display-engine: bound > > > > 1c70000.tcon-top (ops 0xc074e2ac) [ 1.013609] sun4i-drm > > > > display-engine: > > > > bound 1c73000.lcd-controller (ops 0xc0747a28) > > > > [ 1.053988] sun8i-dw-hdmi 1ee0000.hdmi: Detected HDMI TX > > > > controller > > > > v1.32a with HDCP (sun8i_dw_hdmi_phy) > > > > [ 1.063913] sun8i-dw-hdmi 1ee0000.hdmi: registered > > > > DesignWare HDMI > > > > I2C bus driver > > > > [ 1.071683] sun4i-drm display-engine: bound 1ee0000.hdmi > > > > (ops > > > > 0xc074a298) [ 1.078484] [drm] Supports vblank timestamp > > > > caching Rev 2 > > > > (21.10.2013). [ 1.085098] [drm] No driver support for vblank > > > > timestamp query. [ 1.091055] [drm] Cannot find any crtc or > > > > sizes > > > > [ 1.095995] [drm] Initialized sun4i-drm 1.0.0 20150629 for > > > > display-engine on minor 0 > > >=20 > > > This seems like DT issue. Can you post somewhere your V40 DTSI > > > (if it is > > > different to R40) and board DTS? > >=20 > > same dtsi shared between r40 and v40, here is board dts support for > > HDMI[1] > >=20 > > [1] https://paste.ubuntu.com/p/wqVz38BHrM/ >=20 > This patch looks like exactly the same as mine for BananaPi M2U, so > there=20 > should be no issues. As I know, M2B is designed to be compatible with M2U, so most things should be the same. The stock firmware even use the same images for both M2U and M2B. >=20 > What about VCC-HDMI? Is powered? Can you measure it to check? >=20 > Best regards, > Jernej >=20 >=20 >=20 --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout.