RE: [PATCH 1/6] lp8727_charger: change description aboutcompany

[Kim, Milo]

> -----Original Message-----
> From: Alan Cox [mailto:alan@xxxxxxxxxxxxxxxxxxx]
> Sent: Friday, January 27, 2012 8:43 PM
> To: Kim, Milo
> Cc: Anton Vorontsov; linux-kernel@xxxxxxxxxxxxxxx
> Subject: Re: [PATCH 1/6] lp8727_charger: change description about
> company
> 
> On Thu, 26 Jan 2012 22:58:15 -0800
> "Kim, Milo" <Milo.Kim@xxxxxx> wrote:
> 
> > TI acquired National semiconductor at 2011.
> > : National semiconductor -> Texas Instruments
> >
> > Add driver information in the header file.
> >
> > Signed-off-by: Milo(Woogyom) Kim <milo.kim@xxxxxx>
> > ---
> >  drivers/power/lp8727_charger.c |    4 ++--
> >  include/linux/lp8727.h         |    4 +++-
> >  2 files changed, 5 insertions(+), 3 deletions(-)
> >
> > diff --git a/drivers/power/lp8727_charger.c
> b/drivers/power/lp8727_charger.c
> > index b15b575..97be22e 100755
> > --- a/drivers/power/lp8727_charger.c
> > +++ b/drivers/power/lp8727_charger.c
> > @@ -1,7 +1,7 @@
> >  /*
> >   * Driver for LP8727 Micro/Mini USB IC with intergrated charger
> >   *
> > - *			Copyright (C) 2011 National Semiconductor
> > + *			Copyright (C) 2011 Texas Instruments
> >   *
> >   * This program is free software; you can redistribute it and/or
> modify
> >   * it under the terms of the GNU General Public License version 2 as
> > @@ -488,7 +488,7 @@ static void __exit lp8727_exit(void)
> >  module_init(lp8727_init);
> >  module_exit(lp8727_exit);
> >
> > -MODULE_DESCRIPTION("National Semiconductor LP8727 charger driver");
> > +MODULE_DESCRIPTION("Texas Instruments LP8727 charger driver");
> 
> This may confuse people into thinking only the current TI branded
> version
> works ? I wonder if it should give both names ?

OK, adding new company name is better. New patch has been sent.
Please refer to ' [PATCH v2] lp8727_charger: add company name and description'.
Thank you.

Best Regards,
Milo -


--
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/