29 Oct
2020
29 Oct
'20
4:47 p.m.
+static int wsa_macro_set_prim_interpolator_rate(struct snd_soc_dai *dai,
u8 int_prim_fs_rate_reg_val,u32 sample_rate)+{
- u8 int_1_mix1_inp;
- u32 j, port = 0;
nit-pick: initialization of port not needed
- u16 int_mux_cfg0, int_mux_cfg1;
- u16 int_fs_reg;
- u8 int_mux_cfg0_val, int_mux_cfg1_val;
- u8 inp0_sel, inp1_sel, inp2_sel;
- struct snd_soc_component *component = dai->component;
- struct wsa_macro *wsa = snd_soc_component_get_drvdata(component);
- for_each_set_bit(port, &wsa->active_ch_mask[dai->id], WSA_MACRO_RX_MAX) {
int_1_mix1_inp = port;if ((int_1_mix1_inp < WSA_MACRO_RX0) || (int_1_mix1_inp > WSA_MACRO_RX_MIX1)) {dev_err(component->dev, "%s: Invalid RX port, Dai ID is %d\n",__func__, dai->id);return -EINVAL;}int_mux_cfg0 = CDC_WSA_RX_INP_MUX_RX_INT0_CFG0;/** Loop through all interpolator MUX inputs and find out* to which interpolator input, the cdc_dma rx port* is connected*/for (j = 0; j < NUM_INTERPOLATORS; j++) {int_mux_cfg1 = int_mux_cfg0 + WSA_MACRO_MUX_CFG1_OFFSET;int_mux_cfg0_val = snd_soc_component_read(component,int_mux_cfg0);int_mux_cfg1_val = snd_soc_component_read(component,int_mux_cfg1);inp0_sel = int_mux_cfg0_val & WSA_MACRO_MUX_INP_MASK1;inp1_sel = (int_mux_cfg0_val >>WSA_MACRO_MUX_INP_SHFT) &WSA_MACRO_MUX_INP_MASK1;inp2_sel = (int_mux_cfg1_val >>WSA_MACRO_MUX_INP_SHFT) &WSA_MACRO_MUX_INP_MASK1;if ((inp0_sel == int_1_mix1_inp + INTn_1_INP_SEL_RX0) ||(inp1_sel == int_1_mix1_inp + INTn_1_INP_SEL_RX0) ||(inp2_sel == int_1_mix1_inp + INTn_1_INP_SEL_RX0)) {int_fs_reg = CDC_WSA_RX0_RX_PATH_CTL +WSA_MACRO_RX_PATH_OFFSET * j;/* sample_rate is in Hz */snd_soc_component_update_bits(component,int_fs_reg,WSA_MACRO_FS_RATE_MASK,int_prim_fs_rate_reg_val);}int_mux_cfg0 += WSA_MACRO_MUX_CFG_OFFSET;}- }
- return 0;
+}
+static int wsa_macro_set_mix_interpolator_rate(struct snd_soc_dai *dai,
u8 int_mix_fs_rate_reg_val,u32 sample_rate)+{
- u8 int_2_inp;
- u32 j, port = 0;
same here, port is set in the for_each_bit loop
[...]
+static void wsa_macro_mclk_enable(struct wsa_macro *wsa, bool mclk_enable) +{
- struct regmap *regmap = wsa->regmap;
- if (mclk_enable) {
if (wsa->wsa_mclk_users == 0) {regcache_mark_dirty(regmap);regcache_sync_region(regmap, 0x0, WSA_MAX_OFFSET);
I am not a regcache/regmap expert but the sequence seems odd. I could be wrong, but one would typically sync *after* doing changes, no?
/* 9.6MHz MCLK, set value 0x00 if other frequency */regmap_update_bits(regmap, CDC_WSA_TOP_FREQ_MCLK, 0x01, 0x01);regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_MCLK_CONTROL,CDC_WSA_MCLK_EN_MASK,CDC_WSA_MCLK_ENABLE);regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_FS_CNT_CONTROL,CDC_WSA_FS_CNT_EN_MASK,CDC_WSA_FS_CNT_ENABLE);}wsa->wsa_mclk_users++;- } else {
if (wsa->wsa_mclk_users <= 0) {dev_err(wsa->dev, "clock already disabled\n");wsa->wsa_mclk_users = 0;return;}wsa->wsa_mclk_users--;if (wsa->wsa_mclk_users == 0) {regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_FS_CNT_CONTROL,CDC_WSA_FS_CNT_EN_MASK,CDC_WSA_FS_CNT_DISABLE);regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_MCLK_CONTROL,CDC_WSA_MCLK_EN_MASK,CDC_WSA_MCLK_DISABLE);}- }
+}
[...]
+static int wsa_swrm_clock(struct wsa_macro *wsa, bool enable) +{
- struct regmap *regmap = wsa->regmap;
- if (enable) {
wsa_macro_mclk_enable(wsa, true);if (wsa->reset_swr)regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_SWR_CONTROL,0x02, 0x02);regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_SWR_CONTROL,0x01, 0x01);
Touching the same register once conditionally and then unconditionally looks suspicious. Make add a comment to explain this is intentional and not an indentation mistake.
if (wsa->reset_swr)regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_SWR_CONTROL,0x02, 0x00);wsa->reset_swr = false;- } else {
regmap_update_bits(regmap,CDC_WSA_CLK_RST_CTRL_SWR_CONTROL,0x01, 0x00);wsa_macro_mclk_enable(wsa, false);- }
- return 0;
+}