Message ID | 20191024122224.11776-2-geert+renesas@glider.be (mailing list archive) |
---|---|
State | Mainlined |
Commit | b74f0456c120289b026701f849e0fd7de56fd2d7 |
Delegated to: | Geert Uytterhoeven |
Headers | show |
Series | gpio: renesas: Use proper irq_chip name | expand |
On Thu, Oct 24, 2019 at 2:22 PM Geert Uytterhoeven <geert+renesas@glider.be> wrote: > The irq_chip .name field should contain the device's class name, not the > instance's name. > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > --- > Untested due to lack of hardware. > .parent_device not filled in as this driver doesn't use Runtime PM. > > v2: > - No changes. Patch applied. Yours, Linus Walleij
pon., 4 lis 2019 o 16:10 Linus Walleij <linus.walleij@linaro.org> napisał(a): > > On Thu, Oct 24, 2019 at 2:22 PM Geert Uytterhoeven > <geert+renesas@glider.be> wrote: > > > The irq_chip .name field should contain the device's class name, not the > > instance's name. > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > --- > > Untested due to lack of hardware. > > .parent_device not filled in as this driver doesn't use Runtime PM. > > > > v2: > > - No changes. > > Patch applied. > Oops I already have those in my tree. I'll back them out before the next PR. Bart > Yours, > Linus Walleij
On Mon, Nov 4, 2019 at 4:21 PM Bartosz Golaszewski <bgolaszewski@baylibre.com> wrote: > pon., 4 lis 2019 o 16:10 Linus Walleij <linus.walleij@linaro.org> napisał(a): > > > > On Thu, Oct 24, 2019 at 2:22 PM Geert Uytterhoeven > > <geert+renesas@glider.be> wrote: > > > > > The irq_chip .name field should contain the device's class name, not the > > > instance's name. > > > > > > Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> > > > --- > > > Untested due to lack of hardware. > > > .parent_device not filled in as this driver doesn't use Runtime PM. > > > > > > v2: > > > - No changes. > > > > Patch applied. > > > > Oops I already have those in my tree. I'll back them out before the next PR. I noticed they were in your tree later, so I backed them out of mine, just keep your commit history. Linus
diff --git a/drivers/gpio/gpio-em.c b/drivers/gpio/gpio-em.c index 620f25b7efb402c1..cc4416310dc9a6fe 100644 --- a/drivers/gpio/gpio-em.c +++ b/drivers/gpio/gpio-em.c @@ -326,7 +326,7 @@ static int em_gio_probe(struct platform_device *pdev) gpio_chip->ngpio = ngpios; irq_chip = &p->irq_chip; - irq_chip->name = name; + irq_chip->name = "gpio-em"; irq_chip->irq_mask = em_gio_irq_disable; irq_chip->irq_unmask = em_gio_irq_enable; irq_chip->irq_set_type = em_gio_irq_set_type;
The irq_chip .name field should contain the device's class name, not the instance's name. Signed-off-by: Geert Uytterhoeven <geert+renesas@glider.be> --- Untested due to lack of hardware. .parent_device not filled in as this driver doesn't use Runtime PM. v2: - No changes. --- drivers/gpio/gpio-em.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)