Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> writes: > 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> > Acked-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > Tested-by: Randy Dunlap <rdunlap@xxxxxxxxxxxxx> > Tested-by: Ricardo Ribalda <ribalda@xxxxxxxxxxxx> > --- > No change since the RFC, just added the acks. > > scripts/kernel-doc | 9 +++++++++ > 1 file changed, 9 insertions(+) > > diff --git a/scripts/kernel-doc b/scripts/kernel-doc > index e8aefd258a29..23c91b11585a 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) { Sigh ... seeing more indecipherable regexes added to kernel-doc is like seeing another load of plastic bags dumped into the ocean... it doesn't change the basic situation, but it's still sad. Oh well, applied, thanks. jon