Hi,
On 09/16/10 11:00, Felipe Balbi wrote:
> Hi,
>
> On Thu, Sep 16, 2010 at 03:54:39AM -0500, Igor Grinberg wrote:
>
> put a description here.
This is a relatively small patch, is the subject not descriptive enough?
>
>> Signed-off-by: Igor Grinberg <grinberg@xxxxxxxxxxxxxx>
>> ---
>
> [snip]
>
>> +static int cm_t3517_init_usbh(void)
>> +{
>> + int err;
>> +
>> + err = gpio_request(USB_HUB_RESET_GPIO, "usb hub rst");
>> + if (err) {
>> + pr_err("CM-T3517: usb hub reset gpio request failed: %d\n",
>> + err);
>> + } else {
>
> if (err) {
> pr_err("...");
> return err;
> }
>
> gpio_direction_output(...);
>
> this will save you on identation level.
Yes it will, but even if the hub reset gpio is for some reason unavailable,
we don't want to disable ehci completely...
There are other ports, that are not wired through the hub.
--
Regards,
Igor.
--
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
