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 01/16] net: fec: only check queue 0 if
> RXF_0/TXF_0 interrupt is set
> 
> Before queue 0 was always checked if any queue caused an interrupt.
> It is better to just mark queue 0 if queue 0 has caused an interrupt.
> 
> Signed-off-by: Troy Kisky <troy.ki...@boundarydevices.com>
> ---
>  drivers/net/ethernet/freescale/fec_main.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/drivers/net/ethernet/freescale/fec_main.c
> b/drivers/net/ethernet/freescale/fec_main.c
> index bad0ba2..dbac975 100644
> --- a/drivers/net/ethernet/freescale/fec_main.c
> +++ b/drivers/net/ethernet/freescale/fec_main.c
> @@ -1534,14 +1534,14 @@ fec_enet_collect_events(struct fec_enet_private
> *fep, uint int_events)
>       if (int_events == 0)
>               return false;
> 
> -     if (int_events & FEC_ENET_RXF)
> +     if (int_events & FEC_ENET_RXF_0)
>               fep->work_rx |= (1 << 2);
>       if (int_events & FEC_ENET_RXF_1)
>               fep->work_rx |= (1 << 0);
>       if (int_events & FEC_ENET_RXF_2)
>               fep->work_rx |= (1 << 1);
> 
> -     if (int_events & FEC_ENET_TXF)
> +     if (int_events & FEC_ENET_TXF_0)
>               fep->work_tx |= (1 << 2);
>       if (int_events & FEC_ENET_TXF_1)
>               fep->work_tx |= (1 << 0);
> --
> 2.5.0

Acked-by: Fugang Duan <fugang.d...@nxp.com>

Reply via email to