From mboxrd@z Thu Jan 1 00:00:00 1970 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751427AbeAPQUP (ORCPT + 1 other); Tue, 16 Jan 2018 11:20:15 -0500 Received: from mail-ot0-f195.google.com ([74.125.82.195]:41148 "EHLO mail-ot0-f195.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750997AbeAPQUN (ORCPT ); Tue, 16 Jan 2018 11:20:13 -0500 X-Google-Smtp-Source: ACJfBovLXqNBpg82buE94mRt2qmjr7L+RrYYeVs39qmugUqA276dKrXSrOEkvRxtY0pP41CMzfTsU6F9rKN3/Q1wcBI= MIME-Version: 1.0 In-Reply-To: <02874ECE860811409154E81DA85FBB5882BD2802@fmsmsx158.amr.corp.intel.com> References: <20180116091433.2535569-1-arnd@arndb.de> <02874ECE860811409154E81DA85FBB5882BD2802@fmsmsx158.amr.corp.intel.com> From: Arnd Bergmann Date: Tue, 16 Jan 2018 17:20:12 +0100 X-Google-Sender-Auth: eUV2W7OBney05UqkFk-7l1mEtuk Message-ID: Subject: Re: [PATCH] [RESEND net] fm10k: mark PM functions as __maybe_unused To: "Keller, Jacob E" Cc: "Kirsher, Jeffrey T" , "David S. Miller" , "Kwan, Ngai-mint" , "intel-wired-lan@lists.osuosl.org" , "netdev@vger.kernel.org" , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Return-Path: On Tue, Jan 16, 2018 at 5:12 PM, Keller, Jacob E wrote: >> -----Original Message----- >> From: Arnd Bergmann [mailto:arnd@arndb.de] >> Sent: Tuesday, January 16, 2018 1:14 AM >> To: Kirsher, Jeffrey T >> Cc: Arnd Bergmann ; Keller, Jacob E >> ; David S. Miller ; Kwan, >> Ngai-mint ; intel-wired-lan@lists.osuosl.org; >> netdev@vger.kernel.org; linux-kernel@vger.kernel.org >> Subject: [PATCH] [RESEND net] fm10k: mark PM functions as __maybe_unused >> >> A cleanup of the PM code left an incorrect #ifdef in place, leading >> to a harmless build warning: >> >> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:2502:12: error: 'fm10k_suspend' >> defined but not used [-Werror=unused-function] >> drivers/net/ethernet/intel/fm10k/fm10k_pci.c:2475:12: error: 'fm10k_resume' >> defined but not used [-Werror=unused-function] >> >> It's easier to use __maybe_unused attributes here, since you >> can't pick the wrong one. >> >> Fixes: 8249c47c6ba4 ("fm10k: use generic PM hooks instead of legacy PCIe power >> hooks") >> Acked-by: Jacob Keller >> Tested-by: Krishneil Singh >> Signed-off-by: Arnd Bergmann >> --- >> Apparently nobody picked this up the first time around (Oct 2017), >> here is the same patch again. > > Odd. I remember seeing this and thought I ack'd it..? Guess it got missed. > > Acked-by: Jacob Keller Yes, you did give an Ack (see above), it's just that it didn't make it into the tree for some reason. Maybe Jeff assumed you would forward it to him and you thought that he would pick it up from the list? Arnd