On Wed, 27 May 2026 09:51:43 +0200 Ivan Vecera wrote:

On 5/26/26 9:45 AM, Ivan Vecera wrote:

quoted

The change_work was introduced to send device change notifications
from DPLL device callbacks without deadlocking on dpll_lock, since
the callbacks are already invoked under that lock. Now that
__dpll_device_change_ntf() is exported for callers that already
hold dpll_lock, use it directly and remove the change_work
infrastructure entirely.

This eliminates a race condition where change_work could be
re-scheduled after cancel_work_sync() during device teardown,
potentially causing the handler to dereference a freed or NULL
dpll_dev pointer.

Fixes: 9363b4837659 ("dpll: zl3073x: Allow to configure phase offset averaging factor")
Signed-off-by: Ivan Vecera <ivecera@redhat.com>  

Hi Kuba & Paolo,

patchwork reported netdev/build_allmodconfig_warn failure for this
patch. But it looks weird:

https://netdev-ctrl.bots.linux.dev/logs/build/1100767/14594344/build_allmodconfig_warn/stderr

I don't understand how this is related to this patch.

Sparse randomly generates this once a week on a seemingly random patch,
I haven't dug into why. Safe to ignore.