Re: [PATCH 1/2] i2c-i801: Handle multiple instances instead of keeping global state

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

 



On Sat, Oct 30, 2010 at 06:24:58PM +0200, Jean Delvare wrote:
> Hi David,
> 
> On Sat, 30 Oct 2010 14:47:56 +0100 (BST), David Woodhouse wrote:
> > 
> 
> An explanation why this change is needed would be nice.
> 
> > Signed-off-by: David Woodhouse <David.Woodhouse@xxxxxxxxx>
> > ---
> >   drivers/i2c/busses/i2c-i801.c |  329 ++++++++++++++++++++++-------------------
> >   1 files changed, 178 insertions(+), 151 deletions(-)
> > 
> > diff --git a/drivers/i2c/busses/i2c-i801.c b/drivers/i2c/busses/i2c-i801.c
> > index 59d6598..6e8c12c 100644
> > --- a/drivers/i2c/busses/i2c-i801.c
> > +++ b/drivers/i2c/busses/i2c-i801.c
> > @@ -1,8 +1,10 @@
> >   /*
> 
> Note that the patch got corrupted on the way: all leading spaces have
> been doubled. I had to fix it.
> 
> > -    Copyright (c) 1998 - 2002  Frodo Looijaard <frodol@xxxxxx>,
> > -    Philip Edelbrock <phil@xxxxxxxxxxxxx>, and Mark D. Studebaker
> > -    <mdsxyz123@xxxxxxxxx>
> > -    Copyright (C) 2007, 2008   Jean Delvare <khali@xxxxxxxxxxxx>
> > +    Copyright © 1998 - 2002  Frodo Looijaard <frodol@xxxxxx>,
> > +			     Philip Edelbrock <phil@xxxxxxxxxxxxx> and
> > +			     Mark D. Studebaker <mdsxyz123@xxxxxxxxx>
> > +    Copyright © 2007, 2008   Jean Delvare <khali@xxxxxxxxxxxx>
> > +    Copyright © 2010         Intel Corporation
> > +                             David Woodhouse <dwmw2@xxxxxxxxxxxxx>
> 
> I'd rather stick to (C). Using non-ASCII characters is asking for
> trouble, and this doesn't add value.

IIRC, and IANAL, (C) is not sufficient in certain areas (such as the UK)
to declare copyright. It needs to be either the word Copyright in full,
or the proper 'c' in circle. Both is of coures better.
 
[snip]

-- 
Ben

Q:      What's a light-year?
A:      One-third less calories than a regular year.

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


[Index of Archives]     [Linux GPIO]     [Linux SPI]     [Linux Hardward Monitoring]     [LM Sensors]     [Linux USB Devel]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux