On Thu, Jan 19, 2017 at 05:26:03PM -0500, Vivien Didelot wrote:
> Gregory CLEMENT <gregory.clem...@free-electrons.com> writes:
>
> > +static bool mv88e6xxx_6341_family(struct mv88e6xxx_chip *chip)
> > +{
> > + return chip->info->family == MV88E6XXX_FAMILY_6341;
> > +}
>
> I don't want to see these erronous family checks anymore, but I cannot
> blame you for adding it since not all the code is moved to ops yet ;)
>
> > MV88E6XXX_FAMILY_6165, /* 6123 6161 6165 */
> > MV88E6XXX_FAMILY_6185, /* 6108 6121 6122 6131 6152 6155 6182 6185 */
> > MV88E6XXX_FAMILY_6320, /* 6320 6321 */
> > + MV88E6XXX_FAMILY_6341, /* 6141 6341 */
>
> Maybe I missed it, Andrew, can you confirm that 6341 is a proper Marvell
> family of switch chips?
My understand is that it is. Marvell have not added it to DSDT. There
is a new SDK called UMSD, also GPLv2 and BSD. They call this family
Topaz, and the 6390 is Peridot.
Andrew
