[PATCH 1/1] soc: imx: imx8m-blk-ctrl: fix display clock for LCDIF2 power domain

paul.elder at ideasonboard.com paul.elder at ideasonboard.com
Thu May 26 22:02:34 PDT 2022


Hi Alexander,

On Tue, May 24, 2022 at 09:39:34AM +0200, Alexander Stein wrote:
> LCDIF2 has its own display clock, use this one.

Whoops, looks like I missed that :/

> 
> Fixes: 07614fed00e9 ("soc: imx: imx8m-blk-ctrl: Add i.MX8MP media blk-ctrl")
> Signed-off-by: Alexander Stein <alexander.stein at ew.tq-group.com>

Reviewed-by: Paul Elder <paul.elder at ideasonboard.com>

> ---
> I noticed this while trying to get LVDS (on LCDIF2) working. Apparently
> it is not strictly required, but better to have this right.
> 
>  drivers/soc/imx/imx8m-blk-ctrl.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/soc/imx/imx8m-blk-ctrl.c b/drivers/soc/imx/imx8m-blk-ctrl.c
> index 7f49385ed2f8..7ebc28709e94 100644
> --- a/drivers/soc/imx/imx8m-blk-ctrl.c
> +++ b/drivers/soc/imx/imx8m-blk-ctrl.c
> @@ -667,7 +667,7 @@ static const struct imx8m_blk_ctrl_domain_data imx8mp_media_blk_ctl_domain_data[
>  	},
>  	[IMX8MP_MEDIABLK_PD_LCDIF_2] = {
>  		.name = "mediablk-lcdif-2",
> -		.clk_names = (const char *[]){ "disp1", "apb", "axi", },
> +		.clk_names = (const char *[]){ "disp2", "apb", "axi", },
>  		.num_clks = 3,
>  		.gpc_name = "lcdif2",
>  		.rst_mask = BIT(11) | BIT(12) | BIT(24),
> -- 
> 2.25.1
> 



More information about the linux-arm-kernel mailing list