Hi,

On Wed, Dec 22, 2010 at 10:20 PM, Mike Rapoport <[email protected]> wrote:

> Only one comment below, otherwise feel free to add
> Acked-by: Mike Rapoport <[email protected]>

Thanks!

>> +     if (gpio_is_valid(plat->wp_gpio)) {
>> +             rc = gpio_request(plat->wp_gpio, "sdhci_wp");
>> +             if (rc) {
>> +                     dev_err(mmc_dev(host->mmc),
>> +                             "failed to allocate wp gpio\n");
>> +                     goto out_cd;
>> +             }
>> +             tegra_gpio_enable(plat->wp_gpio);
>
> gpio_direction_input?

Yep, for cd_gpio too.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to