From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from metis.whiteo.stw.pengutronix.de (metis.whiteo.stw.pengutronix.de [185.203.201.7]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 629E71B6D14 for ; Mon, 4 Nov 2024 15:46:18 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=185.203.201.7 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730735182; cv=none; b=YviwjdX/0wv6Os7YfFRii3einxv9Stwlp8xlQS/xrt/jbGtr4Z39vkAPAgYCVdgxH38rjUVVgEAQEtx1j0ik5yaMbk9dm9DDoP4YIkK8j/YNlRcqWzBQM9DYHuPZp8VlKgYi2WsCmALc8Qkdh6nuBgkXtAj6RakBAje7Rauimxw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730735182; c=relaxed/simple; bh=QAJdDZFfeKebQ8XuBArUKttiwqQuGrseeLwKbmISSA0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=AjnFDi+R5YomqZ3zBr9QE4iZDRFgMnzBnGogWtf9nXrtfAjEZdCBd6n0PPH0w/dCG1ib1C7d8Iyu5+yDRoMpavumeV6baS2tYCp6yS2X7BYOgP+YztDidJtI9JBOgxXs4ESslM7fZFL3dds/0JIm6h5Tik/Op9U++WXpSflTHkE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de; spf=pass smtp.mailfrom=pengutronix.de; arc=none smtp.client-ip=185.203.201.7 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=pengutronix.de Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=pengutronix.de Received: from ptz.office.stw.pengutronix.de ([2a0a:edc0:0:900:1d::77] helo=[127.0.0.1]) by metis.whiteo.stw.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1t7zHM-0007wV-Fl; Mon, 04 Nov 2024 16:46:08 +0100 Message-ID: <2d950234-b71f-4879-86f9-e36086914e88@pengutronix.de> Date: Mon, 4 Nov 2024 16:46:07 +0100 Precedence: bulk X-Mailing-List: imx@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v4 1/3] dt-bindings: lcdif: Document a imx6sx-lcdif fallback To: Fabio Estevam , shawnguo@kernel.org Cc: marex@denx.de, andreas@kemnade.info, kernel@pengutronix.de, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org, imx@lists.linux.dev, linux-arm-kernel@lists.infradead.org, dri-devel@lists.freedesktop.org, Fabio Estevam References: <20241101135406.47836-1-festevam@gmail.com> Content-Language: en-US From: Ahmad Fatoum In-Reply-To: <20241101135406.47836-1-festevam@gmail.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-SA-Exim-Connect-IP: 2a0a:edc0:0:900:1d::77 X-SA-Exim-Mail-From: a.fatoum@pengutronix.de X-SA-Exim-Scanned: No (on metis.whiteo.stw.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: imx@lists.linux.dev On 01.11.24 14:54, Fabio Estevam wrote: > From: Fabio Estevam > > imx6sx.dtsi has the following lcdif entries: > > compatible = "fsl,imx6sx-lcdif", "fsl,imx28-lcdif"; > > This causes the following dt-schema warning: > > ['fsl,imx6sx-lcdif', 'fsl,imx28-lcdif'] is too long > > To keep DT compatibility, document 'fsl,imx28-lcdif' as a possible > 'fsl,imx6sx-lcdif' fallback. > > Signed-off-by: Fabio Estevam Reviewed-by: Ahmad Fatoum > --- > Changes since v3: > - Also update the example. > > Documentation/devicetree/bindings/display/fsl,lcdif.yaml | 7 +++++-- > 1 file changed, 5 insertions(+), 2 deletions(-) > > diff --git a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > index 8e3a98aeec32..aa3204b6aff9 100644 > --- a/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > +++ b/Documentation/devicetree/bindings/display/fsl,lcdif.yaml > @@ -19,7 +19,6 @@ properties: > - enum: > - fsl,imx23-lcdif > - fsl,imx28-lcdif > - - fsl,imx6sx-lcdif > - fsl,imx8mp-lcdif > - fsl,imx93-lcdif > - items: > @@ -32,6 +31,10 @@ properties: > - fsl,imx8mn-lcdif > - fsl,imx8mq-lcdif > - const: fsl,imx6sx-lcdif > + - items: > + - enum: > + - fsl,imx6sx-lcdif > + - const: fsl,imx28-lcdif > > reg: > maxItems: 1 > @@ -182,7 +185,7 @@ examples: > #include > > display-controller@2220000 { > - compatible = "fsl,imx6sx-lcdif"; > + compatible = "fsl,imx6sx-lcdif", "fsl,imx28-lcdif"; > reg = <0x02220000 0x4000>; > interrupts = ; > clocks = <&clks IMX6SX_CLK_LCDIF1_PIX>, -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |