Re: [PATCH] USB: serial: cp210x: Adding GPIO support for CP2105

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

 



Hi Martyn,

[auto build test WARNING on usb/usb-testing]
[also build test WARNING on next-20160113]
[cannot apply to v4.4]
[if your patch is applied to the wrong git tree, please drop us a note to help improving the system]

url:    https://github.com/0day-ci/linux/commits/Martyn-Welch/USB-serial-cp210x-Adding-GPIO-support-for-CP2105/20160113-183312
base:   https://git.kernel.org/pub/scm/linux/kernel/git/gregkh/usb.git usb-testing
config: sh-allyesconfig (attached as .config)
reproduce:
        wget https://git.kernel.org/cgit/linux/kernel/git/wfg/lkp-tests.git/plain/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # save the attached .config to linux build tree
        make.cross ARCH=sh 

All warnings (new ones prefixed by >>):

   drivers/usb/serial/cp210x.c:211:19: error: field 'gc' has incomplete type
   drivers/usb/serial/cp210x.c: In function 'cp210x_gpio_get':
>> drivers/usb/serial/cp210x.c:1017:5: warning: initialization from incompatible pointer type [enabled by default]
   drivers/usb/serial/cp210x.c:1017:5: warning: (near initialization for 'port_priv') [enabled by default]
   drivers/usb/serial/cp210x.c: In function 'cp210x_gpio_set':
   drivers/usb/serial/cp210x.c:1056:5: warning: initialization from incompatible pointer type [enabled by default]
   drivers/usb/serial/cp210x.c:1056:5: warning: (near initialization for 'port_priv') [enabled by default]
   drivers/usb/serial/cp210x.c: In function 'cp210x_shared_gpio_init':
   drivers/usb/serial/cp210x.c:1164:2: error: implicit declaration of function 'gpiochip_add' [-Werror=implicit-function-declaration]
   drivers/usb/serial/cp210x.c: In function 'cp210x_port_remove':
   drivers/usb/serial/cp210x.c:1217:3: error: implicit declaration of function 'gpiochip_remove' [-Werror=implicit-function-declaration]
   cc1: some warnings being treated as errors

vim +1017 drivers/usb/serial/cp210x.c

  1001			state = BREAK_OFF;
  1002		else
  1003			state = BREAK_ON;
  1004		dev_dbg(&port->dev, "%s - turning break %s\n", __func__,
  1005			state == BREAK_OFF ? "off" : "on");
  1006		cp210x_set_config(port, CP210X_SET_BREAK, &state, 2);
  1007	}
  1008	
  1009	static int cp210x_gpio_direction_get(struct gpio_chip *gc, unsigned gpio)
  1010	{
  1011		return 0;
  1012	}
  1013	
  1014	static int cp210x_gpio_get(struct gpio_chip *gc, unsigned gpio)
  1015	{
  1016		struct cp210x_port_private *port_priv =
> 1017				 container_of(gc, struct cp210x_port_private, gc);
  1018		struct usb_serial *serial = port_priv->serial;
  1019		__le32 *buf;
  1020		int result, i, length;
  1021		int size = 1;
  1022		unsigned int data[size];
  1023	
  1024		/* Number of integers required to contain the array */
  1025		length = (((size - 1) | 3) + 1) / 4;

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

Attachment: .config.gz
Description: Binary data


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

  Powered by Linux