[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <09ad1310-ebc5-7a41-7af6-cdef79f20802@st.com>
Date: Thu, 3 Oct 2019 12:34:17 +0200
From: Alexandre Torgue <alexandre.torgue@...com>
To: Yannick Fertré <yannick.fertre@...com>,
Maxime Coquelin <mcoquelin.stm32@...il.com>,
Rob Herring <robh+dt@...nel.org>,
Mark Rutland <mark.rutland@....com>,
<linux-stm32@...md-mailman.stormreply.com>,
<linux-arm-kernel@...ts.infradead.org>,
<devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
Benjamin Gaignard <benjamin.gaignard@...com>,
Philippe Cornu <philippe.cornu@...com>,
Fabrice Gasnier <fabrice.gasnier@...com>
Subject: Re: [PATCH] ARM: dts: stm32: add focaltech touchscreen on
stm32mp157c-dk2 board
Hi Yannick
On 9/30/19 4:45 PM, Yannick Fertré wrote:
> Enable focaltech ft6236 touchscreen on STM32MP157C-DK2 board.
> This device supports 2 different addresses (0x2a and 0x38)
> depending on the display board version (MB1407).
>
> Signed-off-by: Yannick Fertré <yannick.fertre@...com>
> ---
> arch/arm/boot/dts/stm32mp157c-dk2.dts | 23 +++++++++++++++++++++++
> 1 file changed, 23 insertions(+)
>
> diff --git a/arch/arm/boot/dts/stm32mp157c-dk2.dts b/arch/arm/boot/dts/stm32mp157c-dk2.dts
> index 20ea601..527bb75 100644
> --- a/arch/arm/boot/dts/stm32mp157c-dk2.dts
> +++ b/arch/arm/boot/dts/stm32mp157c-dk2.dts
> @@ -61,6 +61,29 @@
> };
> };
>
> +&i2c1 {
> + touchscreen@2a {
> + compatible = "focaltech,ft6236";
> + reg = <0x2a>;
> + interrupts = <2 2>;
> + interrupt-parent = <&gpiof>;
> + interrupt-controller;
> + touchscreen-size-x = <480>;
> + touchscreen-size-y = <800>;
> + status = "okay";
> + };
> + touchscreen@38 {
> + compatible = "focaltech,ft6236";
> + reg = <0x38>;
> + interrupts = <2 2>;
> + interrupt-parent = <&gpiof>;
> + interrupt-controller;
> + touchscreen-size-x = <480>;
> + touchscreen-size-y = <800>;
> + status = "okay";
> + };
> +};
I'm not confident by this duplication. We should only support the latest
revision of the MB1407. I understand the need but my fear is to
duplicate this node each time we have a new revision (and imagine if we
do that for all i2c devices).
regards
alex
> <dc {
> status = "okay";
>
>
Powered by blists - more mailing lists