Re: [PATCH linux 1/6] hwmon: Add core On-Chip Controller support for POWER CPUs

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

 



Hi Edward,

[auto build test WARNING on hwmon/hwmon-next]
[also build test WARNING on v4.10-rc1 next-20161224]
[cannot apply to linux/master]
[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/eajames-ibm-gmail-com/drivers-hwmon-Add-On-Chip-Controller-driver/20161231-021324
base:   https://git.kernel.org/pub/scm/linux/kernel/git/groeck/linux-staging.git hwmon-next
config: ia64-allmodconfig (attached as .config)
compiler: ia64-linux-gcc (GCC) 6.2.0
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=ia64 

All warnings (new ones prefixed by >>):

   drivers/hwmon/occ/occ.c: In function 'occ_get_all':
>> drivers/hwmon/occ/occ.c:390:44: warning: passing argument 5 of 'occ_send_cmd' discards 'const' qualifier from pointer target type [-Wdiscarded-qualifiers]
     rc = occ_send_cmd(driver, 0, OCC_POLL, 1, &poll_cmd_data, occ_data);
                                               ^
   drivers/hwmon/occ/occ.c:281:11: note: expected 'u8 * {aka unsigned char *}' but argument is of type 'const u8 * {aka const unsigned char *}'
    static u8 occ_send_cmd(struct occ *driver, u8 seq, u8 type, u16 length,
              ^~~~~~~~~~~~

vim +390 drivers/hwmon/occ/occ.c

   374		return rc;
   375	}
   376	
   377	static int occ_get_all(struct occ *driver)
   378	{
   379		int i = 0, rc;
   380		u8 *occ_data;
   381		u16 num_bytes;
   382		const u8 poll_cmd_data = OCC_POLL_STAT_SENSOR;
   383		struct device *dev = driver->dev;
   384		struct occ_response *resp = &driver->response;
   385	
   386		occ_data = devm_kzalloc(dev, OCC_DATA_MAX, GFP_KERNEL);
   387		if (!occ_data)
   388			return -ENOMEM;
   389	
 > 390		rc = occ_send_cmd(driver, 0, OCC_POLL, 1, &poll_cmd_data, occ_data);
   391		if (rc) {
   392			dev_err(dev, "OCC poll failed: %d\n", rc);
   393			goto out;
   394		}
   395	
   396		num_bytes = get_unaligned((u16 *)&occ_data[RESP_DATA_LENGTH]);
   397		num_bytes = be16_to_cpu(num_bytes);
   398		dev_dbg(dev, "OCC data length: %d\n", num_bytes);

---
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 GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux