[PATCH 3/6] ARM: mach-imx/mx27_3ds: Add LCD support
Sascha Hauer
s.hauer at pengutronix.de
Tue Jun 21 18:19:30 EDT 2011
On Tue, Jun 21, 2011 at 10:57:14AM -0300, Fabio Estevam wrote:
> Hi Sascha,
>
> On Tue, Jun 21, 2011 at 9:59 AM, Fabio Estevam <festevam at gmail.com> wrote:
> > Hi Sascha,
> >
> > On Tue, Jun 21, 2011 at 9:11 AM, Sascha Hauer <s.hauer at pengutronix.de> wrote:
> > ...
> >>> mxc_gpio_mode(SPI2_SS0 | GPIO_GPIO | GPIO_OUT);
> >>> mxc_gpio_mode(PMIC_INT | GPIO_GPIO | GPIO_IN);
> >>> + mxc_gpio_mode(SPI1_SS0 | GPIO_GPIO | GPIO_OUT);
> >>> + mxc_gpio_mode(LCD_ENABLE | GPIO_GPIO | GPIO_OUT);
> >>> + mxc_gpio_mode(LCD_RESET | GPIO_GPIO | GPIO_OUT);
> >>
> >> You can add these to the static pin array instead.
> >
> > I tried that initially and it caused all the gpio_request calls to fail.
> >
> > The only way I could make gpio_request to work is when I define the
> > pin as gpio mode via mxc_gpio_mode.
>
>
> I managed to put these definitions in the static pin array as you suggested.
>
> In order to do so, I had to fix iomux-v1.c. I will send a v2 version
> including the iomux-v1 fix.
>
> Please let me know if you have any other comments in the other patches
> so that I can consider them for v2.
The other patches look fine.
Sascha
>
> Thanks,
>
> Fabio Estevam
>
--
Pengutronix e.K. | |
Industrial Linux Solutions | http://www.pengutronix.de/ |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 |
Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 |
More information about the linux-arm-kernel
mailing list