> static void
> @@ -574,9 +670,16 @@ static int mv88e6xxx_setup_devlink_regions_global(struct
> dsa_switch *ds,
> ops = mv88e6xxx_regions[i].ops;
> size = mv88e6xxx_regions[i].size;
>
> - if (i == MV88E6XXX_REGION_ATU)
> + switch (i) {
> + case MV88E6XXX_REGION_ATU:
> size = mv88e6xxx_num_databases(chip) *
> sizeof(struct mv88e6xxx_devlink_atu_entry);
> + break;
> + case MV88E6XXX_REGION_VTU:
> + size = chip->info->max_vid *
> + sizeof(struct mv88e6xxx_devlink_vtu_entry);
> + break;
> + }
Hi Tobias
Maybe as a follow up patch, add a helper to get
chip->info->max_vid. It seems like many of the other members have a
helper, so it would help keep things consistent.
Reviewed-by: Andrew Lunn <[email protected]>
Andrew
