Messages in this thread | | | Date | Mon, 10 Jul 2023 12:40:28 +0530 | Subject | Re: [PATCH][next] media: venus: Use struct_size_t() helper in pkt_session_unset_buffers() | From | Vikash Garodia <> |
| |
On 7/1/2023 10:47 PM, Gustavo A. R. Silva wrote: > Prefer struct_size_t() over struct_size() when no pointer instance > of the structure type is present. > > Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: Vikash Garodia <quic_vgarodia@quicinc.com>
> --- > drivers/media/platform/qcom/venus/hfi_cmds.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/drivers/media/platform/qcom/venus/hfi_cmds.c b/drivers/media/platform/qcom/venus/hfi_cmds.c > index 7f0802a5518c..3418d2dd9371 100644 > --- a/drivers/media/platform/qcom/venus/hfi_cmds.c > +++ b/drivers/media/platform/qcom/venus/hfi_cmds.c > @@ -251,8 +251,8 @@ int pkt_session_unset_buffers(struct hfi_session_release_buffer_pkt *pkt, > > pkt->extradata_size = 0; > pkt->shdr.hdr.size = > - struct_size((struct hfi_session_set_buffers_pkt *)0, > - buffer_info, bd->num_buffers); > + struct_size_t(struct hfi_session_set_buffers_pkt, > + buffer_info, bd->num_buffers); > } > > pkt->response_req = bd->response_required;
| |