From 02df97f4d814b9893eef2c2118b85d2b4b9d61ae Mon Sep 17 00:00:00 2001
From: Soha Jin <[email protected]>
Date: Sat, 1 Oct 2022 00:26:04 +0800
Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
of_irq_get_byname to get irq
Not only platform devices described by OF have named interrupts, but
devices described by ACPI also have named interrupts. The fwnode is an
abstraction to different standards, and using fwnode_irq_get_byname can
support more devices.
Signed-off-by: Soha Jin <[email protected]>
Tested-by: Wende Tan <[email protected]>
---
drivers/base/platform.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/base/platform.c b/drivers/base/platform.c
index 51bb22898..968f3d71e 100644
--- a/drivers/base/platform.c
+++ b/drivers/base/platform.c
@@ -441,8 +441,8 @@ static int __platform_get_irq_byname(struct platform_device *dev,
struct resource *r;
int ret;
- if (IS_ENABLED(CONFIG_OF_IRQ) && dev->dev.of_node) {
- ret = of_irq_get_byname(dev->dev.of_node, name);
+ if (!dev->dev.of_node || IS_ENABLED(CONFIG_OF_IRQ)) {
+ ret = fwnode_irq_get_byname(dev_fwnode(&dev->dev), name);
if (ret > 0 || ret == -EPROBE_DEFER)
return ret;
}
--
2.30.2
> -----Original Message-----
> From: [email protected] <[email protected]>
> Sent: Saturday, October 1, 2022 2:15 AM
> To: [email protected]
> Cc: [email protected]; [email protected]; 'Wende Tan'
> <[email protected]>
> Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
> of_irq_get_byname to get irq
>
> >From 02df97f4d814b9893eef2c2118b85d2b4b9d61ae Mon Sep 17
> 00:00:00 2001
> From: Soha Jin <[email protected]>
> Date: Sat, 1 Oct 2022 00:26:04 +0800
> Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
> of_irq_get_byname to get irq
>
> Not only platform devices described by OF have named interrupts, but
> devices described by ACPI also have named interrupts. The fwnode is an
> abstraction to different standards, and using fwnode_irq_get_byname can
> support more devices.
>
> Signed-off-by: Soha Jin <[email protected]>
> Tested-by: Wende Tan <[email protected]>
> ---
> drivers/base/platform.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/base/platform.c b/drivers/base/platform.c index
> 51bb22898..968f3d71e 100644
> --- a/drivers/base/platform.c
> +++ b/drivers/base/platform.c
> @@ -441,8 +441,8 @@ static int __platform_get_irq_byname(struct
> platform_device *dev,
> struct resource *r;
> int ret;
>
> - if (IS_ENABLED(CONFIG_OF_IRQ) && dev->dev.of_node) {
> - ret = of_irq_get_byname(dev->dev.of_node, name);
> + if (!dev->dev.of_node || IS_ENABLED(CONFIG_OF_IRQ)) {
> + ret = fwnode_irq_get_byname(dev_fwnode(&dev->dev), name);
> if (ret > 0 || ret == -EPROBE_DEFER)
> return ret;
> }
> --
> 2.30.2
>
>
Any updates on this 10-day-ago patch?
Regards,
Soha
On Mon, Oct 10, 2022 at 11:14:46AM +0800, Soha Jin wrote:
> > -----Original Message-----
> > From: [email protected] <[email protected]>
> > Sent: Saturday, October 1, 2022 2:15 AM
> > To: [email protected]
> > Cc: [email protected]; [email protected]; 'Wende Tan'
> > <[email protected]>
> > Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
> > of_irq_get_byname to get irq
> >
> > >From 02df97f4d814b9893eef2c2118b85d2b4b9d61ae Mon Sep 17
> > 00:00:00 2001
> > From: Soha Jin <[email protected]>
> > Date: Sat, 1 Oct 2022 00:26:04 +0800
> > Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
> > of_irq_get_byname to get irq
> >
> > Not only platform devices described by OF have named interrupts, but
> > devices described by ACPI also have named interrupts. The fwnode is an
> > abstraction to different standards, and using fwnode_irq_get_byname can
> > support more devices.
> >
> > Signed-off-by: Soha Jin <[email protected]>
> > Tested-by: Wende Tan <[email protected]>
> > ---
> > drivers/base/platform.c | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/drivers/base/platform.c b/drivers/base/platform.c index
> > 51bb22898..968f3d71e 100644
> > --- a/drivers/base/platform.c
> > +++ b/drivers/base/platform.c
> > @@ -441,8 +441,8 @@ static int __platform_get_irq_byname(struct
> > platform_device *dev,
> > struct resource *r;
> > int ret;
> >
> > - if (IS_ENABLED(CONFIG_OF_IRQ) && dev->dev.of_node) {
> > - ret = of_irq_get_byname(dev->dev.of_node, name);
> > + if (!dev->dev.of_node || IS_ENABLED(CONFIG_OF_IRQ)) {
> > + ret = fwnode_irq_get_byname(dev_fwnode(&dev->dev), name);
> > if (ret > 0 || ret == -EPROBE_DEFER)
> > return ret;
> > }
> > --
> > 2.30.2
> >
> >
>
> Any updates on this 10-day-ago patch?
It's the middle of the merge window, we can't do anything until after
6.1-rc1 is out, please relax and wait for that.
And what is the rush here, what is broken without this change?
thanks,
greg k-h
> -----Original Message-----
> From: Greg KH <[email protected]>
> Sent: Monday, October 10, 2022 2:22 PM
>
> It's the middle of the merge window, we can't do anything until after
> 6.1-rc1 is out, please relax and wait for that.
>
> And what is the rush here, what is broken without this change?
>
Got it, just wondering if there anything I could improve in this patch,
since I have not got any feedback. The bug just affect named interrupts in
ACPI, which seems not used widely and not in a hurry.
Regards,
Soha
On Sat, Oct 01, 2022 at 02:15:29AM +0800, [email protected] wrote:
> >From 02df97f4d814b9893eef2c2118b85d2b4b9d61ae Mon Sep 17 00:00:00 2001
> From: Soha Jin <[email protected]>
> Date: Sat, 1 Oct 2022 00:26:04 +0800
> Subject: [PATCH] platform: use fwnode_irq_get_byname instead of
> of_irq_get_byname to get irq
>
> Not only platform devices described by OF have named interrupts, but
> devices described by ACPI also have named interrupts. The fwnode is an
> abstraction to different standards, and using fwnode_irq_get_byname can
> support more devices.
>
> Signed-off-by: Soha Jin <[email protected]>
> Tested-by: Wende Tan <[email protected]>
> ---
> drivers/base/platform.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/base/platform.c b/drivers/base/platform.c
> index 51bb22898..968f3d71e 100644
> --- a/drivers/base/platform.c
> +++ b/drivers/base/platform.c
> @@ -441,8 +441,8 @@ static int __platform_get_irq_byname(struct platform_device *dev,
> struct resource *r;
> int ret;
>
> - if (IS_ENABLED(CONFIG_OF_IRQ) && dev->dev.of_node) {
> - ret = of_irq_get_byname(dev->dev.of_node, name);
> + if (!dev->dev.of_node || IS_ENABLED(CONFIG_OF_IRQ)) {
> + ret = fwnode_irq_get_byname(dev_fwnode(&dev->dev), name);
> if (ret > 0 || ret == -EPROBE_DEFER)
> return ret;
> }
> --
> 2.30.2
>
>
Hi,
This is the friendly patch-bot of Greg Kroah-Hartman. You have sent him
a patch that has triggered this response. He used to manually respond
to these common problems, but in order to save his sanity (he kept
writing the same thing over and over, yet to different people), I was
created. Hopefully you will not take offence and will fix the problem
in your patch and resubmit it so that it can be accepted into the Linux
kernel tree.
You are receiving this message because of the following common error(s)
as indicated below:
- Your patch is malformed (tabs converted to spaces, linewrapped, etc.)
and can not be applied. Please read the file,
Documentation/email-clients.txt in order to fix this.
- Your patch was attached, please place it inline so that it can be
applied directly from the email message itself.
If you wish to discuss this problem further, or you have questions about
how to resolve this issue, please feel free to respond to this email and
Greg will reply once he has dug out from the pending patches received
from other developers.
thanks,
greg k-h's patch email bot
On Mon, Oct 10, 2022 at 03:07:19PM +0800, Soha Jin wrote:
> > -----Original Message-----
> > From: Greg KH <[email protected]>
> > Sent: Monday, October 10, 2022 2:22 PM
> >
> > It's the middle of the merge window, we can't do anything until after
> > 6.1-rc1 is out, please relax and wait for that.
> >
> > And what is the rush here, what is broken without this change?
> >
>
> Got it, just wondering if there anything I could improve in this patch,
> since I have not got any feedback. The bug just affect named interrupts in
> ACPI, which seems not used widely and not in a hurry.
Your patch is corrupted and can not be applied :(
You could have tested this by picking it up from lore.kernel.org and
verified that it worked.
Please fix your email client and send a new version.
thanks,
greg k-h
Hello Greg,
> -----Original Message-----
> From: 'Greg KH' <[email protected]>
> Sent: Friday, November 11, 2022 1:56 AM
>
> Your patch is corrupted and can not be applied :( You could have tested this
> by picking it up from lore.kernel.org and verified that it worked.
>
> Please fix your email client and send a new version.
>
> thanks,
>
> greg k-h
I sent a PATCH v2* several days before. This patch's format should be
good and I removed a meaningless if-branch.
Regards,
Soha
* https://lore.kernel.org/all/[email protected]/