Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Laurent Pinchart
Hi Jose, On Tuesday 13 Jun 2017 15:11:27 Jose Abreu wrote: > Hi Laurent, > > Sorry for the late reply! No worries. > On 10-06-2017 09:50, Laurent Pinchart wrote: > > On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: > >> On 09-06-2017 12:04, Jose Abreu wrote: > >>> Currently HDMI 2.0 PHYs do

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Laurent Pinchart
Hi Jose, On Tuesday 13 Jun 2017 15:11:27 Jose Abreu wrote: > Hi Laurent, > > Sorry for the late reply! No worries. > On 10-06-2017 09:50, Laurent Pinchart wrote: > > On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: > >> On 09-06-2017 12:04, Jose Abreu wrote: > >>> Currently HDMI 2.0 PHYs do

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Zheng Yang
Hi, Jose: The value of RK3399 reg HDMI_CONFIG2_ID is 0xf3. Using hdmi_phy_configure_dwc_hdmi_3d_tx() for hdmi 2.0 phy, we test all HDMI video mode(including 594MHz) and it woks good. And our customer has pass the HDMI CTS. If you send a patch with a general config function

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Zheng Yang
Hi, Jose: The value of RK3399 reg HDMI_CONFIG2_ID is 0xf3. Using hdmi_phy_configure_dwc_hdmi_3d_tx() for hdmi 2.0 phy, we test all HDMI video mode(including 594MHz) and it woks good. And our customer has pass the HDMI CTS. If you send a patch with a general config function

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Mark yao
Hi Jose Sorry miss your email and Sorry for the late reply I can sure that your patch works on our rk3399 platform. my internal kernel already has similar patch, using hdmi_phy_configure_dwc_hdmi_3d_tx() for hdmi 2.0 phy, good works with many video modes (4k, 1080p, 720p etc.), I'm not

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-22 Thread Mark yao
Hi Jose Sorry miss your email and Sorry for the late reply I can sure that your patch works on our rk3399 platform. my internal kernel already has similar patch, using hdmi_phy_configure_dwc_hdmi_3d_tx() for hdmi 2.0 phy, good works with many video modes (4k, 1080p, 720p etc.), I'm not

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-20 Thread Heiko Stübner
Hi Jose, Am Freitag, 9. Juni 2017, 13:53:12 CEST schrieb Jose Abreu: > On 09-06-2017 12:04, Jose Abreu wrote: > > Currently HDMI 2.0 PHYs do not have a default configuration function. > > > > As these PHYs have the same register layout as the 3D PHYs we can > > safely use the default

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-20 Thread Heiko Stübner
Hi Jose, Am Freitag, 9. Juni 2017, 13:53:12 CEST schrieb Jose Abreu: > On 09-06-2017 12:04, Jose Abreu wrote: > > Currently HDMI 2.0 PHYs do not have a default configuration function. > > > > As these PHYs have the same register layout as the 3D PHYs we can > > safely use the default

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-13 Thread Jose Abreu
Hi Laurent, Sorry for the late reply! On 10-06-2017 09:50, Laurent Pinchart wrote: > Hi Jose, > > On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: >> On 09-06-2017 12:04, Jose Abreu wrote: >>> Currently HDMI 2.0 PHYs do not have a default configuration function. >>> >>> As these PHYs have the

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-13 Thread Jose Abreu
Hi Laurent, Sorry for the late reply! On 10-06-2017 09:50, Laurent Pinchart wrote: > Hi Jose, > > On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: >> On 09-06-2017 12:04, Jose Abreu wrote: >>> Currently HDMI 2.0 PHYs do not have a default configuration function. >>> >>> As these PHYs have the

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-10 Thread Laurent Pinchart
Hi Jose, On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: > On 09-06-2017 12:04, Jose Abreu wrote: > > Currently HDMI 2.0 PHYs do not have a default configuration function. > > > > As these PHYs have the same register layout as the 3D PHYs we can > > safely use the default configuration

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-10 Thread Laurent Pinchart
Hi Jose, On Friday 09 Jun 2017 13:53:12 Jose Abreu wrote: > On 09-06-2017 12:04, Jose Abreu wrote: > > Currently HDMI 2.0 PHYs do not have a default configuration function. > > > > As these PHYs have the same register layout as the 3D PHYs we can > > safely use the default configuration

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-09 Thread Jose Abreu
Hi Mark, On 09-06-2017 12:04, Jose Abreu wrote: > Currently HDMI 2.0 PHYs do not have a default configuration function. > > As these PHYs have the same register layout as the 3D PHYs we can > safely use the default configuration function. I may have been a little to fast arriving at this

Re: [PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-09 Thread Jose Abreu
Hi Mark, On 09-06-2017 12:04, Jose Abreu wrote: > Currently HDMI 2.0 PHYs do not have a default configuration function. > > As these PHYs have the same register layout as the 3D PHYs we can > safely use the default configuration function. I may have been a little to fast arriving at this

[PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-09 Thread Jose Abreu
Currently HDMI 2.0 PHYs do not have a default configuration function. As these PHYs have the same register layout as the 3D PHYs we can safely use the default configuration function. If, for some reason, the PHY is custom this change will not make any impact because in configuration function we

[PATCH] drm: bridge: synopsys/dw-hdmi: Provide default configuration function for HDMI 2.0 PHY

2017-06-09 Thread Jose Abreu
Currently HDMI 2.0 PHYs do not have a default configuration function. As these PHYs have the same register layout as the 3D PHYs we can safely use the default configuration function. If, for some reason, the PHY is custom this change will not make any impact because in configuration function we