[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <170483326744.3193407.6920780554098441341.robh@kernel.org>
Date: Tue, 9 Jan 2024 14:47:48 -0600
From: Rob Herring <robh@...nel.org>
To: Mark Hasemeyer <markhas@...omium.org>
Cc: Sakari Ailus <sakari.ailus@...ux.intel.com>, AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>, Daniel Scally <djrscally@...il.com>, Raul Rangel <rrangel@...omium.org>, Greg Kroah-Hartman <gregkh@...uxfoundation.org>, Heikki Krogerus <heikki.krogerus@...ux.intel.com>, linux-acpi@...r.kernel.org, Andy Shevchenko <andriy.shevchenko@...ux.intel.com>, "Rafael J. Wysocki" <rafael@...nel.org>, Andy Shevchenko <andriy.shevchenko@...el.com>, Len Brown <lenb@...nel.org>, devicetree@...r.kernel.org, Konrad Dybcio <konrad.dybcio@...aro.org>, Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>, Tzung-Bi Shih <tzungbi@...nel.org>, LKML <linux-kernel@...r.kernel.org>, Frank Rowand <frowand.list@...il.com>, Sudeep Holla <sudeep.holla@....com>, Rob Herring <robh+dt@...nel.org>
Subject: Re: [PATCH v4 21/24] device property: Modify fwnode irq_get() to use
resource
On Tue, 02 Jan 2024 14:07:45 -0700, Mark Hasemeyer wrote:
> The underlying ACPI and OF subsystems provide their own APIs which
> provide IRQ information as a struct resource. This allows callers to get
> more information about the IRQ by looking at the resource flags. For
> example, whether or not an IRQ is wake capable.
>
> Suggested-by: Andy Shevchenko <andriy.shevchenko@...ux.intel.com>
> Reviewed-by: Sakari Ailus <sakari.ailus@...ux.intel.com>
> Signed-off-by: Mark Hasemeyer <markhas@...omium.org>
> ---
>
> Changes in v4:
> -Add Sakari's Reviewed-by tag from v2
> -Remove ioport.h dependency in fwnode.h
> -Use Andy's @linux.intel.com email
>
> Changes in v3:
> -Add Suggested-by tag
> -Initialize struct resource to 0 on stack
> -EXPORT_SYMBOL()->EXPORT_SYMBOL_GPL()
> -Remove extra space in commit message
> -Reformat fwnode_irq_get_resource() declaration
>
> Changes in v2:
> -New patch
>
> drivers/acpi/property.c | 11 +++++------
> drivers/base/property.c | 32 +++++++++++++++++++++++++-------
> drivers/of/property.c | 8 ++++----
> include/linux/fwnode.h | 8 +++++---
> include/linux/property.h | 2 ++
> 5 files changed, 41 insertions(+), 20 deletions(-)
>
Reviewed-by: Rob Herring <robh@...nel.org>
Powered by blists - more mailing lists