> >+static int comm_msg_to_mgmt_sync(struct hinic3_hwdev *hwdev, u16 cmd, > >const void *buf_in, > >+ u32 in_size, void *buf_out, u32 *out_size) > >+{ > >+ return hinic3_send_mbox_to_mgmt(hwdev, HINIC3_MOD_COMM, cmd, > >buf_in, > >+ in_size, buf_out, out_size, 0); > >+} > [Suman] Any reason we need this wrapper? We can directly call hinic3_send_m= > box_to_mgmt() from hinic3_func_reset() This wrapper is used frequently (10 times or more) and it reduces number of parameters from 8 to 6. > >+