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;
Date:   Mon, 21 Nov 2022 18:46:48 +0200
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Colin Foster <colin.foster@in-advantage.com>
Cc:     linux-kernel@vger.kernel.org, netdev@vger.kernel.org,
        Paolo Abeni <pabeni@redhat.com>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>, UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <olteanv@gmail.com>
Subject: Re: [PATCH v2 net-next 1/3] net: mscc: ocelot: remove redundant
 stats_layout pointers
Message-ID: <20221121164632.zfbqka5i3haelmou@skbuf>
References: <20221119231406.3167852-1-colin.foster@in-advantage.com>
 <20221119231406.3167852-1-colin.foster@in-advantage.com>
 <20221119231406.3167852-2-colin.foster@in-advantage.com>
 <20221119231406.3167852-2-colin.foster@in-advantage.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20221119231406.3167852-2-colin.foster@in-advantage.com>
 <20221119231406.3167852-2-colin.foster@in-advantage.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?fBtNR1ARGhxglxuq+N2I7f1zYznvsVSnJXd5ho3SuqejwGLKoN3tjYM5TdTJ?=
 =?us-ascii?Q?L14/wkISRd2HXmodIGELQopiEGTLPrejDoL7esAHfGol176jjt8QfM7kstwJ?=
 =?us-ascii?Q?NvsPOi9IdAnd9oKmpd2iJbouaWOgmu1ehnTFM/Y9QgObTPK8fflORgHzsW3i?=
 =?us-ascii?Q?AABhrtQWvq84UK5M/hpSXvCsOvjVDlUB71WYHrFq+rQW2tla/Knfa/1X7GSm?=
 =?us-ascii?Q?cdlTE2xUxtyeDMvmz+DSBLmWs6Pu2vJXp6ZGaxIuMMITYwoV/Feq03Ym6RCb?=
 =?us-ascii?Q?hBCN+nHO06oJ6A5sdHSQA/b5Yzf/GS6xTiPOxJYj8JBnGR4BEu6rV81XAY2B?=
 =?us-ascii?Q?PErOmh32wTKiSdhEujSB1YrLlZ/pGe2E4UshXSeRKqUIvQHjXhYAFVp5mbfd?=
 =?us-ascii?Q?PlgF+7D94ltRw4pKNXy9SejDXD/7iQNlPI4RfVUk1Hxdy2udIJLmpoyAETou?=
 =?us-ascii?Q?nVx9MDfBfXjBFR+eV5MxN7G3PFwPm953ZY0je7dyJJp3NmlVQ+OcOdPr1pBf?=
 =?us-ascii?Q?9UYp0+5L9FkvdWnzFqaFm3CRnLYyicF6ZiWhvsiEzUQh5lE7K2yFfJjrdz6h?=
 =?us-ascii?Q?/I8nTuHNLvCk2tFIyyI49Zk4zEBwKfnGJW+ZR7g3J2Zoc5+CiU+m/4PwSN8V?=
 =?us-ascii?Q?fxalXpdK9s9O5vbpDKNBaQJ2OhDAnsT0FZmL+lBJ19XB4ZXVTAoj0Ja7hwXN?=
 =?us-ascii?Q?Y8UMIEZo6y7Jw1ncjxXdW+xYPlobi+a3MZrVnlPnThdF9pg+gItsakzmKR4X?=
 =?us-ascii?Q?DvZOhx7eMfPOVi0ynhBiyhYoEtCQVeBgGkbkzjrLROST+hiAAYcXpnfZ5wQf?=
 =?us-ascii?Q?upLbvK3RXJv4993Zp9eedPFVaVZWupjjphq1JoVe8of6A82zPQa5Si1WxirH?=
 =?us-ascii?Q?KZBSWJJWwi00ut72UsgDj1Dbpp+naOa3zwXwB5GiGmbCNW9BHVKZGJMmzR8+?=
 =?us-ascii?Q?9wa4ZYu7wYT5nT1mLYdRp3Zd8kNm0FWYUS6nVi3CX43FuJYiMraEZgqVixm9?=
 =?us-ascii?Q?tP+NYVH4oMa7EgDRuNfZVSEBIbpdOy/8Vqn/MVZdYVnj+C9Y144efOG0RcxQ?=
 =?us-ascii?Q?x5c7yg4ia6HJ16dll8UukVNsNTzU25W6a/gvhCuI+iou9Q1xVs6YjCVkVD/7?=
 =?us-ascii?Q?l9djW/e4xkubvsvu8WfdeKVje8jTWJPdPSHhvxRGawVGOyAofBM9yXZlo900?=
 =?us-ascii?Q?063oSuUq+4jZRRKmWDQHQrzUCNXkIf9mx7rhAIGNLt9nuQRNv4xbTDt6znt6?=
 =?us-ascii?Q?00XnUeTGU1s1FyBgJ/LdsEq0vmt5ArKrh3n4hzbCiRNzeNksgSMU1l2+G1Ph?=
 =?us-ascii?Q?iLl6EA86hoU5+KIKejLlSsl0q1Vwl4pL7oEKKOA3IBjBbRc5BmUurDFJMyn9?=
 =?us-ascii?Q?IOXYw7YZ0LaiG2SuGVnVwmjso7mR+W/hdKQdgfnDZLBT5g4uuNpRxRasJkUG?=
 =?us-ascii?Q?NP7NhLq0BILY5TLL1WpQngiGguj3Y1VFs1IoOh3GCjWt7zJC7Kcb/kV+N5bT?=
 =?us-ascii?Q?TnsHOSGCD1B7WKOSIXeLGLB6MhPwxaqtWvKkDK1189CQX7S8XAM+8ztmRkPA?=
 =?us-ascii?Q?wV12/opeeG4/wak9nR2QBA63EqpO1nTXDmDSD3XofYfVP0qPNvVExeUR4eBH?=
 =?us-ascii?Q?Ew=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: eb499f6e-b4a3-4437-3ea8-08dacbdffd69
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 Nov 2022 16:46:51.7587
 (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: 9vLTATdfuFYJYgkpFSGDi2MoljSP9L89NWqJvwCkT2N7Xrjl4PUgqV28CP+9MJ3SCzcCe3mkcSZ5fGIZMVdR5A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PR3PR04MB7356
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 Sat, Nov 19, 2022 at 03:14:04PM -0800, Colin Foster wrote:
> Ever since commit 4d1d157fb6a4 ("net: mscc: ocelot: share the common stat
> definitions between all drivers") the stats_layout entry in ocelot and
> felix drivers have become redundant. Remove the unnecessary code.
> 
> Suggested-by: Vladimir Oltean <olteanv@gmail.com>
> Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> ---
> 
> v1->v2
>     * Fix unused variable build warning in v1

Reviewed-by: Vladimir Oltean <vladimir.oltean@nxp.com>