[net,v2,0/4] bnxt_en: Misc. fixes

Message

Michael Chan Dec. 7, 2023, 12:05 a.m. UTC

4 miscellaneous driver fixes covering PM resume, SKB recycling,
wrong return value check, and PTP HWTSTAMP_FILTER_ALL.

v2: Fix SOB tags in patch 1 and 3.

Kalesh AP (1):
  bnxt_en: Fix wrong return value check in bnxt_close_nic()

Michael Chan (1):
  bnxt_en: Fix HWTSTAMP_FILTER_ALL packet timestamp logic

Somnath Kotur (1):
  bnxt_en: Clear resource reservation during resume

Sreekanth Reddy (1):
  bnxt_en: Fix skb recycling logic in bnxt_deliver_skb()

 drivers/net/ethernet/broadcom/bnxt/bnxt.c | 27 ++++++++++++++++++-----
 drivers/net/ethernet/broadcom/bnxt/bnxt.h |  8 ++++++-
 2 files changed, 29 insertions(+), 6 deletions(-)

Comments

Vadim Fedorenko Dec. 7, 2023, 12:04 p.m. UTC | #1

On 06/12/2023 16:05, Michael Chan wrote:
> 4 miscellaneous driver fixes covering PM resume, SKB recycling,
> wrong return value check, and PTP HWTSTAMP_FILTER_ALL.
> 
> v2: Fix SOB tags in patch 1 and 3.
> 
> Kalesh AP (1):
>    bnxt_en: Fix wrong return value check in bnxt_close_nic()
> 
> Michael Chan (1):
>    bnxt_en: Fix HWTSTAMP_FILTER_ALL packet timestamp logic

Hi Michael!

What about reconfiguration part? I thought this fix was the only
blocker to remove bnxt_close_nic/bnxt_open_nic logic.

> 
> Somnath Kotur (1):
>    bnxt_en: Clear resource reservation during resume
> 
> Sreekanth Reddy (1):
>    bnxt_en: Fix skb recycling logic in bnxt_deliver_skb()
> 
>   drivers/net/ethernet/broadcom/bnxt/bnxt.c | 27 ++++++++++++++++++-----
>   drivers/net/ethernet/broadcom/bnxt/bnxt.h |  8 ++++++-
>   2 files changed, 29 insertions(+), 6 deletions(-)
>

Michael Chan Dec. 7, 2023, 4:24 p.m. UTC | #2

On Thu, Dec 7, 2023 at 4:04 AM Vadim Fedorenko
<[e-mail]> wrote:
>
> On 06/12/2023 16:05, Michael Chan wrote:
> > 4 miscellaneous driver fixes covering PM resume, SKB recycling,
> > wrong return value check, and PTP HWTSTAMP_FILTER_ALL.
> >
> > v2: Fix SOB tags in patch 1 and 3.
> >
> > Kalesh AP (1):
> >    bnxt_en: Fix wrong return value check in bnxt_close_nic()
> >
> > Michael Chan (1):
> >    bnxt_en: Fix HWTSTAMP_FILTER_ALL packet timestamp logic
>
> Hi Michael!
>
> What about reconfiguration part? I thought this fix was the only
> blocker to remove bnxt_close_nic/bnxt_open_nic logic.

The other PTP patches from Pavan will be submitted to net-next in a
few days because those are technically improvements and not bug fixes.
Thanks.