Re: [PATCH] brcmfmac: avoid duplicated suspend/resume operation

From: Arend van Spriel
Date: Fri Jan 23 2015 - 13:09:56 EST


On 01/23/15 16:29, Kalle Valo wrote:
Arend van Spriel<arend@xxxxxxxxxxxx> 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<zhonghui.fu@xxxxxxxxxxxxxxx>
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<arend@xxxxxxxxxxxx>
Acked-by: Sergei Shtylyov<sergei.shtylyov@xxxxxxxxxxxxxxxxxx>
Acked-by: Kalle Valo<kvalo@xxxxxxxxxxxxxx>
Signed-off-by: Zhonghui Fu<zhonghui.fu@xxxxxxxxxxxxxxx>

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
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/