From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 5E33DC46467 for ; Mon, 2 Jan 2023 09:09:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To: Message-ID:Date:Subject:Cc:To:From:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=jN/3WL6DXypjQh5+XAb9NdEaivoIkCkJYym5n+AAoyg=; b=LaJJJuu4k9ATtK Uqku5lfGnZVcxzopnCG0YWe6Fh/FZPEieD49s1lDUj1dVSh9sMr/le3c4fQi9e/vf+pEkG6TFEkqs NcYyBJ9NT7X+Nqp4p8D0zfXUIuI5GnkhCLy9IIaG4yyymUrfk70h3FRbGT0Kf/kJ/CayTxVSHuGBl ggoijk+Hn/7M7o20exfC7R+QuFr0sRNYHYW+fu6jcXe6TVazCSrBVBHRhh84XH1jquS3ahUua3HwR eCK05IYMq2MDVhbpVbfN9HRQ0QXArHQNKcaZgzsTumwnkzxm9SHBXfnTSZOvPrQ2t/Or61SG+09/4 mOAAXDIi97YhzHKJgrQw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1pCGoJ-009hdj-Ua; Mon, 02 Jan 2023 09:08:48 +0000 Received: from mx1.tq-group.com ([93.104.207.81]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1pCGn5-009hB1-Uo for linux-arm-kernel@lists.infradead.org; Mon, 02 Jan 2023 09:07:34 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1672650452; x=1704186452; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=hY0HSlsTCyK5Fcj80DybP31EoEFbnIv5wUs3Grr3MSI=; b=TobRNRtEFGExE+KBJNASbH5eNSc/tbt+/BBawOiBpX5U04vMjxqL0NpO hlrkYqt/E8TB0b2n/U/I6DfHkp8hYHedLZ9CBq01q230ZeWQfrnURg0xK rccc0CIZXA0KDSCzHokdUMpkcO6dGA+21FCRu+U3y/l4T1mNkfMEACi6C S5UNZbw31BOSxdU5cT6iGnxgraYAlDL8drezruh4ChKsev/isR3j7iLDp 8C4i2Uc3fywjN5vgZHkYkRqp3w5FFhVTBHzJ6H43AmOkGjOIy5v3VqERv cq3AslrRF+1Tq8HCR2YQpOTdNki95yIgV6HUMn7aRo0m62iWd9As0CVxh Q==; X-IronPort-AV: E=Sophos;i="5.96,293,1665439200"; d="scan'208";a="28199828" Received: from unknown (HELO tq-pgp-pr1.tq-net.de) ([192.168.6.15]) by mx1-pgp.tq-group.com with ESMTP; 02 Jan 2023 10:07:27 +0100 Received: from mx1.tq-group.com ([192.168.6.7]) by tq-pgp-pr1.tq-net.de (PGP Universal service); Mon, 02 Jan 2023 10:07:27 +0100 X-PGP-Universal: processed; by tq-pgp-pr1.tq-net.de on Mon, 02 Jan 2023 10:07:27 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tq-group.com; i=@tq-group.com; q=dns/txt; s=key1; t=1672650447; x=1704186447; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=hY0HSlsTCyK5Fcj80DybP31EoEFbnIv5wUs3Grr3MSI=; b=fQsDwAyhaqQ15zv+S7RRaYU1Jk1VZ+PuzIu3HWTJ6kXeyWgNxP0Aq9mM MrkFAf0+5YRoe79e1A17sLgUvj/TU1Yc4twnxw2p5V+yF7RZroC3Ji0eT rWXh7tO7fZg9yPBJ5awK3qrWQVYU04EB8a5fxUF+H3k5lN3egSZa21p4R c+iXDCf+LLwM48EY9OT5uZlSb7i/QQbCKngrmMazhQF74vdnh3zMVOjnx e3tPoGaCX98rl9USCqiTMUvWSy7/ojOX6osdWBD123tZJnBuuevhn7nie X0IELiZZ6v4w2/762WErMeTPp9VSXWkHMrHwfvXlGZkFbIwUVvhXH5Zvb Q==; X-IronPort-AV: E=Sophos;i="5.96,293,1665439200"; d="scan'208";a="28199827" Received: from vtuxmail01.tq-net.de ([10.115.0.20]) by mx1.tq-group.com with ESMTP; 02 Jan 2023 10:07:27 +0100 Received: from steina-w.localnet (unknown [10.123.53.21]) by vtuxmail01.tq-net.de (Postfix) with ESMTP id 24ED3280056; Mon, 2 Jan 2023 10:07:27 +0100 (CET) From: Alexander Stein To: Richard Leitner Cc: Rob Herring , Krzysztof Kozlowski , Shawn Guo , Sascha Hauer , Fabio Estevam , Pengutronix Kernel Team , NXP Linux Team , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org Subject: Re: [PATCH 2/2] arm64: dts: freescale: Add LVDS overlay for TQMa8MPxL Date: Mon, 02 Jan 2023 10:07:26 +0100 Message-ID: <2222186.iZASKD2KPV@steina-w> Organization: TQ-Systems GmbH In-Reply-To: References: <20221208090842.2869374-1-alexander.stein@ew.tq-group.com> <20221208090842.2869374-2-alexander.stein@ew.tq-group.com> MIME-Version: 1.0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230102_010732_545595_71CBBE8F X-CRM114-Status: GOOD ( 27.61 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org Hello Richard, Am Dienstag, 20. Dezember 2022, 09:48:45 CET schrieb Richard Leitner: > Hi Alexander, > > thanks for the series. > > On Thu, Dec 08, 2022 at 10:08:42AM +0100, Alexander Stein wrote: > > This adds an overlay for the supported LVDS display tianma tm070jvhg33. > > The display timings have to be specified explicitly. Using the typical > > pixel clock, the LDB clock can not be configured as the 7-fold of that. > > By setting pixel clock to 74.25 MHz, LDB can be configured to exactly > > 519.75 MHz. > > > > Signed-off-by: Alexander Stein > > --- > > To be hold I'm unsure where to list the overlays. I checked other overlays > > and opted to added them at the end of imx8mp section. > > > > arch/arm64/boot/dts/freescale/Makefile | 4 ++ > > .../imx8mp-tqma8mpql-mba8mpxl-lvds.dtso | 61 +++++++++++++++++++ > > 2 files changed, 65 insertions(+) > > create mode 100644 > > arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso > ... > > > diff --git > > a/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso > > b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso new > > file mode 100644 > > index 0000000000000..ea44d605342ba > > --- /dev/null > > +++ b/arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso > > @@ -0,0 +1,61 @@ > > +// SPDX-License-Identifier: (GPL-2.0-or-later OR MIT) > > +/* > > + * Copyright (c) 2022 TQ-Systems GmbH , > > + * D-82229 Seefeld, Germany. > > + * Author: Alexander Stein > > + */ > > + > > +/dts-v1/; > > +/plugin/; > > + > > +&{/} { > > + compatible = "tq,imx8mp-tqma8mpql-mba8mpxl", "tq,imx8mp-tqma8mpql", > > "fsl,imx8mp"; +}; > > + > > +&backlight_lvds { > > + status = "okay"; > > +}; > > + > > +&display { > > + compatible = "tianma,tm070jvhg33"; > > + status = "okay"; > > + > > + panel-timing { > > + clock-frequency = <74250000>; > > + hactive = <1280>; > > + vactive = <800>; > > + hfront-porch = <64>; > > + hback-porch = <5>; > > + hsync-len = <1>; > > + vfront-porch = <40>; > > + vback-porch = <2>; > > + vsync-len = <1>; > > + de-active = <1>; > > + }; > > + > > + port { > > + panel_in_lvds0: endpoint { > > + remote-endpoint = <&ldb_lvds_ch0>; > > + }; > > + }; > > +}; > > + > > +&lcdif2 { > > + status = "okay"; > > +}; > > + > > +&lvds_bridge { > > + status = "okay"; > > + > > + ports { > > + port@1 { > > + ldb_lvds_ch0: endpoint { > > + remote-endpoint = <&panel_in_lvds0>; > > + }; > > + }; > > + }; > > +}; > > Wouldn't it be possible and easier to read if this was &ldb_lvds_ch0 > only instead of the "whole" &lvds_bridge? > > &ldb_lvds_ch0 { > remote-endpoint = <&panel_in_lvds0>; > }; Yes it is possible and easier to read. Unfortunately dtbs_check will raise the follwing warnings when doing so: > arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtbo: Warning (graph_port): /fragment@4: graph port node name should be 'port' > ../arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso: 47.15-49.3: Warning (graph_endpoint): /fragment@4/__overlay__: graph endpoint node name should be 'endpoint' > ../arch/arm64/boot/dts/freescale/imx8mp-tqma8mpql-mba8mpxl-lvds.dtso: 47.15-49.3: Warning (graph_endpoint): /fragment@4/__overlay__: graph connection to node '/fragment@2/__overlay__/port/endpoint' is not bidirectional I'm not sure which way is the preferred/correct one or if this is just an issue in dtc, raising the warnings. Best regards, Alexander > > + > > +&pwm2 { > > + status = "okay"; > > +}; > > regards;rl _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel