Hi,
> -----Original Message-----
> From: Sergei Shtylyov [mailto:[email protected]]
> Sent: Tuesday, November 17, 2009 9:12 PM
> To: Gupta, Ajay Kumar
> Cc: [email protected]; davinci-linux-open-
> [email protected]; [email protected]; [email protected];
> [email protected]; Gadiyar, Anand
> Subject: Re: [PATCH 2/8] musb: Get power (mA) from board data
> 
> Hello.
> 
> Ajay Kumar Gupta wrote:
> 
> > Different board may have different power sourcing capability and
> > now with 'struct musb_hdrc_board_data' in place; pass this data
> > from board files and also modify musb_core.c to get 'power' data
> > from 'plat->board_data'.
> 
>     This should be part of the patch 1/8 to keep the code compiling.
> 
> > Signed-off-by: Ajay Kumar Gupta <[email protected]>
> 
> > diff --git a/drivers/usb/musb/musb_core.c b/drivers/usb/musb/musb_core.c
> > index 3a61ddb..818ccda 100644
> > --- a/drivers/usb/musb/musb_core.c
> > +++ b/drivers/usb/musb/musb_core.c
> > @@ -2033,7 +2033,9 @@ bad_config:
> >             if (is_otg_enabled(musb))
> >                     hcd->self.otg_port = 1;
> >             musb->xceiv->host = &hcd->self;
> > -           hcd->power_budget = 2 * (plat->power ? : 250);
> > +           if (plat->board_data)
> > +                   hcd->power_budget =
> > +                           2 * (plat->board_data->power ? : 250);
> 
>     Shouldn't it be:
> 
> +
> +             hcd->power_budget = 2 * (plat->board_data &&
> +                                      plat->board_data->power ?
> +                                      plat->board_data->power : 250);

But the next patch on 'extvbus' would again need a check for
' plat->board_data ' so added both in same 'if' check.

-Ajay
> 
> WBR, Sergei

--
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

Reply via email to