Re: [PATCH] HID: mcp2221: Keep IIO initialization retries per device

[sashiko-bot@kernel.org]

Thank you for your contribution! Sashiko AI review found 1 potential
issue to consider:
- [High] HID: mcp2221: Signed-off-by and Author use account handle
  instead of real name
--

commit 0750e741db2254c14f91e93fd256287b28060c69
Author: tuhaowen <tuhaowen@uniontech.com>

> HID: mcp2221: Keep IIO initialization retries per device
>
> mcp_init_work() uses a static retry counter, so all MCP2221 devices share
> the same retry budget. Initializing one device can exhaust the retries
> available to another device, including a new instance after unplugging
> and reconnecting the bridge.
>
> When the counter reaches zero, the post-decrement also changes it to -1.
> A subsequent instance that receives -EAGAIN will then keep rescheduling
> its initialization work instead of stopping at the intended limit.
> Different devices can also update the shared counter concurrently.
>
> Store the retry budget in struct mcp2221 and initialize it before queuing
> the IIO initialization work. Check for exhaustion before decrementing so
> the counter stays at zero once the retry limit is reached.
>
> Fixes: 960f9df7c620 ("HID: mcp2221: add ADC/DAC support via iio subsystem")
> Signed-off-by: tuhaowen <tuhaowen@uniontech.com>

[Severity: High]
The Author header and Signed-off-by trailer use the single-word account
handle "tuhaowen" instead of a full real name, which does not meet the
Developer's Certificate of Origin requirements in
Documentation/process/submitting-patches.rst.

-- 
Sashiko AI review · https://sashiko.dev/#/patchset/4861CDE55CCD8365+20261113014036.1430473-1-tuhaowen@uniontech.com?part=1