2012/04/11 18:35, Felipe Balbi wrote:
> On Fri, Mar 16, 2012 at 01:10:12PM +0900, Shimoda, Yoshihiro wrote:
>> The previous code always set to USB_DEVICE_SELF_POWERED in GET_STATUS.
>> So, this patch adds set_selfpowered().
>>
>> Signed-off-by: Yoshihiro Shimoda <yoshihiro.shimoda.uh@xxxxxxxxxxx>
>> ---
>> drivers/usb/gadget/r8a66597-udc.c | 16 +++++++++++++++-
>> drivers/usb/gadget/r8a66597-udc.h | 1 +
>> 2 files changed, 16 insertions(+), 1 deletions(-)
>>
>> diff --git a/drivers/usb/gadget/r8a66597-udc.c b/drivers/usb/gadget/r8a66597-udc.c
>> index c4401e7..683b3d2 100644
>> --- a/drivers/usb/gadget/r8a66597-udc.c
>> +++ b/drivers/usb/gadget/r8a66597-udc.c
>> @@ -1170,7 +1170,7 @@ __acquires(r8a66597->lock)
>>
>> switch (ctrl->bRequestType & USB_RECIP_MASK) {
>> case USB_RECIP_DEVICE:
>> - status = 1 << USB_DEVICE_SELF_POWERED;
>> + status = r8a66597->device_status;
>> break;
>> case USB_RECIP_INTERFACE:
>> status = 0;
>> @@ -1800,11 +1800,25 @@ static int r8a66597_pullup(struct usb_gadget *gadget, int is_on)
>> return 0;
>> }
>>
>> +static int r8a66597_set_selfpowered(struct usb_gadget *gadget, int is_self)
>> +{
>> + struct r8a66597 *r8a66597 = gadget_to_r8a66597(gadget);
>> + unsigned long flags;
>
> compile warning here. Did you even test this patch ???
I'm sorry, I should have checked it carefully.
I will fix and submit soon.
Best regards,
Yoshihiro Shimoda
--
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
