netdev - Re: [PATCH] net: xilinx_emaclite: Fix problem with first incoming packet

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Tue, 22 Sep 2009 09:53:43 -0700
From:	Grant Likely <grant.likely@...retlab.ca>
To:	John Linn <john.linn@...inx.com>
Cc:	netdev@...r.kernel.org, davem@...emloft.net,
	linuxppc-dev@...abs.org, jwboyer@...ux.vnet.ibm.com,
	sadanand.mutyala@...inx.com, Michal Simek <monstr@...str.eu>
Subject: Re: [PATCH] net: xilinx_emaclite: Fix problem with first incoming 
	packet

On Tue, Sep 22, 2009 at 8:24 AM, John Linn <john.linn@...inx.com> wrote:
> From: Michal Simek <monstr@...str.eu>
>
> You can't ping the board or connect to it unless you send
> any packet out from board.
>
> Tested-by: John Williams <john.williams@...alogix.com>
> Signed-off-by: Michal Simek <monstr@...str.eu>
> Acked-by: John Linn <john.linn@...inx.com>

John, Since this patch is being *sent* by you, then you should use a
"signed-off-by" tag instead because it actually passed through your
hands.

Oh, and:
Acked-by: Grant Likely <grant.likely@...retlab.ca>

> ---
>  drivers/net/xilinx_emaclite.c |    7 ++-----
>  1 files changed, 2 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/net/xilinx_emaclite.c b/drivers/net/xilinx_emaclite.c
> index dc22782..83a044d 100644
> --- a/drivers/net/xilinx_emaclite.c
> +++ b/drivers/net/xilinx_emaclite.c
> @@ -134,18 +134,15 @@ static void xemaclite_enable_interrupts(struct net_local *drvdata)
>        }
>
>        /* Enable the Rx interrupts for the first buffer */
> -       reg_data = in_be32(drvdata->base_addr + XEL_RSR_OFFSET);
>        out_be32(drvdata->base_addr + XEL_RSR_OFFSET,
> -                reg_data | XEL_RSR_RECV_IE_MASK);
> +                XEL_RSR_RECV_IE_MASK);
>
>        /* Enable the Rx interrupts for the second Buffer if
>         * configured in HW */
>        if (drvdata->rx_ping_pong != 0) {
> -               reg_data = in_be32(drvdata->base_addr + XEL_BUFFER_OFFSET +
> -                                  XEL_RSR_OFFSET);
>                out_be32(drvdata->base_addr + XEL_BUFFER_OFFSET +
>                         XEL_RSR_OFFSET,
> -                        reg_data | XEL_RSR_RECV_IE_MASK);
> +                        XEL_RSR_RECV_IE_MASK);
>        }
>
>        /* Enable the Global Interrupt Enable */
> --
> 1.6.2.1
>
>
>
> This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately.
>
>
>



-- 
Grant Likely, B.Sc., P.Eng.
Secret Lab Technologies Ltd.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html