netdev - RE: [PATCH v2 7/8] net/macb: fix the peripheral version test

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, 31 Mar 2015 15:08:19 +0000
From:	Punnaiah Choudary Kalluri <punnaiah.choudary.kalluri@...inx.com>
To:	Nicolas Ferre <nicolas.ferre@...el.com>,
	"linux-arm-kernel@...ts.infradead.org" 
	<linux-arm-kernel@...ts.infradead.org>,
	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"David S. Miller" <davem@...emloft.net>
CC:	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"Boris BREZILLON" <boris.brezillon@...e-electrons.com>,
	Cyrille Pitchen <cyrille.pitchen@...el.com>,
	Alexandre Belloni <alexandre.belloni@...e-electrons.com>,
	Michal Simek <michals@...inx.com>,
	"stable@...r.kernel.org" <stable@...r.kernel.org>
Subject: RE: [PATCH v2 7/8] net/macb: fix the peripheral version test



> -----Original Message-----
> From: Nicolas Ferre [mailto:nicolas.ferre@...el.com]
> Sent: Tuesday, March 31, 2015 6:32 PM
> To: linux-arm-kernel@...ts.infradead.org; netdev@...r.kernel.org; David S.
> Miller
> Cc: linux-kernel@...r.kernel.org; Boris BREZILLON; Cyrille Pitchen; Alexandre
> Belloni; Michal Simek; Punnaiah Choudary Kalluri; Nicolas Ferre;
> stable@...r.kernel.org
> Subject: [PATCH v2 7/8] net/macb: fix the peripheral version test
>
> We currently need two checks of the peripheral version in MACB_MID
> register.
> One of them got out of sync after modification by 8a013a9c71b2 (net: macb:
> Include multi queue support for xilinx ZynqMP ethernet version).
> Fix this in macb_configure_caps() so that xilinx ZynqMP will be considered
> as a GEM flavor.
>
> Fixes: 8a013a9c71b2 ("net: macb: Include multi queue support for xilinx
> ZynqMP
> ethernet version")
>
> Signed-off-by: Nicolas Ferre <nicolas.ferre@...el.com>
   Reviewed-by: Punnaiah Choudary Kalluri <punnaia@...inx.com>

Thanks for the patch. We have this change in our tree but somehow missed while sending
this patch to mainline.

Regards,
Punnaiah
> Cc: Michal Simek <michal.simek@...inx.com>
> Cc: Punnaiah Choudary Kalluri <punnaia@...inx.com>
> Cc: <stable@...r.kernel.org> #4.0 (if it doesn't make it for -final)
> ---
>
> Changes in v2:
> - fix peripheral version test
>
>  drivers/net/ethernet/cadence/macb.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/net/ethernet/cadence/macb.c
> b/drivers/net/ethernet/cadence/macb.c
> index babe972a7c32..4412895cf4a8 100644
> --- a/drivers/net/ethernet/cadence/macb.c
> +++ b/drivers/net/ethernet/cadence/macb.c
> @@ -2139,7 +2139,7 @@ static void macb_configure_caps(struct macb *bp,
> const struct macb_config *dt_co
>       if (dt_conf)
>               bp->caps = dt_conf->caps;
>
> -     if (MACB_BFEXT(IDNUM, macb_readl(bp, MID)) == 0x2)
> +     if (MACB_BFEXT(IDNUM, macb_readl(bp, MID)) >= 0x2)
>               bp->caps |= MACB_CAPS_MACB_IS_GEM;
>
>       if (macb_is_gem(bp)) {
> --
> 2.1.3



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.

--
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