diff mbox series

[4/4] dt-bindings: serial: fsl-lpuart: add imx8ulp compatible string

Message ID 20230625094857.29644-5-sherry.sun@nxp.com
State New
Headers show
Series correct the lpuart compatible for imx8dxl/imx8ulp/imx93 | expand

Commit Message

Sherry Sun June 25, 2023, 9:48 a.m. UTC
LPUART driver adds a new compatible string for imx8ulp, and imx93 is
much more compatible with imx8ulp than imx7ulp, so correct the
dt-binding here.

Signed-off-by: Sherry Sun <sherry.sun@nxp.com>
---
 Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

Comments

Krzysztof Kozlowski June 25, 2023, 10:17 a.m. UTC | #1
On 25/06/2023 11:48, Sherry Sun wrote:
> LPUART driver adds a new compatible string for imx8ulp, and imx93 is

What driver adds or not, is not really correct argument for
compatibility. There are plenty of compatible devices which have both
device ID entries in the driver. Why would you drop their compatibility
between each other? It does not work like that... Provide clear
rationale for this.

> much more compatible with imx8ulp than imx7ulp, so correct the
> dt-binding here.
> 

"Much more compatible" is too vague.


Best regards,
Krzysztof
Sherry Sun June 25, 2023, 1:12 p.m. UTC | #2
> -----Original Message-----
> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> Sent: 2023年6月25日 18:18
> To: Sherry Sun <sherry.sun@nxp.com>; robh+dt@kernel.org;
> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org;
> shawnguo@kernel.org; s.hauer@pengutronix.de; festevam@gmail.com;
> Shenwei Wang <shenwei.wang@nxp.com>; gregkh@linuxfoundation.org
> Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
> kernel@pengutronix.de; linux-serial@vger.kernel.org; linux-
> kernel@vger.kernel.org; dl-linux-imx <linux-imx@nxp.com>
> Subject: Re: [PATCH 4/4] dt-bindings: serial: fsl-lpuart: add imx8ulp
> compatible string
> 
> On 25/06/2023 11:48, Sherry Sun wrote:
> > LPUART driver adds a new compatible string for imx8ulp, and imx93 is
> 
> What driver adds or not, is not really correct argument for compatibility.
> There are plenty of compatible devices which have both device ID entries in
> the driver. Why would you drop their compatibility between each other? It
> does not work like that... Provide clear rationale for this.

Hi Krzysztof, 
We take imx7ulp lpuart as the basic version IP, then imx8ulp adds some improved features based on the imx7ulp lpuart IP.
Actually imx8ulp can also work with the imx7ulp version, we can say it is compatible with imx7ulp, but we need to enable those new features by default on imx8ulp, so we added the imx8ulp-lpuart compatible in lpuart driver. So for this case, maybe compatible = "fsl,imx8ulp-lpuart", "fsl,imx7ulp-lpuart" is correct for imx8ulp platform, right?

Then we have the imx93 family with the same lpuart version as imx8ulp, so how should we handle the compatible string for imx93 platform? Maybe compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-lpuart", "fsl,imx7ulp-lpuart"? But three compatible strings looks too long for me, so I want to drop imx7ulp-lpuart and take imx8ulp-lpuart as the basic compatible string for imx93 and later others imx9 series, I am not sure if it is suitable, some suggestions here will be appreciate, thanks!

Best Regards
Sherry

> 
> > much more compatible with imx8ulp than imx7ulp, so correct the
> > dt-binding here.
> >
> 
> "Much more compatible" is too vague.
> 
> 
> Best regards,
> Krzysztof
Krzysztof Kozlowski June 25, 2023, 3:40 p.m. UTC | #3
On 25/06/2023 15:12, Sherry Sun wrote:
> 
> 
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
>> Sent: 2023年6月25日 18:18
>> To: Sherry Sun <sherry.sun@nxp.com>; robh+dt@kernel.org;
>> krzysztof.kozlowski+dt@linaro.org; conor+dt@kernel.org;
>> shawnguo@kernel.org; s.hauer@pengutronix.de; festevam@gmail.com;
>> Shenwei Wang <shenwei.wang@nxp.com>; gregkh@linuxfoundation.org
>> Cc: devicetree@vger.kernel.org; linux-arm-kernel@lists.infradead.org;
>> kernel@pengutronix.de; linux-serial@vger.kernel.org; linux-
>> kernel@vger.kernel.org; dl-linux-imx <linux-imx@nxp.com>
>> Subject: Re: [PATCH 4/4] dt-bindings: serial: fsl-lpuart: add imx8ulp
>> compatible string
>>
>> On 25/06/2023 11:48, Sherry Sun wrote:
>>> LPUART driver adds a new compatible string for imx8ulp, and imx93 is
>>
>> What driver adds or not, is not really correct argument for compatibility.
>> There are plenty of compatible devices which have both device ID entries in
>> the driver. Why would you drop their compatibility between each other? It
>> does not work like that... Provide clear rationale for this.
> 
> Hi Krzysztof, 
> We take imx7ulp lpuart as the basic version IP, then imx8ulp adds some improved features based on the imx7ulp lpuart IP.
> Actually imx8ulp can also work with the imx7ulp version, we can say it is compatible with imx7ulp, but we need to enable those new features by default on imx8ulp, so we added the imx8ulp-lpuart compatible in lpuart driver. So for this case, maybe compatible = "fsl,imx8ulp-lpuart", "fsl,imx7ulp-lpuart" is correct for imx8ulp platform, right?
> 
> Then we have the imx93 family with the same lpuart version as imx8ulp, so how should we handle the compatible string for imx93 platform? Maybe compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-lpuart", "fsl,imx7ulp-lpuart"? But three compatible strings looks too long for me, so I want to drop imx7ulp-lpuart and take imx8ulp-lpuart as the basic compatible string for imx93 and later others imx9 series, I am not sure if it is suitable, some suggestions here will be appreciate, thanks!
> 

Wrap your responses.

So write that the reason of this change is because looks too long for
you and we will judge that rationale. Three compatibles is still fine.

Best regards,
Krzysztof
Sherry Sun June 26, 2023, 2:02 a.m. UTC | #4
> >>
> >> On 25/06/2023 11:48, Sherry Sun wrote:
> >>> LPUART driver adds a new compatible string for imx8ulp, and imx93 is
> >>
> >> What driver adds or not, is not really correct argument for compatibility.
> >> There are plenty of compatible devices which have both device ID
> >> entries in the driver. Why would you drop their compatibility between
> >> each other? It does not work like that... Provide clear rationale for this.
> >
> > Hi Krzysztof,
> > We take imx7ulp lpuart as the basic version IP, then imx8ulp adds some
> improved features based on the imx7ulp lpuart IP.
> > Actually imx8ulp can also work with the imx7ulp version, we can say it is
> compatible with imx7ulp, but we need to enable those new features by
> default on imx8ulp, so we added the imx8ulp-lpuart compatible in lpuart
> driver. So for this case, maybe compatible = "fsl,imx8ulp-lpuart",
> "fsl,imx7ulp-lpuart" is correct for imx8ulp platform, right?
> >
> > Then we have the imx93 family with the same lpuart version as imx8ulp, so
> how should we handle the compatible string for imx93 platform? Maybe
> compatible = "fsl,imx93-lpuart", "fsl,imx8ulp-lpuart", "fsl,imx7ulp-lpuart"?
> But three compatible strings looks too long for me, so I want to drop
> imx7ulp-lpuart and take imx8ulp-lpuart as the basic compatible string for
> imx93 and later others imx9 series, I am not sure if it is suitable, some
> suggestions here will be appreciate, thanks!
> >
> 
> Wrap your responses.
> 
> So write that the reason of this change is because looks too long for you and
> we will judge that rationale. Three compatibles is still fine.
> 
Hi Krzysztof,
Got it, thanks for your suggestion, I will use three compatibles for imx93.

Best Regards
Sherry
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
index 93062403276b..5599be95400b 100644
--- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
+++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
@@ -21,13 +21,12 @@  properties:
           - fsl,ls1021a-lpuart
           - fsl,ls1028a-lpuart
           - fsl,imx7ulp-lpuart
+          - fsl,imx8ulp-lpuart
           - fsl,imx8qxp-lpuart
           - fsl,imxrt1050-lpuart
       - items:
-          - enum:
-              - fsl,imx93-lpuart
-              - fsl,imx8ulp-lpuart
-          - const: fsl,imx7ulp-lpuart
+          - const: fsl,imx93-lpuart
+          - const: fsl,imx8ulp-lpuart
       - items:
           - enum:
               - fsl,imx8qm-lpuart