From: Florian Fainelli <f.faine...@gmail.com> Date: Tue, 17 Jul 2018 08:36:39 -0700
> @@ -755,7 +755,8 @@ static int bcm_sf2_cfp_rule_set(struct dsa_switch *ds, > int port, > port_num = fs->ring_cookie / SF2_NUM_EGRESS_QUEUES; > > if (fs->ring_cookie == RX_CLS_FLOW_DISC || > - !dsa_is_user_port(ds, port_num) || > + !(dsa_is_user_port(ds, port_num) || > + dsa_is_cpu_port(ds, port_num)) || I think the second new line needs to be indented by two more spaces, but I could be wrong :-)