Hi, On Tue, Jul 26, 2011, Nils Faerber wrote: > Am 26.07.2011 16:09, schrieb Prasad Bhat: > > --- /dev/null > > +++ b/audio/vdp.c > > @@ -0,0 +1,2303 @@ > > +/* > > + * > > + * BlueZ - Bluetooth protocol stack for Linux > > + * > > + * Copyright (C) 2006-2010 Nokia Corporation > > + * Copyright (C) 2004-2010 Marcel Holtmann <marcel@xxxxxxxxxxxx> > > > I think this is your code, is it? > > Why do you assign the copyright to Nokia and Marcel then? Looking at the code it seems to a large part copy-paste + minor modifications from a2dp.c, so to that extent those copyright declarations would be correct. However, I would also have expected Prasad to add either his personal copyright or the copyright of the company that he's working for. Couple of other notes: - The patch is huge. Any chance of splitting it up into smaller chunks? - Since there's so much in common with a2dp.c it'd make sense to attempt some refactoring. - Now that audio/*.c does video too we might consider renaming the audio plugin and directory to something else. Johan -- To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html