On Thu, Sep 21, 2023 at 03:32:00PM +0200, Wolfram Sang wrote: > Needed to enable this chip on a Renesas KingFisher board. What is needed? Please make the commit message self-contained. And what GNSS chip/module is this? This should also be included in the commit message. Do you have a link to a datasheet? None of the u-blox modules I've seen have a reset line so I'd like to where this came from and how it is intended to be used. > Description > copied over from the Mediatek driver which already supports it. The mediatek driver does not support managing a reset line, but the binding includes a description of this pin for completeness. Also you don't seem include any description of the property below (which is fine) so perhaps you can just drop this sentence. Johan