From: kbuild test robot <lkp@intel.com>
To: Shuming Fan <shumingf@realtek.com>
Cc: alsa-devel@alsa-project.org, Mark Brown <broonie@kernel.org>,
kbuild-all@01.org
Subject: [asoc:for-5.3 110/113] sound/soc/codecs/rt1011.c:1148:31: sparse: sparse: incorrect type in assignment (different base types)
Date: Wed, 5 Jun 2019 00:25:04 +0800 [thread overview]
Message-ID: <201906050002.8Xgg5xOE%lkp@intel.com> (raw)
tree: https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-5.3
head: 9b8e8b893ff52264df2b54a6820ae3c4bb820624
commit: d6e65bb7ff0d360c4c5462c3d0b237f2a07e5312 [110/113] ASoC: rt1011: Add RT1011 amplifier driver
reproduce:
# apt-get install sparse
# sparse version: v0.6.1-rc1-7-g2b96cd8-dirty
git checkout d6e65bb7ff0d360c4c5462c3d0b237f2a07e5312
make ARCH=x86_64 allmodconfig
make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'
If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp@intel.com>
sparse warnings: (new ones prefixed by >>)
>> sound/soc/codecs/rt1011.c:1148:31: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned short reg @@ got restricted __lunsigned short reg @@
>> sound/soc/codecs/rt1011.c:1148:31: sparse: expected unsigned short reg
>> sound/soc/codecs/rt1011.c:1148:31: sparse: got restricted __le16 [usertype]
>> sound/soc/codecs/rt1011.c:1149:31: sparse: sparse: incorrect type in assignment (different base types) @@ expected unsigned short val @@ got restricted __lunsigned short val @@
>> sound/soc/codecs/rt1011.c:1149:31: sparse: expected unsigned short val
sound/soc/codecs/rt1011.c:1149:31: sparse: got restricted __le16 [usertype]
>> sound/soc/codecs/rt1011.c:1190:41: sparse: sparse: cast to restricted __le16
sound/soc/codecs/rt1011.c:1191:41: sparse: sparse: cast to restricted __le16
vim +1148 sound/soc/codecs/rt1011.c
1117
1118 static int rt1011_bq_drc_coeff_get(struct snd_kcontrol *kcontrol,
1119 struct snd_ctl_elem_value *ucontrol)
1120 {
1121 struct snd_soc_component *component =
1122 snd_soc_kcontrol_component(kcontrol);
1123 struct rt1011_priv *rt1011 =
1124 snd_soc_component_get_drvdata(component);
1125 struct rt1011_bq_drc_params *bq_drc_info;
1126 struct rt1011_bq_drc_params *params =
1127 (struct rt1011_bq_drc_params *)ucontrol->value.integer.value;
1128 unsigned int i, mode_idx = 0;
1129
1130 if (strstr(ucontrol->id.name, "AdvanceMode Initial Set"))
1131 mode_idx = RT1011_ADVMODE_INITIAL_SET;
1132 else if (strstr(ucontrol->id.name, "AdvanceMode SEP BQ Coeff"))
1133 mode_idx = RT1011_ADVMODE_SEP_BQ_COEFF;
1134 else if (strstr(ucontrol->id.name, "AdvanceMode EQ BQ Coeff"))
1135 mode_idx = RT1011_ADVMODE_EQ_BQ_COEFF;
1136 else if (strstr(ucontrol->id.name, "AdvanceMode BQ UI Coeff"))
1137 mode_idx = RT1011_ADVMODE_BQ_UI_COEFF;
1138 else if (strstr(ucontrol->id.name, "AdvanceMode SmartBoost Coeff"))
1139 mode_idx = RT1011_ADVMODE_SMARTBOOST_COEFF;
1140 else
1141 return -EINVAL;
1142
1143 pr_info("%s, id.name=%s, mode_idx=%d\n", __func__,
1144 ucontrol->id.name, mode_idx);
1145 bq_drc_info = rt1011->bq_drc_params[mode_idx];
1146
1147 for (i = 0; i < RT1011_BQ_DRC_NUM; i++) {
> 1148 params[i].reg = cpu_to_le16(bq_drc_info[i].reg);
> 1149 params[i].val = cpu_to_le16(bq_drc_info[i].val);
1150 }
1151
1152 return 0;
1153 }
1154
1155 static int rt1011_bq_drc_coeff_put(struct snd_kcontrol *kcontrol,
1156 struct snd_ctl_elem_value *ucontrol)
1157 {
1158 struct snd_soc_component *component =
1159 snd_soc_kcontrol_component(kcontrol);
1160 struct rt1011_priv *rt1011 =
1161 snd_soc_component_get_drvdata(component);
1162 struct rt1011_bq_drc_params *bq_drc_info;
1163 struct rt1011_bq_drc_params *params =
1164 (struct rt1011_bq_drc_params *)ucontrol->value.integer.value;
1165 unsigned int i, mode_idx = 0;
1166
1167 if (!component->card->instantiated)
1168 return 0;
1169
1170 if (strstr(ucontrol->id.name, "AdvanceMode Initial Set"))
1171 mode_idx = RT1011_ADVMODE_INITIAL_SET;
1172 else if (strstr(ucontrol->id.name, "AdvanceMode SEP BQ Coeff"))
1173 mode_idx = RT1011_ADVMODE_SEP_BQ_COEFF;
1174 else if (strstr(ucontrol->id.name, "AdvanceMode EQ BQ Coeff"))
1175 mode_idx = RT1011_ADVMODE_EQ_BQ_COEFF;
1176 else if (strstr(ucontrol->id.name, "AdvanceMode BQ UI Coeff"))
1177 mode_idx = RT1011_ADVMODE_BQ_UI_COEFF;
1178 else if (strstr(ucontrol->id.name, "AdvanceMode SmartBoost Coeff"))
1179 mode_idx = RT1011_ADVMODE_SMARTBOOST_COEFF;
1180 else
1181 return -EINVAL;
1182
1183 bq_drc_info = rt1011->bq_drc_params[mode_idx];
1184 memset(bq_drc_info, 0,
1185 sizeof(struct rt1011_bq_drc_params) * RT1011_BQ_DRC_NUM);
1186
1187 pr_info("%s, id.name=%s, mode_idx=%d\n", __func__,
1188 ucontrol->id.name, mode_idx);
1189 for (i = 0; i < RT1011_BQ_DRC_NUM; i++) {
> 1190 bq_drc_info[i].reg = le16_to_cpu(params[i].reg);
1191 bq_drc_info[i].val = le16_to_cpu(params[i].val);
1192 }
1193
1194 for (i = 0; i < RT1011_BQ_DRC_NUM; i++) {
1195 if (bq_drc_info[i].reg == 0)
1196 break;
1197 else if (rt1011_validate_bq_drc_coeff(bq_drc_info[i].reg)) {
1198 snd_soc_component_write(component, bq_drc_info[i].reg,
1199 bq_drc_info[i].val);
1200 }
1201 }
1202
1203 return 0;
1204 }
1205
---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation
reply other threads:[~2019-06-04 16:25 UTC|newest]
Thread overview: [no followups] expand[flat|nested] mbox.gz Atom feed
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=201906050002.8Xgg5xOE%lkp@intel.com \
--to=lkp@intel.com \
--cc=alsa-devel@alsa-project.org \
--cc=broonie@kernel.org \
--cc=kbuild-all@01.org \
--cc=shumingf@realtek.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).