Re: [PATCH 3/3] iio: proximity: Add support for everlight pmd16d17 sensor

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi Jan,

kernel test robot noticed the following build warnings:

[auto build test WARNING on jic23-iio/togreg]
[also build test WARNING on robh/for-next linus/master v6.11-rc3 next-20240815]
[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/Jan-Kiszka/dt-bindings-vendor-prefixes-Add-EVERLIGHT/20240814-234801
base:   https://git.kernel.org/pub/scm/linux/kernel/git/jic23/iio.git togreg
patch link:    https://lore.kernel.org/r/abb0c1c0724be733138276f638e43e98784bd191.1723527641.git.jan.kiszka%40siemens.com
patch subject: [PATCH 3/3] iio: proximity: Add support for everlight pmd16d17 sensor
config: x86_64-randconfig-101-20240816 (https://download.01.org/0day-ci/archive/20240816/202408160542.UEjIkjkC-lkp@xxxxxxxxx/config)
compiler: gcc-12 (Debian 12.2.0-14) 12.2.0
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20240816/202408160542.UEjIkjkC-lkp@xxxxxxxxx/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@xxxxxxxxx>
| Closes: https://lore.kernel.org/oe-kbuild-all/202408160542.UEjIkjkC-lkp@xxxxxxxxx/

All warnings (new ones prefixed by >>):

   drivers/iio/proximity/pm16d17.c: In function 'pm16d17_chip_init':
>> drivers/iio/proximity/pm16d17.c:194:13: warning: argument 1 null where non-null expected [-Wnonnull]
     194 |         if (strcmp(conv_time, "0.4") == 0)
         |             ^~~~~~~~~~~~~~~~~~~~~~~~
   In file included from include/linux/bitmap.h:13,
                    from include/linux/cpumask.h:12,
                    from arch/x86/include/asm/tlbbatch.h:5,
                    from include/linux/mm_types_task.h:16,
                    from include/linux/sched.h:38,
                    from include/linux/delay.h:23,
                    from drivers/iio/proximity/pm16d17.c:11:
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:196:18: warning: argument 1 null where non-null expected [-Wnonnull]
     196 |         else if (strcmp(conv_time, "0.8") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:198:18: warning: argument 1 null where non-null expected [-Wnonnull]
     198 |         else if (strcmp(conv_time, "1.6") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:200:18: warning: argument 1 null where non-null expected [-Wnonnull]
     200 |         else if (strcmp(conv_time, "3.2") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:202:18: warning: argument 1 null where non-null expected [-Wnonnull]
     202 |         else if (strcmp(conv_time, "6.3") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:204:18: warning: argument 1 null where non-null expected [-Wnonnull]
     204 |         else if (strcmp(conv_time, "12.6") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:206:18: warning: argument 1 null where non-null expected [-Wnonnull]
     206 |         else if (strcmp(conv_time, "25.2") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:214:13: warning: argument 1 null where non-null expected [-Wnonnull]
     214 |         if (strcmp(wait_time, "12.5") == 0)
         |             ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:216:18: warning: argument 1 null where non-null expected [-Wnonnull]
     216 |         else if (strcmp(wait_time, "25") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:218:18: warning: argument 1 null where non-null expected [-Wnonnull]
     218 |         else if (strcmp(wait_time, "50") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:220:18: warning: argument 1 null where non-null expected [-Wnonnull]
     220 |         else if (strcmp(wait_time, "100") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:222:18: warning: argument 1 null where non-null expected [-Wnonnull]
     222 |         else if (strcmp(wait_time, "200") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:224:18: warning: argument 1 null where non-null expected [-Wnonnull]
     224 |         else if (strcmp(wait_time, "400") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:226:18: warning: argument 1 null where non-null expected [-Wnonnull]
     226 |         else if (strcmp(wait_time, "800") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~
   drivers/iio/proximity/pm16d17.c:228:18: warning: argument 1 null where non-null expected [-Wnonnull]
     228 |         else if (strcmp(wait_time, "1600") == 0)
         |                  ^~~~~~~~~~~~~~~~~~~~~~~~~
   include/linux/string.h:156:12: note: in a call to function 'strcmp' declared 'nonnull'
     156 | extern int strcmp(const char *,const char *);
         |            ^~~~~~


vim +194 drivers/iio/proximity/pm16d17.c

   152	
   153	static int pm16d17_chip_init(struct pm16d17_data *data)
   154	{
   155		struct i2c_client *client = data->client;
   156		struct device_node *np = client->dev.of_node;
   157		const char *conv_time = NULL;
   158		const char *wait_time = NULL;
   159		uint8_t op_mode_setting_val;
   160		uint32_t ps_offset_cancel;
   161		uint8_t offset_lsb;
   162		uint8_t offset_msb;
   163		uint32_t pulse_count;
   164		uint32_t pgain;
   165		unsigned int val;
   166		int ret;
   167	
   168		ret = pm16d17_read_reg(data, PM16D17_DEV_ID, &val);
   169	
   170		if (ret < 0 || (val != DEVICE_ID)) {
   171			dev_err(&client->dev, "Invalid chip id 0x%04x\n", val);
   172			return -ENODEV;
   173		}
   174	
   175		dev_dbg(&client->dev, "Detected PM16D17 with chip id: 0x%04x\n", val);
   176	
   177		ret = pm16d17_write_reg(data, PM16D17_OP_MODE, ENABLE_PS_FUNCTION);
   178		if (ret < 0)
   179			return ret;
   180	
   181		of_property_read_u32(np, "ps-gain", &pgain);
   182		switch (pgain) {
   183		case 1:
   184		case 2:
   185		case 4:
   186		case 8:
   187			op_mode_setting_val |= (ilog2(pgain) << 6) & PS_GAIN_MASK;
   188			break;
   189		default:
   190			break;
   191		}
   192	
   193		of_property_read_string(np, "ps-itime", &conv_time);
 > 194		if (strcmp(conv_time, "0.4") == 0)
   195			op_mode_setting_val |= PITIME_0_POINT_4_MS & PS_ITIME_MASK;
   196		else if (strcmp(conv_time, "0.8") == 0)
   197			op_mode_setting_val |= PITIME_0_POINT_8_MS & PS_ITIME_MASK;
   198		else if (strcmp(conv_time, "1.6") == 0)
   199			op_mode_setting_val |= PITIME_1_POINT_6_MS & PS_ITIME_MASK;
   200		else if (strcmp(conv_time, "3.2") == 0)
   201			op_mode_setting_val |= PITIME_3_POINT_2_MS & PS_ITIME_MASK;
   202		else if (strcmp(conv_time, "6.3") == 0)
   203			op_mode_setting_val |= PITIME_6_POINT_3_MS & PS_ITIME_MASK;
   204		else if (strcmp(conv_time, "12.6") == 0)
   205			op_mode_setting_val |= PITIME_12_POINT_6_MS & PS_ITIME_MASK;
   206		else if (strcmp(conv_time, "25.2") == 0)
   207			op_mode_setting_val |= PITIME_25_POINT_2_MS & PS_ITIME_MASK;
   208		else {
   209			dev_info(&client->dev, "Using default ps itime value\n");
   210			op_mode_setting_val |= PITIME_0_POINT_4_MS & PS_ITIME_MASK;
   211		}
   212	
   213		of_property_read_string(np, "ps-wtime", &wait_time);
   214		if (strcmp(wait_time, "12.5") == 0)
   215			op_mode_setting_val |= PWTIME_12_POINT_5_MS & PS_WTIME_MASK;
   216		else if (strcmp(wait_time, "25") == 0)
   217			op_mode_setting_val |= PWTIME_25_MS & PS_WTIME_MASK;
   218		else if (strcmp(wait_time, "50") == 0)
   219			op_mode_setting_val |= PWTIME_50_MS & PS_WTIME_MASK;
   220		else if (strcmp(wait_time, "100") == 0)
   221			op_mode_setting_val |= PWTIME_100_MS & PS_WTIME_MASK;
   222		else if (strcmp(wait_time, "200") == 0)
   223			op_mode_setting_val |= PWTIME_200_MS & PS_WTIME_MASK;
   224		else if (strcmp(wait_time, "400") == 0)
   225			op_mode_setting_val |= PWTIME_400_MS & PS_WTIME_MASK;
   226		else if (strcmp(wait_time, "800") == 0)
   227			op_mode_setting_val |= PWTIME_800_MS & PS_WTIME_MASK;
   228		else if (strcmp(wait_time, "1600") == 0)
   229			op_mode_setting_val |= PWTIME_1600_MS & PS_WTIME_MASK;
   230		else {
   231			dev_info(&client->dev, "Using default ps wtime value\n");
   232			op_mode_setting_val |= PWTIME_12_POINT_5_MS & PS_WTIME_MASK;
   233		}
   234	
   235		ret = pm16d17_write_reg(data, PM16D17_PS_SETTING, op_mode_setting_val);
   236		if (ret < 0)
   237			return ret;
   238	
   239		of_property_read_u32(np, "ps-ir-led-pulse-count", &pulse_count);
   240		if (pulse_count > 256)
   241			pulse_count = 256;
   242		ret = pm16d17_write_reg(data, PM16D17_VCSEL_DRIVE_PULSE, pulse_count - 1);
   243		if (ret < 0)
   244			return ret;
   245	
   246		of_property_read_u32(np, "ps-offset-cancel", &ps_offset_cancel);
   247		if (ps_offset_cancel != 0) {
   248			ret = pm16d17_write_reg(data, PM16D17_PS_SETTING2, OFFSET_CANCEL_ENABLE);
   249			if (ret < 0)
   250				return ret;
   251	
   252			offset_lsb = ps_offset_cancel & PS_OFFSET_CANCEL_LSB_MASK;
   253			offset_msb = (ps_offset_cancel & PS_OFFSET_CANCEL_MSB_MASK) >> 8;
   254	
   255			ret = pm16d17_write_reg(data, PM16D17_PS_OFFSET_CANCEL_L, offset_lsb);
   256			if (ret < 0)
   257				return ret;
   258	
   259			ret = pm16d17_write_reg(data, PM16D17_PS_OFFSET_CANCEL_H, offset_msb);
   260			if (ret < 0)
   261				return ret;
   262		}
   263	
   264		return 0;
   265	}
   266	

-- 
0-DAY CI Kernel Test Service
https://github.com/intel/lkp-tests/wiki




[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Input]     [Linux Kernel]     [Linux SCSI]     [X.org]

  Powered by Linux