Forgotten to add

On 4/27/20 5:53 PM, Giulio Benetti wrote:
BROM doesn't enable lcdif by default so add clk_enable() after
clk_set_rate().

Signed-off-by: Giulio Benetti <giulio.bene...@benettiengineering.com>

Reviewed-by: Anatolij Gustschin <ag...@denx.de>
Reviewed-by: Fabio Estevam <feste...@gmail.com>

---
V1->V2:
* call clk_enable() after clk_set_rate() as suggested by Fabio
---
  drivers/video/mxsfb.c | 6 ++++++
  1 file changed, 6 insertions(+)

diff --git a/drivers/video/mxsfb.c b/drivers/video/mxsfb.c
index 8a5a61c9fb..12d00b4689 100644
--- a/drivers/video/mxsfb.c
+++ b/drivers/video/mxsfb.c
@@ -77,6 +77,12 @@ static void mxs_lcd_init(struct udevice *dev, u32 fb_addr,
                dev_err(dev, "Failed to set mxs clk: %d\n", ret);
                return;
        }
+
+       ret = clk_enable(&per_clk);
+       if (ret < 0) {
+               dev_err(dev, "Failed to enable mxs clk: %d\n", ret);
+               return;
+       }
  #else
        /* Kick in the LCDIF clock */
        mxs_set_lcdclk(MXS_LCDIF_BASE, timings->pixelclock.typ / 1000);


--
Giulio Benetti
Benetti Engineering sas

Reply via email to