On 2025/1/21 2:19, Stephen Hemminger wrote: > On Mon, 20 Jan 2025 19:14:37 +0800 > "WanRenyong" <wa...@yunsilicon.com> wrote: > >> +static struct xsc_cmdq_req_msg * >> +xsc_cmdq_req_msg_alloc(struct xsc_dev *xdev, int len) >> +{ >> + struct xsc_cmdq_req_msg *msg; >> + struct xsc_cmdq_mbox *head = NULL; >> + int cmd_len, nb_mbox; >> + >> + msg = (struct xsc_cmdq_req_msg *)malloc(sizeof(*msg)); > Malloc returns void *, so cast here is unnecessary in C Thanks for your review.
I will address this issue and any similar issues elsewhere in the next version. -- Best regards, WanRenyong