On Wed, 06 Mar 2024 18:38:04 +0300, Daniil Dulov wrote: > Pointer item is checked fo NULL at mlxreg_hotplug_work_helper() and then > it is dereferenced to produce dev_err(). > This pointer is also dereferenced before calling this function and should > never be NULL except some piece of hardware is broken as it is said in > the comment before the check. So, this check can be safely removed. > > Found by Linux Verification Center (linuxtesting.org) with SVACE. > > [...] Thank you for your contribution, it has been applied to my local review-ilpo branch. Note it will show up in the public platform-drivers-x86/review-ilpo branch only once I've pushed my local branch there, which might take a while. The list of commits applied: [1/1] platform/mellanox: mlxreg-hotplug: Remove redundant NULL-check commit: 576b82c3c9bf021af4984aafc105508c99660667 -- i.