Hi "José,
Thank you for the patch! Yet something to improve:
[auto build test ERROR on hid/for-next]
[also build test ERROR on v5.16-rc2 next-20211124]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url:
https://github.com/0day-ci/linux/commits/Jos-Exp-sito/Do-not-map-BTN_RIGH...
base:
https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next
config: x86_64-rhel-8.3
(
https://download.01.org/0day-ci/archive/20211124/202111241646.G968t85X-lk...)
compiler: gcc-9 (Debian 9.3.0-22) 9.3.0
reproduce (this is a W=1 build):
#
https://github.com/0day-ci/linux/commit/67a7bd322df749f6ef9a142479668db93...
git remote add linux-review
https://github.com/0day-ci/linux
git fetch --no-tags linux-review
Jos-Exp-sito/Do-not-map-BTN_RIGHT-MIDDLE-on-buttonpads/20211124-031407
git checkout 67a7bd322df749f6ef9a142479668db93b93beca
# save the config file to linux build tree
mkdir build_dir
make W=1 O=build_dir ARCH=x86_64 SHELL=/bin/bash drivers/hid/
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp(a)intel.com>
All errors (new ones prefixed by >>):
drivers/hid/hid-multitouch.c: In function 'mt_touch_input_configured':
> drivers/hid/hid-multitouch.c:1291:26: error: 'dev'
undeclared (first use in this function); did you mean 'hdev'?
1291 |
__clear_bit(BTN_RIGHT, dev->keybit);
| ^~~
| hdev
drivers/hid/hid-multitouch.c:1291:26: note: each undeclared identifier is reported only
once for each function it appears in
vim +1291 drivers/hid/hid-multitouch.c
1261
1262 static int mt_touch_input_configured(struct hid_device *hdev,
1263 struct hid_input *hi,
1264 struct mt_application *app)
1265 {
1266 struct mt_device *td = hid_get_drvdata(hdev);
1267 struct mt_class *cls = &td->mtclass;
1268 struct input_dev *input = hi->input;
1269 int ret;
1270
1271 if (!td->maxcontacts)
1272 td->maxcontacts = MT_DEFAULT_MAXCONTACT;
1273
1274 mt_post_parse(td, app);
1275 if (td->serial_maybe)
1276 mt_post_parse_default_settings(td, app);
1277
1278 if (cls->is_indirect)
1279 app->mt_flags |= INPUT_MT_POINTER;
1280
1281 if (app->quirks & MT_QUIRK_NOT_SEEN_MEANS_UP)
1282 app->mt_flags |= INPUT_MT_DROP_UNUSED;
1283
1284 /* check for clickpads */
1285 if ((app->mt_flags & INPUT_MT_POINTER) &&
1286 (app->buttons_count == 1))
1287 td->is_buttonpad = true;
1288
1289 if (td->is_buttonpad) {
1290 __set_bit(INPUT_PROP_BUTTONPAD, input->propbit);
1291 __clear_bit(BTN_RIGHT, dev->keybit);
1292 __clear_bit(BTN_MIDDLE, dev->keybit);
1293 }
1294
1295 app->pending_palm_slots = devm_kcalloc(&hi->input->dev,
1296 BITS_TO_LONGS(td->maxcontacts),
1297 sizeof(long),
1298 GFP_KERNEL);
1299 if (!app->pending_palm_slots)
1300 return -ENOMEM;
1301
1302 ret = input_mt_init_slots(input, td->maxcontacts, app->mt_flags);
1303 if (ret)
1304 return ret;
1305
1306 app->mt_flags = 0;
1307 return 0;
1308 }
1309
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org