[intel-lts:6.18/linux 1/1] drivers/media/platform/intel/ipu-acpi.c:73:25: sparse: sparse: unsigned value that used to be signed checked against zero?

kernel test robot <[email protected]>
Newsgroups dev.linux.lists.oe-kbuild
Message-ID <[email protected]>
:::::: 
:::::: Manual check reason: "low confidence static check first_new_problem: drivers/media/platform/intel/ipu-acpi.c:73:25: sparse: sparse: unsigned value that used to be signed checked against zero?"
:::::: 

BCC: [email protected]
CC: [email protected]
TO: hepengpx <[email protected]>
CC: "sys_oak" <[email protected]>

tree:   https://github.com/intel/linux-intel-lts.git 6.18/linux
head:   5100e1c74efeba843945492f8d5840e6166c0392
commit: d1a4a0a2fedfc816e1f7db99c4858dfc0b8fd300 [1/1] media: pci: intel: support PDATA in Kconfig/Makefile
:::::: branch date: 10 days ago
:::::: commit date: 3 months ago
config: i386-randconfig-r123-20260708 (https://download.01.org/0day-ci/archive/20260709/[email protected]/config)
compiler: gcc-14 (Debian 14.2.0-19) 14.2.0
sparse: v0.6.5-rc1
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260709/[email protected]/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 <[email protected]>
| Closes: https://lore.kernel.org/r/[email protected]/

sparse warnings: (new ones prefixed by >>)
>> drivers/media/platform/intel/ipu-acpi.c:73:25: sparse: sparse: unsigned value that used to be signed checked against zero?
   drivers/media/platform/intel/ipu-acpi.c:79:16: sparse: signed value source
--
>> drivers/media/platform/intel/ipu-acpi-pdata.c:36:30: sparse: sparse: symbol 'acpi_subdev_pdata' was not declared. Should it be static?
>> drivers/media/platform/intel/ipu-acpi-pdata.c:43:21: sparse: sparse: symbol 'serdes_info' was not declared. Should it be static?

vim +73 drivers/media/platform/intel/ipu-acpi.c

02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  68  
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  69  static int get_table_index(const char *acpi_name)
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  70  {
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  71  	int i;
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  72  
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08 @73  	for (i = 0; i < ARRAY_SIZE(supported_devices); i++) {
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  74  		if (!strncmp(supported_devices[i].hid_name, acpi_name,
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  75  			     strlen(supported_devices[i].hid_name)))
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  76  			return i;
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  77  	}
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  78  
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  79  	return -ENODEV;
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  80  }
02f6fe497a2baa Shahidan, Muhammad Shahmil 2026-04-08  81  

:::::: The code at line 73 was first introduced by commit
:::::: 02f6fe497a2baa45f229a6c4039418e423e8ece6 Add IPU ACPI drivers

:::::: TO: Shahidan, Muhammad Shahmil <[email protected]>
:::::: CC: sys_oak <[email protected]>

--
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.