Re: [PATCH Speakup v2] return same error value from spk_set_key_info

From: kbuild test robot
Date: Tue Feb 21 2017 - 10:37:33 EST


Hi Pranay,

[auto build test WARNING on staging/staging-testing]
[also build test WARNING on v4.10 next-20170220]
[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/Pranay-Kr-Srivastava/return-same-error-value-from-spk_set_key_info/20170221-224753
config: i386-randconfig-i1-201708 (attached as .config)
compiler: gcc-4.8 (Debian 4.8.4-1) 4.8.4
reproduce:
# save the attached .config to linux build tree
make ARCH=i386

All warnings (new ones prefixed by >>):

drivers/staging/speakup/main.c: In function 'spk_set_key_info':
>> drivers/staging/speakup/main.c:1228:3: warning: format '%lu' expects argument of type 'long unsigned int', but argument 4 has type 'unsigned int' [-Wformat=]
pr_debug("key_data_len = %d, SHIFT_TBL_SIZE + 4 = %d,\t"
^

vim +1228 drivers/staging/speakup/main.c

1212 {
1213 int i = 0, states, key_data_len;
1214 const u_char *cp = key_info;
1215 u_char *cp1 = k_buffer;
1216 u_char ch, version, num_keys;
1217
1218 version = *cp++;
1219 if (version != KEY_MAP_VER) {
1220 pr_debug("version found %d should be %d\n",
1221 version, KEY_MAP_VER);
1222 return -EINVAL;
1223 }
1224 num_keys = *cp;
1225 states = (int)cp[1];
1226 key_data_len = (states + 1) * (num_keys + 1);
1227 if (key_data_len + SHIFT_TBL_SIZE + 4 >= sizeof(spk_key_buf)) {
> 1228 pr_debug("key_data_len = %d, SHIFT_TBL_SIZE + 4 = %d,\t"
1229 "sizeof(spk_key_buf) = %lu\n", key_data_len,
1230 SHIFT_TBL_SIZE + 4, sizeof(spk_key_buf));
1231 return -EINVAL;
1232 }
1233 memset(k_buffer, 0, SHIFT_TBL_SIZE);
1234 memset(spk_our_keys, 0, sizeof(spk_our_keys));
1235 spk_shift_table = k_buffer;
1236 spk_our_keys[0] = spk_shift_table;

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

Attachment: .config.gz
Description: application/gzip