[PATCH 03/10] staging: kpc2000: remove fileops.c file.

kbuild test robot lkp at intel.com
Fri May 17 16:52:39 UTC 2019


Hi Greg,

I love your patch! Perhaps something to improve:

[auto build test WARNING on staging/staging-testing]
[cannot apply to v5.1 next-20190517]
[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/Greg-Kroah-Hartman/staging-kpc2000-fix-a-bunch-of-orginization-and-header-file-issues/20190517-213909
reproduce:
        # apt-get install sparse
        make ARCH=x86_64 allmodconfig
        make C=1 CF='-fdiagnostic-prefix -D__CHECK_ENDIAN__'

If you fix the issue, kindly add following tag
Reported-by: kbuild test robot <lkp at intel.com>


sparse warnings: (new ones prefixed by >>)

   drivers/staging/kpc2000/kpc2000/core.c:82:1: sparse: sparse: symbol 'dev_attr_ssid' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:83:1: sparse: sparse: symbol 'dev_attr_ddna' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:85:1: sparse: sparse: symbol 'dev_attr_hw_rev' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:86:1: sparse: sparse: symbol 'dev_attr_build' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:87:1: sparse: sparse: symbol 'dev_attr_build_date' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:88:1: sparse: sparse: symbol 'dev_attr_build_time' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:89:1: sparse: sparse: symbol 'dev_attr_cpld_reg' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:90:1: sparse: sparse: symbol 'dev_attr_cpld_reconfigure' was not declared. Should it be static?
   drivers/staging/kpc2000/kpc2000/core.c:194:14: sparse: sparse: symbol 'kp2000_irq_handler' was not declared. Should it be static?
>> drivers/staging/kpc2000/kpc2000/core.c:285:37: sparse: sparse: incorrect type in argument 1 (different address spaces) @@    expected void [noderef] <asn:1> *to @@    got n:1> *to @@
>> drivers/staging/kpc2000/kpc2000/core.c:285:37: sparse:    expected void [noderef] <asn:1> *to
>> drivers/staging/kpc2000/kpc2000/core.c:285:37: sparse:    got void *

vim +285 drivers/staging/kpc2000/kpc2000/core.c

   259	
   260	static long kp2000_cdev_ioctl(struct file *filp, unsigned int ioctl_num,
   261				      unsigned long ioctl_param)
   262	{
   263		struct kp2000_device *pcard = filp->private_data;
   264	
   265		dev_dbg(&pcard->pdev->dev, "kp2000_cdev_ioctl(filp = [%p], ioctl_num = 0x%08x, ioctl_param = 0x%016lx) pcard = [%p]\n", filp, ioctl_num, ioctl_param, pcard);
   266	
   267		switch (ioctl_num){
   268		case KP2000_IOCTL_GET_CPLD_REG:             return readq(pcard->sysinfo_regs_base + REG_CPLD_CONFIG);
   269		case KP2000_IOCTL_GET_PCIE_ERROR_REG:       return readq(pcard->sysinfo_regs_base + REG_PCIE_ERROR_COUNT);
   270	
   271		case KP2000_IOCTL_GET_EVERYTHING: {
   272			struct kp2000_regs temp;
   273			int ret;
   274	
   275			memset(&temp, 0, sizeof(temp));
   276			temp.card_id = pcard->card_id;
   277			temp.build_version = pcard->build_version;
   278			temp.build_datestamp = pcard->build_datestamp;
   279			temp.build_timestamp = pcard->build_timestamp;
   280			temp.hw_rev = pcard->hardware_revision;
   281			temp.ssid = pcard->ssid;
   282			temp.ddna = pcard->ddna;
   283			temp.cpld_reg = readq(pcard->sysinfo_regs_base + REG_CPLD_CONFIG);
   284	
 > 285			ret = copy_to_user((void*)ioctl_param, (void*)&temp, sizeof(temp));
   286			if (ret)
   287				return -EFAULT;
   288	
   289			return sizeof(temp);
   290			}
   291	
   292		default:
   293			return -ENOTTY;
   294		}
   295		return -ENOTTY;
   296	}
   297	

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


More information about the devel mailing list