[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZlXhhPtE0EP8PE-L@pengutronix.de>
Date: Tue, 28 May 2024 15:52:04 +0200
From: Michael Tretter <m.tretter@...gutronix.de>
To: Ricardo Ribalda <ribalda@...omium.org>
Cc: Pengutronix Kernel Team <kernel@...gutronix.de>,
Mauro Carvalho Chehab <mchehab@...nel.org>,
Laurent Pinchart <laurent.pinchart@...asonboard.com>,
Michal Simek <michal.simek@....com>,
Andy Walls <awalls@...metrocast.net>,
Stanimir Varbanov <stanimir.k.varbanov@...il.com>,
Vikash Garodia <quic_vgarodia@...cinc.com>,
Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
Bjorn Andersson <andersson@...nel.org>,
Konrad Dybcio <konrad.dybcio@...aro.org>,
linux-media@...r.kernel.org, linux-kernel@...r.kernel.org,
linux-arm-kernel@...ts.infradead.org, linux-arm-msm@...r.kernel.org,
Hans Verkuil <hverkuil-cisco@...all.nl>
Subject: Re: [PATCH v3 01/18] media: allegro: nal-hevc: Replace array[1] with
arrray[N]
Typo in the subject: arrray -> array
On Mon, 27 May 2024 21:08:51 +0000, Ricardo Ribalda wrote:
> Structures that have a single element array as the last field can be
> mistaken as a "flex array".
>
> We could replace all the single element arrays in the structure with
> single element fields, but this driver prefers to follow the ITU-T H.265
> specification, which defines it as an array.
>
> If we introduce a new define N_HRD_PARAMETERS, we make clear our
> intentions.
N_HRD_PARAMETERS -> N_HRD_PARAMS
>
> This fixes this cocci warning:
> drivers/media/platform/allegro-dvt/nal-hevc.h:102:14-22: WARNING use flexible-array member instead (https://www.kernel.org/doc/html/latest/process/deprecated.html#zero-length-and-one-element-arrays)
>
> Signed-off-by: Ricardo Ribalda <ribalda@...omium.org>
With that fixed
Reviewed-by: Michael Tretter <m.tretter@...gutronix.de>
> ---
> drivers/media/platform/allegro-dvt/nal-hevc.h | 7 ++++---
> 1 file changed, 4 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/media/platform/allegro-dvt/nal-hevc.h b/drivers/media/platform/allegro-dvt/nal-hevc.h
> index eb46f12aae80..361e2f55c254 100644
> --- a/drivers/media/platform/allegro-dvt/nal-hevc.h
> +++ b/drivers/media/platform/allegro-dvt/nal-hevc.h
> @@ -96,10 +96,11 @@ struct nal_hevc_vps {
> unsigned int extension_data_flag;
> };
>
> +#define N_HRD_PARAMS 1
> struct nal_hevc_sub_layer_hrd_parameters {
> - unsigned int bit_rate_value_minus1[1];
> - unsigned int cpb_size_value_minus1[1];
> - unsigned int cbr_flag[1];
> + unsigned int bit_rate_value_minus1[N_HRD_PARAMS];
> + unsigned int cpb_size_value_minus1[N_HRD_PARAMS];
> + unsigned int cbr_flag[N_HRD_PARAMS];
> };
>
> struct nal_hevc_hrd_parameters {
>
> --
> 2.45.1.288.g0e0cd299f1-goog
>
>
Powered by blists - more mailing lists