netdev - Re: [PATCH net-next v2 0/3] net: dsa: b53: Turn on Broadcom tags

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] [day] [month] [year] [list]

Message-ID: <cdc3c83a-daec-edac-4df1-1ec240bea21a@gmail.com>
Date:   Fri, 10 Nov 2017 10:01:58 -0800
From:   Florian Fainelli <f.fainelli@...il.com>
To:     netdev@...r.kernel.org, davem@...emloft.net
Cc:     andrew@...n.ch, vivien.didelot@...oirfairelinux.com
Subject: Re: [PATCH net-next v2 0/3] net: dsa: b53: Turn on Broadcom tags

On 11/09/2017 02:34 PM, Florian Fainelli wrote:
> Hi all,
> 
> This was long overdue, with this patch series, the b53 driver now
> turns on Broadcom tags except for 5325 and 5365 which use an older
> format that we do not support yet (TBD).
> 
> First patch is necessary in order for bgmac, used on BCM5301X and Northstar
> Plus to work correctly and successfully send ARP packets back to the requsester.
> 
> Second patch is actually a bug fix, but because net/master and net-next/master
> diverge in that area, I am targeting net-next/master here.
> 
> Finally, the last patch enables Broadcom tags after checking that the CPU port
> selected is either, 5, 7 or 8, since those are the only valid combinations
> given currently supported HW.

David, I forgot to update drivers/net/dsa/b53/Kconfig to have a select
NET_DSA_TAG_BRCM, let me re-submit that.

> 
> Florian Fainelli (3):
>   net: bgmac: Pad packets to a minimum size
>   net: dsa: b53: Stop using dev->cpu_port incorrectly
>   net: dsa: b53: Turn on Broadcom tags
> 
>  drivers/net/dsa/b53/b53_common.c      | 58 ++++++++++++++++++++++++++---------
>  drivers/net/ethernet/broadcom/bgmac.c | 13 ++++++++
>  2 files changed, 56 insertions(+), 15 deletions(-)
> 


-- 
Florian