Hi Wolfram, On Wed, Jul 12, 2023 at 1:40 PM Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> wrote: > Needed to enable this chip on a Renesas KingFisher board. Description > copied over from the Mediatek driver which already supports it. > > Signed-off-by: Wolfram Sang <wsa+renesas@xxxxxxxxxxxxxxxxxxxx> Thanks for your patch! > --- a/Documentation/devicetree/bindings/gnss/u-blox,neo-6m.yaml > +++ b/Documentation/devicetree/bindings/gnss/u-blox,neo-6m.yaml > @@ -41,6 +41,12 @@ properties: > description: > > Backup voltage regulator > > + reset-gpios: > + maxItems: 1 > + description: > > + An optional reset line, with names such as RESET or NRESET. > + If the line is active low it should be flagged with GPIO_ACTIVE_LOW. I think you can drop the description, as it describes only the obvious. Gr{oetje,eeting}s, Geert -- Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@xxxxxxxxxxxxxx In personal conversations with technical people, I call myself a hacker. But when I'm talking to journalists I just say "programmer" or something like that. -- Linus Torvalds