Re: [PATCH v16 3/5] mfd: Add driver for RAVE Supervisory Processor

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

[-- Attachment #1: Type: text/plain, Size: 1593 bytes --]

Hi Andrey,

Thank you for the patch! Yet something to improve:

[auto build test ERROR on linus/master]
[also build test ERROR on v4.15-rc4]
[cannot apply to next-20171222]
[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/Andrey-Smirnov/ZII-RAVE-platform-driver/20171223-132544
config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025
reproduce:
        # save the attached .config to linux build tree
        make ARCH=i386 

All errors (new ones prefixed by >>):

   drivers/mfd/rave-sp.c: In function 'csum_ccitt':
>> drivers/mfd/rave-sp.c:227:25: error: implicit declaration of function 'crc_ccitt_false'; did you mean 'crc_ccitt_byte'? [-Werror=implicit-function-declaration]
     const u16 calculated = crc_ccitt_false(0xffff, buf, size);
                            ^~~~~~~~~~~~~~~
                            crc_ccitt_byte
   cc1: some warnings being treated as errors

vim +227 drivers/mfd/rave-sp.c

   224	
   225	static void csum_ccitt(const u8 *buf, size_t size, u8 *crc)
   226	{
 > 227		const u16 calculated = crc_ccitt_false(0xffff, buf, size);
   228	
   229		/*
   230		 * While the rest of the wire protocol is little-endian,
   231		 * CCITT-16 CRC in RDU2 device is sent out in big-endian order.
   232		 */
   233		put_unaligned_be16(calculated, crc);
   234	}
   235	

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

[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 62299 bytes --]