Re: [PATCH v1 7/7] ARM: dts: imx: Update i.MXRT1050.dtsi compatibles

["Arnd Bergmann" <arnd@xxxxxxxx>]

On Mon, Nov 7, 2022, at 16:09, Jesse Taube wrote:
> On 11/7/22 02:44, Arnd Bergmann wrote:
>> On Mon, Nov 7, 2022, at 08:15, Jesse Taube wrote:
>>> Remove unused compatibles from i.MXRT1050.dtsi.
>>> Change GPT clock-names to match documentation.
>>>
>>> Cc: Giulio Benetti <giulio.benetti@xxxxxxxxxxxxxxxxxxxxxx>
>>> Signed-off-by: Jesse Taube <Mr.Bossman075@xxxxxxxxx>
>> 
>> Can you make sure your changelog texts explain why you do this?
> Yes, sorry I wasn't clear.
>
>> Are they fundamentally different from the devices you had
>> claimed to be compatible with that need a different driver,
>
> UART and SDHC had drivers added which are better fit.
> The GPT binds to imx6dl which is also the same as imx6sl.

Where are those drivers added? Looking at linux-6.1-rc2
and linux-next, I still see them use the same drivers as
the original ones, and listing both strings would be the
preferred method.

>> or are there drivers in the field that bind to the wrong
>> string first?
> I don't understand?

I mean if you had run into the case where you have
a driver that misbehaves when the fallback string is
present in addition to the most specific one.

     Arnd