From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S1756822AbcBIPPs (ORCPT <rfc822;w@1wt.eu>);
	Tue, 9 Feb 2016 10:15:48 -0500
Received: from mout.kundenserver.de ([212.227.17.13]:53280 "EHLO
	mout.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S1751197AbcBIPPm (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Tue, 9 Feb 2016 10:15:42 -0500
From: Arnd Bergmann <arnd@arndb.de>
To: Gregory CLEMENT <gregory.clement@free-electrons.com>
Cc: Jason Cooper <jason@lakedaemon.net>, Andrew Lunn <andrew@lunn.ch>,
        Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
        arm@kernel.org, Catalin Marinas <catalin.marinas@arm.com>,
        Will Deacon <will.deacon@arm.com>, Jonathan Corbet <corbet@lwn.net>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Jiri Slaby <jslaby@suse.com>, linux-serial@vger.kernel.org,
        Tejun Heo <tj@kernel.org>, Hans de Goede <hdegoede@redhat.com>,
        linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
        Mark Rutland <mark.rutland@arm.com>, devicetree@vger.kernel.org,
        Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
        linux-arm-kernel@lists.infradead.org, Lior Amsalem <alior@marvell.com>,
        Nadav Haklai <nadavh@marvell.com>, Omri Itach <omrii@marvell.com>
Subject: Re: [PATCH v3 01/12] irqchip: armada-370-xp: add Kconfig option for the driver
Date: Tue, 09 Feb 2016 16:14:32 +0100
Message-ID: <7747159.T9eeT0dC65@wuerfel>
User-Agent: KMail/4.11.5 (Linux/3.16.0-10-generic; KDE/4.11.5; x86_64; ; )
In-Reply-To: <1454951660-13289-2-git-send-email-gregory.clement@free-electrons.com>
References: <1454951660-13289-1-git-send-email-gregory.clement@free-electrons.com> <1454951660-13289-2-git-send-email-gregory.clement@free-electrons.com>
MIME-Version: 1.0
Content-Transfer-Encoding: 7Bit
Content-Type: text/plain; charset="us-ascii"
X-Provags-ID: V03:K0:+h+gf6UzDLZSfuiM73TT53VkYrxuw2lrRpFcOm8iI97X4xC8gUy
 jDGJwyBpjbSvcnC1Ylqdn+CXlzGD6ChhoMW5P4UMe/S8GwV/GSVvtNVLW33owsboXe+NUh5
 +HGO1f7GKurExzGQpd30CIGU5KT63XSR7DKEChGyk+CaPbGCd0ZHsuTedNX8xm3X1h0sBxc
 ZG7Y93uZRRASxtnDMW9nA==
X-UI-Out-Filterresults: notjunk:1;V01:K0:4KRtSTo/lMc=:XGfW9EYiX151De6eSolTJz
 drtMZgG/jul48EyoNTibFqS1mdST+kjmOeSDdjKkNLFfUMaM0JI0NAoKp2zUEU5B9APMQ7Tpk
 Ie7xF0zhsYNVe89sLQG3iLvVALx32Bi8UNwOtSyC3BiI1Pz3Y0AXQEKXGXAOrBNiFtcbHGUIr
 gNgDg3D3ygn65c8SH63M3Mr7uhqiasLdwB7w9XD+gg0TfIDbB7LiHDLBR3XSv21jmZF3oXDnK
 Tj2wI69MzDTq4kWy+XGgtsWFIlcYxTiteUhm78HtHojPsUbQIttf4xCKALmkK6l0hUq4WMlvr
 33jJa5c1TSI7qlPenW9p5iGRTjv5FTv6NphP27EOF/KAUKqEnWFkC7lbS6RNijlfjUwJJEpAJ
 /RNuKegi96IRK+Ul2gS7Dz/Yin4St0NIVvBwwNfyv7xTrq64z5dG/MP9rsc+3yY5nOSfcvP8P
 gN3T/3NgiyuHhA6Soc1VmTHQhr4K7vdnYBrF024V1y6MUKf3cMFg1LXJlFIppQ/YIIzzMfUYM
 kVQaJJIYClol+xOylxcgp1iq4gDczLpGfQASVcx70OZ2BvFDltEqw6DVFxqIgfjMMN6iHTeUF
 xk3s9QYyfARzIb3febKUHdbkr2U118p0tHd9tH6eWss2ODvTfJi8sVWyF9Ye964wNbCp9mGGG
 HYnfAmVNpYPgYRLeeUfmW0ZNcm2I9h3Xyxe6g66IUiXN/dJC3Bce+1AOEeir6HcZ6nLZOUnd/
 TFCeUKS5h31KlBas
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Monday 08 February 2016 18:14:09 Gregory CLEMENT wrote:
> 
> +config ARMADA_370_XP_IRQ
> +       bool
> +       default y if ARCH_MVEBU
> +       select GENERIC_IRQ_CHIP
> +
> 

Looks good. I wonder if we should also enable this for COMPILE_TEST, e.g.

config ARMADA_370_XP_IRQ
       bool "Marvell Armada 3xx/XP/3xxx interrupt controller" if COMPILE_TEST && !ARCH_MVEBU
       default ARCH_MVEBU
       select GENERIC_IRQ_CHIP

If we do this, it's probably something outside of your series that we should
do for all patches, so don't worry about it here. Any opinions on the general
question?

	Arnd