RE: [PATCH v2 01/21] usb/gadget: multi: fix error return code in cdc_do_config()

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

 



On Thursday, July 25, 2013 7:33 PM Felipe Balbi wrote:
> On Fri, Jul 19, 2013 at 12:28:13PM +0200, Andrzej Pietrasiewicz wrote:
> > Fix to return a negative error code from the error handling case
> > instead of 0, as returned elsewhere in this function.
> >
> > Introduced by commit 59835a (usb: gadget: multi: use function
> > framework for ACM.)
> >
> > Signed-off-by: Andrzej Pietrasiewicz <andrzej.p@xxxxxxxxxxx>
> > Signed-off-by: Kyungmin Park <kyungmin.park@xxxxxxxxxxx>
> 
> you really don't want to send fixes and new code in the same series.
> Please split this series properly into two series (even if the fixes
> series is a single patch), and resend with Michal's ack in place so I can
> apply.
> 

Done.

The first 2 of the 21 were actually bugfixes, now sent in a separate series,
with ACKs.

I've also implemented changes resulting from Michal's comments (thank you,
Michal!)
but since these are new features meant for 3.12, I will post the three
configfs
support series (mass_storage, acm_ms, multi) in the beginning of the next
week
after I've done some testing.

Thanks,

Andrzej



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