On Wed, 2011-02-09 at 12:23 +0530, G, Manjunath Kondaiah wrote: > On Wed, Feb 9, 2011 at 1:13 AM, Ricardo Salveti de Araujo > <[email protected]> wrote: > > Function beagle_twl_gpio_setup is called after beagle_display_init, what > > 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 powers down DVI in a hardcoded way, as it's > > not necessary anymore. > > > > Tested with Beagle-xM and C4. > > Surprised to see if this code works on C4 since there is gpio_request > for gpio_170 in the beagle init code > (linux omap master branch) which is before beagle_display_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); > > The second gpio_request(for 170) should fail in beagle_display_init. > Pls check and confirm.
That's why this part is also removed at my patch. Cheers, -- Ricardo Salveti de Araujo -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
