lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <d2677180-9e98-4ba0-86c7-7e1e499c4456@amlogic.com>
Date: Fri, 14 Feb 2025 17:51:26 +0800
From: Jiebing Chen <jiebing.chen@...ogic.com>
To: Jerome Brunet <jbrunet@...libre.com>,
 jiebing chen via B4 Relay <devnull+jiebing.chen.amlogic.com@...nel.org>
Cc: Liam Girdwood <lgirdwood@...il.com>, Mark Brown <broonie@...nel.org>,
 Rob Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>,
 Conor Dooley <conor+dt@...nel.org>, Jaroslav Kysela <perex@...ex.cz>,
 Takashi Iwai <tiwai@...e.com>, Neil Armstrong <neil.armstrong@...aro.org>,
 Kevin Hilman <khilman@...libre.com>,
 Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
 Michael Turquette <mturquette@...libre.com>, Stephen Boyd
 <sboyd@...nel.org>, linux-sound@...r.kernel.org, devicetree@...r.kernel.org,
 linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
 linux-amlogic@...ts.infradead.org, linux-clk@...r.kernel.org,
 jian.xu@...ogic.com, shuai.li@...ogic.com, zhe.wang@...ogic.com
Subject: Re: [PATCH v2 3/5] ASoC: meson: s4:support the audio clk


在 2025/2/14 17:44, Jiebing Chen 写道:
>
> 在 2025/2/14 17:17, Jerome Brunet 写道:
>> [ EXTERNAL EMAIL ]
>>
>> On Fri 14 Feb 2025 at 10:14, Jerome Brunet <jbrunet@...libre.com> wrote:
>>
>>> On Fri 14 Feb 2025 at 10:13, jiebing chen via B4 Relay 
>>> <devnull+jiebing.chen.amlogic.com@...nel.org> wrote:
>>>
>>>> From: jiebing chen <jiebing.chen@...ogic.com>
>>>>
>>>> Add the audio clk tree for s4, add new mclk, sclk and lrclk pad.
>>>>
>>> This disregard previous feedback about the compatibility with the
>>> axg/g12 platform.
>>>
>>> Is what follows really the only difference between the 2 ?
>>> You previous reported the difference would be significant and now 
>>> this ?
>>> I don't get it.
>> And the title is wrong
>
> Hi, jerome
>
> The s4 mclk pad  is different from the g12a ,for g12a. the mclk pad 
> not to add the div factor,
>
> for the previous version, we only test the internal codec, it didn't 
> rely on an mclk pad,
>
> if not use the external codec(dac/adc), old clock  id is compatible, 
> so the old clock id still work,
>
> based on your suggestion, we re-added the full clk id for s4, the s4 
> base on sm1,  except for some
>
> minor differences between the mclk pad and toodec and sm1, everything 
> else is exactly the same
>
> This is the s4 mclkpad  register

mclk pad0

bit[15]:en

bit[10:8] pad source

bit[7:0] div

>
>
>>
>>>> Signed-off-by: jiebing chen <jiebing.chen@...ogic.com>
>>>> ---
>>>>   drivers/clk/meson/axg-audio.c | 410 
>>>> +++++++++++++++++++++++++++++++++++++++++-
>>>>   drivers/clk/meson/axg-audio.h |   4 +
>>>>   2 files changed, 412 insertions(+), 2 deletions(-)
>>>>
>>>> diff --git a/drivers/clk/meson/axg-audio.c 
>>>> b/drivers/clk/meson/axg-audio.c
>>>> index 
>>>> 9df627b142f89788966ede0262aaaf39e13f0b49..e70559ee0fd8deaf9858b0592e839fdf49d05886 
>>>> 100644
>>>> --- a/drivers/clk/meson/axg-audio.c
>>>> +++ b/drivers/clk/meson/axg-audio.c
>>>> @@ -323,6 +323,16 @@ static const struct clk_parent_data 
>>>> lrclk_pad_ctrl_parent_data[] = {
>>>>       AUD_MUX(_name, _reg, 0x7, _shift, 0, _parents,          \
>>>>               CLK_SET_RATE_NO_REPARENT)
>>>>
>>>> +#define AUD_MCLK_PAD_MUX(_name, _reg, 
>>>> _shift)                                       \
>>>> +    AUD_MUX(_name##_sel, _reg, 0x7, _shift, 
>>>> CLK_MUX_ROUND_CLOSEST,                  \
>>>> +            mclk_pad_ctrl_parent_data, 0)
>>>> +#define AUD_MCLK_PAD_DIV(_name, _reg, 
>>>> _shift)                                       \
>>>> +    AUD_DIV(_name##_div, _reg, _shift, 8, 
>>>> CLK_DIVIDER_ROUND_CLOSEST,                        \
>>>> +            aud_##_name##_sel, CLK_SET_RATE_PARENT)
>>>> +#define AUD_MCLK_PAD_GATE(_name, _reg, 
>>>> _shift)                                      \
>>>> +    AUD_GATE(_name, _reg, _shift, 
>>>> aud_##_name##_div,                        \
>>>> +             CLK_SET_RATE_PARENT)
>>>> +
>>>>   /* Common Clocks */
>>>>   static struct clk_regmap ddr_arb =
>>>>       AUD_PCLK_GATE(ddr_arb, AUDIO_CLK_GATE_EN, 0);
>>>> @@ -826,6 +836,49 @@ static struct clk_regmap sm1_tdm_sclk_pad_1 = 
>>>> AUD_TDM_PAD_CTRL(
>>>>   static struct clk_regmap sm1_tdm_sclk_pad_2 = AUD_TDM_PAD_CTRL(
>>>>       tdm_sclk_pad_2, AUDIO_SM1_MST_PAD_CTRL1, 8, 
>>>> sclk_pad_ctrl_parent_data);
>>>>
>>>> +static struct clk_regmap s4_tdm_mclk_pad0_sel =
>>>> +    AUD_MCLK_PAD_MUX(mclk_pad0, AUDIO_S4_MCLK_PAD_CTRL0, 8);
>>>> +static struct clk_regmap s4_tdm_mclk_pad1_sel =
>>>> +    AUD_MCLK_PAD_MUX(mclk_pad1, AUDIO_S4_MCLK_PAD_CTRL0, 24);
>>>> +static struct clk_regmap s4_tdm_mclk_pad2_sel =
>>>> +    AUD_MCLK_PAD_MUX(mclk_pad2, AUDIO_S4_MCLK_PAD_CTRL1, 8);
>>>> +
>>>> +static struct clk_regmap s4_tdm_mclk_pad0_div =
>>>> +    AUD_MCLK_PAD_DIV(mclk_pad0, AUDIO_S4_MCLK_PAD_CTRL0, 0);
>>>> +static struct clk_regmap s4_tdm_mclk_pad1_div =
>>>> +    AUD_MCLK_PAD_DIV(mclk_pad1, AUDIO_S4_MCLK_PAD_CTRL0, 16);
>>>> +static struct clk_regmap s4_tdm_mclk_pad2_div =
>>>> +    AUD_MCLK_PAD_DIV(mclk_pad2, AUDIO_S4_MCLK_PAD_CTRL1, 0);
>>>> +
>>>> +static struct clk_regmap s4_tdm_mclk_pad_0 =
>>>> +    AUD_MCLK_PAD_GATE(mclk_pad0, AUDIO_S4_MCLK_PAD_CTRL0, 15);
>>>> +static struct clk_regmap s4_tdm_mclk_pad_1 =
>>>> +    AUD_MCLK_PAD_GATE(mclk_pad1, AUDIO_S4_MCLK_PAD_CTRL0, 31);
>>>> +static struct clk_regmap s4_tdm_mclk_pad_2 =
>>>> +    AUD_MCLK_PAD_GATE(mclk_pad2, AUDIO_S4_MCLK_PAD_CTRL1, 15);
>>>> +
>>>> +static struct clk_regmap s4_tdm_sclk_pad_0 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_sclk_pad_0, AUDIO_S4_SCLK_PAD_CTRL0, 0, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_sclk_pad_1 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_sclk_pad_1, AUDIO_S4_SCLK_PAD_CTRL0, 4, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_sclk_pad_2 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_sclk_pad_2, AUDIO_S4_SCLK_PAD_CTRL0, 8, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_sclk_pad_3 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_sclk_pad_3, AUDIO_S4_SCLK_PAD_CTRL0, 16, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_sclk_pad_4 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_sclk_pad_4, AUDIO_S4_SCLK_PAD_CTRL0, 20, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +
>>>> +static struct clk_regmap s4_tdm_lrclk_pad_0 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_lrclk_pad_0, AUDIO_S4_SCLK_PAD_CTRL1, 0, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_lrclk_pad_1 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_lrclk_pad_1, AUDIO_S4_SCLK_PAD_CTRL1, 4, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_lrclk_pad_2 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_lrclk_pad_2, AUDIO_S4_SCLK_PAD_CTRL1, 8, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_lrclk_pad_3 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_lrclk_pad_3, AUDIO_S4_SCLK_PAD_CTRL1, 16, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +static struct clk_regmap s4_tdm_lrclk_pad_4 =
>>>> +    AUD_TDM_PAD_CTRL(tdm_lrclk_pad_4, AUDIO_S4_SCLK_PAD_CTRL1, 20, 
>>>> lrclk_pad_ctrl_parent_data);
>>>> +
>>>>   /*
>>>>    * Array of all clocks provided by this provider
>>>>    * The input clocks of the controller will be populated at runtime
>>>> @@ -1257,6 +1310,177 @@ static struct clk_hw *sm1_audio_hw_clks[] = {
>>>>       [AUD_CLKID_EARCRX_DMAC]         = &sm1_earcrx_dmac_clk.hw,
>>>>   };
>>>>
>>>> +/*
>>>> + * Array of all S4 clocks provided by this provider
>>>> + * The input clocks of the controller will be populated at runtime
>>>> + */
>>>> +static struct clk_hw *s4_audio_hw_clks[] = {
>>>> +    [AUD_CLKID_DDR_ARB]             = &ddr_arb.hw,
>>>> +    [AUD_CLKID_PDM]                 = &pdm.hw,
>>>> +    [AUD_CLKID_TDMIN_A]             = &tdmin_a.hw,
>>>> +    [AUD_CLKID_TDMIN_B]             = &tdmin_b.hw,
>>>> +    [AUD_CLKID_TDMIN_C]             = &tdmin_c.hw,
>>>> +    [AUD_CLKID_TDMIN_LB]            = &tdmin_lb.hw,
>>>> +    [AUD_CLKID_TDMOUT_A]            = &tdmout_a.hw,
>>>> +    [AUD_CLKID_TDMOUT_B]            = &tdmout_b.hw,
>>>> +    [AUD_CLKID_TDMOUT_C]            = &tdmout_c.hw,
>>>> +    [AUD_CLKID_FRDDR_A]             = &frddr_a.hw,
>>>> +    [AUD_CLKID_FRDDR_B]             = &frddr_b.hw,
>>>> +    [AUD_CLKID_FRDDR_C]             = &frddr_c.hw,
>>>> +    [AUD_CLKID_TODDR_A]             = &toddr_a.hw,
>>>> +    [AUD_CLKID_TODDR_B]             = &toddr_b.hw,
>>>> +    [AUD_CLKID_TODDR_C]             = &toddr_c.hw,
>>>> +    [AUD_CLKID_LOOPBACK]            = &loopback.hw,
>>>> +    [AUD_CLKID_SPDIFIN]             = &spdifin.hw,
>>>> +    [AUD_CLKID_SPDIFOUT]            = &spdifout.hw,
>>>> +    [AUD_CLKID_RESAMPLE]            = &resample.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_B]          = &spdifout_b.hw,
>>>> +    [AUD_CLKID_MST_A_MCLK_SEL]      = &sm1_mst_a_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_B_MCLK_SEL]      = &sm1_mst_b_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_C_MCLK_SEL]      = &sm1_mst_c_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_D_MCLK_SEL]      = &sm1_mst_d_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_E_MCLK_SEL]      = &sm1_mst_e_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_F_MCLK_SEL]      = &sm1_mst_f_mclk_sel.hw,
>>>> +    [AUD_CLKID_MST_A_MCLK_DIV]      = &sm1_mst_a_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_B_MCLK_DIV]      = &sm1_mst_b_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_C_MCLK_DIV]      = &sm1_mst_c_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_D_MCLK_DIV]      = &sm1_mst_d_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_E_MCLK_DIV]      = &sm1_mst_e_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_F_MCLK_DIV]      = &sm1_mst_f_mclk_div.hw,
>>>> +    [AUD_CLKID_MST_A_MCLK]          = &sm1_mst_a_mclk.hw,
>>>> +    [AUD_CLKID_MST_B_MCLK]          = &sm1_mst_b_mclk.hw,
>>>> +    [AUD_CLKID_MST_C_MCLK]          = &sm1_mst_c_mclk.hw,
>>>> +    [AUD_CLKID_MST_D_MCLK]          = &sm1_mst_d_mclk.hw,
>>>> +    [AUD_CLKID_MST_E_MCLK]          = &sm1_mst_e_mclk.hw,
>>>> +    [AUD_CLKID_MST_F_MCLK]          = &sm1_mst_f_mclk.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_CLK_SEL]    = &spdifout_clk_sel.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_CLK_DIV]    = &spdifout_clk_div.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_CLK]        = &spdifout_clk.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_B_CLK_SEL]  = &spdifout_b_clk_sel.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_B_CLK_DIV]  = &spdifout_b_clk_div.hw,
>>>> +    [AUD_CLKID_SPDIFOUT_B_CLK]      = &spdifout_b_clk.hw,
>>>> +    [AUD_CLKID_SPDIFIN_CLK_SEL]     = &spdifin_clk_sel.hw,
>>>> +    [AUD_CLKID_SPDIFIN_CLK_DIV]     = &spdifin_clk_div.hw,
>>>> +    [AUD_CLKID_SPDIFIN_CLK]         = &spdifin_clk.hw,
>>>> +    [AUD_CLKID_PDM_DCLK_SEL]        = &pdm_dclk_sel.hw,
>>>> +    [AUD_CLKID_PDM_DCLK_DIV]        = &pdm_dclk_div.hw,
>>>> +    [AUD_CLKID_PDM_DCLK]            = &pdm_dclk.hw,
>>>> +    [AUD_CLKID_PDM_SYSCLK_SEL]      = &pdm_sysclk_sel.hw,
>>>> +    [AUD_CLKID_PDM_SYSCLK_DIV]      = &pdm_sysclk_div.hw,
>>>> +    [AUD_CLKID_PDM_SYSCLK]          = &pdm_sysclk.hw,
>>>> +    [AUD_CLKID_MST_A_SCLK_PRE_EN]   = &mst_a_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_B_SCLK_PRE_EN]   = &mst_b_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_C_SCLK_PRE_EN]   = &mst_c_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_D_SCLK_PRE_EN]   = &mst_d_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_E_SCLK_PRE_EN]   = &mst_e_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_F_SCLK_PRE_EN]   = &mst_f_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_MST_A_SCLK_DIV]      = &mst_a_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_B_SCLK_DIV]      = &mst_b_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_C_SCLK_DIV]      = &mst_c_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_D_SCLK_DIV]      = &mst_d_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_E_SCLK_DIV]      = &mst_e_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_F_SCLK_DIV]      = &mst_f_sclk_div.hw,
>>>> +    [AUD_CLKID_MST_A_SCLK_POST_EN]  = &mst_a_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_B_SCLK_POST_EN]  = &mst_b_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_C_SCLK_POST_EN]  = &mst_c_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_D_SCLK_POST_EN]  = &mst_d_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_E_SCLK_POST_EN]  = &mst_e_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_F_SCLK_POST_EN]  = &mst_f_sclk_post_en.hw,
>>>> +    [AUD_CLKID_MST_A_SCLK]          = &mst_a_sclk.hw,
>>>> +    [AUD_CLKID_MST_B_SCLK]          = &mst_b_sclk.hw,
>>>> +    [AUD_CLKID_MST_C_SCLK]          = &mst_c_sclk.hw,
>>>> +    [AUD_CLKID_MST_D_SCLK]          = &mst_d_sclk.hw,
>>>> +    [AUD_CLKID_MST_E_SCLK]          = &mst_e_sclk.hw,
>>>> +    [AUD_CLKID_MST_F_SCLK]          = &mst_f_sclk.hw,
>>>> +    [AUD_CLKID_MST_A_LRCLK_DIV]     = &mst_a_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_B_LRCLK_DIV]     = &mst_b_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_C_LRCLK_DIV]     = &mst_c_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_D_LRCLK_DIV]     = &mst_d_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_E_LRCLK_DIV]     = &mst_e_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_F_LRCLK_DIV]     = &mst_f_lrclk_div.hw,
>>>> +    [AUD_CLKID_MST_A_LRCLK]         = &mst_a_lrclk.hw,
>>>> +    [AUD_CLKID_MST_B_LRCLK]         = &mst_b_lrclk.hw,
>>>> +    [AUD_CLKID_MST_C_LRCLK]         = &mst_c_lrclk.hw,
>>>> +    [AUD_CLKID_MST_D_LRCLK]         = &mst_d_lrclk.hw,
>>>> +    [AUD_CLKID_MST_E_LRCLK]         = &mst_e_lrclk.hw,
>>>> +    [AUD_CLKID_MST_F_LRCLK]         = &mst_f_lrclk.hw,
>>>> +    [AUD_CLKID_TDMIN_A_SCLK_SEL]    = &tdmin_a_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMIN_B_SCLK_SEL]    = &tdmin_b_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMIN_C_SCLK_SEL]    = &tdmin_c_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMIN_LB_SCLK_SEL]   = &tdmin_lb_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMOUT_A_SCLK_SEL]   = &tdmout_a_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMOUT_B_SCLK_SEL]   = &tdmout_b_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMOUT_C_SCLK_SEL]   = &tdmout_c_sclk_sel.hw,
>>>> +    [AUD_CLKID_TDMIN_A_SCLK_PRE_EN] = &tdmin_a_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMIN_B_SCLK_PRE_EN] = &tdmin_b_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMIN_C_SCLK_PRE_EN] = &tdmin_c_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMIN_LB_SCLK_PRE_EN] = &tdmin_lb_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_A_SCLK_PRE_EN] = &tdmout_a_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_B_SCLK_PRE_EN] = &tdmout_b_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_C_SCLK_PRE_EN] = &tdmout_c_sclk_pre_en.hw,
>>>> +    [AUD_CLKID_TDMIN_A_SCLK_POST_EN] = &tdmin_a_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMIN_B_SCLK_POST_EN] = &tdmin_b_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMIN_C_SCLK_POST_EN] = &tdmin_c_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMIN_LB_SCLK_POST_EN] = &tdmin_lb_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_A_SCLK_POST_EN] = &tdmout_a_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_B_SCLK_POST_EN] = &tdmout_b_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMOUT_C_SCLK_POST_EN] = &tdmout_c_sclk_post_en.hw,
>>>> +    [AUD_CLKID_TDMIN_A_SCLK]        = &tdmin_a_sclk.hw,
>>>> +    [AUD_CLKID_TDMIN_B_SCLK]        = &tdmin_b_sclk.hw,
>>>> +    [AUD_CLKID_TDMIN_C_SCLK]        = &tdmin_c_sclk.hw,
>>>> +    [AUD_CLKID_TDMIN_LB_SCLK]       = &tdmin_lb_sclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_A_SCLK]       = &g12a_tdmout_a_sclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_B_SCLK]       = &g12a_tdmout_b_sclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_C_SCLK]       = &g12a_tdmout_c_sclk.hw,
>>>> +    [AUD_CLKID_TDMIN_A_LRCLK]       = &tdmin_a_lrclk.hw,
>>>> +    [AUD_CLKID_TDMIN_B_LRCLK]       = &tdmin_b_lrclk.hw,
>>>> +    [AUD_CLKID_TDMIN_C_LRCLK]       = &tdmin_c_lrclk.hw,
>>>> +    [AUD_CLKID_TDMIN_LB_LRCLK]      = &tdmin_lb_lrclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_A_LRCLK]      = &tdmout_a_lrclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_B_LRCLK]      = &tdmout_b_lrclk.hw,
>>>> +    [AUD_CLKID_TDMOUT_C_LRCLK]      = &tdmout_c_lrclk.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD0]       = &s4_tdm_mclk_pad_0.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD1]       = &s4_tdm_mclk_pad_1.hw,
>>>> +    [AUD_CLKID_TDM_LRCLK_PAD0]      = &s4_tdm_lrclk_pad_0.hw,
>>>> +    [AUD_CLKID_TDM_LRCLK_PAD1]      = &s4_tdm_lrclk_pad_1.hw,
>>>> +    [AUD_CLKID_TDM_LRCLK_PAD2]      = &s4_tdm_lrclk_pad_2.hw,
>>>> +    [AUD_CLKID_TDM_SCLK_PAD0]       = &s4_tdm_sclk_pad_0.hw,
>>>> +    [AUD_CLKID_TDM_SCLK_PAD1]       = &s4_tdm_sclk_pad_1.hw,
>>>> +    [AUD_CLKID_TDM_SCLK_PAD2]       = &s4_tdm_sclk_pad_2.hw,
>>>> +    [AUD_CLKID_TOP]                 = &sm1_aud_top.hw,
>>>> +    [AUD_CLKID_TORAM]               = &toram.hw,
>>>> +    [AUD_CLKID_EQDRC]               = &eqdrc.hw,
>>>> +    [AUD_CLKID_RESAMPLE_B]          = &resample_b.hw,
>>>> +    [AUD_CLKID_TOVAD]               = &tovad.hw,
>>>> +    [AUD_CLKID_LOCKER]              = &locker.hw,
>>>> +    [AUD_CLKID_SPDIFIN_LB]          = &spdifin_lb.hw,
>>>> +    [AUD_CLKID_FRDDR_D]             = &frddr_d.hw,
>>>> +    [AUD_CLKID_TODDR_D]             = &toddr_d.hw,
>>>> +    [AUD_CLKID_LOOPBACK_B]          = &loopback_b.hw,
>>>> +    [AUD_CLKID_CLK81_EN]            = &sm1_clk81_en.hw,
>>>> +    [AUD_CLKID_SYSCLK_A_DIV]        = &sm1_sysclk_a_div.hw,
>>>> +    [AUD_CLKID_SYSCLK_A_EN]         = &sm1_sysclk_a_en.hw,
>>>> +    [AUD_CLKID_SYSCLK_B_DIV]        = &sm1_sysclk_b_div.hw,
>>>> +    [AUD_CLKID_SYSCLK_B_EN]         = &sm1_sysclk_b_en.hw,
>>>> +    [AUD_CLKID_EARCRX]              = &earcrx.hw,
>>>> +    [AUD_CLKID_EARCRX_CMDC_SEL]     = &sm1_earcrx_cmdc_clk_sel.hw,
>>>> +    [AUD_CLKID_EARCRX_CMDC_DIV]     = &sm1_earcrx_cmdc_clk_div.hw,
>>>> +    [AUD_CLKID_EARCRX_CMDC]         = &sm1_earcrx_cmdc_clk.hw,
>>>> +    [AUD_CLKID_EARCRX_DMAC_SEL]     = &sm1_earcrx_dmac_clk_sel.hw,
>>>> +    [AUD_CLKID_EARCRX_DMAC_DIV]     = &sm1_earcrx_dmac_clk_div.hw,
>>>> +    [AUD_CLKID_EARCRX_DMAC]         = &sm1_earcrx_dmac_clk.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD0_SEL]           = 
>>>> &s4_tdm_mclk_pad0_sel.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD1_SEL]       = &s4_tdm_mclk_pad1_sel.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD0_DIV]           = 
>>>> &s4_tdm_mclk_pad0_div.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD1_DIV]       = &s4_tdm_mclk_pad1_div.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD2]               = &s4_tdm_mclk_pad_2.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD2_SEL]           = 
>>>> &s4_tdm_mclk_pad2_sel.hw,
>>>> +    [AUD_CLKID_TDM_MCLK_PAD2_DIV]       = &s4_tdm_mclk_pad2_div.hw,
>>>> +    [AUD_CLKID_TDM_SCLK_PAD3]       = &s4_tdm_sclk_pad_3.hw,
>>>> +    [AUD_CLKID_TDM_SCLK_PAD4]       = &s4_tdm_sclk_pad_4.hw,
>>>> +    [AUD_CLKID_TDM_LRCLK_PAD3]      = &s4_tdm_lrclk_pad_3.hw,
>>>> +    [AUD_CLKID_TDM_LRCLK_PAD4]      = &s4_tdm_lrclk_pad_4.hw,
>>>> +};
>>>>
>>>>   /* Convenience table to populate regmap in .probe(). */
>>>>   static struct clk_regmap *const axg_clk_regmaps[] = {
>>>> @@ -1678,6 +1902,174 @@ static struct clk_regmap *const 
>>>> sm1_clk_regmaps[] = {
>>>>       &sm1_earcrx_dmac_clk,
>>>>   };
>>>>
>>>> +static struct clk_regmap *const s4_clk_regmaps[] = {
>>>> +    &ddr_arb,
>>>> +    &pdm,
>>>> +    &tdmin_a,
>>>> +    &tdmin_b,
>>>> +    &tdmin_c,
>>>> +    &tdmin_lb,
>>>> +    &tdmout_a,
>>>> +    &tdmout_b,
>>>> +    &tdmout_c,
>>>> +    &frddr_a,
>>>> +    &frddr_b,
>>>> +    &frddr_c,
>>>> +    &toddr_a,
>>>> +    &toddr_b,
>>>> +    &toddr_c,
>>>> +    &loopback,
>>>> +    &spdifin,
>>>> +    &spdifout,
>>>> +    &resample,
>>>> +    &spdifout_b,
>>>> +    &sm1_mst_a_mclk_sel,
>>>> +    &sm1_mst_b_mclk_sel,
>>>> +    &sm1_mst_c_mclk_sel,
>>>> +    &sm1_mst_d_mclk_sel,
>>>> +    &sm1_mst_e_mclk_sel,
>>>> +    &sm1_mst_f_mclk_sel,
>>>> +    &sm1_mst_a_mclk_div,
>>>> +    &sm1_mst_b_mclk_div,
>>>> +    &sm1_mst_c_mclk_div,
>>>> +    &sm1_mst_d_mclk_div,
>>>> +    &sm1_mst_e_mclk_div,
>>>> +    &sm1_mst_f_mclk_div,
>>>> +    &sm1_mst_a_mclk,
>>>> +    &sm1_mst_b_mclk,
>>>> +    &sm1_mst_c_mclk,
>>>> +    &sm1_mst_d_mclk,
>>>> +    &sm1_mst_e_mclk,
>>>> +    &sm1_mst_f_mclk,
>>>> +    &spdifout_clk_sel,
>>>> +    &spdifout_clk_div,
>>>> +    &spdifout_clk,
>>>> +    &spdifin_clk_sel,
>>>> +    &spdifin_clk_div,
>>>> +    &spdifin_clk,
>>>> +    &pdm_dclk_sel,
>>>> +    &pdm_dclk_div,
>>>> +    &pdm_dclk,
>>>> +    &pdm_sysclk_sel,
>>>> +    &pdm_sysclk_div,
>>>> +    &pdm_sysclk,
>>>> +    &mst_a_sclk_pre_en,
>>>> +    &mst_b_sclk_pre_en,
>>>> +    &mst_c_sclk_pre_en,
>>>> +    &mst_d_sclk_pre_en,
>>>> +    &mst_e_sclk_pre_en,
>>>> +    &mst_f_sclk_pre_en,
>>>> +    &mst_a_sclk_div,
>>>> +    &mst_b_sclk_div,
>>>> +    &mst_c_sclk_div,
>>>> +    &mst_d_sclk_div,
>>>> +    &mst_e_sclk_div,
>>>> +    &mst_f_sclk_div,
>>>> +    &mst_a_sclk_post_en,
>>>> +    &mst_b_sclk_post_en,
>>>> +    &mst_c_sclk_post_en,
>>>> +    &mst_d_sclk_post_en,
>>>> +    &mst_e_sclk_post_en,
>>>> +    &mst_f_sclk_post_en,
>>>> +    &mst_a_sclk,
>>>> +    &mst_b_sclk,
>>>> +    &mst_c_sclk,
>>>> +    &mst_d_sclk,
>>>> +    &mst_e_sclk,
>>>> +    &mst_f_sclk,
>>>> +    &mst_a_lrclk_div,
>>>> +    &mst_b_lrclk_div,
>>>> +    &mst_c_lrclk_div,
>>>> +    &mst_d_lrclk_div,
>>>> +    &mst_e_lrclk_div,
>>>> +    &mst_f_lrclk_div,
>>>> +    &mst_a_lrclk,
>>>> +    &mst_b_lrclk,
>>>> +    &mst_c_lrclk,
>>>> +    &mst_d_lrclk,
>>>> +    &mst_e_lrclk,
>>>> +    &mst_f_lrclk,
>>>> +    &tdmin_a_sclk_sel,
>>>> +    &tdmin_b_sclk_sel,
>>>> +    &tdmin_c_sclk_sel,
>>>> +    &tdmin_lb_sclk_sel,
>>>> +    &tdmout_a_sclk_sel,
>>>> +    &tdmout_b_sclk_sel,
>>>> +    &tdmout_c_sclk_sel,
>>>> +    &tdmin_a_sclk_pre_en,
>>>> +    &tdmin_b_sclk_pre_en,
>>>> +    &tdmin_c_sclk_pre_en,
>>>> +    &tdmin_lb_sclk_pre_en,
>>>> +    &tdmout_a_sclk_pre_en,
>>>> +    &tdmout_b_sclk_pre_en,
>>>> +    &tdmout_c_sclk_pre_en,
>>>> +    &tdmin_a_sclk_post_en,
>>>> +    &tdmin_b_sclk_post_en,
>>>> +    &tdmin_c_sclk_post_en,
>>>> +    &tdmin_lb_sclk_post_en,
>>>> +    &tdmout_a_sclk_post_en,
>>>> +    &tdmout_b_sclk_post_en,
>>>> +    &tdmout_c_sclk_post_en,
>>>> +    &tdmin_a_sclk,
>>>> +    &tdmin_b_sclk,
>>>> +    &tdmin_c_sclk,
>>>> +    &tdmin_lb_sclk,
>>>> +    &g12a_tdmout_a_sclk,
>>>> +    &g12a_tdmout_b_sclk,
>>>> +    &g12a_tdmout_c_sclk,
>>>> +    &tdmin_a_lrclk,
>>>> +    &tdmin_b_lrclk,
>>>> +    &tdmin_c_lrclk,
>>>> +    &tdmin_lb_lrclk,
>>>> +    &tdmout_a_lrclk,
>>>> +    &tdmout_b_lrclk,
>>>> +    &tdmout_c_lrclk,
>>>> +    &spdifout_b_clk_sel,
>>>> +    &spdifout_b_clk_div,
>>>> +    &spdifout_b_clk,
>>>> +    &s4_tdm_mclk_pad_0,
>>>> +    &s4_tdm_mclk_pad_1,
>>>> +    &s4_tdm_lrclk_pad_0,
>>>> +    &s4_tdm_lrclk_pad_1,
>>>> +    &s4_tdm_lrclk_pad_2,
>>>> +    &s4_tdm_sclk_pad_0,
>>>> +    &s4_tdm_sclk_pad_1,
>>>> +    &s4_tdm_sclk_pad_2,
>>>> +    &sm1_aud_top,
>>>> +    &toram,
>>>> +    &eqdrc,
>>>> +    &resample_b,
>>>> +    &tovad,
>>>> +    &locker,
>>>> +    &spdifin_lb,
>>>> +    &frddr_d,
>>>> +    &toddr_d,
>>>> +    &loopback_b,
>>>> +    &sm1_clk81_en,
>>>> +    &sm1_sysclk_a_div,
>>>> +    &sm1_sysclk_a_en,
>>>> +    &sm1_sysclk_b_div,
>>>> +    &sm1_sysclk_b_en,
>>>> +    &earcrx,
>>>> +    &sm1_earcrx_cmdc_clk_sel,
>>>> +    &sm1_earcrx_cmdc_clk_div,
>>>> +    &sm1_earcrx_cmdc_clk,
>>>> +    &sm1_earcrx_dmac_clk_sel,
>>>> +    &sm1_earcrx_dmac_clk_div,
>>>> +    &sm1_earcrx_dmac_clk,
>>>> +    &s4_tdm_mclk_pad0_sel,
>>>> +    &s4_tdm_mclk_pad1_sel,
>>>> +    &s4_tdm_mclk_pad0_div,
>>>> +    &s4_tdm_mclk_pad1_div,
>>>> +    &s4_tdm_mclk_pad_2,
>>>> +    &s4_tdm_mclk_pad2_sel,
>>>> +    &s4_tdm_mclk_pad2_div,
>>>> +    &s4_tdm_sclk_pad_3,
>>>> +    &s4_tdm_sclk_pad_4,
>>>> +    &s4_tdm_lrclk_pad_3,
>>>> +    &s4_tdm_lrclk_pad_4,
>>>> +};
>>>> +
>>>>   struct axg_audio_reset_data {
>>>>       struct reset_controller_dev rstc;
>>>>       struct regmap *map;
>>>> @@ -1822,7 +2214,6 @@ static int axg_audio_clkc_probe(struct 
>>>> platform_device *pdev)
>>>>                       continue;
>>>>
>>>>               name = hw->init->name;
>>>> -
>>>>               ret = devm_clk_hw_register(dev, hw);
>>>>               if (ret) {
>>>>                       dev_err(dev, "failed to register clock %s\n", 
>>>> name);
>>>> @@ -1886,6 +2277,18 @@ static const struct audioclk_data 
>>>> sm1_audioclk_data = {
>>>>       .max_register = AUDIO_EARCRX_DMAC_CLK_CTRL,
>>>>   };
>>>>
>>>> +static const struct audioclk_data s4_audioclk_data = {
>>>> +    .regmap_clks = s4_clk_regmaps,
>>>> +    .regmap_clk_num = ARRAY_SIZE(s4_clk_regmaps),
>>>> +    .hw_clks = {
>>>> +            .hws = s4_audio_hw_clks,
>>>> +            .num = ARRAY_SIZE(s4_audio_hw_clks),
>>>> +    },
>>>> +    .reset_offset = AUDIO_SM1_SW_RESET0,
>>>> +    .reset_num = 39,
>>>> +    .max_register = AUDIO_S4_SCLK_PAD_CTRL1,
>>>> +};
>>>> +
>>>>   static const struct of_device_id clkc_match_table[] = {
>>>>       {
>>>>               .compatible = "amlogic,axg-audio-clkc",
>>>> @@ -1896,7 +2299,10 @@ static const struct of_device_id 
>>>> clkc_match_table[] = {
>>>>       }, {
>>>>               .compatible = "amlogic,sm1-audio-clkc",
>>>>               .data = &sm1_audioclk_data
>>>> -    }, {}
>>>> +    }, {
>>>> +            .compatible = "amlogic,s4-audio-clkc",
>>>> +            .data = &s4_audioclk_data
>>>> +    }, { }
>>>>   };
>>>>   MODULE_DEVICE_TABLE(of, clkc_match_table);
>>>>
>>>> diff --git a/drivers/clk/meson/axg-audio.h 
>>>> b/drivers/clk/meson/axg-audio.h
>>>> index 
>>>> 9e7765b630c96a8029140539ffda789b7db5277a..2dd1c41d775da8f91ed281470d06e9c970cfc92c 
>>>> 100644
>>>> --- a/drivers/clk/meson/axg-audio.h
>>>> +++ b/drivers/clk/meson/axg-audio.h
>>>> @@ -66,5 +66,9 @@
>>>>   #define AUDIO_CLK81_EN              0x034
>>>>   #define AUDIO_EARCRX_CMDC_CLK_CTRL  0x0D0
>>>>   #define AUDIO_EARCRX_DMAC_CLK_CTRL  0x0D4
>>>> +#define AUDIO_S4_MCLK_PAD_CTRL0 0xE80
>>>> +#define AUDIO_S4_MCLK_PAD_CTRL1 0xE84
>>>> +#define AUDIO_S4_SCLK_PAD_CTRL0 0xE88
>>>> +#define AUDIO_S4_SCLK_PAD_CTRL1 0xE8C
>>>>
>>>>   #endif /*__AXG_AUDIO_CLKC_H */
>> -- 
>> Jerome

Powered by blists - more mailing lists

Powered by Openwall GNU/*/Linux Powered by OpenVZ