> -----Original Message-----
> From: Intel-wired-lan <[email protected]> On Behalf
> Of Mauro Carvalho Chehab
> Sent: Thursday, January 29, 2026 9:08 AM
> To: Jonathan Corbet <[email protected]>; Linux Doc Mailing List <linux-
> [email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>;
> [email protected]; [email protected]; linux-
> [email protected]; [email protected];
> [email protected]; Mauro Carvalho Chehab <[email protected]>;
> Randy Dunlap <[email protected]>
> Subject: [Intel-wired-lan] [PATCH v3 09/30] docs: kdoc_parser: add
> some debug for variable parsing
> 
> This is a new parser that we're still fine-tuning. Add some extra
> debug messages to help addressing issues over there.
> 
> Signed-off-by: Mauro Carvalho Chehab <[email protected]>
> Acked-by: Randy Dunlap <[email protected]>
> Tested-by: Randy Dunlap <[email protected]>
> ---
>  tools/lib/python/kdoc/kdoc_parser.py | 11 +++++++++++
>  1 file changed, 11 insertions(+)
> 
> diff --git a/tools/lib/python/kdoc/kdoc_parser.py
> b/tools/lib/python/kdoc/kdoc_parser.py
> index cbfdaba39494..ccee4e0bcaab 100644
> --- a/tools/lib/python/kdoc/kdoc_parser.py
> +++ b/tools/lib/python/kdoc/kdoc_parser.py
> @@ -1033,11 +1033,19 @@ class KernelDoc:
>                  declaration_name = r.group(1)
> 
>              default_val = r.group(2)
> +
> +            self.config.log.debug("Variable proto parser: %s from
> '%s'",
> +                                  r.groups(), proto)
> +
>          else:
>              r= KernRe(OPTIONAL_VAR_ATTR +
> r"(?:[\w_]*)?\s+(?:\*+)?(?:[\w_]+)\s*[\d\]\[]*\s*(=.*)?")
> 
>              if r.match(proto):
>                  default_val = r.group(1)
> +
> +        if default_val:
> +            self.config.log.debug("default: '%s'", default_val)
> +
>          if not declaration_name:
>             self.emit_msg(ln,f"{proto}: can't parse variable")
>             return
> @@ -1045,6 +1053,9 @@ class KernelDoc:
>          if default_val:
>              default_val = default_val.lstrip("=").strip()
> 
> +        self.config.log.debug("'%s' variable prototype: '%s',
> default: %s",
> +                              declaration_name, proto, default_val)
> +
>          self.output_declaration("var", declaration_name,
>                                  full_proto=full_proto,
>                                  default_val=default_val,
> --
> 2.52.0

Reviewed-by: Aleksandr Loktionov <[email protected]>

Reply via email to