On Fri, 15 Jan 2016, Peter Senna wrote: > CC: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx> > > On Friday, January 15, 2016 18:56 CET, Joe Perches <joe@xxxxxxxxxxx> wrote: > > > On Fri, 2016-01-15 at 18:41 +0100, Peter Senna Tschudin wrote: > > > This patch fixes whitespace coding style issues that can be fixed > > > within a single line. > > > > trivia: > > > > > diff --git a/drivers/usb/misc/sisusbvga/sisusb.c b/drivers/usb/misc/sisusbvga/sisusb.c > > [] > > > @@ -549,7 +549,7 @@ static int sisusb_recv_bulk_msg(struct sisusb_usb_data *sisusb, int ep, int len, > > > } > > > > > > static int sisusb_send_packet(struct sisusb_usb_data *sisusb, int len, > > > - struct sisusb_packet *packet) > > > + struct sisusb_packet *packet) > > > > I think all of these should use indentation alignment > > to open parenthesis. > That was my original idea too, but I have the impression things are different for usb, at least for usb-host. Alan, can you comment on this continuation line style issue here? Different files use different alignment for continuation lines. I use the style I like (two extra tabs) for the code I write, but other people make different choices. Unless I happen to be the maintainer or an active contributor for the source file in question, my preferences don't matter much. Alan Stern -- 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
