[U-Boot] [PATCH] Enable HDMI on i.MX6 without VPU

[U-Boot] [PATCH] Enable HDMI on i.MX6 without VPU

[Nikolay Petukhov]

Hi, all

This patch enables HDMI on CPU without VPU.
A similar patch for the mainline kernel:
https://patchwork.kernel.org/patch/9874831/
Tested on MCIMX6Q4AVT10AD.


Signed-off-by: Nikolay Petukhov <nikolaypetukhov@gmail.com>
Cc: Stefano Babic sbabic at denx.de
---
 arch/arm/mach-imx/mx6/soc.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/mach-imx/mx6/soc.c b/arch/arm/mach-imx/mx6/soc.c
index b724668..8b976f7 100644
--- a/arch/arm/mach-imx/mx6/soc.c
+++ b/arch/arm/mach-imx/mx6/soc.c
@@ -655,6 +655,11 @@ void imx_setup_hdmi(void)
     int reg, count;
     u8 val;

+    /* Turn on MIPI core cfg clock */
+    reg = readl(&mxc_ccm->CCGR3);
+    reg |= MXC_CCM_CCGR3_MIPI_CORE_CFG_MASK;
+    writel(reg, &mxc_ccm->CCGR3);
+
     /* Turn on HDMI PHY clock */
     reg = readl(&mxc_ccm->CCGR2);
     reg |=  MXC_CCM_CCGR2_HDMI_TX_IAHBCLK_MASK|
-- 
2.7.4



-- 
Nikolay

[U-Boot] [PATCH] Enable HDMI on i.MX6 without VPU

[Stefano Babic]

Hi Nikolay,

On 10/10/2017 16:27, Nikolay Petukhov wrote:
> Hi, all
> 
> This patch enables HDMI on CPU without VPU.
> A similar patch for the mainline
> kernel:https://patchwork.kernel.org/patch/9874831/
> Tested on MCIMX6Q4AVT10AD.
> 

This is stored in the commit message if I apply. Please rewrite the
commit message to be suitable for inclusion.

> 
> Signed-off-by: Nikolay Petukhov <nikolaypetukhov@gmail.com
> <mailto:nikolaypetukhov@gmail.com>>
> Cc: Stefano Babic sbabic at denx.de <mailto:sbabic@denx.de>
> ---
>  arch/arm/mach-imx/mx6/soc.c | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/arch/arm/mach-imx/mx6/soc.c b/arch/arm/mach-imx/mx6/soc.c
> index b724668..8b976f7 100644
> --- a/arch/arm/mach-imx/mx6/soc.c
> +++ b/arch/arm/mach-imx/mx6/soc.c
> @@ -655,6 +655,11 @@ void imx_setup_hdmi(void)
>      int reg, count;
>      u8 val;
>  
> +    /* Turn on MIPI core cfg clock */
> +    reg = readl(&mxc_ccm->CCGR3);
> +    reg |= MXC_CCM_CCGR3_MIPI_CORE_CFG_MASK;
> +    writel(reg, &mxc_ccm->CCGR3);
> +
>      /* Turn on HDMI PHY clock */
>      reg = readl(&mxc_ccm->CCGR2);
>      reg |=  MXC_CCM_CCGR2_HDMI_TX_IAHBCLK_MASK|
> -- 
> 2.7.4
> 
> 

Best regards,
Stefano Babic

-- 
=====================================================================
DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================

[U-Boot] [PATCH] Enable HDMI on i.MX6 without VPU

[Nikolay Petukhov]

Hi Stefano,

On i.MX6 SoCs without VPU(MCIMX6Q4AVT10AD) the HDMI is not working.
That's because hdmi_isfr's parent clock, video_27m, is not correctly
ungated.

The video_27m clock is gated by CCM_CCGR3[CG8] - mipi_core_cfg_clk_enable.

On i.MX6 SoCs with VPU, the HDMI is working thanks to the
CCM_CMEOR[mod_en_ov_vpu] bit which makes the video_27m ungated whatever
is in CCM_CCGR3[CG8].

This patch make the HDMI to work in every case by gating the mipi_core_cfg
clock.

Signed-off-by: Nikolay Petukhov <nikolaypetukhov@gmail.com>
Cc: Stefano Babic sbabic at denx.de
---
 arch/arm/mach-imx/mx6/soc.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/arch/arm/mach-imx/mx6/soc.c b/arch/arm/mach-imx/mx6/soc.c
index b724668..8b976f7 100644
--- a/arch/arm/mach-imx/mx6/soc.c
+++ b/arch/arm/mach-imx/mx6/soc.c
@@ -655,6 +655,11 @@ void imx_setup_hdmi(void)
     int reg, count;
     u8 val;

+    /* Turn on MIPI core cfg clock */
+    reg = readl(&mxc_ccm->CCGR3);
+    reg |= MXC_CCM_CCGR3_MIPI_CORE_CFG_MASK;
+    writel(reg, &mxc_ccm->CCGR3);
+
     /* Turn on HDMI PHY clock */
     reg = readl(&mxc_ccm->CCGR2);
     reg |=  MXC_CCM_CCGR2_HDMI_TX_IAHBCLK_MASK|
-- 
2.7.4





2017-10-31 15:33 GMT+05:00 Stefano Babic <sbabic@denx.de>:

> Hi Nikolay,
>
> On 10/10/2017 16:27, Nikolay Petukhov wrote:
> > Hi, all
> >
> > This patch enables HDMI on CPU without VPU.
> > A similar patch for the mainline
> > kernel:https://patchwork.kernel.org/patch/9874831/
> > Tested on MCIMX6Q4AVT10AD.
> >
>
> This is stored in the commit message if I apply. Please rewrite the
> commit message to be suitable for inclusion.
>
> >
> > Signed-off-by: Nikolay Petukhov <nikolaypetukhov@gmail.com
> > <mailto:nikolaypetukhov@gmail.com>>
> > Cc: Stefano Babic sbabic at denx.de <mailto:sbabic@denx.de>
> > ---
> >  arch/arm/mach-imx/mx6/soc.c | 5 +++++
> >  1 file changed, 5 insertions(+)
> >
> > diff --git a/arch/arm/mach-imx/mx6/soc.c b/arch/arm/mach-imx/mx6/soc.c
> > index b724668..8b976f7 100644
> > --- a/arch/arm/mach-imx/mx6/soc.c
> > +++ b/arch/arm/mach-imx/mx6/soc.c
> > @@ -655,6 +655,11 @@ void imx_setup_hdmi(void)
> >      int reg, count;
> >      u8 val;
> >
> > +    /* Turn on MIPI core cfg clock */
> > +    reg = readl(&mxc_ccm->CCGR3);
> > +    reg |= MXC_CCM_CCGR3_MIPI_CORE_CFG_MASK;
> > +    writel(reg, &mxc_ccm->CCGR3);
> > +
> >      /* Turn on HDMI PHY clock */
> >      reg = readl(&mxc_ccm->CCGR2);
> >      reg |=  MXC_CCM_CCGR2_HDMI_TX_IAHBCLK_MASK|
> > --
> > 2.7.4
> >
> >
>
> Best regards,
> Stefano Babic
>
> --
> =====================================================================
> DENX Software Engineering GmbH,      Managing Director: Wolfgang Denk
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
> =====================================================================
>



-- 
Nikolay