Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Sean Anderson <sean.anderson@seco.com>
CC:     Vladimir Oltean <olteanv@gmail.com>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        Andrew Lunn <andrew@lunn.ch>,
        Heiner Kallweit <hkallweit1@gmail.com>,
        Russell King <linux@armlinux.org.uk>,
        Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        Alexandru Marginean <alexandru.marginean@nxp.com>,
        "David S . Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        Russell King <rmk+kernel@armlinux.org.uk>
Subject: Re: [PATCH v3 06/11] net: phylink: Add some helpers for working with
 mac caps
Thread-Topic: [PATCH v3 06/11] net: phylink: Add some helpers for working with
 mac caps
Thread-Index: AQHYoDyCh+mIA8W3VEWBqR7iDWfkxa2POWuAgAAAYoCAAAEbgA==
Date:   Mon, 25 Jul 2022 15:46:24 +0000
Message-ID: <20220725154623.ynt64sgphyhm3wgm@skbuf>
References: <20220725153730.2604096-1-sean.anderson@seco.com>
 <20220725153730.2604096-7-sean.anderson@seco.com>
 <20220725154103.e3l4cde3bhgdl65y@skbuf>
 <2c7b01e3-0236-3fae-7680-05a47b9c266a@seco.com>
In-Reply-To: <2c7b01e3-0236-3fae-7680-05a47b9c266a@seco.com>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?yNZmvjAlzG95PAXAPA/8/K1ZttcRBwBiHav8r6ojOiXXNd7WVsD5v65LX3jE?=
 =?us-ascii?Q?ljjYY+hdOHvu8teSFq92ocEnD98SYPL2kREj5lGnhWw06hUpKNpLGYzsxDno?=
 =?us-ascii?Q?4M4aCQ2/u7xT+W3HQdCKljG5JSbpXCn7OqrQayrZ5CSuUh+RaDt4EqNHfI5b?=
 =?us-ascii?Q?QQoNAfd6fWCHZXg7cJtoeNHuA1/J7R0J7j3DgIfwFsEYrEDgf/DPcTuxquyj?=
 =?us-ascii?Q?I71oXqUaEkVaaAGnr2DIgnSMwpVu9Aso3K2u3G7GgLomg4TdNMabIfJEa2Tw?=
 =?us-ascii?Q?HdrMdrfCF6zSVBbW6wh87acr5R2OsobOI9PsVtbIExmXBsm9IYmvNlo1Wd90?=
 =?us-ascii?Q?tQOhN7QC9mgFXsCmPnqhDklosS/XGw0s0HMe2dhDtGMSHx0YXnxrDFJUDpGt?=
 =?us-ascii?Q?tanVio/98BdbsEQ9tZQ71oXch+URsPxo6lnZA0X0JrZFpAaeIjjRtnHo1HFL?=
 =?us-ascii?Q?+XOEHl83eUhq+DYrC/2LUSU7R6mAp3sC3ka9OMlmhTtuiJjeLp9reCIzLwFv?=
 =?us-ascii?Q?P/Y8G9+kUVkoaVT44wCJkNYFb6O+y/wflffPzqGKiscHN6wE3r6TjjKEsQsN?=
 =?us-ascii?Q?s0HJuesBFvBwGsi4eJUqcVKiw2lTmQtLxAwCacL6UgGLDVy1SiQNr95v1cNH?=
 =?us-ascii?Q?8vE4L1oZVY1OdLgEpD+IEflc0jKvckAO49kJtUv8d2K1yjFoO6/hBp4Z076T?=
 =?us-ascii?Q?fzW8oWE/sF7dZcr4CrAjD4jko8hi5ABSmOgdbtcuNa5LF1lSgqZtVhtxq/yH?=
 =?us-ascii?Q?+gBV9RJLiIgHw4972PUnvdI3RUHPmkajMWdOtMTnYB+5hzX7z4zeWXsmWWjI?=
 =?us-ascii?Q?vkSAOtLE5bYLjA7yUqZRy6Mc7tyHi/5tXVb6fPRGXRxgDnuh33iZxM7Q7D/G?=
 =?us-ascii?Q?/JZX1zQdA7qckCBnZY437lXaZmd9vc1fEYYiAxMsNAGnH3yZ+uJNQN+E3dQ7?=
 =?us-ascii?Q?KhnHlAOH2OWBs/kB5H6T69eR/SmkdzRXUArAXRgDe2CbE8/7iaXnSHvkH+hK?=
 =?us-ascii?Q?CQuG0HpbtJv0waKq27Oi55F4njaGi681JwbKsnFfX6jZR7M8CHXJUFJHIHqq?=
 =?us-ascii?Q?nt9Xa6JSKcGzsf6reqI1lEpEXlNx5aDle37yRbSL2Kw5WCmMrjOh0f486nJt?=
 =?us-ascii?Q?JW9aTLMj4Sag0ZO7CyPFyaY66WLwQ69HCkqrBfQXXSfLa6/jbeDqoY7CRFVX?=
 =?us-ascii?Q?FvxDm0M8tlOffr6JeRPctR8oxyoLA9GINGwH8crT+tTErvZMgbpHlemB5wHR?=
 =?us-ascii?Q?8hjDxa4kbS6e2QDyoI9902aXTY+EOl1a6cfVyz9ACi3QlvTcPk9k/d1/WzhR?=
 =?us-ascii?Q?2Bwdhx0yHKodHR5jcZFAxNF8wH/Sn9FH5Uwo9BgEMARPRMPqAi7MZ9U+F89R?=
 =?us-ascii?Q?JF+TaChZjghc2u2+jxtEvGqLZwHWDOz3ZCyZbl1sl6DMKunkrUszMIctX56V?=
 =?us-ascii?Q?gW1sUQM4sB9H6fmM0PFKeBhEAkg+L/ayVGRG6GMy/l+UbEfwDPtXIu3/g6Lh?=
 =?us-ascii?Q?8dyTiQaC9kJ8Lhmw5DYc2zHOSckoDBLRhIUPA0WK0DQNaJr059UscjayYBrS?=
 =?us-ascii?Q?PJuLV7LvfBNI3eNLvcHGHHffMpVn1eKcAYpZodyKnTy7tS+2azd01k7rp9Rw?=
 =?us-ascii?Q?ig=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <59E186851D63C34C8543BEDDAEDF1065@eurprd04.prod.outlook.com>
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: b47af1b4-9df3-43eb-85c4-08da6e54d431
X-MS-Exchange-CrossTenant-originalarrivaltime: 25 Jul 2022 15:46:24.2357
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: 1keuD5IMc+W2sYZw5tPTGA1bK319Bqoek5z3w3ZSAQMbEvggpHAbuSBdfqir2rJFIk++WXi3XOE8nNVkCx6BvQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM6PR04MB6071
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Jul 25, 2022 at 11:42:25AM -0400, Sean Anderson wrote:
> Hi Vladmir,
>=20
> On 7/25/22 11:41 AM, Vladimir Oltean wrote:
> > On Mon, Jul 25, 2022 at 11:37:24AM -0400, Sean Anderson wrote:
> >> This adds a table for converting between speed/duplex and mac
> >> capabilities. It also adds a helper for getting the max speed/duplex
> >> from some caps. It is intended to be used by Russell King's DSA phylin=
k
> >> series. The table will be used directly later in this series.
> >>=20
> >> Co-developed-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> >> Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> >> Co-developed-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
> >> Signed-off-by: Russell King (Oracle) <rmk+kernel@armlinux.org.uk>
> >> [ adapted to live in phylink.c ]
> >> Signed-off-by: Sean Anderson <sean.anderson@seco.com>
> >> ---
> >> This is adapted from [1].
> >>=20
> >> [1] https://lore.kernel.org/netdev/E1oCNlE-006e3z-3T@rmk-PC.armlinux.o=
rg.uk/
> >=20
> > I did not write even one line of code from this patch, please drop my
> > name from the next revision when there will be one.
> >=20
>=20
> I merely retained your CDB/SoB from [1].

Yes, but context matters, the logic that you cropped out from that patch
was exactly my contribution to that change, the result no longer has anythi=
ng
to do with me. Maybe you didn't have any way to know this, but now you do.=