arm: mx6: cm-fx6: setup hdmi only on hdmi enable
authorNikita Kiryanov <nikita@compulab.co.il>
Thu, 23 Jul 2015 14:19:31 +0000 (17:19 +0300)
committerStefano Babic <sbabic@denx.de>
Sun, 2 Aug 2015 08:51:37 +0000 (10:51 +0200)
Refactor display code to only setup hdmi if do_enable_hdmi() is invoked.

Cc: Stefano Babic <sbabic@denx.de>
Cc: Igor Grinberg <grinberg@compulab.co.il>
Signed-off-by: Nikita Kiryanov <nikita@compulab.co.il>
Acked-by: Igor Grinberg <grinberg@compulab.co.il>
board/compulab/cm_fx6/cm_fx6.c

index 3e518c18d41d8d6ecbe2643b4cbbf57e09097ce3..e85c8aba45c781f6235817f52abc0884baf19ae4 100644 (file)
@@ -52,6 +52,9 @@ int splash_screen_prepare(void)
 #ifdef CONFIG_IMX_HDMI
 static void cm_fx6_enable_hdmi(struct display_info_t const *dev)
 {
+       struct mxc_ccm_reg *mxc_ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
+       imx_setup_hdmi();
+       setbits_le32(&mxc_ccm->CCGR3, MXC_CCM_CCGR3_IPU1_IPU_DI0_MASK);
        imx_enable_hdmi_phy();
 }
 
@@ -79,15 +82,9 @@ static struct display_info_t preset_hdmi_1024X768 = {
 
 static void cm_fx6_setup_display(void)
 {
-       struct mxc_ccm_reg *mxc_ccm = (struct mxc_ccm_reg *)CCM_BASE_ADDR;
        struct iomuxc *const iomuxc_regs = (struct iomuxc *)IOMUXC_BASE_ADDR;
-       int reg;
 
        enable_ipu_clock();
-       imx_setup_hdmi();
-       reg = __raw_readl(&mxc_ccm->CCGR3);
-       reg |= MXC_CCM_CCGR3_IPU1_IPU_DI0_MASK;
-       writel(reg, &mxc_ccm->CCGR3);
        clrbits_le32(&iomuxc_regs->gpr[3], MXC_CCM_CCGR3_IPU1_IPU_DI0_MASK);
 }