* [PATCH 1/6] lp8727_charger: change description about company
@ 2012-01-27 6:58 Kim, Milo
2012-01-27 11:42 ` Alan Cox
0 siblings, 1 reply; 3+ messages in thread
From: Kim, Milo @ 2012-01-27 6:58 UTC (permalink / raw)
To: Anton Vorontsov; +Cc: linux-kernel
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
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH 1/6] lp8727_charger: change description about company
2012-01-27 6:58 [PATCH 1/6] lp8727_charger: change description about company Kim, Milo
@ 2012-01-27 11:42 ` Alan Cox
2012-01-30 1:28 ` Kim, Milo
0 siblings, 1 reply; 3+ messages in thread
From: Alan Cox @ 2012-01-27 11:42 UTC (permalink / raw)
To: Kim, Milo; +Cc: Anton Vorontsov, linux-kernel
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 ?
^ permalink raw reply [flat|nested] 3+ messages in thread
* RE: [PATCH 1/6] lp8727_charger: change description about company
2012-01-27 11:42 ` Alan Cox
@ 2012-01-30 1:28 ` Kim, Milo
0 siblings, 0 replies; 3+ messages in thread
From: Kim, Milo @ 2012-01-30 1:28 UTC (permalink / raw)
To: Alan Cox; +Cc: Anton Vorontsov, linux-kernel
> -----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 -
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-01-30 1:28 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-01-27 6:58 [PATCH 1/6] lp8727_charger: change description about company Kim, Milo
2012-01-27 11:42 ` Alan Cox
2012-01-30 1:28 ` Kim, Milo
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).