Hi Sakari
Thanks a lot for the patch :)
On Sat, 27 Jan 2024 at 19:53, Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> wrote:
>
> In a rather unusual arrangement in include/media/v4l2-vp9.h struct
> v4l2_vp9_frame_symbol_counts has fields that are arrays of pointers, not a
> pointer to an array, which is what's usually done.
>
> Add support for such arrays of pointers to kernel-doc.
>
> Signed-off-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
Tested-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx>
> ---
> Hi Ricardo,
>
> I tried to ram it together with processing the rest but it seemed
> to complicate processing the usual fields.
>
> scripts/kernel-doc | 9 +++++++++
> 1 file changed, 9 insertions(+)
>
> diff --git a/scripts/kernel-doc b/scripts/kernel-doc
> index e8aefd258a29..728b1ffc0478 100755
> --- a/scripts/kernel-doc
> +++ b/scripts/kernel-doc
> @@ -1509,6 +1509,15 @@ sub create_parameterlist($$$$) {
> $type =~ s/([^\(]+\(\*?)\s*$param/$1/;
> save_struct_actual($param);
> push_parameter($param, $type, $arg, $file, $declaration_name);
> + } elsif ($arg =~ m/\(.+\)\s*\[/) {
> + # array-of-pointers
> + $arg =~ tr/#/,/;
> + $arg =~ m/[^\(]+\(\s*\*\s*([\w\[\]\.]*?)\s*(\s*\[\s*[\w]+\s*\]\s*)*\)/;
> + $param = $1;
> + $type = $arg;
> + $type =~ s/([^\(]+\(\*?)\s*$param/$1/;
> + save_struct_actual($param);
> + push_parameter($param, $type, $arg, $file, $declaration_name);
> } elsif ($arg) {
> $arg =~ s/\s*:\s*/:/g;
> $arg =~ s/\s*\[/\[/g;
> --
> 2.39.2
>
--
Ricardo Ribalda
