On Thu, 3 May 2012 15:56:18 +0300, Felipe Balbi <balbi@xxxxxx> wrote: > On Wed, May 02, 2012 at 05:13:40PM +0300, Alexander Shishkin wrote: > > In order to function as a "newstyle" udc, move endpoint (de-)initialization > > so first move the endpoint initialization part, then change to newstyle. > It'll be easier to review. Won't it start looking like "Do $X so that the next patch makes sense", which doesn't seem to be a valid reason to change things? And as far as I'm corncerned, it's good to have a reason *why* things are changed in patch descriptions. But you're the maintainer, of course. :) Regards, -- Alex -- 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