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:   Colin Foster <colin.foster@in-advantage.com>
To:     linux-kernel@vger.kernel.org, netdev@vger.kernel.org
Cc:     aolo Abeni <pabeni@redhat.com>, Jakub Kicinski <kuba@kernel.org>,
        Eric Dumazet <edumazet@google.com>,
        "David S. Miller" <davem@davemloft.net>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Andrew Lunn <andrew@lunn.ch>, UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vladimir Oltean <vladimir.oltean@nxp.com>
Subject: [PATCH v1 net-next 0/1] net: ethernet: ocelot: remove num_stats initializer requirement
Date:   Fri, 29 Apr 2022 14:30:35 -0700
Message-Id: <20220429213036.3482333-1-colin.foster@in-advantage.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?nv9jI9vySsNzIxFaeNFALPwr0KUOMktG2xBO+QZHu+NoBs71V4uuP3Jr70Pt?=
 =?us-ascii?Q?Hk4GxojzasjibkiH5VL5EfPkO+E3mgsTqrLNWQp7tk9WU8YXzv1KI0SOr488?=
 =?us-ascii?Q?dknNpvhl934mqRfJ3Opeu2y2GEMyJabtnzLA6n2pfH5uc0PgXzn37tOWVC1L?=
 =?us-ascii?Q?EpI9EPKNsiizK+7WbFDcsr6UaMx+qLODYlkFMl58rAr9HzjghI6oL8t+P7SG?=
 =?us-ascii?Q?5igwtixEPeYdmP+CwqtgiqaWB7iQs/bx+wlLMlM4oWw9hwu7ytDNakX/hCHd?=
 =?us-ascii?Q?lXPsy0GF1E3Jwg41vXeg/FZVPwxXb4K7JS7XNvoZDPPhQmT7hTgi5L1veZao?=
 =?us-ascii?Q?1Y7DX5Y7Rb3ddtstKg+47LZUp4SqjFs1ANIFCst6Hs4xgWzbD4lxH4TPPyeG?=
 =?us-ascii?Q?u4ekV/8OilTU1PRrFpzz5lemWSN2e78A8ied4ew+JdPu82Vu4k+V+3NoY8Z5?=
 =?us-ascii?Q?a2oLtljjkKqaxBr6JvcyhRhYj9urIpTuu1YwFEU+7c1LtNBe/LeyVzj6o07k?=
 =?us-ascii?Q?6LGrq4LIrlFENd0REQ6vMj5gbo1MjK6qr6jPXAGlUcVXJ+Wxb38PLNeElUPl?=
 =?us-ascii?Q?PCPmUm/BTB3QAEH0Gq53En9j5xu8VkM7zwldYzHygI8HkjwdusjW7YxKEKC+?=
 =?us-ascii?Q?o0ZknJcxC9AS2Z7Oi89P08UDV/BbCiFhruVmiFr0kB5e9nv3Our5lsjUissk?=
 =?us-ascii?Q?zH3jBYpExyHT1SK8IcY0Y7cxodVYgurHYmDXfPmz7o+35emY61piT7wckAfj?=
 =?us-ascii?Q?p8q6XI0TRHkuoBSJNCAeKKVXaJP3YbTuIsD+K48namufPFGlukOoQ/pJwnvw?=
 =?us-ascii?Q?Bpmshnv1JKc85MAVW0OP/9Mj0ABUeBlpgU/MNr1JmDrbJlO4rDKQ5IeXGIFa?=
 =?us-ascii?Q?K+ZPFFE7Fm8OWtkSh9cauq9LAHnYWKZZqHjVf9SFaGIpvplhxGIG/iS9k3fI?=
 =?us-ascii?Q?MKmoIJ7FKyiMBRrLT5WYNqyGbuyPSJZ4iDN2o1UWxH2+uj5gxzZOIoHirw8t?=
 =?us-ascii?Q?1YuqFxzRjBASKUyAzYZ+ee9SNDo+yAbIakewx64KPFuzJpSav4oHQJ+7Al3V?=
 =?us-ascii?Q?LbuoBEXtZDi9QpfjHKGyQaO4T0hwdIOWtmJ7Hn610rtc3L7kTo5LnE2JwAAL?=
 =?us-ascii?Q?5Z19NlTOggR++BSpGWxH+0Y0t6JFMNDArk7C28MMrYWMqkzjPplWbiYq5C0j?=
 =?us-ascii?Q?RR2dP73Oi2WPjIMo0BBtFsdsn4SIpR850KhUAWjZqiDpd8vBZ/77WZ11ruwu?=
 =?us-ascii?Q?XnrdhLVtTjGqereE/3vS4wrZUQwHi7HScJ3jC/woDWnX2pQDnl6tefj6cN7Z?=
 =?us-ascii?Q?XjKRVw+fsJdDNBXXgRutesioYGP5+sKaT/NzOgHTduazXyCoZPn/Jkw+OUW6?=
 =?us-ascii?Q?M1A5wq9Gpg+sfAKUGP9xEWY9duDjlfdZr6581xqHlhBP8FkoulLoXvBw2ic1?=
 =?us-ascii?Q?OQc9iHZeVRHp6V+KBLdeabainhCFbnV4Hbo78+0Kd8+oZ3JY3wO1FIE/eeV4?=
 =?us-ascii?Q?OX0yk7JyynMghqNU/CMl1YaSQvr6CphAG19FRfaqZk3HmCMFO7PBQUU4+cLG?=
 =?us-ascii?Q?FanUPakuBfgukuQcc9kEF2a+dwHD7IZV0PBQYvHwUo+RM4kFYu1ZW38X+0Fu?=
 =?us-ascii?Q?+LIxIroofgak/TbcHBeqIdwiMjL3w+Yl2TZ5y0st9NBZsZfe2mv9a5ptJUwt?=
 =?us-ascii?Q?KQp6w8JnZqIPifFCtWbh2y6R+SBdkcIlchdnawwFXNIkzjbmnALxdltE4VxI?=
 =?us-ascii?Q?9WAdOIX8uupI/jYOGkE/omfVHWJBAYvXvwWQg5HF9hb4P8LO5pDP?=
X-OriginatorOrg: in-advantage.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 1f6c852c-9a99-4df3-d16a-08da2a2786a2
X-MS-Exchange-CrossTenant-AuthSource: MWHPR1001MB2351.namprd10.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 29 Apr 2022 21:30:48.1942
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 48e842ca-fbd8-4633-a79d-0c955a7d3aae
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: c553Msb5p38c5DIayaYTXKodQmH/MYuCV0jlTkMgsQTojBWGLsoJKM5pAKfFRTS5U08AVRX7al9ppLRA8uCQJczGOuetY2G1wk0i0n3fadk=
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY4PR10MB1957
X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED,
	DKIM_VALID,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

The ocelot_stats_layout structure array is common with other chips,
specifically the VSC7512. It can only be controlled externally (SPI,
PCIe...)

During the VSC7512 / Felix driver development, it was noticed that
this array can be shared with the Ocelot driver. As with other arrays
shared between the VSC7514 and VSC7512, it makes sense to define them in
drivers/net/ethernet/mscc/vsc7514_regs.c, while declaring them in
include/soc/mscc/vsc7514_regs.h

The thing that makes the stats_layout unique is that it is not accessed
indirectly by way of a shared enumeration index, but instead is looped
over. As such, the num_stats parameter has been used as the loop bounds.

Since the array size isn't necessarily fixed-length, the size has to be
determined. This was done locally in the C file with ARRAY_SIZE, but
that isn't possible if the array is declared via:
extern const struct ocelot_stat_layout ocelot_stats_layout[];

Instead, determine the size by the elemenets of the structure itself.
This way stats can be added / removed as needed (though they rarely
should) without the requirement of dragging a size variable around.

I don't have felix / seville / vsc7514 hardware to test, so I'd
appreciate if someone could give it a test. I've verified functionality
on my in-development kernel, but wouldn't mind feedback from the
existing users who have had the misfortune of having to find my mistakes
when it was too late.

Colin Foster (1):
  net: ethernet: ocelot: remove the need for num_stats initializer

 drivers/net/dsa/ocelot/felix.c             |  1 -
 drivers/net/dsa/ocelot/felix.h             |  1 -
 drivers/net/dsa/ocelot/felix_vsc9959.c     |  2 +-
 drivers/net/dsa/ocelot/seville_vsc9953.c   |  2 +-
 drivers/net/ethernet/mscc/ocelot.c         |  5 +++++
 drivers/net/ethernet/mscc/ocelot_vsc7514.c |  2 +-
 include/soc/mscc/ocelot.h                  | 10 ++++++++++
 7 files changed, 18 insertions(+), 5 deletions(-)

-- 
2.25.1