diff for duplicates of <202602120209.xYKh9QQp-lkp@intel.com> diff --git a/a/1.txt b/N1/1.txt index f9a4366..8d24280 100644 --- a/a/1.txt +++ b/N1/1.txt @@ -1,13 +1,28 @@ +BCC: lkp@intel.com +CC: oe-kbuild-all@lists.linux.dev +In-Reply-To: <20260211053116.GA20357@altimeter-info> +References: <20260211053116.GA20357@altimeter-info> +TO: Ivan Gorinov <linux-kernel@altimeter.info> +TO: Jiri Kosina <jikos@kernel.org> +CC: linux-input@vger.kernel.org +CC: linux-kernel@vger.kernel.org + Hi Ivan, kernel test robot noticed the following build warnings: +[auto build test WARNING on hid/for-next] +[also build test WARNING on linus/master v6.19 next-20260211] +[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#_base_tree_information] url: https://github.com/intel-lab-lkp/linux/commits/Ivan-Gorinov/HID-winwing-Enable-rumble-effects/20260211-133322 base: https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next patch link: https://lore.kernel.org/r/20260211053116.GA20357%40altimeter-info patch subject: [PATCH RESEND] HID: winwing: Enable rumble effects +:::::: branch date: 13 hours ago +:::::: commit date: 13 hours ago config: parisc-randconfig-r071-20260211 (https://download.01.org/0day-ci/archive/20260212/202602120209.xYKh9QQp-lkp@intel.com/config) compiler: hppa-linux-gcc (GCC) 8.5.0 smatch version: v0.5.0-8994-gd50c5a4c @@ -15,7 +30,7 @@ smatch version: v0.5.0-8994-gd50c5a4c If you fix the issue in a separate patch/commit (i.e. not just a new version of the same patch/commit), kindly add following tags | Reported-by: kernel test robot <lkp@intel.com> -| Reported-by: Dan Carpenter <dan.carpenter@linaro.org> +| Reported-by: Dan Carpenter <error27@gmail.com> | Closes: https://lore.kernel.org/r/202602120209.xYKh9QQp-lkp@intel.com/ smatch warnings: @@ -25,6 +40,7 @@ drivers/hid/hid-winwing.c:339 winwing_init_ff() warn: variable dereferenced befo vim +235 drivers/hid/hid-winwing.c +c4c97b07cd09c03 Ivan Gorinov 2026-02-11 220 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 221 static int winwing_haptic_rumble(struct winwing_drv_data *data) c4c97b07cd09c03 Ivan Gorinov 2026-02-11 222 { c4c97b07cd09c03 Ivan Gorinov 2026-02-11 223 __u8 *buf; @@ -40,10 +56,6 @@ c4c97b07cd09c03 Ivan Gorinov 2026-02-11 232 return -EINVAL; c4c97b07cd09c03 Ivan Gorinov 2026-02-11 233 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 234 if (!data->hdev) { c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @235 hid_err(data->hdev, "data->hdev == NULL\n"); - ^^^^^^^^^^ -This doesn't end up getting dereferenced because of dev_err() magic -but passing a NULL here is pointless. - c4c97b07cd09c03 Ivan Gorinov 2026-02-11 236 return -EINVAL; c4c97b07cd09c03 Ivan Gorinov 2026-02-11 237 } c4c97b07cd09c03 Ivan Gorinov 2026-02-11 238 @@ -117,9 +129,6 @@ c4c97b07cd09c03 Ivan Gorinov 2026-02-11 305 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 306 data = container_of(work, struct winwing_drv_data, rumble_work); c4c97b07cd09c03 Ivan Gorinov 2026-02-11 307 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @308 if (data) - -"data" can't be NULL. It's work minus an offset. - c4c97b07cd09c03 Ivan Gorinov 2026-02-11 309 winwing_haptic_rumble(data); c4c97b07cd09c03 Ivan Gorinov 2026-02-11 310 } c4c97b07cd09c03 Ivan Gorinov 2026-02-11 311 @@ -147,21 +156,16 @@ c4c97b07cd09c03 Ivan Gorinov 2026-02-11 332 data = (struct winwing_drv_data * c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @333 data->report_rumble = devm_kzalloc(&hdev->dev, MAX_REPORT, GFP_KERNEL); c4c97b07cd09c03 Ivan Gorinov 2026-02-11 334 data->rumble_left = -1; c4c97b07cd09c03 Ivan Gorinov 2026-02-11 335 data->rumble_right = -1; - ^^^^^^^^^^^^^^^^^^ -Dereferences here. - c4c97b07cd09c03 Ivan Gorinov 2026-02-11 336 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 337 input_set_capability(hidinput->input, EV_FF, FF_RUMBLE); c4c97b07cd09c03 Ivan Gorinov 2026-02-11 338 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @339 if (!data) - ^^^^^^ -Checked too late. - c4c97b07cd09c03 Ivan Gorinov 2026-02-11 340 return -EINVAL; c4c97b07cd09c03 Ivan Gorinov 2026-02-11 341 c4c97b07cd09c03 Ivan Gorinov 2026-02-11 342 return input_ff_create_memless(hidinput->input, data, c4c97b07cd09c03 Ivan Gorinov 2026-02-11 343 winwing_play_effect); c4c97b07cd09c03 Ivan Gorinov 2026-02-11 344 } +c4c97b07cd09c03 Ivan Gorinov 2026-02-11 345 -- 0-DAY CI Kernel Test Service diff --git a/a/content_digest b/N1/content_digest index a3bd486..22dfb61 100644 --- a/a/content_digest +++ b/N1/content_digest @@ -1,26 +1,36 @@ - "ref\020260211053116.GA20357@altimeter-info\0" - "From\0Dan Carpenter <dan.carpenter@linaro.org>\0" + "From\0kernel test robot <lkp@intel.com>\0" "Subject\0Re: [PATCH RESEND] HID: winwing: Enable rumble effects\0" - "Date\0Thu, 12 Feb 2026 08:31:11 +0300\0" - "To\0oe-kbuild@lists.linux.dev" - Ivan Gorinov <linux-kernel@altimeter.info> - " Jiri Kosina <jikos@kernel.org>\0" + "Date\0Thu, 12 Feb 2026 02:47:34 +0800\0" + "To\0oe-kbuild@lists.linux.dev\0" "Cc\0lkp@intel.com" - oe-kbuild-all@lists.linux.dev - linux-input@vger.kernel.org - " linux-kernel@vger.kernel.org\0" + " Dan Carpenter <error27@gmail.com>\0" "\00:1\0" "b\0" + "BCC: lkp@intel.com\n" + "CC: oe-kbuild-all@lists.linux.dev\n" + "In-Reply-To: <20260211053116.GA20357@altimeter-info>\n" + "References: <20260211053116.GA20357@altimeter-info>\n" + "TO: Ivan Gorinov <linux-kernel@altimeter.info>\n" + "TO: Jiri Kosina <jikos@kernel.org>\n" + "CC: linux-input@vger.kernel.org\n" + "CC: linux-kernel@vger.kernel.org\n" + "\n" "Hi Ivan,\n" "\n" "kernel test robot noticed the following build warnings:\n" "\n" + "[auto build test WARNING on hid/for-next]\n" + "[also build test WARNING on linus/master v6.19 next-20260211]\n" + "[If your patch is applied to the wrong git tree, kindly drop us a note.\n" + "And when submitting patch, we suggest to use '--base' as documented in\n" "https://git-scm.com/docs/git-format-patch#_base_tree_information]\n" "\n" "url: https://github.com/intel-lab-lkp/linux/commits/Ivan-Gorinov/HID-winwing-Enable-rumble-effects/20260211-133322\n" "base: https://git.kernel.org/pub/scm/linux/kernel/git/hid/hid.git for-next\n" "patch link: https://lore.kernel.org/r/20260211053116.GA20357%40altimeter-info\n" "patch subject: [PATCH RESEND] HID: winwing: Enable rumble effects\n" + ":::::: branch date: 13 hours ago\n" + ":::::: commit date: 13 hours ago\n" "config: parisc-randconfig-r071-20260211 (https://download.01.org/0day-ci/archive/20260212/202602120209.xYKh9QQp-lkp@intel.com/config)\n" "compiler: hppa-linux-gcc (GCC) 8.5.0\n" "smatch version: v0.5.0-8994-gd50c5a4c\n" @@ -28,7 +38,7 @@ "If you fix the issue in a separate patch/commit (i.e. not just a new version of\n" "the same patch/commit), kindly add following tags\n" "| Reported-by: kernel test robot <lkp@intel.com>\n" - "| Reported-by: Dan Carpenter <dan.carpenter@linaro.org>\n" + "| Reported-by: Dan Carpenter <error27@gmail.com>\n" "| Closes: https://lore.kernel.org/r/202602120209.xYKh9QQp-lkp@intel.com/\n" "\n" "smatch warnings:\n" @@ -38,6 +48,7 @@ "\n" "vim +235 drivers/hid/hid-winwing.c\n" "\n" + "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 220 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 221 static int winwing_haptic_rumble(struct winwing_drv_data *data)\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 222 {\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 223 \t__u8 *buf;\n" @@ -53,10 +64,6 @@ "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 233 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 234 \tif (!data->hdev) {\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @235 \t\thid_err(data->hdev, \"data->hdev == NULL\\n\");\n" - " ^^^^^^^^^^\n" - "This doesn't end up getting dereferenced because of dev_err() magic\n" - "but passing a NULL here is pointless.\n" - "\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 236 \t\treturn -EINVAL;\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 237 \t}\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 238 \n" @@ -130,9 +137,6 @@ "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 306 \tdata = container_of(work, struct winwing_drv_data, rumble_work);\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 307 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @308 \tif (data)\n" - "\n" - "\"data\" can't be NULL. It's work minus an offset.\n" - "\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 309 \t\twinwing_haptic_rumble(data);\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 310 }\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 311 \n" @@ -160,24 +164,19 @@ "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @333 \tdata->report_rumble = devm_kzalloc(&hdev->dev, MAX_REPORT, GFP_KERNEL);\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 334 \tdata->rumble_left = -1;\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 335 \tdata->rumble_right = -1;\n" - " ^^^^^^^^^^^^^^^^^^\n" - "Dereferences here.\n" - "\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 336 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 337 \tinput_set_capability(hidinput->input, EV_FF, FF_RUMBLE);\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 338 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 @339 \tif (!data)\n" - " ^^^^^^\n" - "Checked too late.\n" - "\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 340 \t\treturn -EINVAL;\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 341 \n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 342 \treturn input_ff_create_memless(hidinput->input, data,\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 343 \t\t\twinwing_play_effect);\n" "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 344 }\n" + "c4c97b07cd09c03 Ivan Gorinov 2026-02-11 345 \n" "\n" "-- \n" "0-DAY CI Kernel Test Service\n" https://github.com/intel/lkp-tests/wiki -94f42de4faa0a061c87d4b0f999d34d2830b2425350c33e60ea29b7a4c27a56b +ff959848f89df5d08e4d0aae1a5c8d83872279e120e346a78307d92c8655d0f0
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.