[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <alpine.DEB.2.20.1702131138480.3619@nanos>
Date: Mon, 13 Feb 2017 11:40:56 +0100 (CET)
From: Thomas Gleixner <tglx@...utronix.de>
To: Marc Zyngier <marc.zyngier@....com>
cc: linux-kernel@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
kvmarm@...ts.cs.columbia.edu,
Christoffer Dall <christoffer.dall@...aro.org>,
Jason Cooper <jason@...edaemon.net>,
Eric Auger <eric.auger@...hat.com>
Subject: Re: [RFC PATCH 20/33] irqchip/gic-v3-its: Add VPE domain
infrastructure
On Tue, 17 Jan 2017, Marc Zyngier wrote:
> @@ -2266,6 +2294,8 @@ int __init its_init(struct fwnode_handle *handle, struct rdists *rdists,
> struct irq_domain *parent_domain)
> {
> struct device_node *of_node;
> + struct its_node *its;
> + bool has_v4 = false;
>
> its_parent = parent_domain;
> of_node = to_of_node(handle);
> @@ -2283,5 +2313,11 @@ int __init its_init(struct fwnode_handle *handle, struct rdists *rdists,
> its_alloc_lpi_tables();
> its_lpi_init(rdists->id_bits);
>
> + list_for_each_entry(its, &its_nodes, entry)
> + has_v4 |= its->is_v4;
Hmm, can you have mixed version its nodes? If not, you probably should have
some sanity check (not necessarily here). If yes, then how does that work?
Thanks,
tglx
Powered by blists - more mailing lists