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

From: Alan Cox
Date: Fri Jan 27 2012 - 06:41:13 EST


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