Date: Thu, 27 Jul 2017 10:57:15 +0200
From: Robert Richter <robert.richter@cavium.com>
To: Thomas Gleixner <tglx@linutronix.de>,
        Jason Cooper <jason@lakedaemon.net>,
        Marc Zyngier <marc.zyngier@arm.com>
Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] irqchip/gic-v3-its: Remove ACPICA version check for ACPI
 NUMA
Message-ID: <20170727085715.GJ17961@rric.localdomain>
References: <20170712132509.11281-1-rrichter@cavium.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20170712132509.11281-1-rrichter@cavium.com>
User-Agent: Mutt/1.5.23 (2014-03-12)
SpamDiagnosticOutput: 1:99
SpamDiagnosticMetadata: NSPM
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 27 Jul 2017 08:57:27.5187 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-Transport-CrossTenantHeadersStamped: BY2PR07MB2343
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Length: 1020
Lines: 37

Marc,

I think this can go in as fix for 4.13.

Please review. Thanks,

-Robert

On 12.07.17 15:25:09, Robert Richter wrote:
> The version check was added due to dependency to
> 
>  a618c7f89a02 ACPICA: Add support for new SRAT subtable
> 
> Now, that this code is in the kernel, remove the check. This is esp.
> useful to enable backports.
> 
> Signed-off-by: Robert Richter <rrichter@cavium.com>
> ---
>  drivers/irqchip/irq-gic-v3-its.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/irqchip/irq-gic-v3-its.c b/drivers/irqchip/irq-gic-v3-its.c
> index bb33ba1837a9..745c5772e99b 100644
> --- a/drivers/irqchip/irq-gic-v3-its.c
> +++ b/drivers/irqchip/irq-gic-v3-its.c
> @@ -1858,7 +1858,7 @@ static int __init its_of_probe(struct device_node *node)
>  
>  #define ACPI_GICV3_ITS_MEM_SIZE (SZ_128K)
>  
> -#if defined(CONFIG_ACPI_NUMA) && (ACPI_CA_VERSION >= 0x20170531)
> +#ifdef CONFIG_ACPI_NUMA
>  struct its_srat_map {
>  	/* numa node id */
>  	u32	numa_node;
> -- 
> 2.11.0
>