Re: [PATCH v3 5/5] net: asix: autoneg will set WRITE_MEDIUM reg

On Mon, 2016-08-29 at 09:32 -0400, robert.foss@collabora.com wrote:

From: Robert Foss <redacted>

From: Grant Grundler <redacted>

The miii_nway_restart() causes a PHY link change activity and
ax88772_link_reset will be called. link_reset will set
AX_CMD_WRITE_MEDIUM_MODE register correctly.

The asix_write_medium_mode in reset() fills in a default value to the register
which may be different from the negotiation result. So do this first.

Ignore the ret value since it's ignored in XXX_link_reset() functions.

Signed-off-by: Grant Grundler <redacted>
Signed-off-by: Robert Foss <redacted>
Tested-by: Robert Foss <redacted>
---

This is _really_ confusing Robert.

Why having two 'From:' clauses ?

Who wrote the patch in the first place ? You or Grant ?



End result is :

commit 535baf8588d04b177cb33700f81499f2b5203c2d
Author: Robert Foss [off-list ref]
Date:   Mon Aug 29 09:32:19 2016 -0400

    net: asix: autoneg will set WRITE_MEDIUM reg
    
    From: Grant Grundler [off-list ref]
    
    The miii_nway_restart() causes a PHY link change activity and
    ax88772_link_reset will be called. link_reset will set
    AX_CMD_WRITE_MEDIUM_MODE register correctly.
    
    The asix_write_medium_mode in reset() fills in a default value to the register
    which may be different from the negotiation result. So do this first.
    
    Ignore the ret value since it's ignored in XXX_link_reset() functions.
    
    Signed-off-by: Grant Grundler [off-list ref]
    Signed-off-by: Robert Foss [off-list ref]
    Tested-by: Robert Foss [off-list ref]
    Signed-off-by: David S. Miller [off-list ref]



I guess Grant wrote the patch, but attribution is wrong.