* [PATCH 1/2] irqchip: zevio: use irq_data_get_chip_type
@ 2015-12-30 14:16 Geliang Tang
2015-12-30 14:16 ` [PATCH 2/2] gpio: davinci: " Geliang Tang
2015-12-30 17:33 ` [tip:irq/core] irqchip/zevio: Use irq_data_get_chip_type() helper tip-bot for Geliang Tang
0 siblings, 2 replies; 7+ messages in thread
From: Geliang Tang @ 2015-12-30 14:16 UTC (permalink / raw)
To: Thomas Gleixner, Jason Cooper, Marc Zyngier; +Cc: Geliang Tang, linux-kernel
Use irq_data_get_chip_type() instead of container_of().
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/irqchip/irq-zevio.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-zevio.c b/drivers/irqchip/irq-zevio.c
index 4c48fa8..cb9d8ec 100644
--- a/drivers/irqchip/irq-zevio.c
+++ b/drivers/irqchip/irq-zevio.c
@@ -43,8 +43,7 @@ static void __iomem *zevio_irq_io;
static void zevio_irq_ack(struct irq_data *irqd)
{
struct irq_chip_generic *gc = irq_data_get_irq_chip_data(irqd);
- struct irq_chip_regs *regs =
- &container_of(irqd->chip, struct irq_chip_type, chip)->regs;
+ struct irq_chip_regs *regs = &irq_data_get_chip_type(irqd)->regs;
readl(gc->reg_base + regs->ack);
}
--
2.5.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] gpio: davinci: use irq_data_get_chip_type
2015-12-30 14:16 [PATCH 1/2] irqchip: zevio: use irq_data_get_chip_type Geliang Tang
@ 2015-12-30 14:16 ` Geliang Tang
2016-01-27 13:56 ` Linus Walleij
2016-01-28 14:29 ` Linus Walleij
2015-12-30 17:33 ` [tip:irq/core] irqchip/zevio: Use irq_data_get_chip_type() helper tip-bot for Geliang Tang
1 sibling, 2 replies; 7+ messages in thread
From: Geliang Tang @ 2015-12-30 14:16 UTC (permalink / raw)
To: Linus Walleij, Alexandre Courbot; +Cc: Geliang Tang, linux-gpio, linux-kernel
Use irq_data_get_chip_type() instead of container_of().
Signed-off-by: Geliang Tang <geliangtang@163.com>
---
drivers/gpio/gpio-davinci.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/gpio/gpio-davinci.c b/drivers/gpio/gpio-davinci.c
index 65ebaef..aaa1a249 100644
--- a/drivers/gpio/gpio-davinci.c
+++ b/drivers/gpio/gpio-davinci.c
@@ -435,8 +435,7 @@ static struct irq_chip *davinci_gpio_get_irq_chip(unsigned int irq)
{
static struct irq_chip_type gpio_unbanked;
- gpio_unbanked = *container_of(irq_get_chip(irq),
- struct irq_chip_type, chip);
+ gpio_unbanked = *irq_data_get_chip_type(irq_get_irq_data(irq));
return &gpio_unbanked.chip;
};
--
2.5.0
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [tip:irq/core] irqchip/zevio: Use irq_data_get_chip_type() helper
2015-12-30 14:16 [PATCH 1/2] irqchip: zevio: use irq_data_get_chip_type Geliang Tang
2015-12-30 14:16 ` [PATCH 2/2] gpio: davinci: " Geliang Tang
@ 2015-12-30 17:33 ` tip-bot for Geliang Tang
1 sibling, 0 replies; 7+ messages in thread
From: tip-bot for Geliang Tang @ 2015-12-30 17:33 UTC (permalink / raw)
To: linux-tip-commits
Cc: mingo, hpa, geliangtang, jason, marc.zyngier, tglx, linux-kernel
Commit-ID: 1fd9a71076ccbcf731cf02408122600a6f2b5d17
Gitweb: http://git.kernel.org/tip/1fd9a71076ccbcf731cf02408122600a6f2b5d17
Author: Geliang Tang <geliangtang@163.com>
AuthorDate: Wed, 30 Dec 2015 22:16:37 +0800
Committer: Thomas Gleixner <tglx@linutronix.de>
CommitDate: Wed, 30 Dec 2015 18:29:02 +0100
irqchip/zevio: Use irq_data_get_chip_type() helper
Use irq_data_get_chip_type() instead of container_of().
Signed-off-by: Geliang Tang <geliangtang@163.com>
Cc: Jason Cooper <jason@lakedaemon.net>
Cc: Marc Zyngier <marc.zyngier@arm.com>
Link: http://lkml.kernel.org/r/4cc3a3a7a74c7a1894892a85aa7eabbd1534fe96.1451484758.git.geliangtang@163.com
Signed-off-by: Thomas Gleixner <tglx@linutronix.de>
---
drivers/irqchip/irq-zevio.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/irqchip/irq-zevio.c b/drivers/irqchip/irq-zevio.c
index 4c48fa8..cb9d8ec 100644
--- a/drivers/irqchip/irq-zevio.c
+++ b/drivers/irqchip/irq-zevio.c
@@ -43,8 +43,7 @@ static void __iomem *zevio_irq_io;
static void zevio_irq_ack(struct irq_data *irqd)
{
struct irq_chip_generic *gc = irq_data_get_irq_chip_data(irqd);
- struct irq_chip_regs *regs =
- &container_of(irqd->chip, struct irq_chip_type, chip)->regs;
+ struct irq_chip_regs *regs = &irq_data_get_chip_type(irqd)->regs;
readl(gc->reg_base + regs->ack);
}
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] gpio: davinci: use irq_data_get_chip_type
2015-12-30 14:16 ` [PATCH 2/2] gpio: davinci: " Geliang Tang
@ 2016-01-27 13:56 ` Linus Walleij
2016-01-27 16:35 ` Grygorii Strashko
2016-01-28 14:29 ` Linus Walleij
1 sibling, 1 reply; 7+ messages in thread
From: Linus Walleij @ 2016-01-27 13:56 UTC (permalink / raw)
To: Geliang Tang, Grygorii Strashko, Santosh Shilimkar
Cc: Alexandre Courbot, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org
On Wed, Dec 30, 2015 at 3:16 PM, Geliang Tang <geliangtang@163.com> wrote:
> Use irq_data_get_chip_type() instead of container_of().
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
> ---
> drivers/gpio/gpio-davinci.c | 3 +--
> 1 file changed, 1 insertion(+), 2 deletions(-)
>
> diff --git a/drivers/gpio/gpio-davinci.c b/drivers/gpio/gpio-davinci.c
> index 65ebaef..aaa1a249 100644
> --- a/drivers/gpio/gpio-davinci.c
> +++ b/drivers/gpio/gpio-davinci.c
> @@ -435,8 +435,7 @@ static struct irq_chip *davinci_gpio_get_irq_chip(unsigned int irq)
> {
> static struct irq_chip_type gpio_unbanked;
>
> - gpio_unbanked = *container_of(irq_get_chip(irq),
> - struct irq_chip_type, chip);
> + gpio_unbanked = *irq_data_get_chip_type(irq_get_irq_data(irq));
>
> return &gpio_unbanked.chip;
> };
Grygorii/Santosh, can you ACK/comment on this patch?
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] gpio: davinci: use irq_data_get_chip_type
2016-01-27 13:56 ` Linus Walleij
@ 2016-01-27 16:35 ` Grygorii Strashko
2016-01-27 19:03 ` santosh shilimkar
0 siblings, 1 reply; 7+ messages in thread
From: Grygorii Strashko @ 2016-01-27 16:35 UTC (permalink / raw)
To: Linus Walleij, Geliang Tang, Santosh Shilimkar, Sekhar Nori
Cc: Alexandre Courbot, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org
On 01/27/2016 03:56 PM, Linus Walleij wrote:
> On Wed, Dec 30, 2015 at 3:16 PM, Geliang Tang <geliangtang@163.com> wrote:
>
>> Use irq_data_get_chip_type() instead of container_of().
>>
>> Signed-off-by: Geliang Tang <geliangtang@163.com>
>> ---
>> drivers/gpio/gpio-davinci.c | 3 +--
>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>
>> diff --git a/drivers/gpio/gpio-davinci.c b/drivers/gpio/gpio-davinci.c
>> index 65ebaef..aaa1a249 100644
>> --- a/drivers/gpio/gpio-davinci.c
>> +++ b/drivers/gpio/gpio-davinci.c
>> @@ -435,8 +435,7 @@ static struct irq_chip *davinci_gpio_get_irq_chip(unsigned int irq)
>> {
>> static struct irq_chip_type gpio_unbanked;
>>
>> - gpio_unbanked = *container_of(irq_get_chip(irq),
>> - struct irq_chip_type, chip);
>> + gpio_unbanked = *irq_data_get_chip_type(irq_get_irq_data(irq));
>>
>> return &gpio_unbanked.chip;
>> };
>
> Grygorii/Santosh, can you ACK/comment on this patch?
>
Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
--
regards,
-grygorii
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] gpio: davinci: use irq_data_get_chip_type
2016-01-27 16:35 ` Grygorii Strashko
@ 2016-01-27 19:03 ` santosh shilimkar
0 siblings, 0 replies; 7+ messages in thread
From: santosh shilimkar @ 2016-01-27 19:03 UTC (permalink / raw)
To: Grygorii Strashko, Linus Walleij, Geliang Tang, Santosh Shilimkar,
Sekhar Nori
Cc: Alexandre Courbot, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org
On 1/27/2016 8:35 AM, Grygorii Strashko wrote:
> On 01/27/2016 03:56 PM, Linus Walleij wrote:
>> On Wed, Dec 30, 2015 at 3:16 PM, Geliang Tang <geliangtang@163.com>
>> wrote:
>>
>>> Use irq_data_get_chip_type() instead of container_of().
>>>
>>> Signed-off-by: Geliang Tang <geliangtang@163.com>
>>> ---
>>> drivers/gpio/gpio-davinci.c | 3 +--
>>> 1 file changed, 1 insertion(+), 2 deletions(-)
>>>
>>> diff --git a/drivers/gpio/gpio-davinci.c b/drivers/gpio/gpio-davinci.c
>>> index 65ebaef..aaa1a249 100644
>>> --- a/drivers/gpio/gpio-davinci.c
>>> +++ b/drivers/gpio/gpio-davinci.c
>>> @@ -435,8 +435,7 @@ static struct irq_chip
>>> *davinci_gpio_get_irq_chip(unsigned int irq)
>>> {
>>> static struct irq_chip_type gpio_unbanked;
>>>
>>> - gpio_unbanked = *container_of(irq_get_chip(irq),
>>> - struct irq_chip_type, chip);
>>> + gpio_unbanked = *irq_data_get_chip_type(irq_get_irq_data(irq));
>>>
>>> return &gpio_unbanked.chip;
>>> };
>>
>> Grygorii/Santosh, can you ACK/comment on this patch?
>>
>
> Reviewed-by: Grygorii Strashko <grygorii.strashko@ti.com>
>
Acked-by: Santosh Shilimkar <ssantosh@kernel.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 2/2] gpio: davinci: use irq_data_get_chip_type
2015-12-30 14:16 ` [PATCH 2/2] gpio: davinci: " Geliang Tang
2016-01-27 13:56 ` Linus Walleij
@ 2016-01-28 14:29 ` Linus Walleij
1 sibling, 0 replies; 7+ messages in thread
From: Linus Walleij @ 2016-01-28 14:29 UTC (permalink / raw)
To: Geliang Tang
Cc: Alexandre Courbot, linux-gpio@vger.kernel.org,
linux-kernel@vger.kernel.org
On Wed, Dec 30, 2015 at 3:16 PM, Geliang Tang <geliangtang@163.com> wrote:
> Use irq_data_get_chip_type() instead of container_of().
>
> Signed-off-by: Geliang Tang <geliangtang@163.com>
Patch applied with Grygorii's and Santosh's tags.
Yours,
Linus Walleij
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2016-01-28 14:29 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-12-30 14:16 [PATCH 1/2] irqchip: zevio: use irq_data_get_chip_type Geliang Tang
2015-12-30 14:16 ` [PATCH 2/2] gpio: davinci: " Geliang Tang
2016-01-27 13:56 ` Linus Walleij
2016-01-27 16:35 ` Grygorii Strashko
2016-01-27 19:03 ` santosh shilimkar
2016-01-28 14:29 ` Linus Walleij
2015-12-30 17:33 ` [tip:irq/core] irqchip/zevio: Use irq_data_get_chip_type() helper tip-bot for Geliang Tang
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).