> >+static void airoha_set_port_fwd_cfg(struct airoha_eth *eth, u32 addr, u32 val) > >+{ > >+ airoha_fe_rmw(eth, addr, GDM1_OCFQ_MASK, > >+ FIELD_PREP(GDM1_OCFQ_MASK, val)); > > Not sure this is problem of my email client, but none of the functions args are aligning with above parenthesis > airoha_fe_rmw(eth, addr, GDM1_OCFQ_MASK, > FIELD_PREP(GDM1_OCFQ_MASK, val)); They look good to me. But they can I see from your mail headers you are using Exchange and other M$ things. Outlook is well known to corrupt emails, destroying white space, wrapping lines etc. Your reply appears to of had some tabs replaced by spaces, etc. Outlook is unusable for software engineers handling patches. Andrew