From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arend van Spriel Subject: Re: [PATCH] brcmfmac: avoid duplicated suspend/resume operation Date: Fri, 23 Jan 2015 19:09:47 +0100 Message-ID: <54C28E6B.8090408@broadcom.com> References: <54BDCC06.8090107@linux.intel.com> <87twzincvz.fsf@kamboji.qca.qualcomm.com> <54C1012F.1020400@cogentembedded.com> <54C1646B.4080409@broadcom.com> <87y4otldln.fsf@kamboji.qca.qualcomm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Cc: Sergei Shtylyov , "Fu, Zhonghui" , , Franky Lin , , , , , , , , , "linux-kernel@vger.kernel.org" To: Kalle Valo Return-path: In-Reply-To: <87y4otldln.fsf@kamboji.qca.qualcomm.com> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org On 01/23/15 16:29, Kalle Valo wrote: > Arend van Spriel writes: > >> On 01/22/15 14:54, Sergei Shtylyov wrote: >>> Hello. >>> >>> On 1/22/2015 4:49 PM, Kalle Valo wrote: >>> >>>>> > From 04d3fa673897ca4ccbea6c76836d0092dba2484a Mon Sep 17 00:00:00 2001 >>>>> From: Zhonghui Fu >>>>> Date: Tue, 20 Jan 2015 11:14:13 +0800 >>>>> Subject: [PATCH] brcmfmac: avoid duplicated suspend/resume operation >>> >>>>> WiFi chip has 2 SDIO functions, and PM core will trigger >>>>> twice suspend/resume operations for one WiFi chip to do >>>>> the same things. This patch avoid this case. >>> >>>>> Acked-by: Arend van Spriel >>>>> Acked-by: Sergei Shtylyov >>>>> Acked-by: Kalle Valo >>>>> Signed-off-by: Zhonghui Fu >>> >>>> I don't remember giving Acked-by to this (or for matter to anything for >>>> a long time). What about Sergei or Arend? >>> >>> I haven't ACK'ed this patch either. >> >> I did ACK the initial patch and felt it still valid for this 'V2' patch. > > Ok, thanks. So the patch is good, Zhonghui just needs to remove the two > acked-by lines. Indeed. Regards, Arend