[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20130114135020.BEF873E232D@localhost>
Date: Mon, 14 Jan 2013 13:50:20 +0000
From: Grant Likely <grant.likely@...retlab.ca>
To: Linus Walleij <linus.walleij@...ricsson.com>,
linux-kernel@...r.kernel.org,
Rob Herring <rob.herring@...xeda.com>,
Thomas Gleixner <tglx@...utronix.de>
Cc: Anmar Oueja <anmar.oueja@...aro.org>,
Linus Walleij <linus.walleij@...aro.org>
Subject: Re: [PATCH] irqdomain: document the simple domain first_irq
On Mon, 7 Jan 2013 01:18:32 +0100, Linus Walleij <linus.walleij@...ricsson.com> wrote:
> From: Linus Walleij <linus.walleij@...aro.org>
>
> The first_irq needs to be zero to get a linear domain and that
> comes with special semantics. We want to simplify this going
> forward but some documentation never hurts.
>
> Signed-off-by: Linus Walleij <linus.walleij@...aro.org>
Applied, thanks.
g.
> ---
> kernel/irq/irqdomain.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/kernel/irq/irqdomain.c b/kernel/irq/irqdomain.c
> index 96f3a1d..059a280 100644
> --- a/kernel/irq/irqdomain.c
> +++ b/kernel/irq/irqdomain.c
> @@ -143,7 +143,10 @@ static unsigned int irq_domain_legacy_revmap(struct irq_domain *domain,
> * irq_domain_add_simple() - Allocate and register a simple irq_domain.
> * @of_node: pointer to interrupt controller's device tree node.
> * @size: total number of irqs in mapping
> - * @first_irq: first number of irq block assigned to the domain
> + * @first_irq: first number of irq block assigned to the domain,
> + * pass zero to assign irqs on-the-fly. This will result in a
> + * linear IRQ domain so it is important to use irq_create_mapping()
> + * for each used IRQ, especially when SPARSE_IRQ is enabled.
> * @ops: map/unmap domain callbacks
> * @host_data: Controller private data pointer
> *
> --
> 1.7.11.3
>
--
Grant Likely, B.Sc, P.Eng.
Secret Lab Technologies, Ltd.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists