Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758854AbbGHS2m (ORCPT ); Wed, 8 Jul 2015 14:28:42 -0400 Received: from arrakis.dune.hu ([78.24.191.176]:41839 "EHLO arrakis.dune.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751621AbbGHS2i (ORCPT ); Wed, 8 Jul 2015 14:28:38 -0400 MIME-Version: 1.0 In-Reply-To: <1436379071-31574-1-git-send-email-albeu@free.fr> References: <1436379071-31574-1-git-send-email-albeu@free.fr> From: Jonas Gorski Date: Wed, 8 Jul 2015 20:28:05 +0200 Message-ID: Subject: Re: [PATCH] MIPS: ath79: irq: Remove the include of drivers/irqchip/irqchip.h To: Alban Bedel Cc: MIPS Mailing List , Thomas Gleixner , Ralf Baechle , Andrew Bresticker , Qais Yousef , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1359 Lines: 41 On Wed, Jul 8, 2015 at 8:11 PM, Alban Bedel wrote: > We shouldn't include irqchip.h from outside of the drivers/irqchip > directory. The irq driver should idealy be there, however this not > trivial at the moment. We still need to support platforms without DT > support and the interface to the DDR controller still use a custom > arch specific API. > > For now just redefine the IRQCHIP_DECLARE macro to avoid the cross > tree include. > > Signed-off-by: Alban Bedel The define was moved into linux/irqchip.h in 91e20b5040c67c51aad88cf87db4305c5bd7f79d, so all you can/need to do is ... > --- > arch/mips/ath79/irq.c | 8 +++++++- > 1 file changed, 7 insertions(+), 1 deletion(-) > > diff --git a/arch/mips/ath79/irq.c b/arch/mips/ath79/irq.c > index afb0096..c5ad737 100644 > --- a/arch/mips/ath79/irq.c > +++ b/arch/mips/ath79/irq.c > @@ -17,7 +17,6 @@ > #include > #include > #include > -#include "../../../drivers/irqchip/irqchip.h" > > #include > #include this removal ;) Jonas -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/