On 3/4/2016 2:11 AM, Fugang Duan wrote: > From: Troy Kisky <troy.ki...@boundarydevices.com>Sent: Thursday, February 25, > 2016 8:37 AM >> To: netdev@vger.kernel.org; da...@davemloft.net; b38...@freescale.com >> Cc: fabio.este...@freescale.com; l.st...@pengutronix.de; and...@lunn.ch; >> trem...@gmail.com; li...@arm.linux.org.uk; linux-arm- >> ker...@lists.infradead.org; l...@boundarydevices.com; shawn...@kernel.org; >> johan...@sipsolutions.net; stillcompil...@gmail.com; >> sergei.shtyl...@cogentembedded.com; a...@arndb.de; Troy Kisky >> <troy.ki...@boundarydevices.com> >> Subject: [PATCH net-next V2 06/16] net: fec: don't clear all rx queue bits >> when >> just one is being checked >> >> FEC_ENET_RXF is 3 separate bits, we only check one queue at a time. So, when >> the last queue is being checked, it is bad to remove the interrupt on the 1st >> queue. >> >> Also, since this is now done in the napi routine and not the interrupt, it >> is not >> needed. >> >> Signed-off-by: Troy Kisky <troy.ki...@boundarydevices.com> >> --- >> drivers/net/ethernet/freescale/fec_main.c | 2 -- >> 1 file changed, 2 deletions(-) >> >> diff --git a/drivers/net/ethernet/freescale/fec_main.c >> b/drivers/net/ethernet/freescale/fec_main.c >> index 610cf6c..791f385 100644 >> --- a/drivers/net/ethernet/freescale/fec_main.c >> +++ b/drivers/net/ethernet/freescale/fec_main.c >> @@ -1338,8 +1338,6 @@ static int fec_rxq(struct net_device *ndev, struct >> fec_enet_private *fep, >> break; >> pkt_received++; >> >> - writel(FEC_ENET_RXF, fep->hwp + FEC_IEVENT); >> - > > We should clear the related rx queue ievent, not remove the code. > Pls see commit: db3421c114cf that was submitted by Russell King. > > No ack the patch.
This is now done in patch #4 "net: fec: reduce interrupts" and you could argue that it should be squashed into that patch. But I like separating changes as much as possible. Russell, this patch and patch #4 will likely need your ack before it will be applied. Can you take a look please? http://www.spinics.net/lists/netdev/msg361927.html Thanks Troy
