iv/linux
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

net: mscc: ocelot: strengthen type of "u32 reg" and "u32 base" in ocelot_stats.c

Browse Source 
Use the specific enum ocelot_reg to make it clear that the region
registers are encoded and not plain addresses.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
Reviewed-by: Jacob Keller <jacob.e.keller@intel.com>
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Vladimir Oltean 2023-04-12 15:47:35 +03:00 committed by Jakub Kicinski
parent a9afc3e41c
commit eae0b9d15b
1 changed files with 3 additions and 3 deletions

6
drivers/net/ethernet/mscc/ocelot_stats.c

@ -145,7 +145,7 @@ enum ocelot_stat {
}; };
struct ocelot_stat_layout { struct ocelot_stat_layout {
u32 reg; enum ocelot_reg reg;
char name[ETH_GSTRING_LEN]; char name[ETH_GSTRING_LEN];
}; };
@ -257,7 +257,7 @@ struct ocelot_stat_layout {
struct ocelot_stats_region { struct ocelot_stats_region {
struct list_head node; struct list_head node;
u32 base; enum ocelot_reg base;
enum ocelot_stat first_stat; enum ocelot_stat first_stat;
int count; int count;
u32 *buf; u32 *buf;
@ -889,7 +889,7 @@ static int ocelot_prepare_stats_regions(struct ocelot *ocelot)
{ {
struct ocelot_stats_region *region = NULL; struct ocelot_stats_region *region = NULL;
const struct ocelot_stat_layout *layout; const struct ocelot_stat_layout *layout;
unsigned int last = 0; enum ocelot_reg last = 0;
int i; int i;
INIT_LIST_HEAD(&ocelot->stats_regions); INIT_LIST_HEAD(&ocelot->stats_regions);