linux-kernel - RE: [PATCH 4/4] dt-bindings: serial: fsl-lpuart: add imx8ulp compatible string

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <AS8PR04MB8404E78E7DB663FC639CD0749226A@AS8PR04MB8404.eurprd04.prod.outlook.com>
Date:   Mon, 26 Jun 2023 02:02:17 +0000
From:   Sherry Sun <sherry.sun@....com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        "robh+dt@...nel.org" <robh+dt@...nel.org>,
        "krzysztof.kozlowski+dt@...aro.org" 
        <krzysztof.kozlowski+dt@...aro.org>,
        "conor+dt@...nel.org" <conor+dt@...nel.org>,
        "shawnguo@...nel.org" <shawnguo@...nel.org>,
        "s.hauer@...gutronix.de" <s.hauer@...gutronix.de>,
        "festevam@...il.com" <festevam@...il.com>,
        Shenwei Wang <shenwei.wang@....com>,
        "gregkh@...uxfoundation.org" <gregkh@...uxfoundation.org>
CC:     "devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
        "linux-arm-kernel@...ts.infradead.org" 
        <linux-arm-kernel@...ts.infradead.org>,
        "kernel@...gutronix.de" <kernel@...gutronix.de>,
        "linux-serial@...r.kernel.org" <linux-serial@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        dl-linux-imx <linux-imx@....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.
> 
Hi Krzysztof,
Got it, thanks for your suggestion, I will use three compatibles for imx93.

Best Regards
Sherry