Hello Leandro,
On 02/19/2017 04:11 PM, Leandro Santiago wrote:
> Thank you for the tips. I have just added linux-samsung-soc@xxxxxxxxxxxxxxx
> to cc, even though get_mantainer.pl did not suggested such mailing list.
>
> About the interrupts property, on the original chromeos-3.8, it was defined
> as <2 0>, I just replaced the second zero by it by IRQ_TYPE_NONE as it
> evaluates to zero anyways and is compatible with the existing trackpad
> node. I have fixed it by setting it to IRQ_TYPE_EDGE_FALLING as you
> suggested. I have no access to the platform data though, so could not check
> if it's really correct or not. But still works fine on my chromebook.
>
> Here is the updated version of the patch:
>
> From 458b5b891a98e6163321396514b9d9227ae50e69 Mon Sep 17 00:00:00 2001
> From: Leandro Santiago <leandrosansilva@xxxxxxxxx>
> Date: Sun, 19 Feb 2017 16:50:18 +0100
> Subject: [PATCH] ARM: dts: Add trackpad to exynos5250-snow-rev5
>
> The Samsung Chromebook uses an Atmel maxTouch as trackpad.
>
> The keymap was based on Exynos5250-based Google Spring and most of
> the other values were based on chromeos-3.8 branch from the ChromeOS
> kernel.
>
> Signed-off-by: Leandro Santiago <leandrosansilva@xxxxxxxxx>
> ---
> arch/arm/boot/dts/exynos5250-snow-common.dtsi | 14 ++++++++++++++
> 1 file changed, 14 insertions(+)
>
> diff --git a/arch/arm/boot/dts/exynos5250-snow-common.dtsi
> b/arch/arm/boot/dts/exynos5250-snow-common.dtsi
> index 8f3a80430748..183ff57c6455 100644
> --- a/arch/arm/boot/dts/exynos5250-snow-common.dtsi
> +++ b/arch/arm/boot/dts/exynos5250-snow-common.dtsi
> @@ -434,6 +434,20 @@
> interrupt-parent = <&gpx1>;
> wakeup-source;
> };
> +
> + trackpad@4b {
> + compatible = "atmel,atmel_mxt_tp";
> + reg = <0x4b>;
> + interrupt-parent = <&gpx1>;
> + interrupts = <2 IRQ_TYPE_EDGE_FALLING>;
> + linux,gpio-keymap = <KEY_RESERVED
> + KEY_RESERVED
> + KEY_RESERVED
> + KEY_RESERVED
> + KEY_RESERVED
> + BTN_LEFT>;
> + wakeup-source;
> + };
> };
>
I noticed that the downstream ChromiumOS tree does the same, but I don't
think that's correct to add both trackpad devices under the same I2C bus.
DT is a hardware description and so it should describe what the device
truly has and not add all the possible combinations and make the driver
fail or succeed if according to what's found on a particular board.
So this is a DT hack and instead all the possible variations should be
properly described in different DT with (possibly) common DTS snippets.
I thought that all Snow variations used the Cypress cyapa chip but now
recall a conversation on #linux-exynos about some Snow rev5 variations
using the same Atmel chip used in the Exynos5420/5800 Peach Chromebooks.
That definitely isn't the case for all Snow rev5 though since some have
the Cypress chip like in previous revisions, according to other users.
Best regards,
--
Javier Martinez Canillas
Open Source Group
Samsung Research America
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
