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


Reply via email to