Date:   Sat, 18 Dec 2021 23:58:46 +0200
From:   Abel Vesa <abel.vesa@nxp.com>
To:     Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc:     Rob Herring <robh@kernel.org>, Dong Aisheng <aisheng.dong@nxp.com>,
        Shawn Guo <shawnguo@kernel.org>,
        Sascha Hauer <s.hauer@pengutronix.de>,
        Fabio Estevam <festevam@gmail.com>,
        Pengutronix Kernel Team <kernel@pengutronix.de>,
        linux-i2c@vger.kernel.org, linux-serial@vger.kernel.org,
        NXP Linux Team <linux-imx@nxp.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org
Subject: Re: [RESEND v4 10/10] dt-bindings: serial: fsl-lpuart: Add i.MX8DXL
 compatible
Message-ID: <Yb5ZlnEm3EYCD9h8@ryzen>
References: <1639680494-23183-1-git-send-email-abel.vesa@nxp.com>
 <1639680494-23183-11-git-send-email-abel.vesa@nxp.com>
 <YbzB7mspRBonT9jJ@kroah.com>
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <YbzB7mspRBonT9jJ@kroah.com>
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?s/HlxPNNLdkKvE5853DMGv/tlhDmv1iBHdl2ppI63CsfO5/UBrlThvGhlNBc?=
 =?us-ascii?Q?ECnadbWkHo66Ip11pBualYzcEJwwnOKkUmwVWRnA9gq78wBGH/rPDqBaY6nn?=
 =?us-ascii?Q?H7xS+P5f8eExtWznJfk0hIbf5waG+3rarDeyGJJGysqzIQe5G0A3fvTj0qwY?=
 =?us-ascii?Q?MK4IREzVe45IHoKpyNrd/9QooP1uKUxNC75xEg7VGpTv3krBQBAH2MKrU6G7?=
 =?us-ascii?Q?JinSp/1BjGcjBKutsQNCR5De5vewrcM5CEqQOMNzgWtqwekSDW088S3rb/uj?=
 =?us-ascii?Q?3R3jYrVr0K7UmSKwM9MJW/XnM/NFe0vad19mpSW0vJ/QaF6SU7wZIliXBL5e?=
 =?us-ascii?Q?8MeYb2syXPmOAalO6OJNbsPMDP4e/LDCHmV93WmggxuNrYkWdf0TO1moPRkM?=
 =?us-ascii?Q?qyTE2VkFb4JUeBYK3DfoAAFq3bdPEEWR7+Ijy7LuwZ6qei6dSaKHZDQsD3mI?=
 =?us-ascii?Q?9+oKSKKzVJxmilNUkY+2An3lecry+EC3YQWckbZrl4r8BjphZctGvcaqSttB?=
 =?us-ascii?Q?u97fEpuzn2NnMQ98oRj7PuSldENAH5HyWGnMbOyTapX++MWvdkIjmFqhW6em?=
 =?us-ascii?Q?QrsP3/i6dpwiOS/9jAo4EvKI883Bxf/Q6Z9DudCRZsT7FzmyDL1XFwRfTS03?=
 =?us-ascii?Q?gjQdAL/S1LvyBPoZ2YfOx3GNF8FfB8Yyw1yLFFPVbWUdkSAzUEPK6EVewJuK?=
 =?us-ascii?Q?r6r5EWmmcqD2c1fmIC/HlGSyzNDbmS5IsglkRNDLcvNpPhAKEspaBj3AlYKN?=
 =?us-ascii?Q?X3lJEMWSpS1ZNhOzMnJr8WxIiFoEeioHVAbl4tFddnr0CsE6QkHZB30PhvZ2?=
 =?us-ascii?Q?xEfuYqNv3nq/C3lVsvls1L54lg6zBekAP2IHKBXFStRq/VVXJauVPCdMwDDn?=
 =?us-ascii?Q?UmghefkdTznDVVwqwQeVJWHZh/aTUYyTZxpR8ul3+ybTrfyvFruWxWNBAKFy?=
 =?us-ascii?Q?Akdm7xKzh8gJzmX7PJVRtUDP16t+ACiRM9/APp1UOvBg1Kb4LnOve7YnpjJR?=
 =?us-ascii?Q?bvP8L5zi2+A82pgmg0e7IUJUGOVBjwU3bHRk+G0s3fJwQufICgvt+3/doOXX?=
 =?us-ascii?Q?2118NLlcVUtUgCs5kEzEPNOdbYu4tc7sG479t1m8Fz1qyfWQrrWCZ9qZ32kg?=
 =?us-ascii?Q?0ZcAn8LTGqDNb6S69p9IrcL8OLWxy/3ICfNzVyXU5jySdtCRCcQ0kR/a3QAL?=
 =?us-ascii?Q?yRpJfRnwbwub3/ps0dpMHbziC1rv3lEpMmUfdgpneRiKa5Fc1GZIr0yKs73j?=
 =?us-ascii?Q?74IRNfMuGR7HC0iSekZHTjWwPv5uD5GqcaabQjMvouGnkgXvE/OS8KVqQczc?=
 =?us-ascii?Q?wtUVx1xa7+iIKNtdVHfACYYaUb3WCjhhPgUnUuHJVRiNqT6J3o0UHGDIg0ZN?=
 =?us-ascii?Q?+pU/8i/hWyim64LJEA9Mps1Hapk1jabaVp+0ORl/7RcKtXqoNa4vu5DxPh7e?=
 =?us-ascii?Q?r8w5muHKAPFtpPe0HLMekfoUMxPe0PZBBwQdnuqYttANk8RpNV3ssiI0pwYp?=
 =?us-ascii?Q?/c8qYXQ1ZvVbymsPkTJlfwiUU3kpsWFoVedcbJdyaeoEdz5swpCXnxLbkI4i?=
 =?us-ascii?Q?0zApHL9TRt9v2qp3+EoEenRWauRLh/xaB/RMyUqHjeSK6ezgbce8AscGl09x?=
 =?us-ascii?Q?PvZTJtyLb4xpdvhO+iJ7DWU=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: f41ab1c2-e128-4fb4-7271-08d9c27191f3
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB4688.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Dec 2021 21:58:48.6848
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: +PvJ5K5euzLHiXNFhB1Ewq4cJ8FHyprbjfyIKbXToTh2x9NI7uqCKUMKSTH4MrKMUJG/z8fI0SyjhxNPaj7KTw==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: VI1PR04MB3198
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 21-12-17 17:59:26, Greg Kroah-Hartman wrote:
> On Thu, Dec 16, 2021 at 08:48:14PM +0200, Abel Vesa wrote:
> > Add i.MX8DXL lpuart compatible to the bindings documentation.
> > 
> > Signed-off-by: Abel Vesa <abel.vesa@nxp.com>
> > ---
> >  Documentation/devicetree/bindings/serial/fsl-lpuart.yaml | 4 ++++
> >  1 file changed, 4 insertions(+)
> > 
> > diff --git a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > index fa23c60a01d0..ee37aa125c86 100644
> > --- a/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > +++ b/Documentation/devicetree/bindings/serial/fsl-lpuart.yaml
> > @@ -29,6 +29,10 @@ properties:
> >        - items:
> >            - const: fsl,imx8qm-lpuart
> >            - const: fsl,imx8qxp-lpuart
> > +      - items:
> > +          - const: fsl,imx8dxl-lpuart
> > +          - const: fsl,imx8qxp-lpuart
> > +          - const: fsl,imx7ulp-lpuart
> 
> Why is "- items:" listed twice here?

There are actually 3 "- items:" and there is a "oneOf:" 
that encloses all of these.

Unfortunately, there are special cases within the driver
that forces the devicetree nodes to have a different list of
compatibles for almost each SoC.

> 
> thanks,
> 
> greg k-h