[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <202410051744.I4PVYrMx-lkp@intel.com>
Date: Sat, 5 Oct 2024 17:50:34 +0800
From: kernel test robot <lkp@...el.com>
To: Manuel Quintero F <sakunix@...oo.com>, duncan.sands@...e.fr
Cc: oe-kbuild-all@...ts.linux.dev, gregkh@...uxfoundation.org,
linux-usb@...r.kernel.org, linux-kernel@...r.kernel.org,
Manuel Quintero F <sakunix@...oo.com>
Subject: Re: [PATCH] USB: atm: speedtch: do not use assignment in if condition
Hi Manuel,
kernel test robot noticed the following build warnings:
[auto build test WARNING on usb/usb-testing]
[also build test WARNING on usb/usb-next usb/usb-linus linus/master v6.12-rc1 next-20241004]
[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#_base_tree_information]
url: https://github.com/intel-lab-lkp/linux/commits/Manuel-Quintero-F/USB-atm-speedtch-do-not-use-assignment-in-if-condition/20241004-113643
base: https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
patch link: https://lore.kernel.org/r/20241004030512.2036-1-sakunix%40yahoo.com
patch subject: [PATCH] USB: atm: speedtch: do not use assignment in if condition
config: sh-allmodconfig (https://download.01.org/0day-ci/archive/20241005/202410051744.I4PVYrMx-lkp@intel.com/config)
compiler: sh4-linux-gcc (GCC) 14.1.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241005/202410051744.I4PVYrMx-lkp@intel.com/reproduce)
If you fix the issue in a separate patch/commit (i.e. not just a new version of
the same patch/commit), kindly add following tags
| Reported-by: kernel test robot <lkp@...el.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202410051744.I4PVYrMx-lkp@intel.com/
All warnings (new ones prefixed by >>):
drivers/usb/atm/speedtch.c: In function 'speedtch_upload_firmware':
drivers/usb/atm/speedtch.c:327:85: error: expected ')' before 'if'
327 | ret = usb_set_interface(usb_dev, INTERFACE_DATA, instance->params.altsetting
| ~ ^
| )
328 |
329 | if (ret < 0) {
| ~~
drivers/usb/atm/speedtch.c:346:20: error: expected ';' before '}' token
346 | return ret;
| ^
| ;
347 | }
| ~
drivers/usb/atm/speedtch.c:317:17: error: label 'out_free' used but not defined
317 | goto out_free;
| ^~~~
drivers/usb/atm/speedtch.c:247:17: error: label 'out' used but not defined
247 | goto out;
| ^~~~
>> drivers/usb/atm/speedtch.c:347:1: warning: control reaches end of non-void function [-Wreturn-type]
347 | }
| ^
drivers/usb/atm/speedtch.c: At top level:
drivers/usb/atm/speedtch.c:160:13: warning: 'speedtch_test_sequence' defined but not used [-Wunused-function]
160 | static void speedtch_test_sequence(struct speedtch_instance_data *instance)
| ^~~~~~~~~~~~~~~~~~~~~~
drivers/usb/atm/speedtch.c:144:13: warning: 'speedtch_set_swbuff' defined but not used [-Wunused-function]
144 | static void speedtch_set_swbuff(struct speedtch_instance_data *instance, int state)
| ^~~~~~~~~~~~~~~~~~~
vim +347 drivers/usb/atm/speedtch.c
^1da177e4c3f41 Linus Torvalds 2005-04-16 229
48da7267ff1631 Duncan Sands 2005-05-11 230 static int speedtch_upload_firmware(struct speedtch_instance_data *instance,
48da7267ff1631 Duncan Sands 2005-05-11 231 const struct firmware *fw1,
48da7267ff1631 Duncan Sands 2005-05-11 232 const struct firmware *fw2)
^1da177e4c3f41 Linus Torvalds 2005-04-16 233 {
48da7267ff1631 Duncan Sands 2005-05-11 234 unsigned char *buffer;
48da7267ff1631 Duncan Sands 2005-05-11 235 struct usbatm_data *usbatm = instance->usbatm;
48da7267ff1631 Duncan Sands 2005-05-11 236 struct usb_device *usb_dev = usbatm->usb_dev;
48da7267ff1631 Duncan Sands 2005-05-11 237 int actual_length;
48da7267ff1631 Duncan Sands 2005-05-11 238 int ret = 0;
48da7267ff1631 Duncan Sands 2005-05-11 239 int offset;
48da7267ff1631 Duncan Sands 2005-05-11 240
48da7267ff1631 Duncan Sands 2005-05-11 241 usb_dbg(usbatm, "%s entered\n", __func__);
48da7267ff1631 Duncan Sands 2005-05-11 242
3383ee4c3abf2e Greg Kroah-Hartman 2015-04-30 243 buffer = (unsigned char *)__get_free_page(GFP_KERNEL);
3383ee4c3abf2e Greg Kroah-Hartman 2015-04-30 244 if (!buffer) {
48da7267ff1631 Duncan Sands 2005-05-11 245 ret = -ENOMEM;
48da7267ff1631 Duncan Sands 2005-05-11 246 usb_dbg(usbatm, "%s: no memory for buffer!\n", __func__);
48da7267ff1631 Duncan Sands 2005-05-11 247 goto out;
48da7267ff1631 Duncan Sands 2005-05-11 248 }
48da7267ff1631 Duncan Sands 2005-05-11 249
011db815231f40 Micah Gruber 2007-09-05 250 if (!usb_ifnum_to_if(usb_dev, 2)) {
48da7267ff1631 Duncan Sands 2005-05-11 251 ret = -ENODEV;
48da7267ff1631 Duncan Sands 2005-05-11 252 usb_dbg(usbatm, "%s: interface not found!\n", __func__);
48da7267ff1631 Duncan Sands 2005-05-11 253 goto out_free;
48da7267ff1631 Duncan Sands 2005-05-11 254 }
48da7267ff1631 Duncan Sands 2005-05-11 255
48da7267ff1631 Duncan Sands 2005-05-11 256 /* URB 7 */
48da7267ff1631 Duncan Sands 2005-05-11 257 if (dl_512_first) { /* some modems need a read before writing the firmware */
48da7267ff1631 Duncan Sands 2005-05-11 258 ret = usb_bulk_msg(usb_dev, usb_rcvbulkpipe(usb_dev, ENDPOINT_FIRMWARE),
48da7267ff1631 Duncan Sands 2005-05-11 259 buffer, 0x200, &actual_length, 2000);
48da7267ff1631 Duncan Sands 2005-05-11 260
48da7267ff1631 Duncan Sands 2005-05-11 261 if (ret < 0 && ret != -ETIMEDOUT)
0ec3c7e856319b Duncan Sands 2006-01-17 262 usb_warn(usbatm, "%s: read BLOCK0 from modem failed (%d)!\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 263 else
48da7267ff1631 Duncan Sands 2005-05-11 264 usb_dbg(usbatm, "%s: BLOCK0 downloaded (%d bytes)\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 265 }
48da7267ff1631 Duncan Sands 2005-05-11 266
48da7267ff1631 Duncan Sands 2005-05-11 267 /* URB 8 : both leds are static green */
48da7267ff1631 Duncan Sands 2005-05-11 268 for (offset = 0; offset < fw1->size; offset += PAGE_SIZE) {
48da7267ff1631 Duncan Sands 2005-05-11 269 int thislen = min_t(int, PAGE_SIZE, fw1->size - offset);
48da7267ff1631 Duncan Sands 2005-05-11 270 memcpy(buffer, fw1->data + offset, thislen);
48da7267ff1631 Duncan Sands 2005-05-11 271
48da7267ff1631 Duncan Sands 2005-05-11 272 ret = usb_bulk_msg(usb_dev, usb_sndbulkpipe(usb_dev, ENDPOINT_FIRMWARE),
48da7267ff1631 Duncan Sands 2005-05-11 273 buffer, thislen, &actual_length, DATA_TIMEOUT);
48da7267ff1631 Duncan Sands 2005-05-11 274
48da7267ff1631 Duncan Sands 2005-05-11 275 if (ret < 0) {
0ec3c7e856319b Duncan Sands 2006-01-17 276 usb_err(usbatm, "%s: write BLOCK1 to modem failed (%d)!\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 277 goto out_free;
48da7267ff1631 Duncan Sands 2005-05-11 278 }
48da7267ff1631 Duncan Sands 2005-05-11 279 usb_dbg(usbatm, "%s: BLOCK1 uploaded (%zu bytes)\n", __func__, fw1->size);
48da7267ff1631 Duncan Sands 2005-05-11 280 }
48da7267ff1631 Duncan Sands 2005-05-11 281
48da7267ff1631 Duncan Sands 2005-05-11 282 /* USB led blinking green, ADSL led off */
48da7267ff1631 Duncan Sands 2005-05-11 283
48da7267ff1631 Duncan Sands 2005-05-11 284 /* URB 11 */
48da7267ff1631 Duncan Sands 2005-05-11 285 ret = usb_bulk_msg(usb_dev, usb_rcvbulkpipe(usb_dev, ENDPOINT_FIRMWARE),
48da7267ff1631 Duncan Sands 2005-05-11 286 buffer, 0x200, &actual_length, DATA_TIMEOUT);
48da7267ff1631 Duncan Sands 2005-05-11 287
48da7267ff1631 Duncan Sands 2005-05-11 288 if (ret < 0) {
0ec3c7e856319b Duncan Sands 2006-01-17 289 usb_err(usbatm, "%s: read BLOCK2 from modem failed (%d)!\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 290 goto out_free;
48da7267ff1631 Duncan Sands 2005-05-11 291 }
48da7267ff1631 Duncan Sands 2005-05-11 292 usb_dbg(usbatm, "%s: BLOCK2 downloaded (%d bytes)\n", __func__, actual_length);
48da7267ff1631 Duncan Sands 2005-05-11 293
48da7267ff1631 Duncan Sands 2005-05-11 294 /* URBs 12 to 139 - USB led blinking green, ADSL led off */
48da7267ff1631 Duncan Sands 2005-05-11 295 for (offset = 0; offset < fw2->size; offset += PAGE_SIZE) {
48da7267ff1631 Duncan Sands 2005-05-11 296 int thislen = min_t(int, PAGE_SIZE, fw2->size - offset);
48da7267ff1631 Duncan Sands 2005-05-11 297 memcpy(buffer, fw2->data + offset, thislen);
48da7267ff1631 Duncan Sands 2005-05-11 298
48da7267ff1631 Duncan Sands 2005-05-11 299 ret = usb_bulk_msg(usb_dev, usb_sndbulkpipe(usb_dev, ENDPOINT_FIRMWARE),
48da7267ff1631 Duncan Sands 2005-05-11 300 buffer, thislen, &actual_length, DATA_TIMEOUT);
48da7267ff1631 Duncan Sands 2005-05-11 301
48da7267ff1631 Duncan Sands 2005-05-11 302 if (ret < 0) {
0ec3c7e856319b Duncan Sands 2006-01-17 303 usb_err(usbatm, "%s: write BLOCK3 to modem failed (%d)!\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 304 goto out_free;
48da7267ff1631 Duncan Sands 2005-05-11 305 }
48da7267ff1631 Duncan Sands 2005-05-11 306 }
48da7267ff1631 Duncan Sands 2005-05-11 307 usb_dbg(usbatm, "%s: BLOCK3 uploaded (%zu bytes)\n", __func__, fw2->size);
48da7267ff1631 Duncan Sands 2005-05-11 308
48da7267ff1631 Duncan Sands 2005-05-11 309 /* USB led static green, ADSL led static red */
48da7267ff1631 Duncan Sands 2005-05-11 310
48da7267ff1631 Duncan Sands 2005-05-11 311 /* URB 142 */
48da7267ff1631 Duncan Sands 2005-05-11 312 ret = usb_bulk_msg(usb_dev, usb_rcvbulkpipe(usb_dev, ENDPOINT_FIRMWARE),
48da7267ff1631 Duncan Sands 2005-05-11 313 buffer, 0x200, &actual_length, DATA_TIMEOUT);
^1da177e4c3f41 Linus Torvalds 2005-04-16 314
^1da177e4c3f41 Linus Torvalds 2005-04-16 315 if (ret < 0) {
0ec3c7e856319b Duncan Sands 2006-01-17 316 usb_err(usbatm, "%s: read BLOCK4 from modem failed (%d)!\n", __func__, ret);
48da7267ff1631 Duncan Sands 2005-05-11 317 goto out_free;
48da7267ff1631 Duncan Sands 2005-05-11 318 }
48da7267ff1631 Duncan Sands 2005-05-11 319
48da7267ff1631 Duncan Sands 2005-05-11 320 /* success */
48da7267ff1631 Duncan Sands 2005-05-11 321 usb_dbg(usbatm, "%s: BLOCK4 downloaded (%d bytes)\n", __func__, actual_length);
48da7267ff1631 Duncan Sands 2005-05-11 322
48da7267ff1631 Duncan Sands 2005-05-11 323 /* Delay to allow firmware to start up. We can do this here
48da7267ff1631 Duncan Sands 2005-05-11 324 because we're in our own kernel thread anyway. */
48da7267ff1631 Duncan Sands 2005-05-11 325 msleep_interruptible(1000);
48da7267ff1631 Duncan Sands 2005-05-11 326
a23b48ae442d5f Manuel Quintero F 2024-10-03 327 ret = usb_set_interface(usb_dev, INTERFACE_DATA, instance->params.altsetting
a23b48ae442d5f Manuel Quintero F 2024-10-03 328
a23b48ae442d5f Manuel Quintero F 2024-10-03 329 if (ret < 0) {
6a4f1b41357d2b Duncan Sands 2006-10-05 330 usb_err(usbatm, "%s: setting interface to %d failed (%d)!\n", __func__, instance->params.altsetting, ret);
6f7494759870ec Duncan Sands 2006-01-13 331 goto out_free;
6f7494759870ec Duncan Sands 2006-01-13 332 }
6f7494759870ec Duncan Sands 2006-01-13 333
48da7267ff1631 Duncan Sands 2005-05-11 334 /* Enable software buffering, if requested */
48da7267ff1631 Duncan Sands 2005-05-11 335 if (sw_buffering)
48da7267ff1631 Duncan Sands 2005-05-11 336 speedtch_set_swbuff(instance, 1);
48da7267ff1631 Duncan Sands 2005-05-11 337
48da7267ff1631 Duncan Sands 2005-05-11 338 /* Magic spell; don't ask us what this does */
48da7267ff1631 Duncan Sands 2005-05-11 339 speedtch_test_sequence(instance);
48da7267ff1631 Duncan Sands 2005-05-11 340
48da7267ff1631 Duncan Sands 2005-05-11 341 ret = 0;
48da7267ff1631 Duncan Sands 2005-05-11 342
48da7267ff1631 Duncan Sands 2005-05-11 343 out_free:
48da7267ff1631 Duncan Sands 2005-05-11 344 free_page((unsigned long)buffer);
48da7267ff1631 Duncan Sands 2005-05-11 345 out:
^1da177e4c3f41 Linus Torvalds 2005-04-16 346 return ret;
^1da177e4c3f41 Linus Torvalds 2005-04-16 @347 }
^1da177e4c3f41 Linus Torvalds 2005-04-16 348
--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists