Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:18 as permitted sender) client-ip=2620:137:e000::1:18;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     Colin Foster <colin.foster@in-advantage.com>
CC:     "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "David S. Miller" <davem@davemloft.net>,
        "UNGLinuxDriver@microchip.com" <UNGLinuxDriver@microchip.com>,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>
Subject: Re: [PATCH v1 net-next 1/2] net: mscc: ocelot: remove unnecessary
 variable
Thread-Topic: [PATCH v1 net-next 1/2] net: mscc: ocelot: remove unnecessary
 variable
Thread-Index: AQHYXOlXKyVUt7/ZYUirbL8KuFWiB60J3cKA
Date:   Sun, 1 May 2022 11:08:32 +0000
Message-ID: <20220501110831.klfcbqiapumdndyz@skbuf>
References: <20220430232327.4091825-1-colin.foster@in-advantage.com>
 <20220430232327.4091825-2-colin.foster@in-advantage.com>
In-Reply-To: <20220430232327.4091825-2-colin.foster@in-advantage.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?7bOMVmEgNRID2XOmUskP72cYSLS7BsqYGhGKSLnX9hw20dCDrY+5MpCvVy0K?=
 =?us-ascii?Q?+UH/VhEv+ce6R87r4OXq64Q7h3xJwAKQf+oQ+hd2ViMaqbUB5fC/8VZBAJpx?=
 =?us-ascii?Q?w1XePENic1nrtQCF/KYzRxsCZaRCCRasy9pL2420FEroGIahLyqTU0VPBeqG?=
 =?us-ascii?Q?nFV9K/K97dHGMtlhLZwDw4Fr/CZSc1VKHhLrp514Rw9+2kgl+MU3c+nsBhC6?=
 =?us-ascii?Q?l/9mx6F89PRmBkklovVd33UuqANmMqrJgv6jAkEqQb3A8dKkZ9JV9fXycAdc?=
 =?us-ascii?Q?marTAQnTbKwsakY7jyLV3k+vjJ4ypfjXjXvfaz55zFeh3mZmXbRGeV73OVSg?=
 =?us-ascii?Q?Hu/VkmHHMDCVgrKiLks/JosSZGAHDCNG+rL/5gD5hPBbNfrj6JOGTyjPGML/?=
 =?us-ascii?Q?u8eEdM/y/rE8pwRzUxKJahu/eKAPY7Xe5SMRtvFss4ndn+F1h0aZYNFDhJ+A?=
 =?us-ascii?Q?hhi0Uom+0TG593GpARVB8RfDc1RIw8L1MT7r/fomOC3ULdNoLyUtAcSNCF7G?=
 =?us-ascii?Q?Y8m6yeU/FT94CpaNUwP8WLNYdPvdJIE0qV4LlL3nnMLJBcuydL8o6PL8+OJZ?=
 =?us-ascii?Q?aeStj9albwB53sBl45KkDcZggrGrDd7sLotUTRr8EfMeDobTaMd7TFDOEgwh?=
 =?us-ascii?Q?2GuqhXDxs3PLstJwGfKObRrsGPt/xAuP1AXFvTp/OLaIhqNF+bx5tqG5tjqH?=
 =?us-ascii?Q?NX4IZYqXuJt5OdSay3rv67hQqkXpWHiEpG+NMeqTL79xXGxbcPUOjYGI1Hyb?=
 =?us-ascii?Q?BXHq/cdpjxvrsftNu9+GTs7S1SV7Cbgn04NK6f7VDN25VHC/VgsbZpmROoQ8?=
 =?us-ascii?Q?xeUgkvMwQo9yiOUGJcOrVD9G4JlXh1zcfcxSmVMYJb7kdIJ+xr1HIxW8MPBB?=
 =?us-ascii?Q?G9Y9hm+GYHZ7rartbbjxyqFYAn3vWhyknQftq4dXJNbQ65h6OnQNEIO2efG1?=
 =?us-ascii?Q?BPsiEbVmoDXsBW/MZRrhjvdV+OpnW2Bf6CaFTHaiv1geJya9+hd5j8FM0Gti?=
 =?us-ascii?Q?YP+GUp/DM8Ykw9IzCpxxXjZwdTpYmcA1Yk/9aOasiusGVFEi1UOY8eX4VaBd?=
 =?us-ascii?Q?GCewO+vI0E7vo1ulI/pI18fbGgJ84Dzv28dEmojC/AzF78ySXxGQqeWls5j4?=
 =?us-ascii?Q?zj/txWSgptTMh9cQCfyZ3/0VFqT0VmzeeZBGzdiD0sZq62n70v8BidEW74sU?=
 =?us-ascii?Q?H03NJ68yVAO3+grfUoDrZqEyoTTq/8oT7/f+V4I+/OeljFpeWocPtX6T9E4r?=
 =?us-ascii?Q?I4711hCQ88A8URzJn4ADrzCnv7/EVHtcjx+6ji3DusQhfgYXDTezXJGaqH6P?=
 =?us-ascii?Q?X4J1SwVSzyxv+yk8qbOHqhh0MbgPlqp+Lm0/pIyCHN+ud4CMkTSR2TjNOOY6?=
 =?us-ascii?Q?t4XdhU6ABLq+u9Hx4FMIylYGfY+AusJhZa/I1+hpdtP965j1PTAfLbA3XaKv?=
 =?us-ascii?Q?L5Va1e/zJ66prfDfugzEziP87aYlaBx7DsiGcyuAFo9HrI3kdFrd3JIrsj8L?=
 =?us-ascii?Q?P0o5o14ut2wjLloCOYi8dsbPVzIjwbjPjrFrBtDC4kvhx59C94Nt3YoYj7s2?=
 =?us-ascii?Q?Grbrh92yclXp/U8caqhty2n+JewC6dMxU1KqsYbknREXXdFB1Kv57pik80XR?=
 =?us-ascii?Q?O2x7PVSDNZiJpVG3XADMeZImowjJETy8V1m0M3/OwoUNc8TS4brPhMM1JcEw?=
 =?us-ascii?Q?T0+0nNduiRwPlcFMl9o8HkUdVdz2TcFO2hWSreom3ABpFHzUxtls2rJ05mpB?=
 =?us-ascii?Q?Z8tvCqK+G4wZUk2n2ny+BKcuwpCKcUQ=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-ID: <B6464C7F17DAB14FBA427848C19521F1@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: aa4c944a-6e8e-4958-aaca-08da2b62edbd
X-MS-Exchange-CrossTenant-originalarrivaltime: 01 May 2022 11:08:32.1609
 (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: E4ERtltoKlqPm/jxWE4wOA+uqhje95awu+a7YfKpunTlhY1VP1jtMHADulQ2w3uiUMXVrNYgNhK1p9z0fm4ynw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB6862
X-Spam-Status: No, score=-2.0 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,
	MAILING_LIST_MULTI,RDNS_NONE,SPF_HELO_NONE,T_SCC_BODY_TEXT_LINE
	autolearn=no 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, Apr 30, 2022 at 04:23:26PM -0700, Colin Foster wrote:
> Commit 2f187bfa6f35 ("net: ethernet: ocelot: remove the need for num_stat=
s
> initializer") added a flags field to the ocelot stats structure. The same
> behavior can be achieved without this additional field taking up extra
> memory.
>=20
> Remove this structure element to free up RAM
>=20
> Suggested-by: Vladimir Oltean <vladimir.oltean@nxp.com>
> Signed-off-by: Colin Foster <colin.foster@in-advantage.com>
> ---

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