On Mon, 9 Nov 2020 18:44:15 +0100 Andrew Lunn wrote:
> > 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;
> > + }
>
[...]
> Reviewed-by: Andrew Lunn <and...@lunn.ch>
Applied, thanks!
