> Extending the Felix driver to probe a PCI as well as a platform device > would have introduced unnecessary complexity. The 'meat' of both drivers > is in drivers/net/ethernet/mscc/ocelot*.c anyway, so let's just > duplicate the Felix driver, s/Felix/Seville/, and define the low-level > bits in seville_vsc9953.c.
Hi Vladimir That has resulted in a lot of duplicated code. Is there an overall family name for these switch? Could you add foo_set_ageing_time() with both felix and saville share? Andrew