On Tue, Jan 16, 2018 at 5:12 PM, Keller, Jacob E
<[email protected]> wrote:
>> -----Original Message-----
>> From: Arnd Bergmann [mailto:[email protected]]
>> Sent: Tuesday, January 16, 2018 1:14 AM
>> To: Kirsher, Jeffrey T <[email protected]>
>> Cc: Arnd Bergmann <[email protected]>; Keller, Jacob E
>> <[email protected]>; David S. Miller <[email protected]>; Kwan,
>> Ngai-mint <[email protected]>; [email protected];
>> [email protected]; [email protected]
>> 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 <[email protected]>
>> Tested-by: Krishneil Singh <[email protected]>
>> Signed-off-by: Arnd Bergmann <[email protected]>
>> ---
>> 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 <[email protected]>
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
