RE: [PATCH V6 7/7] ARM: dts: imx6sx-sabreauto: add egalax touch screen support

[Anson Huang]

Hi, Fabio

Anson Huang
Best Regards!


> -----Original Message-----
> From: Fabio Estevam [mailto:festevam@xxxxxxxxx]
> Sent: Saturday, May 5, 2018 11:18 PM
> To: Anson Huang <anson.huang@xxxxxxx>
> Cc: Shawn Guo <shawnguo@xxxxxxxxxx>; Sascha Hauer
> <kernel@xxxxxxxxxxxxxx>; Fabio Estevam <fabio.estevam@xxxxxxx>; Rob
> Herring <robh+dt@xxxxxxxxxx>; Mark Rutland <mark.rutland@xxxxxxx>;
> Haibo Chen <haibo.chen@xxxxxxxxxxxxx>; Andy Duan
> <fugang.duan@xxxxxxx>; A.s. Dong <aisheng.dong@xxxxxxx>; Robin Gong
> <yibin.gong@xxxxxxx>; dl-linux-imx <linux-imx@xxxxxxx>; moderated
> list:ARM/FREESCALE IMX / MXC ARM ARCHITECTURE
> <linux-arm-kernel@xxxxxxxxxxxxxxxxxxx>; open list:OPEN FIRMWARE AND
> FLATTENED DEVICE TREE BINDINGS <devicetree@xxxxxxxxxxxxxxx>; linux-kernel
> <linux-kernel@xxxxxxxxxxxxxxx>
> Subject: Re: [PATCH V6 7/7] ARM: dts: imx6sx-sabreauto: add egalax touch
> screen support
> 
> On Sat, May 5, 2018 at 5:29 AM, Anson Huang <Anson.Huang@xxxxxxx>
> wrote:
> 
> >  &iomuxc {
> > +       pinctrl_egalax_int: egalax-intgrp {
> > +               fsl,pins = <
> > +                       MX6SX_PAD_SD4_RESET_B__GPIO6_IO_22
> 0x80000000
> 
> Please avoid using 0x80000000 and use the real IOMUX value instead.
 
Will use the expected pad settings in next version. Thanks.

Anson.