Re: [PATCH 1/1] HID: multitouch: only map BTN_LEFT on buttonpads

From: kernel test robot
Date: Wed Nov 24 2021 - 03:44:31 EST


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_RIGHT-MIDDLE-on-buttonpads/20211124-031407
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-lkp@xxxxxxxxx/config)
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/67a7bd322df749f6ef9a142479668db93b93beca
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@xxxxxxxxx>

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@xxxxxxxxxxxx