> -----Original Message----- > From: Eric Dumazet <[email protected]> > On 6/13/20 11:11 PM, Greg Thelen wrote: > > Commit e086ba2fccda ("e1000e: disable s0ix entry and exit flows for ME > > systems") added e1000e_check_me() but it's only called from > > CONFIG_PM_SLEEP protected code. Thus builds without CONFIG_PM_SLEEP > > see: > > drivers/net/ethernet/intel/e1000e/netdev.c:137:13: warning: > > 'e1000e_check_me' defined but not used [-Wunused-function] > > > > Add CONFIG_PM_SLEEP ifdef guard to avoid dead code. > > > > Fixes: e086ba2fccda ("e1000e: disable s0ix entry and exit flows for ME > > systems") > > Signed-off-by: Greg Thelen <[email protected]> > > --- > > drivers/net/ethernet/intel/e1000e/netdev.c | 2 ++ > > 1 file changed, 2 insertions(+) > > Reviewed-by: Eric Dumazet <[email protected]> [Kirsher, Jeffrey T]
This patch is not necessary, after Arnd's patch. Here is his patch: http://patchwork.ozlabs.org/project/intel-wired-lan/patch/[email protected]/ and I will be pushing it to Dave's net tree later tonight.
