Re: [PATCH v4 2/5] resource: Add irqresource_disabled()

[Rafael J. Wysocki]

On Mon, Nov 30, 2020 at 6:41 PM John Garry <john.garry@xxxxxxxxxx> wrote:
>
> Add a common function to set the fields for a irq resource to disabled,
> which mimics what is done in acpi_dev_irqresource_disabled(), with a view
> to replace that function.
>
> Signed-off-by: John Garry <john.garry@xxxxxxxxxx>

Reviewed-by: Rafael J. Wysocki <rafael.j.wysocki@xxxxxxxxx>

> ---
>  include/linux/ioport.h | 7 +++++++
>  1 file changed, 7 insertions(+)
>
> diff --git a/include/linux/ioport.h b/include/linux/ioport.h
> index 5135d4b86cd6..f9bf374f9633 100644
> --- a/include/linux/ioport.h
> +++ b/include/linux/ioport.h
> @@ -307,6 +307,13 @@ struct resource *devm_request_free_mem_region(struct device *dev,
>  struct resource *request_free_mem_region(struct resource *base,
>                 unsigned long size, const char *name);
>
> +static inline void irqresource_disabled(struct resource *res, u32 irq)
> +{
> +       res->start = irq;
> +       res->end = irq;
> +       res->flags = IORESOURCE_IRQ | IORESOURCE_DISABLED | IORESOURCE_UNSET;
> +}
> +
>  #ifdef CONFIG_IO_STRICT_DEVMEM
>  void revoke_devmem(struct resource *res);
>  #else
> --
> 2.26.2
>