> >+static void mv88e6xxx_phy_destroy(struct mv88e6xxx_chip *chip)
> >+{
> >+ if (mv88e6xxx_has(chip, MV88E6XXX_FLAG_PPU)) {
> >+ mv88e6xxx_ppu_state_destroy(chip);
> >+ }
>
> {} not needed here. See Documentation/CodingStyle, chapter 3.
Agreed, I will send a follow up patch, since the patchset has been
accepted.
Thanks
Andrew
