> >+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