From: Kefeng Wang <wangkefeng.wang@huawei.com> To: Rob Herring <robh+dt@kernel.org> Cc: Saravana Kannan <saravanak@google.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Frank Rowand <frowand.list@gmail.com>, <devicetree@vger.kernel.org>, Russell King <linux@armlinux.org.uk>, "Linus Walleij" <linus.walleij@linaro.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Ruizhe Lin <linruizhe@huawei.com> Subject: Re: [PATCH 3/3] amba: Properly handle device probe without IRQ domain Date: Wed, 25 Aug 2021 22:41:15 +0800 [thread overview] Message-ID: <d7a1d3a8-bda4-7a29-b66d-22ed4926f9c8@huawei.com> (raw) In-Reply-To: <CAL_Jsq+0rq6u5d7itETOnQWx_V+J3aP1m1Zgehi5QKVoKvdbvQ@mail.gmail.com> On 2021/8/25 20:33, Rob Herring wrote: > On Tue, Aug 24, 2021 at 11:05 PM Kefeng Wang <wangkefeng.wang@huawei.com> wrote: ... >>> Similar to other resources the AMBA bus "gets" for the device, I think >>> this should be moved into amba_probe() and not here. There's no reason >>> to delay the addition of the device (and loading its module) because >>> the IRQ isn't ready yet. >> The following code in the amba_device_try_add() will be called, it uses irq[0] >> and irq[1], so I put of_amba_device_decode_irq() into amba_device_try_add(). >> >> 470 if (dev->irq[0]) >> 471 ret = device_create_file(&dev->dev, &dev_attr_irq0); >> 472 if (ret == 0 && dev->irq[1]) >> 473 ret = device_create_file(&dev->dev, &dev_attr_irq1); >> 474 if (ret == 0) >> 475 return ret; > I wonder if we could just remove these. Why does userspace need them > in the first place? It's only an ABI if someone notices. Looking at > the history, AMBA bus was added in 2003 with just 'irq' and then > changed (ABI break) in 2004 to 'irq0' and 'irq1'. > > Rob Ok, I will kill all irq parts, diff --git a/drivers/amba/bus.c b/drivers/amba/bus.c index 962041148482..c08e8b30e02c 100644 --- a/drivers/amba/bus.c +++ b/drivers/amba/bus.c @@ -20,8 +20,6 @@ #include <linux/platform_device.h> #include <linux/reset.h> -#include <asm/irq.h> - #define to_amba_driver(d) container_of(d, struct amba_driver, drv) /* called on periphid match and class 0x9 coresight device. */ @@ -135,8 +133,6 @@ static ssize_t name##_show(struct device *_dev, \ static DEVICE_ATTR_RO(name) amba_attr_func(id, "%08x\n", dev->periphid); -amba_attr_func(irq0, "%u\n", dev->irq[0]); -amba_attr_func(irq1, "%u\n", dev->irq[1]); amba_attr_func(resource, "\t%016llx\t%016llx\t%016lx\n", (unsigned long long)dev->res.start, (unsigned long long)dev->res.end, dev->res.flags); @@ -467,10 +463,6 @@ static int amba_device_try_add(struct amba_device *dev, struct resource *parent) if (ret) goto err_release; - if (dev->irq[0]) - ret = device_create_file(&dev->dev, &dev_attr_irq0); - if (ret == 0 && dev->irq[1]) - ret = device_create_file(&dev->dev, &dev_attr_irq1); and do some cleanup about error handling in the next version. > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git/log/arch/arm/common/amba.c > . >
WARNING: multiple messages have this Message-ID (diff)
From: Kefeng Wang <wangkefeng.wang@huawei.com> To: Rob Herring <robh+dt@kernel.org> Cc: Saravana Kannan <saravanak@google.com>, "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>, Frank Rowand <frowand.list@gmail.com>, <devicetree@vger.kernel.org>, Russell King <linux@armlinux.org.uk>, "Linus Walleij" <linus.walleij@linaro.org>, linux-arm-kernel <linux-arm-kernel@lists.infradead.org>, Ruizhe Lin <linruizhe@huawei.com> Subject: Re: [PATCH 3/3] amba: Properly handle device probe without IRQ domain Date: Wed, 25 Aug 2021 22:41:15 +0800 [thread overview] Message-ID: <d7a1d3a8-bda4-7a29-b66d-22ed4926f9c8@huawei.com> (raw) In-Reply-To: <CAL_Jsq+0rq6u5d7itETOnQWx_V+J3aP1m1Zgehi5QKVoKvdbvQ@mail.gmail.com> On 2021/8/25 20:33, Rob Herring wrote: > On Tue, Aug 24, 2021 at 11:05 PM Kefeng Wang <wangkefeng.wang@huawei.com> wrote: ... >>> Similar to other resources the AMBA bus "gets" for the device, I think >>> this should be moved into amba_probe() and not here. There's no reason >>> to delay the addition of the device (and loading its module) because >>> the IRQ isn't ready yet. >> The following code in the amba_device_try_add() will be called, it uses irq[0] >> and irq[1], so I put of_amba_device_decode_irq() into amba_device_try_add(). >> >> 470 if (dev->irq[0]) >> 471 ret = device_create_file(&dev->dev, &dev_attr_irq0); >> 472 if (ret == 0 && dev->irq[1]) >> 473 ret = device_create_file(&dev->dev, &dev_attr_irq1); >> 474 if (ret == 0) >> 475 return ret; > I wonder if we could just remove these. Why does userspace need them > in the first place? It's only an ABI if someone notices. Looking at > the history, AMBA bus was added in 2003 with just 'irq' and then > changed (ABI break) in 2004 to 'irq0' and 'irq1'. > > Rob Ok, I will kill all irq parts, diff --git a/drivers/amba/bus.c b/drivers/amba/bus.c index 962041148482..c08e8b30e02c 100644 --- a/drivers/amba/bus.c +++ b/drivers/amba/bus.c @@ -20,8 +20,6 @@ #include <linux/platform_device.h> #include <linux/reset.h> -#include <asm/irq.h> - #define to_amba_driver(d) container_of(d, struct amba_driver, drv) /* called on periphid match and class 0x9 coresight device. */ @@ -135,8 +133,6 @@ static ssize_t name##_show(struct device *_dev, \ static DEVICE_ATTR_RO(name) amba_attr_func(id, "%08x\n", dev->periphid); -amba_attr_func(irq0, "%u\n", dev->irq[0]); -amba_attr_func(irq1, "%u\n", dev->irq[1]); amba_attr_func(resource, "\t%016llx\t%016llx\t%016lx\n", (unsigned long long)dev->res.start, (unsigned long long)dev->res.end, dev->res.flags); @@ -467,10 +463,6 @@ static int amba_device_try_add(struct amba_device *dev, struct resource *parent) if (ret) goto err_release; - if (dev->irq[0]) - ret = device_create_file(&dev->dev, &dev_attr_irq0); - if (ret == 0 && dev->irq[1]) - ret = device_create_file(&dev->dev, &dev_attr_irq1); and do some cleanup about error handling in the next version. > > [1] https://git.kernel.org/pub/scm/linux/kernel/git/tglx/history.git/log/arch/arm/common/amba.c > . > _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
next prev parent reply other threads:[~2021-08-25 14:41 UTC|newest] Thread overview: 52+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-08-16 7:46 [PATCH 0/3] amba: Properly handle device probe without IRQ domain Kefeng Wang 2021-08-16 7:46 ` Kefeng Wang 2021-08-16 7:46 ` [PATCH 1/3] amba: Drop unused functions about APB/AHB devices add Kefeng Wang 2021-08-16 7:46 ` Kefeng Wang 2021-08-16 7:46 ` [PATCH 2/3] Revert "ARM: amba: make use of -1 IRQs warn" Kefeng Wang 2021-08-16 7:46 ` Kefeng Wang 2021-08-16 7:46 ` [PATCH 3/3] amba: Properly handle device probe without IRQ domain Kefeng Wang 2021-08-16 7:46 ` Kefeng Wang 2021-08-24 20:05 ` Rob Herring 2021-08-24 20:05 ` Rob Herring 2021-08-24 20:08 ` Saravana Kannan 2021-08-24 20:08 ` Saravana Kannan 2021-08-25 4:05 ` Kefeng Wang 2021-08-25 4:05 ` Kefeng Wang 2021-08-25 8:04 ` Saravana Kannan 2021-08-25 8:04 ` Saravana Kannan 2021-08-25 8:39 ` Kefeng Wang 2021-08-25 8:39 ` Kefeng Wang 2021-08-26 2:45 ` Kefeng Wang 2021-08-26 2:45 ` Kefeng Wang 2021-08-26 4:45 ` Saravana Kannan 2021-08-26 4:45 ` Saravana Kannan 2021-08-26 6:22 ` Kefeng Wang 2021-08-26 6:22 ` Kefeng Wang 2021-08-26 8:22 ` [BUG] amba: Remove deferred device addition Kefeng Wang 2021-08-27 0:04 ` Saravana Kannan 2021-08-27 0:04 ` Saravana Kannan 2021-08-27 14:38 ` Kefeng Wang 2021-08-27 19:09 ` Saravana Kannan 2021-08-27 19:09 ` Saravana Kannan 2021-08-28 1:09 ` Kefeng Wang 2021-08-28 1:09 ` Kefeng Wang 2021-09-09 3:30 ` Saravana Kannan 2021-09-09 3:30 ` Saravana Kannan 2021-09-10 7:59 ` Kefeng Wang 2021-09-10 7:59 ` Kefeng Wang 2022-07-05 19:25 ` Saravana Kannan 2022-07-05 19:25 ` Saravana Kannan 2022-08-27 10:26 ` Leizhen (ThunderTown) 2022-08-27 10:26 ` Leizhen (ThunderTown) 2021-08-25 12:33 ` [PATCH 3/3] amba: Properly handle device probe without IRQ domain Rob Herring 2021-08-25 12:33 ` Rob Herring 2021-08-25 14:41 ` Kefeng Wang [this message] 2021-08-25 14:41 ` Kefeng Wang 2021-08-17 22:27 ` [PATCH 0/3] " Rob Herring 2021-08-17 22:27 ` Rob Herring 2021-08-23 2:19 ` Kefeng Wang 2021-08-23 2:19 ` Kefeng Wang 2021-08-23 9:05 ` Russell King (Oracle) 2021-08-23 9:05 ` Russell King (Oracle) 2021-08-23 10:57 ` Kefeng Wang 2021-08-23 10:57 ` Kefeng Wang
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=d7a1d3a8-bda4-7a29-b66d-22ed4926f9c8@huawei.com \ --to=wangkefeng.wang@huawei.com \ --cc=devicetree@vger.kernel.org \ --cc=frowand.list@gmail.com \ --cc=linruizhe@huawei.com \ --cc=linus.walleij@linaro.org \ --cc=linux-arm-kernel@lists.infradead.org \ --cc=linux-kernel@vger.kernel.org \ --cc=linux@armlinux.org.uk \ --cc=robh+dt@kernel.org \ --cc=saravanak@google.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.