Hi Marijn,
Thank you for the patch! Perhaps something to improve:
[auto build test WARNING on lee-backlight/for-backlight-next]
[also build test WARNING on robh/for-next v5.16-rc1 next-20211115]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Marijn-Suijten/backlight-qcom-wl...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/lee/backlight.git
for-backlight-next
config: x86_64-randconfig-s022-20211115 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce:
# apt-get install sparse
# sparse version: v0.6.4-dirty
#
https://github.com/0day-ci/linux/commit/94425d366b39b6eea4acc5623f8a66046...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
Marijn-Suijten/backlight-qcom-wled-fix-and-solidify-handling-of-enabled-strings/20211112-083005
git checkout 94425d366b39b6eea4acc5623f8a66046eb62bdb
# save the attached .config to linux build tree
make W=1 C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__' ARCH=x86_64
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
sparse warnings: (new ones prefixed by >>)
> drivers/video/backlight/qcom-wled.c:236:11: sparse: sparse:
incorrect type in assignment (different base types) @@ expected unsigned short
[usertype] v @@ got restricted __le16 [usertype] @@
drivers/video/backlight/qcom-wled.c:236:11: sparse: expected unsigned short [usertype]
v
drivers/video/backlight/qcom-wled.c:236:11: sparse: got restricted __le16
[usertype]
drivers/video/backlight/qcom-wled.c:258:11: sparse: sparse: incorrect type in
assignment (different base types) @@ expected unsigned short [usertype] v @@ got
restricted __le16 [usertype] @@
drivers/video/backlight/qcom-wled.c:258:11: sparse: expected unsigned short
[usertype] v
drivers/video/backlight/qcom-wled.c:258:11: sparse: got restricted __le16
[usertype]
drivers/video/backlight/qcom-wled.c:280:11: sparse: sparse: incorrect type in
assignment (different base types) @@ expected unsigned short [usertype] v @@ got
restricted __le16 [usertype] @@
drivers/video/backlight/qcom-wled.c:280:11: sparse: expected unsigned short
[usertype] v
drivers/video/backlight/qcom-wled.c:280:11: sparse: got restricted __le16
[usertype]
vim +236 drivers/video/backlight/qcom-wled.c
230
231 static int wled3_set_brightness(struct wled *wled, u16 brightness)
232 {
233 int rc, i;
234 u16 v;
235
236 v = cpu_to_le16(brightness & WLED3_SINK_REG_BRIGHT_MAX);
237
238 for (i = 0; i < wled->cfg.num_strings; ++i) {
239 rc = regmap_bulk_write(wled->regmap, wled->ctrl_addr +
240 WLED3_SINK_REG_BRIGHT(i),
241 &v, sizeof(v));
242 if (rc < 0)
243 return rc;
244 }
245
246 return 0;
247 }
248
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org