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 5FA18ECAAD5 for ; Mon, 5 Sep 2022 21:18:44 +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:In-Reply-To:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=LKsh71V31p/Y0xRRldX1gyhL3RP47MyC+ziGKURgLhM=; b=xMm9WfzC8sf5VE u69UU2yD2WdgMHGqwVWFBUaTKhpuaWzMqZDfeQsoITouB/qFa/UadUXTbetTl90n+D1LoIiEfOE5M lLB0jSQZ8ca/Rpuw1lU8nB9NBXrjmOu+iSTflGSRCbDtj9NmGVwwKAM8+QCgFCL0MXsg/HvTCYbWw QLiY7a++Hdo8lw+jj7r21HCHHZa0WS8pS+kSfh16dAYYB9Lpb7RuiZd+gMp/4ddfpPmYMJkqFpZ7l Rz8AsVgsF7pF1Je1OJb7+y1sQkBSBmHwF9DxLjwfN00ob7wjRHvBGTGIYncH+cin0hCGp+w5S877L Kq1z2kcSRmpvkgh5oWbA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVJTC-00EU63-Q3; Mon, 05 Sep 2022 21:17:27 +0000 Received: from de-smtp-delivery-113.mimecast.com ([194.104.109.113]) by bombadil.infradead.org with esmtps (Exim 4.94.2 #2 (Red Hat Linux)) id 1oVJT8-00ESdo-BM for linux-arm-kernel@lists.infradead.org; Mon, 05 Sep 2022 21:17:24 +0000 Received: from CHE01-GV0-obe.outbound.protection.outlook.com (mail-gv0che01lp2048.outbound.protection.outlook.com [104.47.22.48]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id de-mta-41-4-nydSdGN-Wfi091Q1beeA-1; Mon, 05 Sep 2022 23:17:05 +0200 X-MC-Unique: 4-nydSdGN-Wfi091Q1beeA-1 Received: from ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:2e::8) by GV0P278MB0148.CHEP278.PROD.OUTLOOK.COM (2603:10a6:710:2f::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5588.10; Mon, 5 Sep 2022 21:17:04 +0000 Received: from ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM ([fe80::6c6d:333:ab23:3f5b]) by ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM ([fe80::6c6d:333:ab23:3f5b%2]) with mapi id 15.20.5588.018; Mon, 5 Sep 2022 21:17:03 +0000 Date: Mon, 5 Sep 2022 23:17:03 +0200 From: Francesco Dolcini To: Laurent Pinchart Cc: Francesco Dolcini , Francesco Dolcini , Max Krummenacher , Max Krummenacher , Fabio Estevam , Krzysztof Kozlowski , Marcel Ziswiler , NXP Linux Team , Pengutronix Kernel Team , Philippe Schenker , Rob Herring , Sascha Hauer , Shawn Guo , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH 1/2] arm64: dts: imx8mp-verdin: add dsi to hdmi functionality Message-ID: <20220905211703.GA6180@francesco-nb.int.toradex.com> References: <20220901154051.1885509-1-max.oss.09@gmail.com> <20220902155720.GB5699@francesco-nb.int.toradex.com> In-Reply-To: X-ClientProxiedBy: ZR2P278CA0017.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:46::11) To ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:2e::8) MIME-Version: 1.0 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: e17bbf86-f956-4d1b-5f99-08da8f83fab0 X-MS-TrafficTypeDiagnostic: GV0P278MB0148:EE_ X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0 X-Microsoft-Antispam-Message-Info: 5pOKL82w3G4zFICp5ECySGk0Euugzjv/iw5q55Io7+xpN3gCH0gFW920Ib6tTGxESQJ2Ga0XtTO7S6Aq1TF/tqyIAQhmHAVoLaGcvM8/N++r40Bv0019xhmLx3ioSAcmt90TO5LpP1dw+1jbNjmn4SKuPMy+CFCoKJQ0vLVXIj4joRaABrg0RyBzMw17YKGKiHyz9Hf9l9/YjmP2Vbi56hJtgYNJxdr8ukyhxr+cImySqd26Q8escOMxkdTKSnjYi5RLy4HT9iGUWsYjvvDg4Ks0x4HDVh+OR9BNvvi+az1qoR0ngRCVxgTDewfoo6Iy36He9oXlUuRJaUzBh2dS60hydDbww3okltT7X43I1GflMGVt0DI9SAErTM2dhCF89VvVsbIjKytaNJZMxbI6tqO7Gz4dy62oqqmSiEwyi2qEkGI+3b8GycHBq1rqzWEyqJKXneH+gP17ulIjYlKH8+VNSXsa1ZLlhL7MvvJ2AUPoyyyxUaqf15Sk4UJmFJDv1W3LP7Ugv0vurw4cA5gM4thTsPq1fLsL+FsCZZHk3gVZCdkwzKtPTFTwy27IUnxefJ8EJwCyTCu+FUaOHYIOk5Wq8/2Uv00gCyjbJZYYk0kcpyxjWokkcpE9lWUz2KsGNhwedfW29tCkvU/ov9eMIO0JEB/GBn36Rt/8fzRXw6SWXrN5Cswd/czZr13CQt62Oy2Ec77nL4BZRYCRz81uaeQkxEaa+mPQiyoMda+i1BDEvp05wTJCIEhIwGIFZfiM X-Forefront-Antispam-Report: CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM;PTR:;CAT:NONE;SFS:(13230016)(4636009)(39840400004)(136003)(366004)(376002)(346002)(396003)(4326008)(66556008)(8676002)(33656002)(66946007)(8936002)(5660300002)(7416002)(6506007)(83380400001)(52116002)(478600001)(6486002)(41300700001)(1076003)(186003)(86362001)(316002)(6916009)(54906003)(66476007)(2906002)(38100700002)(6512007)(44832011)(32563001);DIR:OUT;SFP:1102 X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?NS2pg3e12TRoWLkNE96juh7Va/ghs3YfR9qbAdiDsqcO87MUcaC/XZu4hYrp?= =?us-ascii?Q?VHSG2YnhC5CRmDGsmkI0svDHAYu3KXSVHRREwX8UdyWugvL90T8nOKowaMB2?= =?us-ascii?Q?JaogMurWzMizd6g9eDuC/koA2NEiSgm24c+0pLQwjpU621acI68TJfbgqhot?= =?us-ascii?Q?PKG8BpTPc6k71yxLu+JYvntupQK3CPAxWKtqAcLadkPRVMqjoXkqZ3Qevdx6?= =?us-ascii?Q?8A8pnmOiTj+GzxgiYmNUy3+PITXoortg12vTnVadkgJMEqMSeW2b+GEv/z0a?= =?us-ascii?Q?WZ1Uyfc4OmQ7cVIbVxgF7XMCtIDLnIZPXaxzpf+45dwe+HiBoDYzsaV+gF+A?= =?us-ascii?Q?5gzRvp83BIKPzyjtMfHW0Hi0GZm9qrVKP/kyhSTVrlWDe2dvflwK2aN8ZhYQ?= =?us-ascii?Q?fh0WJuDurHMJ1CFRHpITzVfLpRc5gYly64HE7LXo5knjx8Yzlvwy4oPRYPkk?= =?us-ascii?Q?TufTum5sm+zpy8INYGdJ4/g9zKI6+HOjiefahoqM4OHrbh7jsbyuDD27S617?= =?us-ascii?Q?vJbEslNBiqTVBeQuXrW+PMH14KqpSHK3RcZ9ECIsZSPDd4h5onitgGt6udOH?= =?us-ascii?Q?65aIZfMVygM14qFZq0dqnvknQ9Tjdh/dczYGtsCVYk/beticMbGPV6u8N3FN?= =?us-ascii?Q?gyNst41Qu00ln4dRQ1/O+0rxLiM6GEiCQxeS1ItpCNKR5LaPE96r6r3B23rn?= =?us-ascii?Q?GzmJIDfVLlyEsz0CfoUKItZNZFBL7JhiFnUaS7pcoLGGSWnOwODp95yd/2PQ?= =?us-ascii?Q?Akk/ZB+zh4dGZXJ9s8bdZWJmqRTtBD05nLPQ0ovFMsFd3P5FCz8eyPqPVVpz?= =?us-ascii?Q?U0JP2NG77fVO73ykaiG/TVMgOPmPyWRQrvsCy4gBXUQWDmcDQaMsrZjpIhzH?= =?us-ascii?Q?fDCWTbKJYT9eEGX9SBS/rjFuz1qqotRzG7y/uWs1cn1q8IHSQ85TtNIuTLo5?= =?us-ascii?Q?HQbUiZVI4qQfeh3CLBZnXO8b8odb4Afwlrjo2t4xlW/25e+OMKEdeOsVe+no?= =?us-ascii?Q?A4IBDaeDt6JgP8yFfD2Xw2Q5k9A4LRwrGzzohxeoxh+2R8423B5a4zNTWUDx?= =?us-ascii?Q?s1U8hMzHvP8B2OJyx0G4dRq545G6qS0zhkF9AiYAUSXW4vFRGYdfxVLGNVyL?= =?us-ascii?Q?4l6UtA0+Rk0dH3D+VOnN5dhq3Qgz0eIZNqs3bhmEb1wSRIpHj3pYHzzqtubs?= =?us-ascii?Q?w66PfWSHTz536LUywu5Ap1lI9y0gP697gYqR5IyY7zYCn8KzSqz0Cn7VKbcV?= =?us-ascii?Q?VPx/b2XwpYDomSfm20v7JNOfYjasF/Iks+vR7LlyUyf9U+45rQ6a3invEZPz?= =?us-ascii?Q?hqdGPidAhqkil/0gMTHJB5VLfz0oUJb1gewsc44XdHaw5PUiF8GxVL5frd3q?= =?us-ascii?Q?rvlDqsvZMerLymOTFW2sX9IebvpTrFsQJTFjD0WQDTyESEyirFX6yGijSwG3?= =?us-ascii?Q?2FEwPI4bxVQ6asx1oJK3MdA5UsZMqo4CAEfMhSS5fJzQE8IXXFPuTnuM7+N8?= =?us-ascii?Q?O4seKTN6VX0Q8tzUfSkfi1p9Kc80CYmbcY/MVygW15sDu/eKa1TmzlZQtCOF?= =?us-ascii?Q?p3YNd7FKEDQK5nZAEV4VQQ8pbelA7kVEkM6rfjU2ZJNj+Or66ANjf0vOfuSP?= =?us-ascii?Q?C+G9+qikr5K+M428DU0pAuOQw3GfQ0pS06vGiKGr0ae+bNS79rP5rYMlhbvv?= =?us-ascii?Q?TDPfcw=3D=3D?= X-OriginatorOrg: toradex.com X-MS-Exchange-CrossTenant-Network-Message-Id: e17bbf86-f956-4d1b-5f99-08da8f83fab0 X-MS-Exchange-CrossTenant-AuthSource: ZRAP278MB0495.CHEP278.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 05 Sep 2022 21:17:03.7259 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: d9995866-0d9b-4251-8315-093f062abab4 X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: SHs6d1bf4y9HIzBh7Veb9We+OgnHjwvUwM2aFmfkTlvdIglkjril35n6MtxJCowlnUUFYwYME/oXfDE5O7HjrhCgy0D2Ca7fYlJhqvC2EL4= X-MS-Exchange-Transport-CrossTenantHeadersStamped: GV0P278MB0148 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: toradex.com Content-Disposition: inline X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220905_141722_752612_0CF9CCDF X-CRM114-Status: GOOD ( 26.44 ) 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 Laurent, On Mon, Sep 05, 2022 at 10:26:14PM +0300, Laurent Pinchart wrote: > On Sat, Sep 03, 2022 at 02:47:43PM +0200, Francesco Dolcini wrote: > > On Sat, Sep 03, 2022 at 03:24:51AM +0300, Laurent Pinchart wrote: > > > On Fri, Sep 02, 2022 at 05:57:20PM +0200, Francesco Dolcini wrote: > > > > On Thu, Sep 01, 2022 at 09:07:49PM +0300, Laurent Pinchart wrote: > > > Someone can integrate a Verdin SoM with a carrier board that has no DSI > > > to HDMI (or LVDS) bridge, there should thus be no such device in the > > > device tree. The SoM has DSI signals present on its connector, that's > > > what the SoM .dtsi should expose. > > > > Just for the record Verdin i.MX8M Plus do have both HDMI and LVDS on the > > connector (in addition to DSI) [1], of course we do have also the option to > > have LVDS or HDMI using an external add-on DSI bridge as this patches are > > about. > > > > Said that it's true that sometime we describe peripherals that are part of the > > SOM family into the SOM dtsi, this avoid quite a lot of duplications given the > > amount of carrier board that are available on the market that use just the same > > building blocks (and this was one of the 2 points I mentioned as a reasoning > > for our current DTS files structure). > > If those "SoM family" peripherals are on the carrier board, what's the > issue with describing them in the carrier board .dtsi ? And if they're > on an add-on board (such as, if I understand correctly, the DSI to HDMI > encoder for the Dahlia carrier board), what's the issue with describing > them in an overlay ? These SOM family peripherals are in multiples(!) carrier boards AND on accessories. The drawback of being strict as you are asking is that we would end-up with a massive duplication of this small DTS building blocks, therefore the decision in the past to put those in the base SOM dtsi file. Maybe adding something like imx8mp-verdin-dsi-hdmi.dtsi and imx8mp-verdin-dsi-lvds.dtsi that can be included by both overlay and carrier dts files as needed would solve both the need of being strict on the board definition in the dts file and avoid duplications? Not sure if that would work smoothly, it looks like adding some complexity and maintenance overhead, but maybe is the correct solution. Anyway, while I fully understand your reasoning, I'm still not happy to change this for the current toradex products, since users of our dts file currently rely on the expectations I tried to explain in this email thread and Max patches are implementing (and this is currently uniform over the whole toradex product range). > Maybe I'm missing something ? I tried to give more insights. Francesco _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel