mbox series

[v1,net-next,0/2] clean up ocelot_reset() routine

Message ID 20220917175127.161504-1-colin.foster@in-advantage.com (mailing list archive)
Headers show
Series clean up ocelot_reset() routine | expand

Message

Colin Foster Sept. 17, 2022, 5:51 p.m. UTC 
ocelot_reset() will soon be exported to a common library to be used by
the ocelot_ext system. This will make error values from regmap calls
possible, so they must be checked. Additionally, readx_poll_timeout()
can be substituted for the custom loop, as a simple cleanup.

I don't have hardware to verify this set directly, but there shouldn't
be any functional changes.

v2:
    Fix 64-bit compiler warning (1/2) (kernel test robot)
    Remove unnecessary variable assignment (2/2)
    Add Reviewed tag (1/2)

Colin Foster (2):
  net: mscc: ocelot: utilize readx_poll_timeout() for chip reset
  net: mscc: ocelot: check return values of writes during reset

 drivers/net/ethernet/mscc/ocelot_vsc7514.c | 46 ++++++++++++++++------
 1 file changed, 33 insertions(+), 13 deletions(-)

Comments

patchwork-bot+netdevbpf@kernel.org Sept. 22, 2022, 1:40 a.m. UTC | #1 
Hello:

This series was applied to netdev/net-next.git (master)
by Jakub Kicinski <kuba@kernel.org>:

On Sat, 17 Sep 2022 10:51:25 -0700 you wrote:
> ocelot_reset() will soon be exported to a common library to be used by
> the ocelot_ext system. This will make error values from regmap calls
> possible, so they must be checked. Additionally, readx_poll_timeout()
> can be substituted for the custom loop, as a simple cleanup.
> 
> I don't have hardware to verify this set directly, but there shouldn't
> be any functional changes.
> 
> [...]

Here is the summary with links:
  - [v2,net-next,1/2] net: mscc: ocelot: utilize readx_poll_timeout() for chip reset
    https://git.kernel.org/netdev/net-next/c/21bb08cd2cda
  - [v2,net-next,2/2] net: mscc: ocelot: check return values of writes during reset
    https://git.kernel.org/netdev/net-next/c/fa1d90b048c2

You are awesome, thank you!