Re: [PATCH] serial: sirf: update copyright years to 2014

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

 



On Wed, Feb 12, 2014 at 10:47:42PM +0800, Barry Song wrote:
> 2014-02-12 22:37 GMT+08:00 Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>:
> > On Wed, Feb 12, 2014 at 10:26:32PM +0800, Barry Song wrote:
> >> 2014-02-12 22:19 GMT+08:00 Greg KH <gregkh@xxxxxxxxxxxxxxxxxxx>:
> >> > On Wed, Feb 12, 2014 at 10:08:44PM +0800, Barry Song wrote:
> >> >> From: Barry Song <Baohua.Song@xxxxxxx>
> >> >>
> >> >> Signed-off-by: Barry Song <Baohua.Song@xxxxxxx>
> >> >> ---
> >> >>  drivers/tty/serial/sirfsoc_uart.c |    3 ++-
> >> >>  1 file changed, 2 insertions(+), 1 deletion(-)
> >> >>
> >> >> diff --git a/drivers/tty/serial/sirfsoc_uart.c b/drivers/tty/serial/sirfsoc_uart.c
> >> >> index 49a2ffd..7079b5c 100644
> >> >> --- a/drivers/tty/serial/sirfsoc_uart.c
> >> >> +++ b/drivers/tty/serial/sirfsoc_uart.c
> >> >> @@ -1,7 +1,8 @@
> >> >>  /*
> >> >>   * Driver for CSR SiRFprimaII onboard UARTs.
> >> >>   *
> >> >> - * Copyright (c) 2011 Cambridge Silicon Radio Limited, a CSR plc group company.
> >> >> + * Copyright (c) 2011 - 2014 Cambridge Silicon Radio Limited, a CSR plc group
> >> >> + * company.
> >> >
> >> > I do not see contributions from csr for all of these years, so no, this
> >> > is not allowed.  Please go talk with your corporate lawyer if you have
> >> > questions about how/when to update the copyright of your files, as I
> >> > don't think they would agree that this change is correct.
> >>
> >> Greg, as i run "git log sirfsoc_uart.c", there are commits from csr
> >> for touched file from 2011 to 2014:
> >>
> >> 2011:
> >> commit 161e773cbd0c3d1b5b8cc00602e1f72de61ed4f7
> >> Author: Rong Wang <Rong.Wang@xxxxxxx>
> >> Date:   Thu Nov 17 23:17:04 2011 +0800
> >>
> >> 2012:
> >> commit 5425e03f97d1e5847372aae0b895d8d1c9bf2741
> >> Author: Barry Song <Baohua.Song@xxxxxxx>
> >> Date:   Tue Dec 25 17:32:04 2012 +0800
> >>
> >> 2013:
> >> commit ac4ce718893c546f7a2d34ab55a8f75842399f86
> >> Author: Barry Song <Baohua.Song@xxxxxxx>
> >> Date:   Wed Jan 16 14:49:27 2013 +0800
> >> ...
> >> commit 59f8a62c25b9c2a53e7c359ba9ca611639a4c0b0
> >> Author: Qipan Li <Qipan.Li@xxxxxxx>
> >> Date:   Sat Sep 21 09:02:10 2013 +0800
> >>
> >>
> >> 2014:
> >> commit 388faf9ffdaf92c81243514a2dd4c6ce04d28874
> >> Author: Qipan Li <Qipan.Li@xxxxxxx>
> >> Date:   Fri Jan 3 15:44:07 2014 +0800
> >>
> >> commit df8d4aa0d84995bf7fb8d8a978a0d67fff6ca53a
> >> Author: Qipan Li <Qipan.Li@xxxxxxx>
> >> Date:   Fri Jan 3 15:44:08 2014 +0800
> >
> > And do all of these changes fall under the "copyright is allowed to be
> > updated" rule that your corporation follows?  Again, please talk to your
> > corporate lawyer about this, as the rules I have been told to follow do
> > not seem to match up here.  I'll be glad to discuss this with your
> > lawyer through email if they have further questions.
> 
> ok. it seems it is very confused now. i was told we can update
> copyright years once we have any valid change in one module.
> i will talk with the lawyer of csr. BTW, would you like to share your
> idea why it is not ok here since we have changes for all of 2011,
> 2012, 2013 and 2014. i am not debating this with you, i just want to
> collect your input so that i can output to csr lawyer and customers.

Well, your 2012 change doesn't seem to be "significant" to warrent a
normal copyright update, but the update is usually "less strict" than an
original mark, so that might be ok, now that I review these closer.

But I'd still prefer to get the opinion of your lawyer about this.

thanks,

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-serial" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html




[Index of Archives]     [Kernel Newbies]     [Security]     [Netfilter]     [Bugtraq]     [Linux PPP]     [Linux FS]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Linmodem]     [Device Mapper]     [Linux Kernel for ARM]

  Powered by Linux