diff mbox

ARM: ux500: rename ab8500 to abx500 for hwmon driver

Message ID 1359531747-3636-1-git-send-email-hongbo.zhang@linaro.org (mailing list archive)
State New, archived
Headers show

Commit Message

Hongbo Zhang Jan. 30, 2013, 7:42 a.m. UTC
We are using a generic abx500 hwmon layer, so rename specific ab8500 to generic
abx500 for hwmon device and driver matching.

Signed-off-by: Hongbo Zhang <hongbo.zhang@linaro.org>
---
 drivers/mfd/ab8500-core.c | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

Comments

Hongbo Zhang Jan. 30, 2013, 7:56 a.m. UTC | #1
Hi Linus Walleij,
This is for upstreaming AB8500 hwmon, see another Email:
[PATCH] hwmon: add ST-Ericsson ABX500 hwmon driver


On 30 January 2013 15:42, Hongbo Zhang <hongbo.zhang@linaro.org> wrote:
> We are using a generic abx500 hwmon layer, so rename specific ab8500 to generic
> abx500 for hwmon device and driver matching.
>
> Signed-off-by: Hongbo Zhang <hongbo.zhang@linaro.org>
> ---
>  drivers/mfd/ab8500-core.c | 6 +++---
>  1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/mfd/ab8500-core.c b/drivers/mfd/ab8500-core.c
> index e1650ba..579753f 100644
> --- a/drivers/mfd/ab8500-core.c
> +++ b/drivers/mfd/ab8500-core.c
> @@ -922,7 +922,7 @@ static struct resource ab8505_iddet_resources[] = {
>
>  static struct resource ab8500_temp_resources[] = {
>         {
> -               .name  = "AB8500_TEMP_WARM",
> +               .name  = "ABX500_TEMP_WARM",
>                 .start = AB8500_INT_TEMP_WARM,
>                 .end   = AB8500_INT_TEMP_WARM,
>                 .flags = IORESOURCE_IRQ,
> @@ -998,8 +998,8 @@ static struct mfd_cell abx500_common_devs[] = {
>                 .of_compatible = "stericsson,ab8500-denc",
>         },
>         {
> -               .name = "ab8500-temp",
> -               .of_compatible = "stericsson,ab8500-temp",
> +               .name = "abx500-temp",
> +               .of_compatible = "stericsson,abx500-temp",
>                 .num_resources = ARRAY_SIZE(ab8500_temp_resources),
>                 .resources = ab8500_temp_resources,
>         },
> --
> 1.8.0
>
Linus Walleij Jan. 30, 2013, 9:45 a.m. UTC | #2
On Wed, Jan 30, 2013 at 8:42 AM, Hongbo Zhang <hongbo.zhang@linaro.org> wrote:

> We are using a generic abx500 hwmon layer, so rename specific ab8500 to generic
> abx500 for hwmon device and driver matching.
>
> Signed-off-by: Hongbo Zhang <hongbo.zhang@linaro.org>

You need to have the ACK:ed by the MFD subsystem maintainer
so that Günther can merge it later, along with the HWMON patch.

Also combine them into a series, this one as 1/2 and the other
patch as 2/2.

And remember to send both to the relevant maintainers
(see MAINTAINERS file):

HARDWARE MONITORING
M:      Jean Delvare <khali@linux-fr.org>
M:      Guenter Roeck <linux@roeck-us.net>

MULTIFUNCTION DEVICES (MFD)
M:      Samuel Ortiz <sameo@linux.intel.com>

Both patches should also go into the linux-kernel@vger.kernel.org mailing
list "LKML".

Thanks!
Linus Walleij
Hongbo Zhang Jan. 30, 2013, 9:52 a.m. UTC | #3
On 30 January 2013 17:45, Linus Walleij <linus.walleij@linaro.org> wrote:
> On Wed, Jan 30, 2013 at 8:42 AM, Hongbo Zhang <hongbo.zhang@linaro.org> wrote:
>
>> We are using a generic abx500 hwmon layer, so rename specific ab8500 to generic
>> abx500 for hwmon device and driver matching.
>>
>> Signed-off-by: Hongbo Zhang <hongbo.zhang@linaro.org>
>
> You need to have the ACK:ed by the MFD subsystem maintainer
> so that Günther can merge it later, along with the HWMON patch.
>
> Also combine them into a series, this one as 1/2 and the other
> patch as 2/2.
>
> And remember to send both to the relevant maintainers
> (see MAINTAINERS file):
>
> HARDWARE MONITORING
> M:      Jean Delvare <khali@linux-fr.org>
> M:      Guenter Roeck <linux@roeck-us.net>
>
> MULTIFUNCTION DEVICES (MFD)
> M:      Samuel Ortiz <sameo@linux.intel.com>
>
> Both patches should also go into the linux-kernel@vger.kernel.org mailing
> list "LKML".
>
Thanks for your instructions, will follow them.

> Thanks!
> Linus Walleij
diff mbox

Patch

diff --git a/drivers/mfd/ab8500-core.c b/drivers/mfd/ab8500-core.c
index e1650ba..579753f 100644
--- a/drivers/mfd/ab8500-core.c
+++ b/drivers/mfd/ab8500-core.c
@@ -922,7 +922,7 @@  static struct resource ab8505_iddet_resources[] = {
 
 static struct resource ab8500_temp_resources[] = {
 	{
-		.name  = "AB8500_TEMP_WARM",
+		.name  = "ABX500_TEMP_WARM",
 		.start = AB8500_INT_TEMP_WARM,
 		.end   = AB8500_INT_TEMP_WARM,
 		.flags = IORESOURCE_IRQ,
@@ -998,8 +998,8 @@  static struct mfd_cell abx500_common_devs[] = {
 		.of_compatible = "stericsson,ab8500-denc",
 	},
 	{
-		.name = "ab8500-temp",
-		.of_compatible = "stericsson,ab8500-temp",
+		.name = "abx500-temp",
+		.of_compatible = "stericsson,abx500-temp",
 		.num_resources = ARRAY_SIZE(ab8500_temp_resources),
 		.resources = ab8500_temp_resources,
 	},