Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751997AbbDRW3M (ORCPT ); Sat, 18 Apr 2015 18:29:12 -0400 Received: from arrakis.dune.hu ([78.24.191.176]:35397 "EHLO arrakis.dune.hu" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750818AbbDRW3J (ORCPT ); Sat, 18 Apr 2015 18:29:09 -0400 MIME-Version: 1.0 In-Reply-To: <1429274178-4337-5-git-send-email-albeu@free.fr> References: <1429274178-4337-1-git-send-email-albeu@free.fr> <1429274178-4337-5-git-send-email-albeu@free.fr> From: Jonas Gorski Date: Sun, 19 Apr 2015 00:28:39 +0200 Message-ID: Subject: Re: [PATCH 4/5] MIPS: ath79: Fix the PCI memory size and offset of window 7 To: Alban Bedel Cc: MIPS Mailing List , Ralf Baechle , Andrew Bresticker , Qais Yousef , Wolfram Sang , Sergey Ryazanov , "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: 2218 Lines: 61 Hi, On Fri, Apr 17, 2015 at 2:36 PM, Alban Bedel wrote: > The define AR71XX_PCI_MEM_SIZE miss one window, there is 7 windows, > not 6. To make things clearer, and allow simpler code, derive > AR71XX_PCI_MEM_SIZE from the newly introduced AR71XX_PCI_WIN_COUNT > and AR71XX_PCI_WIN_SIZE. > > The define AR71XX_PCI_WIN7_OFFS also add a typo, fix it. I think this will break PCI on ar71xx. > > Signed-off-by: Alban Bedel > --- > arch/mips/include/asm/mach-ath79/ar71xx_regs.h | 6 ++++-- > 1 file changed, 4 insertions(+), 2 deletions(-) > > diff --git a/arch/mips/include/asm/mach-ath79/ar71xx_regs.h b/arch/mips/include/asm/mach-ath79/ar71xx_regs.h > index aa3800c..e2669a8 100644 > --- a/arch/mips/include/asm/mach-ath79/ar71xx_regs.h > +++ b/arch/mips/include/asm/mach-ath79/ar71xx_regs.h > @@ -41,7 +41,9 @@ > #define AR71XX_RESET_SIZE 0x100 > > #define AR71XX_PCI_MEM_BASE 0x10000000 > -#define AR71XX_PCI_MEM_SIZE 0x07000000 > +#define AR71XX_PCI_WIN_COUNT 8 > +#define AR71XX_PCI_WIN_SIZE 0x01000000 > +#define AR71XX_PCI_MEM_SIZE (AR71XX_PCI_WIN_COUNT * AR71XX_PCI_WIN_SIZE) > > #define AR71XX_PCI_WIN0_OFFS 0x10000000 > #define AR71XX_PCI_WIN1_OFFS 0x11000000 > @@ -50,7 +52,7 @@ > #define AR71XX_PCI_WIN4_OFFS 0x14000000 > #define AR71XX_PCI_WIN5_OFFS 0x15000000 > #define AR71XX_PCI_WIN6_OFFS 0x16000000 > -#define AR71XX_PCI_WIN7_OFFS 0x07000000 > +#define AR71XX_PCI_WIN7_OFFS 0x17000000 These values are used in exactly one place, for writing into the PCI address space offset registers. The 7th PCI window is a special one for accessing the configuration space registers, which requires to be set to 0x07000000 for that purpose. So by changing this value you likely break access to these values. > > #define AR71XX_PCI_CFG_BASE \ > (AR71XX_PCI_MEM_BASE + AR71XX_PCI_WIN7_OFFS + 0x10000) Also this macro would now be wrong, and calculate a wrong address. Regards 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/