netdev - Re: [PATCH net 1/2] Revert "net: bcmgenet: use RGMII loopback for MAC reset"

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]

Message-ID: <4f4a7695-0c6c-c5e8-84b2-602d0ee4fd29@gmail.com>
Date:   Tue, 17 Mar 2020 12:43:09 -0700
From:   Doug Berger <opendmb@...il.com>
To:     Florian Fainelli <florian.fainelli@...adcom.com>,
        "David S. Miller" <davem@...emloft.net>
Cc:     Florian Fainelli <f.fainelli@...il.com>,
        bcm-kernel-feedback-list@...adcom.com, netdev@...r.kernel.org,
        linux-kernel@...r.kernel.org
Subject: Re: [PATCH net 1/2] Revert "net: bcmgenet: use RGMII loopback for MAC
 reset"

On 3/16/2020 7:21 PM, Florian Fainelli wrote:
> 
> 
> On 3/16/2020 2:44 PM, Doug Berger wrote:
>> This reverts commit 3a55402c93877d291b0a612d25edb03d1b4b93ac.
>>
>> This is not a good solution when connecting to an external switch
>> that may not support the isolation of the TXC signal resulting in
>> output driver contention on the pin.
>>
>> A different solution is necessary.
>>
>> Signed-off-by: Doug Berger <opendmb@...il.com>
> 
> Acked-by: Florian Fainelli <f.fainelli@...il.com>
> 
> Did you want this to be tagged with:
> 
> Fixes: 3a55402c9387 ("net: bcmgenet: use RGMII loopback for MAC reset")
> 
> so as to make it more explicit how the two commits relate to each other?
I wasn't sure how best to tag this commit.

It seems odd to indicate that the reversion of a commit fixes the commit
that is reverted, but maybe that is the best way. It is more accurate to
say that the reversion of the commit reintroduces the problem that it
was intended to address, and therefore doesn't fix anything but rather
trades one problem for another.

It is clearer that the second commit fixes the original issue and so I
tagged it accordingly, but it is far less clear how a reversion like
this should be tagged.

If you think it is clearer to tag this as you describe I have no objection.

> 
> Thanks!
> --
> Florian
> 
Thank you for taking a look at this and for the feedback,
    Doug