netdev - Re: [PATCH] Revert "net/ibm/emac: wrong bit is used for STA control"

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: <20181207.223740.800602386798041681.davem@davemloft.net>
Date:   Fri, 07 Dec 2018 22:37:40 -0800 (PST)
From:   David Miller <davem@...emloft.net>
To:     benh@...nel.crashing.org
Cc:     netdev@...r.kernel.org, michaele@....ibm.com, fr0st61te@...il.com
Subject: Re: [PATCH] Revert "net/ibm/emac: wrong bit is used for STA
 control"

From: Benjamin Herrenschmidt <benh@...nel.crashing.org>
Date: Fri, 07 Dec 2018 15:05:04 +1100

> This reverts commit 624ca9c33c8a853a4a589836e310d776620f4ab9.
> 
> This commit is completely bogus. The STACR register has two formats, old
> and new, depending on the version of the IP block used. There's a pair of
> device-tree properties that can be used to specify the format used:
> 
> 	has-inverted-stacr-oc
> 	has-new-stacr-staopc
> 
> What this commit did was to change the bit definition used with the old
> parts to match the new parts. This of course breaks the driver on all
> the old ones.
> 
> Instead, the author should have set the appropriate properties in the
> device-tree for the variant used on his board.
> 
> Signed-off-by: Benjamin Herrenschmidt <benh@...nel.crashing.org>
> ---
> 
> Found while setting up some old ppc440 boxes for test/CI

Applied, thanks.