On Wed, Feb 9, 2011 at 5:17 PM, Ricardo Salveti de Araujo <[email protected]> wrote: > 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.
Thanks. Changes looks fine. Reviewed-by: Manjunath G Kondaiah <[email protected]> -Manjunath -- 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
