diff mbox

[6/8] ARM: dts: sun8i: Add touchscreen node for sun8i-a23-polaroid-mid2407pxe03

Message ID 1470685398-14568-6-git-send-email-hdegoede@redhat.com (mailing list archive)
State New, archived
Headers show

Commit Message

Hans de Goede Aug. 8, 2016, 7:43 p.m. UTC
Add a node enabling the gsl1680 touchscreen controller found on
sun8i-a23-polaroid-mid2407pxe03 tablets.

Signed-off-by: Hans de Goede <hdegoede@redhat.com>
---
 arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts | 17 +++++++++++++++++
 1 file changed, 17 insertions(+)

Comments

Chen-Yu Tsai Aug. 16, 2016, 3:03 a.m. UTC | #1
Hi,

On Tue, Aug 9, 2016 at 3:43 AM, Hans de Goede <hdegoede@redhat.com> wrote:
> Add a node enabling the gsl1680 touchscreen controller found on
> sun8i-a23-polaroid-mid2407pxe03 tablets.
>
> Signed-off-by: Hans de Goede <hdegoede@redhat.com>
> ---
>  arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts | 17 +++++++++++++++++
>  1 file changed, 17 insertions(+)
>
> diff --git a/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts b/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
> index e300442..cd64fd61 100644
> --- a/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
> +++ b/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
> @@ -48,3 +48,20 @@
>         model = "Polaroid MID2407PXE03 tablet";
>         compatible = "polaroid,mid2407pxe03", "allwinner,sun8i-a23";
>  };
> +
> +&gsl1680 {
> +       touchscreen-size-x = <960>;
> +       touchscreen-size-y = <640>;
> +       touchscreen-inverted-x;
> +       touchscreen-inverted-y;
> +       touchscreen-fw-name = "silead/gsl1680-polaroid-mid2407pxe03.fw";
> +       status = "okay";
> +};
> +
> +&reg_ldo_io1 {
> +       regulator-always-on;
> +       regulator-min-microvolt = <3300000>;
> +       regulator-max-microvolt = <3300000>;
> +       regulator-name = "vcc-touchscreen";
> +       status = "okay";
> +};

This looks good, but it would be better if the touchscreen node
took the regulator as a supply.

Regards
ChenYu
diff mbox

Patch

diff --git a/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts b/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
index e300442..cd64fd61 100644
--- a/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
+++ b/arch/arm/boot/dts/sun8i-a23-polaroid-mid2407pxe03.dts
@@ -48,3 +48,20 @@ 
 	model = "Polaroid MID2407PXE03 tablet";
 	compatible = "polaroid,mid2407pxe03", "allwinner,sun8i-a23";
 };
+
+&gsl1680 {
+	touchscreen-size-x = <960>;
+	touchscreen-size-y = <640>;
+	touchscreen-inverted-x;
+	touchscreen-inverted-y;
+	touchscreen-fw-name = "silead/gsl1680-polaroid-mid2407pxe03.fw";
+	status = "okay";
+};
+
+&reg_ldo_io1 {
+	regulator-always-on;
+	regulator-min-microvolt = <3300000>;
+	regulator-max-microvolt = <3300000>;
+	regulator-name = "vcc-touchscreen";
+	status = "okay";
+};