> -----Original Message----- > From: Yury Norov <[email protected]> > Sent: Thursday, September 3, 2026 3:21 PM > To: Nguyen, Anthony L <[email protected]>; Kitszel, Przemyslaw > <[email protected]>; Andrew Lunn <[email protected]>; > David S. Miller <[email protected]>; Eric Dumazet > <[email protected]>; Jakub Kicinski <[email protected]>; Paolo Abeni > <[email protected]>; [email protected]; > [email protected]; open list <[email protected]> > Cc: Yury Norov <[email protected]> > Subject: [PATCH 2/2] ice: drop pf == NULL check in ice_pf_state_is_nominal() > > The function has 3 callers, all except one explicitly require pf != NULL. Add > the > corresponding check to the remaining caller, and remove it from the > ice_pf_state_is_nominal(). It makse the fun function a one-liner. > > Signed-off-by: Yury Norov <[email protected]> > --- > drivers/net/ethernet/intel/ice/ice_lib.c | 3 --- > drivers/net/ethernet/intel/ice/ice_main.c | 2 +- > 2 files changed, 1 insertion(+), 4 deletions(-) > > diff --git a/drivers/net/ethernet/intel/ice/ice_lib.c > b/drivers/net/ethernet/intel/ice/ice_lib.c > index 73996e3022ef..6d5b8acc4764 100644 > --- a/drivers/net/ethernet/intel/ice/ice_lib.c > +++ b/drivers/net/ethernet/intel/ice/ice_lib.c > @@ -1705,9 +1705,6 @@ static void ice_vsi_set_rss_flow_fld(struct ice_vsi > *vsi) > */ > bool ice_pf_state_is_nominal(struct ice_pf *pf) { > - if (!pf) > - return false; > - > return bitmap_empty(pf->state, ICE_STATE_NOMINAL_CHECK_BITS); > } > > diff --git a/drivers/net/ethernet/intel/ice/ice_main.c > b/drivers/net/ethernet/intel/ice/ice_main.c > index d88835482d3a..168d85f9477f 100644 > --- a/drivers/net/ethernet/intel/ice/ice_main.c > +++ b/drivers/net/ethernet/intel/ice/ice_main.c > @@ -5521,7 +5521,7 @@ static int ice_suspend(struct device *dev) > > pf = pci_get_drvdata(pdev); > > - if (!ice_pf_state_is_nominal(pf)) { > + if (!pf || !ice_pf_state_is_nominal(pf)) { > dev_err(dev, "Device is not ready, no need to suspend it\n"); > return -EBUSY; > } > -- > 2.53.0
Reviewed-by: Sergey Temerkhanov <[email protected]>
