[lkml]   [2017]   [Dec]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
SubjectRe: [PATCH v16 3/5] mfd: Add driver for RAVE Supervisory Processor
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]

config: i386-allmodconfig (attached as .config)
compiler: gcc-7 (Debian 7.2.0-12) 7.2.1 20171025
# 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);
cc1: some warnings being treated as errors

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

225 static void csum_ccitt(const u8 *buf, size_t size, u8 *crc)
226 {
> 227 const u16 calculated = crc_ccitt_false(0xffff, buf, size);
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 }

0-DAY kernel test infrastructure Open Source Technology Center Intel Corporation
[unhandled content-type:application/gzip]
 \ /
  Last update: 2017-12-23 08:24    [W:0.065 / U:2.776 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site