Re: [PATCH 2/2] usb: typec: ucsi: add support for Cypress CCGx

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

 



Hi Ajay,

I love your patch! Yet something to improve:

[auto build test ERROR on wsa/i2c/for-next]
[also build test ERROR on v4.19-rc1 next-20180831]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Ajay-Gupta/i2c-buses-add-i2c-bus-driver-for-NVIDIA-GPU/20180827-092936
base:   https://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux.git i2c/for-next
config: i386-allyesconfig (attached as .config)
compiler: gcc-7 (Debian 7.3.0-16) 7.3.0
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/usb//typec/ucsi/ucsi_i2c_ccg.c: In function 'ucsi_i2c_ccg_resume':
>> drivers/usb//typec/ucsi/ucsi_i2c_ccg.c:559:8: error: implicit declaration of function 'ucsi_run_command'; did you mean 'ucsi_i2c_ccg_cmd'? [-Werror=implicit-function-declaration]
     ret = ucsi_run_command(ui->ucsi, &c, NULL, 0);
           ^~~~~~~~~~~~~~~~
           ucsi_i2c_ccg_cmd
   cc1: some warnings being treated as errors

vim +559 drivers/usb//typec/ucsi/ucsi_i2c_ccg.c

   544	
   545	static int ucsi_i2c_ccg_resume(struct device *dev)
   546	{
   547		struct i2c_client *client = to_i2c_client(dev);
   548		struct ucsi_i2c_ccg *ui = i2c_get_clientdata(client);
   549		struct ucsi_control c;
   550		int ret;
   551	
   552		if (device_may_wakeup(dev) && ui->wake_enabled) {
   553			disable_irq_wake(ui->irq);
   554			ui->wake_enabled = false;
   555		}
   556	
   557		/* restore UCSI notification enable mask */
   558		UCSI_CMD_SET_NTFY_ENABLE(c, UCSI_ENABLE_NTFY_ALL);
 > 559		ret = ucsi_run_command(ui->ucsi, &c, NULL, 0);
   560		if (ret) {
   561			dev_err(ui->dev, "%s: failed to set notification enable - %d\n",
   562				__func__, ret);
   563		}
   564	
   565		return 0;
   566	}
   567	

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

Attachment: .config.gz
Description: application/gzip


[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux