linux-next: build warning after merge of the net-next tree

2 messages, 2 authors, 2021-03-17 · open the first message on its own page

linux-next: build warning after merge of the net-next tree

From: Stephen Rothwell <hidden>
Date: 2021-03-17 07:57:04

Hi all,

After merging the net-next tree, today's linux-next build (sparc64
defconfig) produced this warning:

drivers/net/ethernet/intel/e1000e/netdev.c:6926:12: warning: 'e1000e_pm_prepare' defined but not used [-Wunused-function]
 static int e1000e_pm_prepare(struct device *dev)
            ^~~~~~~~~~~~~~~~~

Introduced by commit

  ccf8b940e5fd ("e1000e: Leverage direct_complete to speed up s2ram")

CONFIG_PM_SLEEP is not set for this build.

-- 
Cheers,
Stephen Rothwell

Re: linux-next: build warning after merge of the net-next tree

From: Chen Yu <yu.c.chen@intel.com>
Date: 2021-03-17 08:17:49

Hi Stephen,
On Wed, Mar 17, 2021 at 06:56:05PM +1100, Stephen Rothwell wrote:
Hi all,

After merging the net-next tree, today's linux-next build (sparc64
defconfig) produced this warning:

drivers/net/ethernet/intel/e1000e/netdev.c:6926:12: warning: 'e1000e_pm_prepare' defined but not used [-Wunused-function]
 static int e1000e_pm_prepare(struct device *dev)
            ^~~~~~~~~~~~~~~~~

Introduced by commit

  ccf8b940e5fd ("e1000e: Leverage direct_complete to speed up s2ram")

CONFIG_PM_SLEEP is not set for this build.
Thanks for reporting. I think we need to add the '__maybe_unused' attribute as in case
CONFIG_PM_SLEEP is not set. Tony, it seems that the original v1 patch should eliminate
this warning, could you please help double check and apply that version?

thanks,
Chenyu
 
-- 
Cheers,
Stephen Rothwell

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help