| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <CA+4VgcKXQ4padSHBpnmw_vN5WWL+GfcJqOCtthN88C1hwicb6A@mail.gmail.com>
Date: Thu, 7 Nov 2024 08:36:16 +0800
From: Yu-Hsian Yang <j2anfernee@...il.com>
To: kernel test robot <lkp@...el.com>
Cc: avifishman70@...il.com, tmaimon77@...il.com, tali.perry1@...il.com,
venture@...gle.com, yuenn@...gle.com, benjaminfair@...gle.com,
jic23@...nel.org, lars@...afoo.de, robh@...nel.org, krzk+dt@...nel.org,
conor+dt@...nel.org, nuno.sa@...log.com, dlechner@...libre.com,
javier.carrasco.cruz@...il.com, andy@...nel.org, marcelo.schmitt@...log.com,
olivier.moysan@...s.st.com, mitrutzceclan@...il.com,
matteomartelli3@...il.com, alisadariana@...il.com, joao.goncalves@...adex.com,
marius.cristea@...rochip.com, mike.looijmans@...ic.nl,
chanh@...amperecomputing.com, KWLIU@...oton.com, yhyang2@...oton.com,
llvm@...ts.linux.dev, oe-kbuild-all@...ts.linux.dev, openbmc@...ts.ozlabs.org,
linux-iio@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v1 2/2] iio: adc: add Nuvoton NCT720x ADC driver
Dear kernel test robot,
Thank you for your response.
I would check these warnings.
kernel test robot <lkp@...el.com> 於 2024年11月6日 週三 下午8:32寫道：
>
> Hi Eason,
>
> kernel test robot noticed the following build warnings:
>
> [auto build test WARNING on jic23-iio/togreg]
> [also build test WARNING on linus/master v6.12-rc6 next-20241106]
> [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/Eason-Yang/dt-bindings-iio-adc-Add-binding-for-Nuvoton-NCT720x-ADCs/20241106-104046
> base: https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
> patch link: https://lore.kernel.org/r/20241106023916.440767-3-j2anfernee%40gmail.com
> patch subject: [PATCH v1 2/2] iio: adc: add Nuvoton NCT720x ADC driver
> config: s390-allmodconfig (https://download.01.org/0day-ci/archive/20241106/202411062051.TLRkJSSL-lkp@intel.com/config)
> compiler: clang version 20.0.0git (https://github.com/llvm/llvm-project 592c0fe55f6d9a811028b5f3507be91458ab2713)
> reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20241106/202411062051.TLRkJSSL-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/202411062051.TLRkJSSL-lkp@intel.com/
>
> All warnings (new ones prefixed by >>):
>
> In file included from drivers/iio/adc/nct720x.c:9:
> In file included from include/linux/device.h:32:
> In file included from include/linux/device/driver.h:21:
> In file included from include/linux/module.h:19:
> In file included from include/linux/elf.h:6:
> In file included from arch/s390/include/asm/elf.h:181:
> In file included from arch/s390/include/asm/mmu_context.h:11:
> In file included from arch/s390/include/asm/pgalloc.h:18:
> In file included from include/linux/mm.h:2213:
> include/linux/vmstat.h:504:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
> 504 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
> | ~~~~~~~~~~~~~~~~~~~~~ ^
> 505 | item];
> | ~~~~
> include/linux/vmstat.h:511:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
> 511 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
> | ~~~~~~~~~~~~~~~~~~~~~ ^
> 512 | NR_VM_NUMA_EVENT_ITEMS +
> | ~~~~~~~~~~~~~~~~~~~~~~
> include/linux/vmstat.h:518:36: warning: arithmetic between different enumeration types ('enum node_stat_item' and 'enum lru_list') [-Wenum-enum-conversion]
> 518 | return node_stat_name(NR_LRU_BASE + lru) + 3; // skip "nr_"
> | ~~~~~~~~~~~ ^ ~~~
> include/linux/vmstat.h:524:43: warning: arithmetic between different enumeration types ('enum zone_stat_item' and 'enum numa_stat_item') [-Wenum-enum-conversion]
> 524 | return vmstat_text[NR_VM_ZONE_STAT_ITEMS +
> | ~~~~~~~~~~~~~~~~~~~~~ ^
> 525 | NR_VM_NUMA_EVENT_ITEMS +
> | ~~~~~~~~~~~~~~~~~~~~~~
> >> drivers/iio/adc/nct720x.c:526:16: warning: cast to smaller integer type 'enum nct720x_chips' from 'const void *' [-Wvoid-pointer-to-enum-cast]
> 526 | chip->type = (enum nct720x_chips)device_get_match_data(&client->dev);
> | ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> 5 warnings generated.
>
>
> vim +526 drivers/iio/adc/nct720x.c
>
> 511
> 512 static int nct720x_probe(struct i2c_client *client)
> 513 {
> 514 const struct i2c_device_id *id = i2c_client_get_device_id(client);
> 515 struct nct720x_chip_info *chip;
> 516 struct iio_dev *indio_dev;
> 517 int ret;
> 518 u32 tmp;
> 519
> 520 indio_dev = devm_iio_device_alloc(&client->dev, sizeof(*chip));
> 521 if (!indio_dev)
> 522 return -ENOMEM;
> 523 chip = iio_priv(indio_dev);
> 524
> 525 if (client->dev.of_node)
> > 526 chip->type = (enum nct720x_chips)device_get_match_data(&client->dev);
> 527 else
> 528 chip->type = i2c_match_id(nct720x_id, client)->driver_data;
> 529
> 530 chip->vin_max = (chip->type == nct7201) ? NCT7201_VIN_MAX : NCT7202_VIN_MAX;
> 531
> 532 ret = of_property_read_u32(client->dev.of_node, "read-vin-data-size", &tmp);
> 533 if (ret < 0) {
> 534 pr_err("read-vin-data-size property not found\n");
> 535 return ret;
> 536 }
> 537
> 538 if (tmp == 8) {
> 539 chip->use_read_byte_vin = true;
> 540 } else if (tmp == 16) {
> 541 chip->use_read_byte_vin = false;
> 542 } else {
> 543 pr_err("invalid read-vin-data-size (%d)\n", tmp);
> 544 return -EINVAL;
> 545 }
> 546
> 547 mutex_init(&chip->access_lock);
> 548
> 549 /* this is only used for device removal purposes */
> 550 i2c_set_clientdata(client, indio_dev);
> 551
> 552 chip->client = client;
> 553
> 554 ret = nct720x_init_chip(chip);
> 555 if (ret < 0)
> 556 return ret;
> 557
> 558 indio_dev->name = id->name;
> 559 indio_dev->channels = nct720x_channels;
> 560 indio_dev->num_channels = ARRAY_SIZE(nct720x_channels);
> 561 indio_dev->info = &nct720x_info;
> 562 indio_dev->modes = INDIO_DIRECT_MODE;
> 563
> 564 iio_device_register(indio_dev);
> 565
> 566 return 0;
> 567 }
> 568
>
> --
> 0-DAY CI Kernel Test Service
> https://github.com/intel/lkp-tests/wiki
Powered by blists - more mailing lists