* [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller @ 2024-09-10 15:22 Emanuele Ghidoli 2024-09-16 16:10 ` Krzysztof Kozlowski 2024-10-16 7:34 ` Shawn Guo 0 siblings, 2 replies; 6+ messages in thread From: Emanuele Ghidoli @ 2024-09-10 15:22 UTC (permalink / raw) To: linux-kernel, linux-arm-kernel, devicetree Cc: Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini From: Emanuele Ghidoli <emanuele.ghidoli@toradex.com> The device tree defines the touchscreen controller, but it cannot be enabled because it lacks a reference label. This commit adds a label to allow it to be referenced and enabled. Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@toradex.com> --- arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi index edba5b582414..4ea47a576b1c 100644 --- a/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi +++ b/arch/arm64/boot/dts/freescale/imx8x-colibri.dtsi @@ -166,7 +166,7 @@ sgtl5000_a: audio-codec@a { }; /* Touch controller */ - touchscreen@2c { + ad7879_ts: touchscreen@2c { compatible = "adi,ad7879-1"; pinctrl-names = "default"; pinctrl-0 = <&pinctrl_ad7879_int>; -- 2.34.1 ^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller 2024-09-10 15:22 [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller Emanuele Ghidoli @ 2024-09-16 16:10 ` Krzysztof Kozlowski 2024-09-17 7:45 ` Francesco Dolcini 2024-10-16 7:34 ` Shawn Guo 1 sibling, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2024-09-16 16:10 UTC (permalink / raw) To: Emanuele Ghidoli, linux-kernel, linux-arm-kernel, devicetree Cc: Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini On 10/09/2024 17:22, Emanuele Ghidoli wrote: > From: Emanuele Ghidoli <emanuele.ghidoli@toradex.com> > > The device tree defines the touchscreen controller, but it cannot be > enabled because it lacks a reference label. It can be. Just enable it... > 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. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller 2024-09-16 16:10 ` Krzysztof Kozlowski @ 2024-09-17 7:45 ` Francesco Dolcini 2024-09-17 8:21 ` Krzysztof Kozlowski 0 siblings, 1 reply; 6+ messages in thread From: Francesco Dolcini @ 2024-09-17 7:45 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Emanuele Ghidoli, linux-kernel, linux-arm-kernel, devicetree, Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini Hello Krzysztof, it seems that this patch requires more explanation, maybe something that should be part of the commit message, let's see ... 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 <emanuele.ghidoli@toradex.com> > > > > 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. > > 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? Francesco [1] https://lore.kernel.org/all/20240222093607.3085545-1-sumit.garg@linaro.org/ ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller 2024-09-17 7:45 ` Francesco Dolcini @ 2024-09-17 8:21 ` Krzysztof Kozlowski 2024-09-17 13:29 ` Francesco Dolcini 0 siblings, 1 reply; 6+ messages in thread From: Krzysztof Kozlowski @ 2024-09-17 8:21 UTC (permalink / raw) To: Francesco Dolcini Cc: Emanuele Ghidoli, linux-kernel, linux-arm-kernel, devicetree, Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini On 17/09/2024 09:45, Francesco Dolcini wrote: > Hello Krzysztof, > it seems that this patch requires more explanation, maybe something > that should be part of the commit message, let's see ... > > 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 <emanuele.ghidoli@toradex.com> >>> >>> 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. Best regards, Krzysztof ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller 2024-09-17 8:21 ` Krzysztof Kozlowski @ 2024-09-17 13:29 ` Francesco Dolcini 0 siblings, 0 replies; 6+ messages in thread From: Francesco Dolcini @ 2024-09-17 13:29 UTC (permalink / raw) To: Krzysztof Kozlowski Cc: Francesco Dolcini, Emanuele Ghidoli, linux-kernel, linux-arm-kernel, devicetree, Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini 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 <emanuele.ghidoli@toradex.com> > >>> > >>> 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 ^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller 2024-09-10 15:22 [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller Emanuele Ghidoli 2024-09-16 16:10 ` Krzysztof Kozlowski @ 2024-10-16 7:34 ` Shawn Guo 1 sibling, 0 replies; 6+ messages in thread From: Shawn Guo @ 2024-10-16 7:34 UTC (permalink / raw) To: Emanuele Ghidoli Cc: linux-kernel, linux-arm-kernel, devicetree, Emanuele Ghidoli, Pengutronix Kernel Team, Fabio Estevam, NXP Linux Team, Shawn Guo, Rob Herring, Krzysztof Kozlowski, Sascha Hauer, Francesco Dolcini On Tue, Sep 10, 2024 at 05:22:12PM +0200, Emanuele Ghidoli wrote: > From: Emanuele Ghidoli <emanuele.ghidoli@toradex.com> > > The device tree defines the touchscreen controller, but it cannot be > enabled because it lacks a reference label. > This commit adds a label to allow it to be referenced and enabled. > > Signed-off-by: Emanuele Ghidoli <emanuele.ghidoli@toradex.com> Applied, thanks! ^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2024-10-16 7:35 UTC | newest] Thread overview: 6+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2024-09-10 15:22 [PATCH v1] arm64: dts: colibri-imx8x: Add ad7879_ts label to touchscreen controller Emanuele Ghidoli 2024-09-16 16:10 ` Krzysztof Kozlowski 2024-09-17 7:45 ` Francesco Dolcini 2024-09-17 8:21 ` Krzysztof Kozlowski 2024-09-17 13:29 ` Francesco Dolcini 2024-10-16 7:34 ` Shawn Guo
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).