From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ducie-dc1.codethink.co.uk ([37.128.190.40]:35056 "EHLO ducie-dc1.codethink.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751335AbaA2JNK (ORCPT ); Wed, 29 Jan 2014 04:13:10 -0500 Message-ID: <52E8C622.5030603@codethink.co.uk> Date: Wed, 29 Jan 2014 09:13:06 +0000 From: Ben Dooks MIME-Version: 1.0 To: Valentine CC: linux-kernel@lists.codethink.co.uk, linux-sh@vger.kernel.org, Bjorn Helgaas , Simon Horman , linux-pci@vger.kernel.org Subject: Re: [PATCH 3/3] PCI: rcar: fix bridge logic configuration accesses References: <1390903616-8073-1-git-send-email-ben.dooks@codethink.co.uk> <1390903616-8073-4-git-send-email-ben.dooks@codethink.co.uk> <52E807E9.8060900@cogentembedded.com> In-Reply-To: <52E807E9.8060900@cogentembedded.com> Content-Type: text/plain; charset=ISO-8859-1; format=flowed Sender: linux-pci-owner@vger.kernel.org List-ID: On 28/01/14 19:41, Valentine wrote: > On 01/28/2014 02:06 PM, Ben Dooks wrote: >> The bridge logic at slot 0 only supports reads up to 0x40 and the >> rest of the PCI configuration space for this slot is marked as >> reserved in the manual. >> >> Trying a read from offset 0x100 is producing an error from the >> bridge. With error interrupts enabled, the following is printed: > > I don't think this is a critical error. > The bridge works fine after an attempt to access the unsupported > PCIe/PCI-X 2 area. > > If you want to prevent the access, I'm OK with it. > But I think it's better to do "if (where <= 0x100)" > and drop the slot check since all other slots do not support > access beyond 0x100 as well. The bridge specifically says that the registers end at 0x40 so that really should stay. We're seeing external aborts being generated from somewhere, and it seems that either the bridge or the USB drivers are responsible for generating them. > The PCI code attempts to access beyond 0x100 only once when probing PCI > bridges > to see if they support PCIe/PCI-X 2 area which is 4K. > The area beyond 0x40 is never accessed because the bridge does not > expose any PCI capabilities. > >> >> pci-rcar-gen2 ee0d0000.pci: error irq: status 00000014 > > Did you experience any problems other than this message > printed by the IRQ handler introduced by the previous patch? > -- Ben Dooks http://www.codethink.co.uk/ Senior Engineer Codethink - Providing Genius