Re: [PATCH net-next 4/4] net: dsa: mv88e6xxx: refactor FDB routines

[Andrew Lunn]

On Tue, Aug 04, 2015 at 02:33:38AM -0400, Vivien Didelot wrote:
> Refactor mv88e6xxx_port_fdb_{add,del,getnext} to respect the new DSA
> switch driver FDB access routines.
> 
> The Marvell 88E6xxx switches support up to 4094 FIDs (from 1 to 0xfff;
> FID 0 means that multiple address databases are not being used). So
> change the fid_mask for a fid_bitmap of 4096 bits.
> 
> FIDs 1 to num_ports will be reserved for non-bridged ports and bridge
> groups (a bridge group gets the FID of its first member). The remaining
> bits will then be used for VLANs.
> 
> Also do not consider an address (yet) if it is trunk mapped.
> 
> This change is a need to welcome the future support for hardware VLANs.

This patch seems to contain more than one logical set of changes. That
is making it hard to review. Can you split it up a bit.

The GLOBAL_ATU_DATA_TRUNK stuff can be in a patch of its own.
unsigned char *addr to u8 addr[ETH_ALEN].
The change fid_mask to fid_bitmap
...
...
The refactoring

Basically, we want to see lots of little patches which are obvious
correct, rather than one big patch where it is not obvious.

Thanks
	Andrew
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/