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 17408C3601A for ; Tue, 17 Sep 2024 13:30:58 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender:List-Subscribe:List-Help :List-Post:List-Archive:List-Unsubscribe:List-Id:In-Reply-To:Content-Type: MIME-Version:References:Message-ID:Subject:Cc:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=+17PJtQ9hXw0xsvyjaFHf6l+N2HVSiJMd9TkH1OyWKk=; b=kXnnbBxVhV9qVizayOOKMhrxaa jU2bMROJGSBhbkFudIYkd6vpEXkMYuYqaCThmRjV2TnXL4uCVSnnLRY5f9P2pUBaxpe3Tu4vG5Us9 5zcFNM2Wmu60ZDRMpLQZbZ60JnoZQqHlHixnCE2o6XUgr4fZ4m3thlmQ05mhp8v5qeoTd1PrBWsRW iF3K9TBGEE8rejQFBH8AN4oCnIqdwQMjsiKEQlTjtx/h4+Lvk7BkjtpbfjfoKToMRyyMfL9NFs0/4 8B34OUdMaPRBtn18k3BJ5/oqlgKNtNmRDfdubGmSztSzpCDIaXG3HLcEETREKdYxAAoWoYDecVOsA FnMo45Qg==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.98 #2 (Red Hat Linux)) id 1sqYI3-00000006ETM-3dsm; Tue, 17 Sep 2024 13:30:47 +0000 Received: from mail11.truemail.it ([217.194.8.81]) by bombadil.infradead.org with esmtps (Exim 4.98 #2 (Red Hat Linux)) id 1sqYGv-00000006EJC-3V65 for linux-arm-kernel@lists.infradead.org; Tue, 17 Sep 2024 13:29:39 +0000 Received: from francesco-nb (93-49-2-63.ip317.fastwebnet.it [93.49.2.63]) by mail11.truemail.it (Postfix) with ESMTPA id CAA241F910; Tue, 17 Sep 2024 15:29:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=dolcini.it; s=default; t=1726579773; bh=+17PJtQ9hXw0xsvyjaFHf6l+N2HVSiJMd9TkH1OyWKk=; h=From:To:Subject; b=wkFhk6gikRjOdmodnIppIFD8v8E76qRGG2r+Tgrj2tgiMaa4EAs8DdShu4APg8TOV pauHmxvvUXRRVOIG2dJOejLP/CjWsuEgjyHTX33AETbeey49Iw9mjb7GFuWdClUomR qpexSmcZQ3bDrBwW6aQ7SfZDsWT8xi8Puxy2fhNfB1K+sfPm4rYElF2pMLwtn0DcjA 2+6xwVO46djGMEwYHuK7CjNn2ibFV43AuHHQJQ5m/thOTcNjWNYScLqTdCMhgtUmWo LCkD8cCQbFQ0kFYzv54YB53inbEjjunk6wwBpdsSAMU+7K2Wx8jR/jgV6ZIPcpkxdO SVLhuCAWWIuUA== Date: Tue, 17 Sep 2024 15:29:28 +0200 From: Francesco Dolcini To: Krzysztof Kozlowski Cc: Francesco Dolcini , Emanuele Ghidoli , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, Emanuele Ghidoli , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , Shawn Guo , Rob Herring , Krzysztof Kozlowski , Sascha Hauer , Francesco Dolcini Subject: Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller Message-ID: <20240917132928.GA5292@francesco-nb> References: <20240910152213.2072743-1-ghidoliemanuele@gmail.com> <1cfe3f3a-28a3-4030-b6ba-3892a2a7bc79@linaro.org> <20240917074544.GA7916@francesco-nb> <99a6eec3-ed24-47b2-af07-b4764c6de6a9@linaro.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <99a6eec3-ed24-47b2-af07-b4764c6de6a9@linaro.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20240917_062938_508619_B592DB77 X-CRM114-Status: GOOD ( 27.21 ) 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: , Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Sep 17, 2024 at 10:21:16AM +0200, Krzysztof Kozlowski wrote: > On 17/09/2024 09:45, Francesco Dolcini wrote: > > On Mon, Sep 16, 2024 at 06:10:55PM +0200, Krzysztof Kozlowski wrote: > >> On 10/09/2024 17:22, Emanuele Ghidoli wrote: > >>> From: Emanuele Ghidoli > >>> > >>> The device tree defines the touchscreen controller, but it cannot be > >>> enabled because it lacks a reference label. > >> > >> It can be. Just enable it... > > > > colibri-imx8x is a SoM, it's not a complete self-contained device. The > > display touch controller is part of the SoM, however it is kept disabled > > since you need an actual touchscreen to make any use of it. > > > > This label would be used where an actual touchscreen is defined, this can > > happen with a DT overlay, for example. > > DT overlay should be in the tree. > > > > >>> This commit adds a label to allow it to be referenced and enabled. > >> > >> You changed here nothing. For me this patch is churn and pointless. > >> You add the label when you need to use it. > > > > DT files from the Linux kernel GIT are used also outside the Linux kernel > > sources, see for example U-Boot OF_UPSTREAM [1], to me it's fair to add > > a label for an out-of-tree user, am I wrong? > > > For U-Boot or any other upstream project: yes. It's enough to point to > lore patch or lore link. For downstream projects: I don't care, just > churn. Downstream should upstream their stuff to be considered. In the specific case this is a SOM dtsi that can be included in an actual whole self-contained product DTS file. It's not just about DT overlay. In the Linux kernel DT files we do have labels all the time into SOC dtsi files to allow the actual board user to augment/edit the node, even when not used (yet). As a user of SOC dtsi file this is great to me, and the same reasoning just apply to a SOM dtsi. While I understand you do not care, IMO it has some valid use case and should be considered for merging. Francesco