lkml.org 
[lkml]   [2020]   [Aug]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    From
    Date
    SubjectRe: [PATCH v2 12/17] clk: imx8mp: Add audio blk_ctrl clocks and resets
    On Fri, Aug 14, 2020 at 8:12 PM Abel Vesa <abel.vesa@nxp.com> wrote:
    >
    > Add audio blk_ctrl clocks and resets in the i.MX8MP clock
    > driver to be picked up by the clk-blk-ctrl driver.
    >
    > Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
    > ---
    > drivers/clk/imx/clk-blk-ctrl.c | 4 ++
    > drivers/clk/imx/clk-imx8mp.c | 138 +++++++++++++++++++++++++++++++++++++++++

    As i mentioned in Patch 11, I wonder we probably better to put blk
    ctrl clk data in blk ctrl driver.
    Otherwise, i'm okay with the code.

    Regards
    Aisheng

    > 2 files changed, 142 insertions(+)
    >
    > diff --git a/drivers/clk/imx/clk-blk-ctrl.c b/drivers/clk/imx/clk-blk-ctrl.c
    > index 1672646..1c2991c 100644
    > --- a/drivers/clk/imx/clk-blk-ctrl.c
    > +++ b/drivers/clk/imx/clk-blk-ctrl.c
    > @@ -312,6 +312,10 @@ static const struct dev_pm_ops imx_blk_ctrl_pm_ops = {
    > };
    >
    > static const struct of_device_id imx_blk_ctrl_of_match[] = {
    > + {
    > + .compatible = "fsl,imx8mp-audio-blk-ctrl",
    > + .data = &imx8mp_audio_blk_ctrl_dev_data
    > + },
    > { /* Sentinel */ }
    > };
    > MODULE_DEVICE_TABLE(of, imx_blk_ctrl_of_match);
    > diff --git a/drivers/clk/imx/clk-imx8mp.c b/drivers/clk/imx/clk-imx8mp.c
    > index 462c558..00e7f5e 100644
    > --- a/drivers/clk/imx/clk-imx8mp.c
    > +++ b/drivers/clk/imx/clk-imx8mp.c
    > @@ -4,6 +4,7 @@
    > */
    >
    > #include <dt-bindings/clock/imx8mp-clock.h>
    > +#include <dt-bindings/reset/imx8mp-reset.h>
    > #include <linux/clk-provider.h>
    > #include <linux/err.h>
    > #include <linux/io.h>
    > @@ -14,11 +15,148 @@
    > #include <linux/types.h>
    >
    > #include "clk.h"
    > +#include "clk-blk-ctrl.h"
    > +
    > +#define IMX_AUDIO_BLK_CTRL_CLKEN0 0x0
    > +#define IMX_AUDIO_BLK_CTRL_CLKEN1 0x4
    > +#define IMX_AUDIO_BLK_CTRL_EARC 0x200
    > +#define IMX_AUDIO_BLK_CTRL_SAI1_MCLK_SEL 0x300
    > +#define IMX_AUDIO_BLK_CTRL_SAI2_MCLK_SEL 0x304
    > +#define IMX_AUDIO_BLK_CTRL_SAI3_MCLK_SEL 0x308
    > +#define IMX_AUDIO_BLK_CTRL_SAI5_MCLK_SEL 0x30C
    > +#define IMX_AUDIO_BLK_CTRL_SAI6_MCLK_SEL 0x310
    > +#define IMX_AUDIO_BLK_CTRL_SAI7_MCLK_SEL 0x314
    > +#define IMX_AUDIO_BLK_CTRL_PDM_CLK 0x318
    > +#define IMX_AUDIO_BLK_CTRL_SAI_PLL_GNRL_CTL 0x400
    > +#define IMX_AUDIO_BLK_CTRL_SAI_PLL_FDIVL_CTL0 0x404
    > +#define IMX_AUDIO_BLK_CTRL_SAI_PLL_FDIVL_CTL1 0x408
    > +#define IMX_AUDIO_BLK_CTRL_SAI_PLL_SSCG_CTL 0x40C
    > +#define IMX_AUDIO_BLK_CTRL_SAI_PLL_MNIT_CTL 0x410
    > +#define IMX_AUDIO_BLK_CTRL_IPG_LP_CTRL 0x504
    > +
    > +#define IMX_MEDIA_BLK_CTRL_SFT_RSTN 0x0
    > +#define IMX_MEDIA_BLK_CTRL_CLK_EN 0x4
    >
    > static u32 share_count_nand;
    > static u32 share_count_media;
    > static u32 share_count_audio;
    >
    > +static int shared_count_pdm;
    > +
    > +static const struct imx_pll14xx_rate_table imx_blk_ctrl_sai_pll_tbl[] = {
    > + PLL_1443X_RATE(650000000U, 325, 3, 2, 0),
    > +};
    > +
    > +static const struct imx_pll14xx_clk imx_blk_ctrl_sai_pll = {
    > + .type = PLL_1443X,
    > + .rate_table = imx_blk_ctrl_sai_pll_tbl,
    > +};
    > +
    > +static const char * const imx_sai_mclk2_sels[] = {"sai1_root", "sai2_root", "sai3_root", "dummy",
    > + "sai5_root", "sai6_root", "sai7_root", "sai1_mclk",
    > + "sai2_mclk", "sai3_mclk", "dummy",
    > + "sai5_mclk", "sai6_mclk", "sai7_mclk", "spdif1_ext_clk"};
    > +static const char * const imx_sai1_mclk1_sels[] = {"sai1_root", "sai1_mclk", };
    > +static const char * const imx_sai2_mclk1_sels[] = {"sai2_root", "sai2_mclk", };
    > +static const char * const imx_sai3_mclk1_sels[] = {"sai3_root", "sai3_mclk", };
    > +static const char * const imx_sai5_mclk1_sels[] = {"sai5_root", "sai5_mclk", };
    > +static const char * const imx_sai6_mclk1_sels[] = {"sai6_root", "sai6_mclk", };
    > +static const char * const imx_sai7_mclk1_sels[] = {"sai7_root", "sai7_mclk", };
    > +static const char * const imx_pdm_sels[] = {"pdm_root", "sai_pll_div2", "dummy", "dummy" };
    > +static const char * const imx_sai_pll_ref_sels[] = {"osc_24m", "dummy", "dummy", "dummy", };
    > +static const char * const imx_sai_pll_bypass_sels[] = {"sai_pll", "sai_pll_ref_sel", };
    > +
    > +static struct imx_blk_ctrl_hw imx8mp_audio_blk_ctrl_hws[] = {
    > + /* clocks */
    > + IMX_BLK_CTRL_CLK_MUX("sai_pll_ref_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI_PLL_REF_SEL, 0x400, 0, 2, imx_sai_pll_ref_sels),
    > + IMX_BLK_CTRL_CLK_PLL14XX("sai_pll", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI_PLL, 0x400, "sai_pll_ref_sel", &imx_blk_ctrl_sai_pll),
    > + IMX_BLK_CTRL_CLK_MUX_FLAGS("sai_pll_bypass", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI_PLL_BYPASS, 0x400, 4, 1, imx_sai_pll_bypass_sels, CLK_SET_RATE_PARENT),
    > + IMX_BLK_CTRL_CLK_GATE("sai_pll_out", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI_PLL_OUT, 0x400, 13, "sai_pll_bypass"),
    > + IMX_BLK_CTRL_CLK_MUX_FLAGS("sai1_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_MCLK1_SEL, 0x300, 0, 1, imx_sai1_mclk1_sels, CLK_SET_RATE_PARENT),
    > + IMX_BLK_CTRL_CLK_MUX("sai1_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_MCLK2_SEL, 0x300, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_MUX_FLAGS("sai2_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_MCLK1_SEL, 0x304, 0, 1, imx_sai2_mclk1_sels, CLK_SET_RATE_PARENT),
    > + IMX_BLK_CTRL_CLK_MUX("sai2_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_MCLK2_SEL, 0x304, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_MUX_FLAGS("sai3_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_MCLK1_SEL, 0x308, 0, 1, imx_sai3_mclk1_sels, CLK_SET_RATE_PARENT),
    > + IMX_BLK_CTRL_CLK_MUX("sai3_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_MCLK2_SEL, 0x308, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai5_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_MCLK1_SEL, 0x30C, 0, 1, imx_sai5_mclk1_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai5_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_MCLK2_SEL, 0x30C, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai6_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_MCLK1_SEL, 0x310, 0, 1, imx_sai6_mclk1_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai6_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_MCLK2_SEL, 0x310, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai7_mclk1_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_MCLK1_SEL, 0x314, 0, 1, imx_sai7_mclk1_sels),
    > + IMX_BLK_CTRL_CLK_MUX("sai7_mclk2_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_MCLK2_SEL, 0x314, 1, 4, imx_sai_mclk2_sels),
    > + IMX_BLK_CTRL_CLK_GATE("sai1_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_IPG, 0, 0, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai1_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_MCLK1, 0, 1, "sai1_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai1_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_MCLK2, 0, 2, "sai1_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai1_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI1_MCLK3, 0, 3, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("sai2_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_IPG, 0, 4, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai2_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_MCLK1, 0, 5, "sai2_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai2_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_MCLK2, 0, 6, "sai2_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai2_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI2_MCLK3, 0, 7, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("sai3_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_IPG, 0, 8, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai3_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_MCLK1, 0, 9, "sai3_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai3_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_MCLK2, 0, 10, "sai3_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai3_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI3_MCLK3, 0, 11, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("sai5_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_IPG, 0, 12, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai5_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_MCLK1, 0, 13, "sai5_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai5_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_MCLK2, 0, 14, "sai5_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai5_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI5_MCLK3, 0, 15, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("sai6_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_IPG, 0, 16, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai6_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_MCLK1, 0, 17, "sai6_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai6_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_MCLK2, 0, 18, "sai6_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai6_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI6_MCLK3, 0, 19, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("sai7_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_IPG, 0, 20, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sai7_mclk1_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_MCLK1, 0, 21, "sai7_mclk1_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai7_mclk2_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_MCLK2, 0, 22, "sai7_mclk2_sel"),
    > + IMX_BLK_CTRL_CLK_GATE("sai7_mclk3_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SAI7_MCLK3, 0, 23, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_GATE("asrc_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_ASRC_IPG, 0, 24, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_SHARED_GATE("pdm_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_PDM_IPG, 0, 25, "audio_ahb_root", &shared_count_pdm),
    > + IMX_BLK_CTRL_CLK_SHARED_GATE("pdm_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_PDM_ROOT, 0, 25, "pdm_root", &shared_count_pdm),
    > + IMX_BLK_CTRL_CLK_GATE("sdma2_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SDMA2_ROOT, 0, 26, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("sdma3_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SDMA3_ROOT, 0, 27, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("spba2_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_SPBA2_ROOT, 0, 28, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("dsp_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_DSP_ROOT, 0, 29, "audio_axi_root"),
    > + IMX_BLK_CTRL_CLK_GATE("dsp_dbg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_DSPDBG_ROOT, 0, 30, "audio_axi_root"),
    > + IMX_BLK_CTRL_CLK_GATE("earc_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_EARC_IPG, 0, 31, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("ocram_a_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_OCRAMA_IPG, 4, 0, "audio_axi_root"),
    > + IMX_BLK_CTRL_CLK_GATE("aud2htx_ipg_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_AUD2HTX_IPG, 4, 1, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("edma_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_EDMA_ROOT, 4, 2, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("aud_pll_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_AUDPLL_ROOT, 4, 3, "osc_24m"),
    > + IMX_BLK_CTRL_CLK_GATE("mu2_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_MU2_ROOT, 4, 4, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("mu3_root_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_MU3_ROOT, 4, 5, "audio_ahb_root"),
    > + IMX_BLK_CTRL_CLK_GATE("earc_phy_clk", IMX8MP_CLK_AUDIO_BLK_CTRL_EARC_PHY, 4, 6, "sai_pll_out"),
    > + IMX_BLK_CTRL_CLK_MUX("pdm_sel", IMX8MP_CLK_AUDIO_BLK_CTRL_PDM_SEL, 0x318, 1, 4, imx_pdm_sels),
    > +
    > + /* resets */
    > + IMX_BLK_CTRL_RESET(IMX8MP_AUDIO_BLK_CTRL_EARC_RESET, 0x200, 0),
    > + IMX_BLK_CTRL_RESET(IMX8MP_AUDIO_BLK_CTRL_EARC_PHY_RESET, 0x200, 1),
    > +};
    > +
    > +const struct imx_blk_ctrl_dev_data imx8mp_audio_blk_ctrl_dev_data __initconst = {
    > + .hws = imx8mp_audio_blk_ctrl_hws,
    > + .hws_num = ARRAY_SIZE(imx8mp_audio_blk_ctrl_hws),
    > + .clocks_max = IMX8MP_CLK_AUDIO_BLK_CTRL_END,
    > + .resets_max = IMX8MP_AUDIO_BLK_CTRL_RESET_NUM,
    > + .pm_runtime_saved_regs_num = 16,
    > + .pm_runtime_saved_regs = {
    > + IMX_AUDIO_BLK_CTRL_CLKEN0,
    > + IMX_AUDIO_BLK_CTRL_CLKEN1,
    > + IMX_AUDIO_BLK_CTRL_EARC,
    > + IMX_AUDIO_BLK_CTRL_SAI1_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_SAI2_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_SAI3_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_SAI5_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_SAI6_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_SAI7_MCLK_SEL,
    > + IMX_AUDIO_BLK_CTRL_PDM_CLK,
    > + IMX_AUDIO_BLK_CTRL_SAI_PLL_GNRL_CTL,
    > + IMX_AUDIO_BLK_CTRL_SAI_PLL_FDIVL_CTL0,
    > + IMX_AUDIO_BLK_CTRL_SAI_PLL_FDIVL_CTL1,
    > + IMX_AUDIO_BLK_CTRL_SAI_PLL_SSCG_CTL,
    > + IMX_AUDIO_BLK_CTRL_SAI_PLL_MNIT_CTL,
    > + IMX_AUDIO_BLK_CTRL_IPG_LP_CTRL
    > + },
    > +};
    > +
    > static const char * const pll_ref_sels[] = { "osc_24m", "dummy", "dummy", "dummy", };
    > static const char * const audio_pll1_bypass_sels[] = {"audio_pll1", "audio_pll1_ref_sel", };
    > static const char * const audio_pll2_bypass_sels[] = {"audio_pll2", "audio_pll2_ref_sel", };
    > --
    > 2.7.4
    >

    \
     
     \ /
      Last update: 2020-08-18 13:46    [W:6.279 / U:0.020 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site