On 3/4/25 8:21 AM, Wei Fang wrote: > +static void enetc_mac_list_del_matched_entries(struct enetc_pf *pf, u16 > si_bit, > + struct enetc_mac_addr *mac, > + int mac_cnt) > +{ > + struct enetc_mac_list_entry *entry; > + int i; > + > + for (i = 0; i < mac_cnt; i++) { > + entry = enetc_mac_list_lookup_entry(pf, mac[i].addr); > + if (entry) { > + entry->si_bitmap &= ~si_bit; > + if (!entry->si_bitmap) {
Minor nit: here and elsewhere you could reduce the level of indentation restructoring the code as: if (!entry) continue; entry->si_bitmap &= ~si_bit; if (entry->si_bitmap) continue; /P