Date:   Mon, 20 Feb 2023 13:55:20 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Kurt Kanzenbach <kurt@linutronix.de>
Cc:     netdev@vger.kernel.org, "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Jamal Hadi Salim <jhs@mojatatu.com>,
        Cong Wang <xiyou.wangcong@gmail.com>,
        Jiri Pirko <jiri@resnulli.us>,
        Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        Gerhard Engleder <gerhard@engleder-embedded.com>,
        Amritha Nambiar <amritha.nambiar@intel.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        UNGLinuxDriver@microchip.com, Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Ferenc Fejes <ferenc.fejes@ericsson.com>,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        Roger Quadros <rogerq@kernel.org>,
        Pranavi Somisetty <pranavi.somisetty@amd.com>,
        Harini Katakam <harini.katakam@amd.com>,
        linux-kernel@vger.kernel.org,
        Giuseppe Cavallaro <peppe.cavallaro@st.com>,
        Alexandre Torgue <alexandre.torgue@foss.st.com>,
        Jose Abreu <joabreu@synopsys.com>,
        Wong Vee Khee <vee.khee.wong@linux.intel.com>,
        Michael Sit Wei Hong <michael.wei.hong.sit@intel.com>,
        Jacob Keller <jacob.e.keller@intel.com>
Subject: Re: [PATCH v2 net-next 04/12] net: ethtool: fix
 __ethtool_dev_mm_supported() implementation
Message-ID: <20230220115520.oidzuwzc6ifytd2s@skbuf>
References: <20230219135309.594188-1-vladimir.oltean@nxp.com>
 <20230219135309.594188-5-vladimir.oltean@nxp.com>
 <87sff0ftdg.fsf@kurt>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <87sff0ftdg.fsf@kurt>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?xEUE3sIzC2JTNH4TLhSITqR0EJoth9rvmGl0CoF7xW7kfe7GepCvijwlVNbg?=
 =?us-ascii?Q?MggagCySe//P0sa+V4nqJD3lNRVOenNMgOolfvKok1Zerm+yhpvIFz/asVqH?=
 =?us-ascii?Q?uZECgzRV9bn0KLH+z+TbsECQQfwGCd17NHBchCeMGfFkmhhNV7pNaGjBcKjd?=
 =?us-ascii?Q?UvNY2amD3pYzTPxGnm3Ml+ceHrJ9eZp0GTWe42e+7t1weWvAXVBzotwOEyrc?=
 =?us-ascii?Q?Ufo8HbD6GQ5My0m3zwcrzv20b2tQwy5gT+/7F21F0LSe9A4i5Xrbuh0kFtV3?=
 =?us-ascii?Q?eBUTTp+25KEQrozhI7bTWY20Vet1SLujdvBvWK06n4UqLgAYYwNNQJf/W1Me?=
 =?us-ascii?Q?9zWLTSC6Gqt6WH6TeUwYK0bFYeAbS2NIlYQwDQkotOYQMxkDNcSEuwQ9Qp6S?=
 =?us-ascii?Q?Ld4le/9f/caX6A4DTVhn/PZpZV2iWHfwgI7TU8YgmcO0OWLwea7sg6KDWBov?=
 =?us-ascii?Q?3Uz0Ew7meHtCpzYoU7wxHStxcTISfOtDXuYaICv/OxahFPH9F7e/h0Baf6PF?=
 =?us-ascii?Q?Tqltk1Ldncqfe6FjXpPuQOtg5zzPjxMgw4HKbyA1b8WaNi/+mlGaM5QuzAB1?=
 =?us-ascii?Q?H1xvjYHsWvbjIuR/UVzLLmCxsw4IFTMMxzFz0Cl8IET+47de3bYPiH94SfsF?=
 =?us-ascii?Q?mstAAiXF5AeKdS35YT6XsjhO+hwgyjMclZaBPxXfdNEkoMG/zhHQ2vJJAX6/?=
 =?us-ascii?Q?ypkFBy/R1OdVbbWicWidTEgly5Mpo4bMQs+z6OtciuD7c66U6w3fD/dZGntr?=
 =?us-ascii?Q?Gry5jYMVbb/OyUeSh87uFzj8uMduuEvf8dw/W2mnM5yqTrJbmzd1aK+yL4bc?=
 =?us-ascii?Q?ADLwNhi5f5oAuDy+zlt7ivtmTeFH2BST3eEZwzrMmJzZO1BH/Kw3b9cgyTZL?=
 =?us-ascii?Q?flpLPXKLVtgkHcrz3TJCpG1EHZdT4Ju7EwSsfiZDedhWhp3lJrwkHzKWuF71?=
 =?us-ascii?Q?tYONXwcSM5AldsURm5melDfQkABtaxJ+LjfGB1gQzcONabqYva2cw57KyAOF?=
 =?us-ascii?Q?lDnlFOW8rtKTKauFIJadxbmmr9JWN1GAkmiXr1h9Y9ttqEkS69+kRGMa4o5O?=
 =?us-ascii?Q?WtivvxD4+acUuI5hEPt8TmLvbe6hU/o9sA/bhct1OIZLR3xubwLHcNlQz982?=
 =?us-ascii?Q?q2YDnZ4RgjMyNL9dezkG4KAzPCDevR0ulodNSuzlvXlphHmzdheJ0QbzoCAS?=
 =?us-ascii?Q?HddWkzmYE/fi1T3WaA7GNKGGz9ozI9jRidrnds4tzQzDgj97HFM/RH69LMsm?=
 =?us-ascii?Q?0pqUC6JkQOOX6a8jGsmYhkWdfh8hO8qgBrzaSz/PdC1gyMQJZzPMxZXfwcte?=
 =?us-ascii?Q?cZWxpfv6lygHKhPouwEd6ETJdnlEoEAIF2FS7b4v4EP3PR1/A4PLTDvSZs1G?=
 =?us-ascii?Q?wKMOw676+4M2Dbz8rxt6VBKBvqci6Ajg8L4IkgtxtA+0W6ZOvr1OvhrZyiWx?=
 =?us-ascii?Q?K2wDK5BJcIyOVwV9hZxJcsTqkUdYT6UC8hXLnD4xDW+3CDP471fFLZIKu29R?=
 =?us-ascii?Q?9Wz63GKvU9K5T40lIcbIS+tBNR/EZ3j23Fq+xENyUvDBbdel6xDbmRbqNdks?=
 =?us-ascii?Q?0Aq8F1Zne6hm3HXmcWpqHPvbJgBRDDI5E7dTbB0CsfdJETWyfcZt/DAfEZZs?=
 =?us-ascii?Q?iQ=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 7aace47c-c376-4121-5eaa-08db13395a13
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 20 Feb 2023 11:55:26.9431
 (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: GVF/kTUoUJVbybD4F9o7IAMyt8H7SM3jw/k/mQGArA4aF1AUQ2aEf6vtkV2E0yIXeRDeViZBsJqb/wk/vmQZOw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB8909
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, Feb 20, 2023 at 12:50:35PM +0100, Kurt Kanzenbach wrote:
> On Sun Feb 19 2023, Vladimir Oltean wrote:
> > The MAC Merge layer is supported when ops->get_mm() returns 0.
> > The implementation was changed during review, and in this process, a bug
> > was introduced.
> >
> > Link: https://patchwork.kernel.org/project/netdevbpf/patch/20230111161706.1465242-5-vladimir.oltean@nxp.com/
> 
> Nit:
> 
> Link: https://lore.kernel.org/r/20230111161706.1465242-5-vladimir.oltean@nxp.com/
> 
> is preferred, because it is supposed to be stable. Same for patch #8.
> 
> Thanks,
> Kurt

Thanks for taking a look, I've made the change.
FWIW, your /r/ link redirected me to /all/. I've replaced it with /netdev/.