Re: [PATCH net-next v6 01/10] r8152: split r8156_init per chip and factor out wait_autoload_done

[netdev-bot+sashiko@kernel.org]

Thank you for your contribution! Sashiko AI review found more severe
issue(s) in other patches of this series. This mail is only an FYI, in case
you want to address the nit picks below when you respin.

Low priority / nit picks for your consideration:

- [Low] The commit message says r8152b_init() was converted to
  wait_autoload_done(), along with r8153_init(), r8153b_init(),…

--

r8152: split r8156_init per chip and factor out wait_autoload_done

This splits the shared r8156_init() into r8156_init(), r8156b_init(),
r8157_init() and r8159_init(), one per chip family. It also adds a
wait_autoload_done() helper built on read_poll_timeout(). The helper
replaces the open-coded PLA_BOOT_CTRL/AUTOLOAD_DONE polling loops in
several init and UPS functions.

> r8152: split r8156_init per chip and factor out wait_autoload_done

[ ... ]

> Also factor the repeated "poll PLA_BOOT_CTRL for AUTOLOAD_DONE" loop
> into wait_autoload_done(), reusing read_poll_timeout() instead of the
> 500 * msleep(20) busy loops. r8152b_init(), r8153_init(),
> r8153b_init(), r8153c_init(), r8153b_ups_en() and r8153c_ups_en() are
> converted to use it too.

[Severity: Low]
Should r8152b_init() be in this list?

r8152b_init() in drivers/net/usb/r8152.c has no PLA_BOOT_CTRL/AUTOLOAD_DONE
polling loop, either before or after this patch, and the patch does not
change it. Its name only shows up in the diff as the hunk header for the
r8153_init() change:

  @@ -7225,22 +7226,14 @@ static void r8152b_init(struct r8152 *tp)
   static void r8153_init(struct r8152 *tp)

After this patch, wait_autoload_done() is called from r8153b_ups_en(),
r8153c_ups_en(), r8153_init(), r8153b_init(), r8153c_init(), r8156_init(),
r8156b_init(), r8157_init() and r8159_init().

Could r8152b_init() be removed from the list of converted functions in the
commit message?

-- 
Sashiko AI review · https://netdev-ai.bots.linux.dev/sashiko/#/patchset/20261008081556.47901-1-nic_swsd%40realtek.com