RE: [PATCH 2/8] musb: Get power (mA) from board data

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi,
> -----Original Message-----
> From: Sergei Shtylyov [mailto:sshtylyov@xxxxxxxxxxxxx]
> Sent: Tuesday, November 17, 2009 9:12 PM
> To: Gupta, Ajay Kumar
> Cc: linux-usb@xxxxxxxxxxxxxxx; davinci-linux-open-
> source@xxxxxxxxxxxxxxxxxxxx; cooloney@xxxxxxxxxx; felipe.balbi@xxxxxxxxx;
> linux-omap@xxxxxxxxxxxxxxx; 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 <ajay.gupta@xxxxxx>
> 
> > 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-usb" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux