Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]

Groups > linux.kernel > #1432948 > unrolled thread

[PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table

Started byKeerthy <j-keerthy@ti.com>
First post2016-06-28 14:40 +0200
Last post2016-07-04 12:00 +0200
Articles 5 — 3 participants

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.

Contents

  [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table Keerthy <j-keerthy@ti.com> - 2016-06-28 14:40 +0200
    Re: [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table Linus Walleij <linus.walleij@linaro.org> - 2016-06-29 11:00 +0200
      Re: [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table Keerthy <a0393675@ti.com> - 2016-06-29 11:40 +0200
    Re: [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table Linus Walleij <linus.walleij@linaro.org> - 2016-07-04 12:00 +0200
      Re: [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table Keerthy <a0393675@ti.com> - 2016-07-04 12:00 +0200

#1432948 — [PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table

FromKeerthy <j-keerthy@ti.com>
Date2016-06-28 14:40 +0200
Subject[PATCH v6 4/6] gpio: tps65218-gpio: Add platform_device_id table
Message-ID<rP0t4-730-3@gated-at.bofh.it>
platform_device_id table is needed for adding the tps65218-gpio
module to the mfd_cell array.

Signed-off-by: Keerthy <j-keerthy@ti.com>
---

Changes in v4:

  * Retained the compatible strings in order to interrogate the DT in the
    normal way.

 drivers/gpio/gpio-tps65218.c | 7 +++++++
 1 file changed, 7 insertions(+)

diff --git a/drivers/gpio/gpio-tps65218.c b/drivers/gpio/gpio-tps65218.c
index 0f9d9bd..0961f3a 100644
--- a/drivers/gpio/gpio-tps65218.c
+++ b/drivers/gpio/gpio-tps65218.c
@@ -231,6 +231,12 @@ static const struct of_device_id tps65218_dt_match[] = {
 };
 MODULE_DEVICE_TABLE(of, tps65218_dt_match);
 
+static const struct platform_device_id tps65218_gpio_id_table[] = {
+	{ "tps65218-gpio", },
+	{ /* sentinel */ }
+};
+MODULE_DEVICE_TABLE(platform, tps65218_gpio_id_table);
+
 static struct platform_driver tps65218_gpio_driver = {
 	.driver = {
 		.name = "tps65218-gpio",
@@ -238,6 +244,7 @@ static struct platform_driver tps65218_gpio_driver = {
 	},
 	.probe = tps65218_gpio_probe,
 	.remove = tps65218_gpio_remove,
+	.id_table = tps65218_gpio_id_table,
 };
 
 module_platform_driver(tps65218_gpio_driver);
-- 
1.9.1

[toc] | [next] | [standalone]

#1433583

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-06-29 11:00 +0200
Message-ID<rPjvI-1Zc-9@gated-at.bofh.it>
In reply to#1432948
On Tue, Jun 28, 2016 at 2:30 PM, Keerthy <j-keerthy@ti.com> wrote:

> platform_device_id table is needed for adding the tps65218-gpio
> module to the mfd_cell array.
>
> Signed-off-by: Keerthy <j-keerthy@ti.com>
> ---
>
> Changes in v4:
>
>   * Retained the compatible strings in order to interrogate the DT in the
>     normal way.

So do I just apply this or do you need it applied through another tree?

It looks standalone, should I just merge it?

Acked-by: Linus Walleij <linus.walleij@linaro.org>

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]

#1433614

FromKeerthy <a0393675@ti.com>
Date2016-06-29 11:40 +0200
Message-ID<rPk8q-2rm-5@gated-at.bofh.it>
In reply to#1433583

On Wednesday 29 June 2016 02:26 PM, Linus Walleij wrote:
> On Tue, Jun 28, 2016 at 2:30 PM, Keerthy <j-keerthy@ti.com> wrote:
>
>> platform_device_id table is needed for adding the tps65218-gpio
>> module to the mfd_cell array.
>>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>> ---
>>
>> Changes in v4:
>>
>>    * Retained the compatible strings in order to interrogate the DT in the
>>      normal way.
>
> So do I just apply this or do you need it applied through another tree?
>
> It looks standalone, should I just merge it?
>
> Acked-by: Linus Walleij <linus.walleij@linaro.org>

Thanks Linus. I guess this can be merged by you. Is that fine Lee Jones?


>
> Yours,
> Linus Walleij
>

[toc] | [prev] | [next] | [standalone]

#1436185

FromLinus Walleij <linus.walleij@linaro.org>
Date2016-07-04 12:00 +0200
Message-ID<rR8Pv-419-3@gated-at.bofh.it>
In reply to#1432948
On Tue, Jun 28, 2016 at 2:30 PM, Keerthy <j-keerthy@ti.com> wrote:

> platform_device_id table is needed for adding the tps65218-gpio
> module to the mfd_cell array.
>
> Signed-off-by: Keerthy <j-keerthy@ti.com>

Patch applied.

Yours,
Linus Walleij

[toc] | [prev] | [next] | [standalone]

#1436226

FromKeerthy <a0393675@ti.com>
Date2016-07-04 12:00 +0200
Message-ID<rR8Pw-419-45@gated-at.bofh.it>
In reply to#1436185

On Monday 04 July 2016 03:20 PM, Linus Walleij wrote:
> On Tue, Jun 28, 2016 at 2:30 PM, Keerthy <j-keerthy@ti.com> wrote:
>
>> platform_device_id table is needed for adding the tps65218-gpio
>> module to the mfd_cell array.
>>
>> Signed-off-by: Keerthy <j-keerthy@ti.com>
>
> Patch applied.

Thanks Linus.

>
> Yours,
> Linus Walleij
>

[toc] | [prev] | [standalone]

Back to top | Article view | linux.kernel

csiph-web