Re: [RFC PATCH] i2c: i2c-gpio Enhance driver for buses with shared SCL

[kernel test robot <lkp@intel.com>]

Hi Markus,

[This is a private test report for your RFC patch.]
kernel test robot noticed the following build errors:

[auto build test ERROR on andi-shyti/i2c/i2c-host]
[also build test ERROR on linus/master v7.1-rc3 next-20260508]
[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/Markus-Stockhausen/i2c-i2c-gpio-Enhance-driver-for-buses-with-shared-SCL/20260514-233812
base:   https://git.kernel.org/pub/scm/linux/kernel/git/andi.shyti/linux.git i2c/i2c-host
patch link:    https://lore.kernel.org/r/20260514092042.3265986-1-markus.stockhausen%40gmx.de
patch subject: [RFC PATCH] i2c: i2c-gpio Enhance driver for buses with shared SCL
config: arm64-randconfig-004-20260516 (https://download.01.org/0day-ci/archive/20260516/202605161950.P3hdEfkq-lkp@intel.com/config)
compiler: clang version 23.0.0git (https://github.com/llvm/llvm-project 5bac06718f502014fade905512f1d26d578a18f3)
reproduce (this is a W=1 build): (https://download.01.org/0day-ci/archive/20260516/202605161950.P3hdEfkq-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@intel.com>
| Closes: https://lore.kernel.org/oe-kbuild-all/202605161950.P3hdEfkq-lkp@intel.com/

All errors (new ones prefixed by >>):

>> drivers/i2c/busses/i2c-gpio.c:197:30: error: incompatible pointer types passing 'struct i2c_gpio_scl_data *' to parameter of type 'const struct gpio_desc *' [-Wincompatible-pointer-types]
     197 |         int ret, irq = gpiod_to_irq(priv->scl);
         |                                     ^~~~~~~~~
   include/linux/gpio/consumer.h:169:42: note: passing argument to parameter 'desc' here
     169 | int gpiod_to_irq(const struct gpio_desc *desc);
         |                                          ^
>> drivers/i2c/busses/i2c-gpio.c:204:30: error: incompatible pointer types passing 'struct i2c_gpio_scl_data *' to parameter of type 'struct gpio_desc *' [-Wincompatible-pointer-types]
     204 |         ret = gpiod_direction_input(priv->scl);
         |                                     ^~~~~~~~~
   include/linux/gpio/consumer.h:114:45: note: passing argument to parameter 'desc' here
     114 | int gpiod_direction_input(struct gpio_desc *desc);
         |                                             ^
   drivers/i2c/busses/i2c-gpio.c:219:31: error: incompatible pointer types passing 'struct i2c_gpio_scl_data *' to parameter of type 'struct gpio_desc *' [-Wincompatible-pointer-types]
     219 |         ret = gpiod_direction_output(priv->scl, 1) ?: ret;
         |                                      ^~~~~~~~~
   include/linux/gpio/consumer.h:115:46: note: passing argument to parameter 'desc' here
     115 | int gpiod_direction_output(struct gpio_desc *desc, int value);
         |                                              ^
   3 errors generated.


vim +197 drivers/i2c/busses/i2c-gpio.c

bbe899700a44af Wolfram Sang 2018-06-29  193  
63e57b6f191db9 Wolfram Sang 2019-02-19  194  static int i2c_gpio_fi_act_on_scl_irq(struct i2c_gpio_private_data *priv,
63e57b6f191db9 Wolfram Sang 2019-02-19  195  				       irqreturn_t handler(int, void*))
63e57b6f191db9 Wolfram Sang 2019-02-19  196  {
63e57b6f191db9 Wolfram Sang 2019-02-19 @197  	int ret, irq = gpiod_to_irq(priv->scl);
63e57b6f191db9 Wolfram Sang 2019-02-19  198  
63e57b6f191db9 Wolfram Sang 2019-02-19  199  	if (irq < 0)
63e57b6f191db9 Wolfram Sang 2019-02-19  200  		return irq;
63e57b6f191db9 Wolfram Sang 2019-02-19  201  
63e57b6f191db9 Wolfram Sang 2019-02-19  202  	i2c_lock_bus(&priv->adap, I2C_LOCK_ROOT_ADAPTER);
63e57b6f191db9 Wolfram Sang 2019-02-19  203  
63e57b6f191db9 Wolfram Sang 2019-02-19 @204  	ret = gpiod_direction_input(priv->scl);
63e57b6f191db9 Wolfram Sang 2019-02-19  205  	if (ret)
63e57b6f191db9 Wolfram Sang 2019-02-19  206  		goto unlock;
63e57b6f191db9 Wolfram Sang 2019-02-19  207  
63e57b6f191db9 Wolfram Sang 2019-02-19  208  	reinit_completion(&priv->scl_irq_completion);
63e57b6f191db9 Wolfram Sang 2019-02-19  209  
63e57b6f191db9 Wolfram Sang 2019-02-19  210  	ret = request_irq(irq, handler, IRQF_TRIGGER_FALLING,
63e57b6f191db9 Wolfram Sang 2019-02-19  211  			  "i2c_gpio_fault_injector_scl_irq", priv);
63e57b6f191db9 Wolfram Sang 2019-02-19  212  	if (ret)
63e57b6f191db9 Wolfram Sang 2019-02-19  213  		goto output;
63e57b6f191db9 Wolfram Sang 2019-02-19  214  
63e57b6f191db9 Wolfram Sang 2019-02-19  215  	wait_for_completion_interruptible(&priv->scl_irq_completion);
63e57b6f191db9 Wolfram Sang 2019-02-19  216  
63e57b6f191db9 Wolfram Sang 2019-02-19  217  	free_irq(irq, priv);
63e57b6f191db9 Wolfram Sang 2019-02-19  218   output:
63e57b6f191db9 Wolfram Sang 2019-02-19  219  	ret = gpiod_direction_output(priv->scl, 1) ?: ret;
63e57b6f191db9 Wolfram Sang 2019-02-19  220   unlock:
63e57b6f191db9 Wolfram Sang 2019-02-19  221  	i2c_unlock_bus(&priv->adap, I2C_LOCK_ROOT_ADAPTER);
63e57b6f191db9 Wolfram Sang 2019-02-19  222  
63e57b6f191db9 Wolfram Sang 2019-02-19  223  	return ret;
63e57b6f191db9 Wolfram Sang 2019-02-19  224  }
63e57b6f191db9 Wolfram Sang 2019-02-19  225  

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