diff for duplicates of <202201241944.dodYYcTk-lkp@intel.com> diff --git a/a/1.txt b/N1/1.txt index 7686009..183261d 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,15 +1,6 @@ -CC: kbuild-all(a)lists.01.org -CC: linux-kernel(a)vger.kernel.org -TO: Antoniu Miclaus <antoniu.miclaus@analog.com> -CC: Jonathan Cameron <Jonathan.Cameron@huawei.com> -CC: Cai Huoqing <cai.huoqing@linux.dev> - tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dd81e1c7d5fb126e5fbc5c9e334d7b3ec29a16a0 commit: 0bb12606c05fe9737e3056fe76d6e4b9c2a87b57 iio:dac:ad7293: add support for AD7293 -date: 7 weeks ago -:::::: branch date: 19 hours ago -:::::: commit date: 7 weeks ago config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp(a)intel.com/config) compiler: nios2-linux-gcc (GCC) 11.2.0 @@ -22,41 +13,6 @@ drivers/iio/dac/ad7293.c:524 ad7293_read_raw() error: uninitialized symbol 'data vim +/data +524 drivers/iio/dac/ad7293.c -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 475 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 476 static int ad7293_read_raw(struct iio_dev *indio_dev, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 477 struct iio_chan_spec const *chan, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 478 int *val, int *val2, long info) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 479 { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 480 struct ad7293_state *st = iio_priv(indio_dev); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 481 int ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 482 u16 data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 483 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 484 switch (info) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 485 case IIO_CHAN_INFO_RAW: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 486 switch (chan->type) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 487 case IIO_VOLTAGE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 488 if (chan->output) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 489 ret = ad7293_ch_read_raw(st, AD7293_DAC, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 490 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 491 else -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 492 ret = ad7293_ch_read_raw(st, AD7293_ADC_VINX, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 493 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 494 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 495 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 496 case IIO_CURRENT: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 497 ret = ad7293_ch_read_raw(st, AD7293_ADC_ISENSE, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 498 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 499 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 500 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 501 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 502 ret = ad7293_ch_read_raw(st, AD7293_ADC_TSENSE, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 503 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 504 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 505 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 506 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 507 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 508 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 509 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 510 if (ret) 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 511 return ret; 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 512 @@ -70,6 +26,9 @@ vim +/data +524 drivers/iio/dac/ad7293.c 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 520 ret = ad7293_get_offset(st, 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 521 chan->channel + AD7293_VOUT_MIN_OFFSET_CH, 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 522 &data); + +ad7293_get_offset() can fail + 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 523 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 @524 data = FIELD_GET(AD7293_REG_VOUT_OFFSET_MSK, data); 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 525 } else { @@ -83,52 +42,6 @@ vim +/data +524 drivers/iio/dac/ad7293.c 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 533 &data); 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 534 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 535 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 536 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 537 ret = ad7293_get_offset(st, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 538 chan->channel + AD7293_TSENSE_MIN_OFFSET_CH, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 539 &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 540 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 541 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 542 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 543 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 544 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 545 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 546 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 547 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 548 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 549 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 550 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 551 case IIO_CHAN_INFO_SCALE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 552 switch (chan->type) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 553 case IIO_VOLTAGE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 554 ret = ad7293_adc_get_scale(st, chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 555 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 556 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 557 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 558 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 559 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 560 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 561 case IIO_CURRENT: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 562 ret = ad7293_isense_get_scale(st, chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 563 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 564 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 565 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 566 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 567 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 568 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 569 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 570 *val = 1; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 571 *val2 = 8; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 572 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 573 return IIO_VAL_FRACTIONAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 574 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 575 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 576 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 577 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 578 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 579 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 580 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 581 --- 0-DAY CI Kernel Test Service, Intel Corporation diff --git a/a/content_digest b/N1/content_digest index d5b4012..6072cc3 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,21 +1,12 @@ - "From\0kernel test robot <lkp@intel.com>\0" + "From\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Subject\0drivers/iio/dac/ad7293.c:524 ad7293_read_raw() error: uninitialized symbol 'data'.\0" - "Date\0Mon, 24 Jan 2022 19:10:08 +0800\0" - "To\0kbuild@lists.01.org\0" + "Date\0Mon, 24 Jan 2022 15:05:43 +0300\0" + "To\0kbuild-all@lists.01.org\0" "\01:1\0" "b\0" - "CC: kbuild-all(a)lists.01.org\n" - "CC: linux-kernel(a)vger.kernel.org\n" - "TO: Antoniu Miclaus <antoniu.miclaus@analog.com>\n" - "CC: Jonathan Cameron <Jonathan.Cameron@huawei.com>\n" - "CC: Cai Huoqing <cai.huoqing@linux.dev>\n" - "\n" "tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master\n" "head: dd81e1c7d5fb126e5fbc5c9e334d7b3ec29a16a0\n" "commit: 0bb12606c05fe9737e3056fe76d6e4b9c2a87b57 iio:dac:ad7293: add support for AD7293\n" - "date: 7 weeks ago\n" - ":::::: branch date: 19 hours ago\n" - ":::::: commit date: 7 weeks ago\n" "config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp(a)intel.com/config)\n" "compiler: nios2-linux-gcc (GCC) 11.2.0\n" "\n" @@ -28,41 +19,6 @@ "\n" "vim +/data +524 drivers/iio/dac/ad7293.c\n" "\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 475 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 476 static int ad7293_read_raw(struct iio_dev *indio_dev,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 477 \t\t\t struct iio_chan_spec const *chan,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 478 \t\t\t int *val, int *val2, long info)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 479 {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 480 \tstruct ad7293_state *st = iio_priv(indio_dev);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 481 \tint ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 482 \tu16 data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 483 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 484 \tswitch (info) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 485 \tcase IIO_CHAN_INFO_RAW:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 486 \t\tswitch (chan->type) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 487 \t\tcase IIO_VOLTAGE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 488 \t\t\tif (chan->output)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 489 \t\t\t\tret = ad7293_ch_read_raw(st, AD7293_DAC,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 490 \t\t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 491 \t\t\telse\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 492 \t\t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_VINX,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 493 \t\t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 494 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 495 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 496 \t\tcase IIO_CURRENT:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 497 \t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_ISENSE,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 498 \t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 499 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 500 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 501 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 502 \t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_TSENSE,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 503 \t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 504 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 505 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 506 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 507 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 508 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 509 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 510 \t\tif (ret)\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 511 \t\t\treturn ret;\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 512 \n" @@ -76,6 +32,9 @@ "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 520 \t\t\t\tret = ad7293_get_offset(st,\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 521 \t\t\t\t\t\t\tchan->channel + AD7293_VOUT_MIN_OFFSET_CH,\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 522 \t\t\t\t\t\t\t&data);\n" + "\n" + "ad7293_get_offset() can fail\n" + "\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 523 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 @524 \t\t\t\tdata = FIELD_GET(AD7293_REG_VOUT_OFFSET_MSK, data);\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 525 \t\t\t} else {\n" @@ -89,55 +48,9 @@ "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 533 \t\t\t\t\t\t&data);\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 534 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 535 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 536 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 537 \t\t\tret = ad7293_get_offset(st,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 538 \t\t\t\t\t\tchan->channel + AD7293_TSENSE_MIN_OFFSET_CH,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 539 \t\t\t\t\t\t&data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 540 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 541 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 542 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 543 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 544 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 545 \t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 546 \t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 547 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 548 \t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 549 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 550 \t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 551 \tcase IIO_CHAN_INFO_SCALE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 552 \t\tswitch (chan->type) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 553 \t\tcase IIO_VOLTAGE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 554 \t\t\tret = ad7293_adc_get_scale(st, chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 555 \t\t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 556 \t\t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 557 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 558 \t\t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 559 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 560 \t\t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 561 \t\tcase IIO_CURRENT:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 562 \t\t\tret = ad7293_isense_get_scale(st, chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 563 \t\t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 564 \t\t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 565 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 566 \t\t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 567 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 568 \t\t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 569 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 570 \t\t\t*val = 1;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 571 \t\t\t*val2 = 8;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 572 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 573 \t\t\treturn IIO_VAL_FRACTIONAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 574 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 575 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 576 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 577 \tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 578 \t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 579 \t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 580 }\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 581 \n" "\n" "---\n" "0-DAY CI Kernel Test Service, Intel Corporation\n" https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org -a501509345c0bc7ac9b646643192b006ea50bcb01fec3f83c244ad1d5502cd52 +a2d703c1ebf8c5ef8bae83b6cd292f226eedf8e8ba7a5d7d30ade211407bde1a
diff --git a/a/1.txt b/N2/1.txt index 7686009..4aeb191 100644 --- a/a/1.txt +++ b/N2/1.txt @@ -1,16 +1,7 @@ -CC: kbuild-all(a)lists.01.org -CC: linux-kernel(a)vger.kernel.org -TO: Antoniu Miclaus <antoniu.miclaus@analog.com> -CC: Jonathan Cameron <Jonathan.Cameron@huawei.com> -CC: Cai Huoqing <cai.huoqing@linux.dev> - tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master head: dd81e1c7d5fb126e5fbc5c9e334d7b3ec29a16a0 commit: 0bb12606c05fe9737e3056fe76d6e4b9c2a87b57 iio:dac:ad7293: add support for AD7293 -date: 7 weeks ago -:::::: branch date: 19 hours ago -:::::: commit date: 7 weeks ago -config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp(a)intel.com/config) +config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp@intel.com/config) compiler: nios2-linux-gcc (GCC) 11.2.0 If you fix the issue, kindly add following tag as appropriate @@ -22,41 +13,6 @@ drivers/iio/dac/ad7293.c:524 ad7293_read_raw() error: uninitialized symbol 'data vim +/data +524 drivers/iio/dac/ad7293.c -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 475 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 476 static int ad7293_read_raw(struct iio_dev *indio_dev, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 477 struct iio_chan_spec const *chan, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 478 int *val, int *val2, long info) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 479 { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 480 struct ad7293_state *st = iio_priv(indio_dev); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 481 int ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 482 u16 data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 483 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 484 switch (info) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 485 case IIO_CHAN_INFO_RAW: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 486 switch (chan->type) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 487 case IIO_VOLTAGE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 488 if (chan->output) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 489 ret = ad7293_ch_read_raw(st, AD7293_DAC, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 490 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 491 else -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 492 ret = ad7293_ch_read_raw(st, AD7293_ADC_VINX, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 493 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 494 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 495 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 496 case IIO_CURRENT: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 497 ret = ad7293_ch_read_raw(st, AD7293_ADC_ISENSE, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 498 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 499 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 500 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 501 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 502 ret = ad7293_ch_read_raw(st, AD7293_ADC_TSENSE, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 503 chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 504 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 505 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 506 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 507 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 508 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 509 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 510 if (ret) 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 511 return ret; 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 512 @@ -70,6 +26,9 @@ vim +/data +524 drivers/iio/dac/ad7293.c 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 520 ret = ad7293_get_offset(st, 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 521 chan->channel + AD7293_VOUT_MIN_OFFSET_CH, 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 522 &data); + +ad7293_get_offset() can fail + 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 523 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 @524 data = FIELD_GET(AD7293_REG_VOUT_OFFSET_MSK, data); 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 525 } else { @@ -83,53 +42,7 @@ vim +/data +524 drivers/iio/dac/ad7293.c 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 533 &data); 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 534 0bb12606c05fe9 Antoniu Miclaus 2021-12-02 535 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 536 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 537 ret = ad7293_get_offset(st, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 538 chan->channel + AD7293_TSENSE_MIN_OFFSET_CH, -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 539 &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 540 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 541 break; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 542 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 543 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 544 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 545 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 546 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 547 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 548 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 549 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 550 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 551 case IIO_CHAN_INFO_SCALE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 552 switch (chan->type) { -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 553 case IIO_VOLTAGE: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 554 ret = ad7293_adc_get_scale(st, chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 555 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 556 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 557 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 558 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 559 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 560 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 561 case IIO_CURRENT: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 562 ret = ad7293_isense_get_scale(st, chan->channel, &data); -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 563 if (ret) -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 564 return ret; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 565 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 566 *val = data; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 567 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 568 return IIO_VAL_INT; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 569 case IIO_TEMP: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 570 *val = 1; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 571 *val2 = 8; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 572 -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 573 return IIO_VAL_FRACTIONAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 574 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 575 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 576 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 577 default: -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 578 return -EINVAL; -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 579 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 580 } -0bb12606c05fe9 Antoniu Miclaus 2021-12-02 581 --- 0-DAY CI Kernel Test Service, Intel Corporation -https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org +https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org diff --git a/a/content_digest b/N2/content_digest index d5b4012..527bb74 100644 --- a/a/content_digest +++ b/N2/content_digest @@ -1,22 +1,19 @@ - "From\0kernel test robot <lkp@intel.com>\0" + "From\0Dan Carpenter <dan.carpenter@oracle.com>\0" "Subject\0drivers/iio/dac/ad7293.c:524 ad7293_read_raw() error: uninitialized symbol 'data'.\0" - "Date\0Mon, 24 Jan 2022 19:10:08 +0800\0" - "To\0kbuild@lists.01.org\0" - "\01:1\0" + "Date\0Mon, 24 Jan 2022 15:05:43 +0300\0" + "To\0kbuild@lists.01.org" + " Antoniu Miclaus <antoniu.miclaus@analog.com>\0" + "Cc\0lkp@intel.com" + kbuild-all@lists.01.org + linux-kernel@vger.kernel.org + Jonathan Cameron <Jonathan.Cameron@huawei.com> + " Cai Huoqing <cai.huoqing@linux.dev>\0" + "\00:1\0" "b\0" - "CC: kbuild-all(a)lists.01.org\n" - "CC: linux-kernel(a)vger.kernel.org\n" - "TO: Antoniu Miclaus <antoniu.miclaus@analog.com>\n" - "CC: Jonathan Cameron <Jonathan.Cameron@huawei.com>\n" - "CC: Cai Huoqing <cai.huoqing@linux.dev>\n" - "\n" "tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master\n" "head: dd81e1c7d5fb126e5fbc5c9e334d7b3ec29a16a0\n" "commit: 0bb12606c05fe9737e3056fe76d6e4b9c2a87b57 iio:dac:ad7293: add support for AD7293\n" - "date: 7 weeks ago\n" - ":::::: branch date: 19 hours ago\n" - ":::::: commit date: 7 weeks ago\n" - "config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp(a)intel.com/config)\n" + "config: nios2-randconfig-m031-20220124 (https://download.01.org/0day-ci/archive/20220124/202201241944.dodYYcTk-lkp@intel.com/config)\n" "compiler: nios2-linux-gcc (GCC) 11.2.0\n" "\n" "If you fix the issue, kindly add following tag as appropriate\n" @@ -28,41 +25,6 @@ "\n" "vim +/data +524 drivers/iio/dac/ad7293.c\n" "\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 475 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 476 static int ad7293_read_raw(struct iio_dev *indio_dev,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 477 \t\t\t struct iio_chan_spec const *chan,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 478 \t\t\t int *val, int *val2, long info)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 479 {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 480 \tstruct ad7293_state *st = iio_priv(indio_dev);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 481 \tint ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 482 \tu16 data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 483 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 484 \tswitch (info) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 485 \tcase IIO_CHAN_INFO_RAW:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 486 \t\tswitch (chan->type) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 487 \t\tcase IIO_VOLTAGE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 488 \t\t\tif (chan->output)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 489 \t\t\t\tret = ad7293_ch_read_raw(st, AD7293_DAC,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 490 \t\t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 491 \t\t\telse\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 492 \t\t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_VINX,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 493 \t\t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 494 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 495 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 496 \t\tcase IIO_CURRENT:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 497 \t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_ISENSE,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 498 \t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 499 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 500 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 501 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 502 \t\t\tret = ad7293_ch_read_raw(st, AD7293_ADC_TSENSE,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 503 \t\t\t\t\t\t chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 504 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 505 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 506 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 507 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 508 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 509 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 510 \t\tif (ret)\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 511 \t\t\treturn ret;\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 512 \n" @@ -76,6 +38,9 @@ "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 520 \t\t\t\tret = ad7293_get_offset(st,\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 521 \t\t\t\t\t\t\tchan->channel + AD7293_VOUT_MIN_OFFSET_CH,\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 522 \t\t\t\t\t\t\t&data);\n" + "\n" + "ad7293_get_offset() can fail\n" + "\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 523 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 @524 \t\t\t\tdata = FIELD_GET(AD7293_REG_VOUT_OFFSET_MSK, data);\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 525 \t\t\t} else {\n" @@ -89,55 +54,9 @@ "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 533 \t\t\t\t\t\t&data);\n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 534 \n" "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 535 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 536 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 537 \t\t\tret = ad7293_get_offset(st,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 538 \t\t\t\t\t\tchan->channel + AD7293_TSENSE_MIN_OFFSET_CH,\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 539 \t\t\t\t\t\t&data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 540 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 541 \t\t\tbreak;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 542 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 543 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 544 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 545 \t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 546 \t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 547 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 548 \t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 549 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 550 \t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 551 \tcase IIO_CHAN_INFO_SCALE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 552 \t\tswitch (chan->type) {\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 553 \t\tcase IIO_VOLTAGE:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 554 \t\t\tret = ad7293_adc_get_scale(st, chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 555 \t\t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 556 \t\t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 557 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 558 \t\t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 559 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 560 \t\t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 561 \t\tcase IIO_CURRENT:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 562 \t\t\tret = ad7293_isense_get_scale(st, chan->channel, &data);\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 563 \t\t\tif (ret)\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 564 \t\t\t\treturn ret;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 565 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 566 \t\t\t*val = data;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 567 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 568 \t\t\treturn IIO_VAL_INT;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 569 \t\tcase IIO_TEMP:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 570 \t\t\t*val = 1;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 571 \t\t\t*val2 = 8;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 572 \n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 573 \t\t\treturn IIO_VAL_FRACTIONAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 574 \t\tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 575 \t\t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 576 \t\t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 577 \tdefault:\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 578 \t\treturn -EINVAL;\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 579 \t}\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 580 }\n" - "0bb12606c05fe9 Antoniu Miclaus 2021-12-02 581 \n" "\n" "---\n" "0-DAY CI Kernel Test Service, Intel Corporation\n" - https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org + https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org -a501509345c0bc7ac9b646643192b006ea50bcb01fec3f83c244ad1d5502cd52 +8b4d19ce1c3f53272049660d53cdf6a2b2085708f711ff7432e56005a3a98ec7
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.