Hi,

This is fixing a bug introduced in b596229448dd2

Acked-by: Allan W. Nielsen <allan.niel...@microchip.com>

/Allan


The 08/20/2019 12:20, Yangbo Lu wrote:
> External E-Mail
> 
> 
> The trap action should be copying the frame to CPU and
> dropping it for forwarding, but current setting was just
> copying frame to CPU.
> 
> Signed-off-by: Yangbo Lu <yangbo...@nxp.com>
> ---
>  drivers/net/ethernet/mscc/ocelot_ace.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/mscc/ocelot_ace.c 
> b/drivers/net/ethernet/mscc/ocelot_ace.c
> index 39aca1a..86fc6e6 100644
> --- a/drivers/net/ethernet/mscc/ocelot_ace.c
> +++ b/drivers/net/ethernet/mscc/ocelot_ace.c
> @@ -317,7 +317,7 @@ static void is2_action_set(struct vcap_data *data,
>               break;
>       case OCELOT_ACL_ACTION_TRAP:
>               VCAP_ACT_SET(PORT_MASK, 0x0);
> -             VCAP_ACT_SET(MASK_MODE, 0x0);
> +             VCAP_ACT_SET(MASK_MODE, 0x1);
>               VCAP_ACT_SET(POLICE_ENA, 0x0);
>               VCAP_ACT_SET(POLICE_IDX, 0x0);
>               VCAP_ACT_SET(CPU_QU_NUM, 0x0);
> -- 
> 2.7.4
> 
> 

-- 
/Allan

Reply via email to