net: ch9200: fix various bugs and improve qinheng ch9200 driver
Get series with b4
b4 am [email protected]Versions newest first
- v1 Latest5 patches 14 repliesIn review
Patches in v1 No applies recorded yet
| # | Subject and files | Size | Replies | Applied |
|---|---|---|---|---|
| 1/5 | net: ch9200: fix uninitialised access bug during mii_nway_restartdrivers/ | +5 | 1 reply | |
| 2/5 | net: ch9200: remove extraneous return that prevents error propagationdrivers/ | +0 | 0 replies | |
| 3/5 | net: ch9200: fail fast on control_read() failures during get_mac_address()drivers/ | +19 | 0 replies | |
| 4/5 | net: ch9200: add missing error handling in ch9200_bind()drivers/ | +15 | 2 replies | |
| 5/5 | net: ch9200: avoid triggering NWay restart on non-zero PHY IDdrivers/ | +1 | 11 replies |
Reviews trailers per patch
4 of 5 patches have a Reviewed-by or Acked-by.
| Patch | syzbot | Simon Horman | Andrew Lunn |
|---|---|---|---|
| 1/5 | T-b | R-b | |
| 2/5 | R-b | ||
| 3/5 | R-b | ||
| 4/5 | R-b | ||
| 5/5 |
Trailers given in replies are filled; those carried in the patches from earlier versions are outlined with dashes. Scroll the table sideways for all 3 reviewers.
Cover letter [PATCH 0/5] net: ch9200: fix various bugs and improve qinheng ch9200 driver
This patch series aims to fix various issues throughout the QinHeng CH9200 driver. This driver fails to handle various failures, which in one case has lead to a uninit access bug found via syzbot. Upon reviewing the driver I fixed a few more issues which I have included in this patch series.
Parts of this series are the product of discussions and suggestions I had from others like Andrew Lunn, Simon Horman and Jakub Kicinski you can view those discussions below:
Qasim Ijaz (5):
fix uninitialised access bug during mii_nway_restart remove extraneous return that prevents error propagation fail fast on control_read() failures during get_mac_address() add missing error handling in ch9200_bind()
avoid triggering NWay restart on non-zero PHY ID
drivers/net/usb/ch9200.c | 61 ++++++++++++++++++++++++++-------------- 1 file changed, 40 insertions(+), 21 deletions(-)
Signature · 1 line
2.39.5