On Sat, 26 Sep 2020 12:27:00 +0530 Geetha sowjanya wrote: > From: Subbaraya Sundeep <sbha...@marvell.com> > > Packet replication feature present in Octeontx2 > is a hardware linked list of PF and its VF > interfaces so that broadcast packets are sent > to all interfaces present in the list. It is > driver job to add and delete a PF/VF interface > to/from the list when the interface is brought > up and down. This patch fixes the > npc_enadis_default_entries function to handle > broadcast replication properly if packet replication > feature is present. > > Fixes: 40df309e4166 > ("octeontx2-af: Support to enable/disable default MCAM entries") > > Signed-off-by: Subbaraya Sundeep <sbha...@marvell.com> > Signed-off-by: Geetha sowjanya <gak...@marvell.com> > Signed-off-by: Sunil Goutham <sgout...@marvell.com>
The patches look good, but could you please resend them with fixed "Fixes" tags? The tag should be on one line with no empty line between the tag and the signoffs. So the above should have looked like: Fixes: 40df309e4166 ("octeontx2-af: Support to enable/disable default MCAM entries") Signed-off-by: Subbaraya Sundeep <sbha...@marvell.com> Signed-off-by: Geetha sowjanya <gak...@marvell.com> Signed-off-by: Sunil Goutham <sgout...@marvell.com>