On Fri, Jan 28, 2011 at 09:19, Ricardo Salveti de Araujo
<ricardo.salveti@xxxxxxxxxxxxx> wrote:
> Function beagle_twl_gpio_setup is called after beagle_display_init, what
> makes lets reset_gpio with an invalid value at the time it request the
> gpio. As a side effect the DVI reset GPIO is not properly set.
>
> Also removing old code that power down DVI in a hardcoded way, as it's
> not necessary anymore.
>
> Tested with Beagle-xM and C4.
>
> Signed-off-by: Ricardo Salveti de Araujo <ricardo.salveti@xxxxxxxxxxxxx>
> ---
> arch/arm/mach-omap2/board-omap3beagle.c | 17 ++++++-----------
> 1 files changed, 6 insertions(+), 11 deletions(-)
>
> diff --git a/arch/arm/mach-omap2/board-omap3beagle.c b/arch/arm/mach-omap2/board-omap3beagle.c
> index 685ac06..c8ab478 100644
> --- a/arch/arm/mach-omap2/board-omap3beagle.c
> +++ b/arch/arm/mach-omap2/board-omap3beagle.c
> @@ -248,6 +248,12 @@ static void __init beagle_display_init(void)
> {
> int r;
>
> + /* DVI reset GPIO is different between beagle revisions */
> + if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
> + beagle_dvi_device.reset_gpio = 129;
Define a macro for this gpio and use it here.
> + else
> + beagle_dvi_device.reset_gpio = 170;
Ditto.
> +
Where is the mux configuration (omap_mux_init_gpio()) done for this gpio?
-V Charulatha
> r = gpio_request(beagle_dvi_device.reset_gpio, "DVI reset");
> if (r < 0) {
> printk(KERN_ERR "Unable to get DVI reset GPIO\n");
> @@ -324,12 +330,6 @@ static int beagle_twl_gpio_setup(struct device *dev,
> else
> gpio_direction_output(gpio + TWL4030_GPIO_MAX, 0);
>
> - /* DVI reset GPIO is different between beagle revisions */
> - if (omap3_beagle_get_rev() == OMAP3BEAGLE_BOARD_XM)
> - beagle_dvi_device.reset_gpio = 129;
> - else
> - beagle_dvi_device.reset_gpio = 170;
> -
> /* TWL4030_GPIO_MAX + 1 == ledB, PMU_STAT (out, active low LED) */
> gpio_leds[2].gpio = gpio + TWL4030_GPIO_MAX + 1;
>
> @@ -620,11 +620,6 @@ static void __init omap3_beagle_init(void)
> ARRAY_SIZE(omap3_beagle_devices));
> omap_serial_init();
>
> - omap_mux_init_gpio(170, OMAP_PIN_INPUT);
> - gpio_request(170, "DVI_nPD");
> - /* REVISIT leave DVI powered down until it's needed ... */
> - gpio_direction_output(170, true);
> -
> usb_musb_init(&musb_board_data);
> usb_ehci_init(&ehci_pdata);
> omap3beagle_flash_init();
> --
> 1.7.2.3
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
