RE: [RFC/PATCH 1/2] Add usb_endpoint_descriptor to be part of the struct usb_ep

From: Tanya Brokhman
Date: Thu Nov 11 2010 - 01:31:51 EST




-----Original Message-----
From: David Brown [mailto:davidb@xxxxxxxxxxxxxx]
Sent: Thursday, November 11, 2010 1:41 AM
To: Alan Stern
Cc: tlinder; USB list; David Brownell; Greg Kroah-Hartman; Michal
Nazarewicz; Andrew Morton; Kernel development list
Subject: Re: [RFC/PATCH 1/2] Add usb_endpoint_descriptor to be part of the
struct usb_ep

On Sun, Oct 03, 2010 at 12:35:07PM -0400, Alan Stern wrote:

> > Signed-off-by: Tatyana Linder <tlinder@xxxxxxxxxxxxxx>
> ...
>
> > diff --git a/drivers/usb/gadget/file_storage.c
b/drivers/usb/gadget/file_storage.c
> > index a857b7a..6264fba 100644
> > --- a/drivers/usb/gadget/file_storage.c
> > +++ b/drivers/usb/gadget/file_storage.c
> > @@ -3,6 +3,7 @@
> > *
> > * Copyright (C) 2003-2008 Alan Stern
> > * All rights reserved.
> > + * Copyright (C) 2010 Code Aurora Forum. All rights reserved.
> > *
> > * Redistribution and use in source and binary forms, with or without
> > * modification, are permitted provided that the following conditions
>
> Speaking for myself only, I don't think people should add their own
> copyright notices in files to which they have not made substantial
> changes. A little code motion and a small API adjustment don't seem
> like large enough changes to justify this. Some of the other files in
> this patch have even smaller changes!

BTW, we agree on this. People aren't supposed to be putting these
copyrights in, but they've gotten used to it up until now.

Tatyana, please be careful to not insert a Code Aurora Forum copyright
unless you change a significant portion of the file.

[Brokhman, Tanya] - Yes, this was already removed in the latest version of
the patch. Thanks!

David

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/