Re: [PATCH v3] dt-bindings: gnss: add u-blox,neo-9m compatible

[Rob Herring <robh@kernel.org>]

On Tue, May 27, 2025 at 01:14:54PM +0200, Krzysztof Kozlowski wrote:
> On 27/05/2025 13:02, Johan Hovold wrote:
> > On Tue, May 27, 2025 at 12:51:12PM +0200, Krzysztof Kozlowski wrote:
> >> On 27/05/2025 12:44, Johan Hovold wrote:
> >>> On Tue, May 27, 2025 at 10:35:14AM +0200, Krzysztof Kozlowski wrote:
> >>>> On 23/05/2025 13:52, Krzysztof Kozlowski wrote:
> >>>>> On 23/05/2025 13:19, Alejandro Enrique via B4 Relay wrote:
> >>>>>> From: Alejandro Enrique <alejandroe1@geotab.com>
> >>>>>>
> >>>>>> Add compatible for u-blox NEO-9M GPS module.
> >>>>>>
> >>>>>> Signed-off-by: Alejandro Enrique <alejandroe1@geotab.com>
> >>>>>> ---
> >>>>>> This series just add the compatible string for u-blox NEO-9M module,
> >>>>>> using neo-m8 as fallback. I have tested the driver with such a module
> >>>>>> and it is working fine.
> >>>>>> ---
> >>>>>
> >>>>> I assume there is a user somewhere?
> >>>>>
> >>>>> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> >>>> Un-reviewed. Please drop the patch. It turns out there is no user for
> >>>> this binding. We don't take bindings for every possible device out there
> >>>> - you need users of that binding.
> >>>
> >>> No, we don't require manufacturers to upstream their machine dts.
> >>
> >> No, we don't take bindings for whatever is there. In any case, drop my
> >> review tag.
> > 
> > Perhaps not for whatever, but here we have an actual user that needs
> > this binding do I'll take it.
> 
> Great, I understand above that I can send you bindings for multiple
> devices I have (or had and still have interest in or my previous
> employer has interest in), which are used in downstream products, and
> you will take these bindings?
> 
> That would be cool, because I have bunch of GNSS devices related to my
> pre-previous job, which I would really like to upstream.
> 
> Is my understanding correct?

What's the issue here? We have a driver and that's the user. The 
requirement is a driver OR .dts for the user. Are we now discouraging 
having a specific compatible for a new device that's backwards 
compatible with an existing device? No!

Rob