[PATCH 1/6] lp8727_charger: change description about company

[PATCH 1/6] lp8727_charger: change description about company

[Kim, Milo]

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@ti.com>
---
 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");
 MODULE_AUTHOR
     ("Woogyom Kim <milo.kim@ti.com>, Daniel Jeong <daniel.jeong@ti.com>");
 MODULE_LICENSE("GPL");
diff --git a/include/linux/lp8727.h b/include/linux/lp8727.h
index d21fa28..43d4242 100644
--- a/include/linux/lp8727.h
+++ b/include/linux/lp8727.h
@@ -1,5 +1,7 @@
 /*
- *			Copyright (C) 2011 National Semiconductor
+ * LP8727 Micro/Mini USB IC with intergrated charger
+ *
+ *			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
-- 
1.7.4.1


Best Regards,
Milo (Woogyom) Kim
Texas Instruments Incorporated

Re: [PATCH 1/6] lp8727_charger: change description about company

[Alan Cox]

On Thu, 26 Jan 2012 22:58:15 -0800
"Kim, Milo" <Milo.Kim@ti.com> 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@ti.com>
> ---
>  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 ?

RE: [PATCH 1/6] lp8727_charger: change description about company

[Kim, Milo]

> -----Original Message-----
> From: Alan Cox [mailto:alan@lxorguk.ukuu.org.uk]
> Sent: Friday, January 27, 2012 8:43 PM
> To: Kim, Milo
> Cc: Anton Vorontsov; linux-kernel@vger.kernel.org
> Subject: Re: [PATCH 1/6] lp8727_charger: change description about
> company
> 
> On Thu, 26 Jan 2012 22:58:15 -0800
> "Kim, Milo" <Milo.Kim@ti.com> 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@ti.com>
> > ---
> >  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 -