lkml.org 
[lkml]   [2022]   [Jan]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 5.10 245/563] clk: imx8mn: Fix imx8mn_clko1_sels
    Date
    From: Adam Ford <aford173@gmail.com>

    [ Upstream commit 570727e9acfac1c2330a01dd5e1272e9c3acec08 ]

    When attempting to use sys_pll1_80m as the parent for clko1, the
    system hangs. This is due to the fact that the source select
    for sys_pll1_80m was incorrectly pointing to m7_alt_pll_clk, which
    doesn't yet exist.

    According to Rev 3 of the TRM, The imx8mn_clko1_sels also incorrectly
    references an osc_27m which does not exist, nor does an entry for
    source select bits 010b. Fix both by inserting a dummy clock into
    the missing space in the table and renaming the incorrectly name clock
    with dummy.

    Fixes: 96d6392b54db ("clk: imx: Add support for i.MX8MN clock driver")
    Signed-off-by: Adam Ford <aford173@gmail.com>
    Reviewed-by: Fabio Estevam <festevam@gmail.com>
    Link: https://lore.kernel.org/r/20211117133202.775633-1-aford173@gmail.com
    Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/clk/imx/clk-imx8mn.c | 6 +++---
    1 file changed, 3 insertions(+), 3 deletions(-)

    diff --git a/drivers/clk/imx/clk-imx8mn.c b/drivers/clk/imx/clk-imx8mn.c
    index 33a7ddc23cd24..db122d94db583 100644
    --- a/drivers/clk/imx/clk-imx8mn.c
    +++ b/drivers/clk/imx/clk-imx8mn.c
    @@ -274,9 +274,9 @@ static const char * const imx8mn_pdm_sels[] = {"osc_24m", "sys_pll2_100m", "audi

    static const char * const imx8mn_dram_core_sels[] = {"dram_pll_out", "dram_alt_root", };

    -static const char * const imx8mn_clko1_sels[] = {"osc_24m", "sys_pll1_800m", "osc_27m",
    - "sys_pll1_200m", "audio_pll2_out", "vpu_pll",
    - "sys_pll1_80m", };
    +static const char * const imx8mn_clko1_sels[] = {"osc_24m", "sys_pll1_800m", "dummy",
    + "sys_pll1_200m", "audio_pll2_out", "sys_pll2_500m",
    + "dummy", "sys_pll1_80m", };
    static const char * const imx8mn_clko2_sels[] = {"osc_24m", "sys_pll2_200m", "sys_pll1_400m",
    "sys_pll2_166m", "sys_pll3_out", "audio_pll1_out",
    "video_pll1_out", "osc_32k", };
    --
    2.34.1


    \
     
     \ /
      Last update: 2022-01-24 22:41    [W:2.088 / U:0.024 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site