From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Tue, 07 May 2013 12:20:17 +0200 (CEST) Received: from gate.crashing.org ([63.228.1.57]:59831 "EHLO gate.crashing.org" rhost-flags-OK-OK-OK-OK) by eddie.linux-mips.org with ESMTP id S6822675Ab3EGKUN0oNE9 (ORCPT ); Tue, 7 May 2013 12:20:13 +0200 Received: from [127.0.0.1] (localhost.localdomain [127.0.0.1]) by gate.crashing.org (8.14.1/8.13.8) with ESMTP id r47AJIJ0028838; Tue, 7 May 2013 05:19:19 -0500 Message-ID: <1367921958.25488.8.camel@pasglop> Subject: Re: [PATCH v8 1/3] of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC From: Benjamin Herrenschmidt To: Andrew Murray Cc: "linux-mips@linux-mips.org" , linuxppc-dev , "jgunthorpe@obsidianresearch.com" , "linux@arm.linux.org.uk" , "siva.kallam@samsung.com" , "linux-pci@vger.kernel.org" , "devicetree-discuss@lists.ozlabs.org" , "jg1.han@samsung.com" , Liviu Dudau , "linux-kernel@vger.kernel.org" , "linux-samsung-soc@vger.kernel.org" , "kgene.kim@samsung.com" , "bhelgaas@google.com" , "suren.reddy@samsung.com" , "linux-arm-kernel@lists.infradead.org" , "monstr@monstr.eu" , "paulus@samba.org" , "thomas.petazzoni@free-electrons.com" , "thierry.reding@avionic-design.de" , "thomas.abraham@linaro.org" , "arnd@arndb.de" , "linus.walleij@linaro.org" , "juhosg@openwrt.org" , "grant.likely@linaro.org" , Rob Herring Date: Tue, 07 May 2013 20:19:18 +1000 In-Reply-To: <20130507080142.GA8808@arm.com> References: <1366627295-16964-1-git-send-email-Andrew.Murray@arm.com> <1366627295-16964-2-git-send-email-Andrew.Murray@arm.com> <1367721709.11982.37.camel@pasglop> <20130507080142.GA8808@arm.com> Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.6.4-0ubuntu1 Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 36337 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: benh@kernel.crashing.org Precedence: bulk List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: linux-mips X-List-ID: linux-mips List-subscribe: List-owner: List-post: List-archive: X-list: linux-mips On Tue, 2013-05-07 at 09:01 +0100, Andrew Murray wrote: > > There were no objections to this latest revision until now and it is > currently sitting with Jason Cooper (mvebu-next/pcie). [1] Ok, well I've just sent Linus a pull request for my changes so at least drop the powerpc changes from your tree for the time being. > This is a view that was also shared by Bjorn [2] when I attempted to > submit a patchset which moves struct pci_controller to asm-generic. Right, it's the logical way to go > The motativation for my patchsets were to give a way for ARM PCI host > bridge drivers to parse the DT ranges property, but this snow-balled > into unifying pci_process_bridge_OF_ranges. Which I understand, I would probably have done the same thing in your shoes :-) > My v8 patchset provides a of_pci_range_parser which is used directly > by a few ARM PCI DT host bridge drivers, this has been generally > accepted and tested. I don't see why this can't remain and so I'd > really like to keep this around. Sure, no objection, in fact I should/could probably update my new code to use it as well. > Grant, Benjamin would you be happy for me to resubmit this series > which provides the of_pci_range_parser which will be used by the > separate implementations of pci_process_bridge_OF_ranges in > PowerPC/Microblaze? Sure, in fact feel free to update my new code if you have more bandwidth than I do, it should hit Linus tree soon hopefully unless he objects to me having a second pull request this merge window... > Benjamin are you able to still use of_pci_range_parser in your > 'Support per-aperture memory offset' patch? I see no reason why not. I just haven't looked into it much, I admit, being bogged down with a pile of new HW bringup in the lab etc... Cheers, Ben. From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benjamin Herrenschmidt Subject: Re: [PATCH v8 1/3] of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC Date: Tue, 07 May 2013 20:19:18 +1000 Message-ID: <1367921958.25488.8.camel@pasglop> References: <1366627295-16964-1-git-send-email-Andrew.Murray@arm.com> <1366627295-16964-2-git-send-email-Andrew.Murray@arm.com> <1367721709.11982.37.camel@pasglop> <20130507080142.GA8808@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20130507080142.GA8808@arm.com> Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-subscribe: List-owner: List-post: List-archive: To: Andrew Murray Cc: "linux-mips@linux-mips.org" , linuxppc-dev , "jgunthorpe@obsidianresearch.com" , "linux@arm.linux.org.uk" , "siva.kallam@samsung.com" , "linux-pci@vger.kernel.org" , "devicetree-discuss@lists.ozlabs.org" , "jg1.han@samsung.com" , Liviu Dudau , "linux-kernel@vger.kernel.org" , "linux-samsung-soc@vger.kernel.org" , "kgene.kim@samsung.com" , "bhelgaas@google.com" , "suren.reddy@samsung.com" List-Id: linux-samsung-soc@vger.kernel.org On Tue, 2013-05-07 at 09:01 +0100, Andrew Murray wrote: > > There were no objections to this latest revision until now and it is > currently sitting with Jason Cooper (mvebu-next/pcie). [1] Ok, well I've just sent Linus a pull request for my changes so at least drop the powerpc changes from your tree for the time being. > This is a view that was also shared by Bjorn [2] when I attempted to > submit a patchset which moves struct pci_controller to asm-generic. Right, it's the logical way to go > The motativation for my patchsets were to give a way for ARM PCI host > bridge drivers to parse the DT ranges property, but this snow-balled > into unifying pci_process_bridge_OF_ranges. Which I understand, I would probably have done the same thing in your shoes :-) > My v8 patchset provides a of_pci_range_parser which is used directly > by a few ARM PCI DT host bridge drivers, this has been generally > accepted and tested. I don't see why this can't remain and so I'd > really like to keep this around. Sure, no objection, in fact I should/could probably update my new code to use it as well. > Grant, Benjamin would you be happy for me to resubmit this series > which provides the of_pci_range_parser which will be used by the > separate implementations of pci_process_bridge_OF_ranges in > PowerPC/Microblaze? Sure, in fact feel free to update my new code if you have more bandwidth than I do, it should hit Linus tree soon hopefully unless he objects to me having a second pull request this merge window... > Benjamin are you able to still use of_pci_range_parser in your > 'Support per-aperture memory offset' patch? I see no reason why not. I just haven't looked into it much, I admit, being bogged down with a pile of new HW bringup in the lab etc... Cheers, Ben. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Message-ID: <1367921958.25488.8.camel@pasglop> Subject: Re: [PATCH v8 1/3] of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC From: Benjamin Herrenschmidt To: Andrew Murray Date: Tue, 07 May 2013 20:19:18 +1000 In-Reply-To: <20130507080142.GA8808@arm.com> References: <1366627295-16964-1-git-send-email-Andrew.Murray@arm.com> <1366627295-16964-2-git-send-email-Andrew.Murray@arm.com> <1367721709.11982.37.camel@pasglop> <20130507080142.GA8808@arm.com> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Cc: "linux-mips@linux-mips.org" , "siva.kallam@samsung.com" , "linux-pci@vger.kernel.org" , "linus.walleij@linaro.org" , "thierry.reding@avionic-design.de" , Liviu Dudau , "juhosg@openwrt.org" , "paulus@samba.org" , "linux-samsung-soc@vger.kernel.org" , "linux@arm.linux.org.uk" , "jg1.han@samsung.com" , "jgunthorpe@obsidianresearch.com" , "thomas.abraham@linaro.org" , "grant.likely@linaro.org" , Rob Herring , "arnd@arndb.de" , "devicetree-discuss@lists.ozlabs.org" , "kgene.kim@samsung.com" , "bhelgaas@google.com" , "linux-arm-kernel@lists.infradead.org" , "thomas.petazzoni@free-electrons.com" , "monstr@monstr.eu" , "linux-kernel@vger.kernel.org" , "suren.reddy@samsung.com" , linuxppc-dev List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Tue, 2013-05-07 at 09:01 +0100, Andrew Murray wrote: > > There were no objections to this latest revision until now and it is > currently sitting with Jason Cooper (mvebu-next/pcie). [1] Ok, well I've just sent Linus a pull request for my changes so at least drop the powerpc changes from your tree for the time being. > This is a view that was also shared by Bjorn [2] when I attempted to > submit a patchset which moves struct pci_controller to asm-generic. Right, it's the logical way to go > The motativation for my patchsets were to give a way for ARM PCI host > bridge drivers to parse the DT ranges property, but this snow-balled > into unifying pci_process_bridge_OF_ranges. Which I understand, I would probably have done the same thing in your shoes :-) > My v8 patchset provides a of_pci_range_parser which is used directly > by a few ARM PCI DT host bridge drivers, this has been generally > accepted and tested. I don't see why this can't remain and so I'd > really like to keep this around. Sure, no objection, in fact I should/could probably update my new code to use it as well. > Grant, Benjamin would you be happy for me to resubmit this series > which provides the of_pci_range_parser which will be used by the > separate implementations of pci_process_bridge_OF_ranges in > PowerPC/Microblaze? Sure, in fact feel free to update my new code if you have more bandwidth than I do, it should hit Linus tree soon hopefully unless he objects to me having a second pull request this merge window... > Benjamin are you able to still use of_pci_range_parser in your > 'Support per-aperture memory offset' patch? I see no reason why not. I just haven't looked into it much, I admit, being bogged down with a pile of new HW bringup in the lab etc... Cheers, Ben. From mboxrd@z Thu Jan 1 00:00:00 1970 From: benh@kernel.crashing.org (Benjamin Herrenschmidt) Date: Tue, 07 May 2013 20:19:18 +1000 Subject: [PATCH v8 1/3] of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC In-Reply-To: <20130507080142.GA8808@arm.com> References: <1366627295-16964-1-git-send-email-Andrew.Murray@arm.com> <1366627295-16964-2-git-send-email-Andrew.Murray@arm.com> <1367721709.11982.37.camel@pasglop> <20130507080142.GA8808@arm.com> Message-ID: <1367921958.25488.8.camel@pasglop> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org On Tue, 2013-05-07 at 09:01 +0100, Andrew Murray wrote: > > There were no objections to this latest revision until now and it is > currently sitting with Jason Cooper (mvebu-next/pcie). [1] Ok, well I've just sent Linus a pull request for my changes so at least drop the powerpc changes from your tree for the time being. > This is a view that was also shared by Bjorn [2] when I attempted to > submit a patchset which moves struct pci_controller to asm-generic. Right, it's the logical way to go > The motativation for my patchsets were to give a way for ARM PCI host > bridge drivers to parse the DT ranges property, but this snow-balled > into unifying pci_process_bridge_OF_ranges. Which I understand, I would probably have done the same thing in your shoes :-) > My v8 patchset provides a of_pci_range_parser which is used directly > by a few ARM PCI DT host bridge drivers, this has been generally > accepted and tested. I don't see why this can't remain and so I'd > really like to keep this around. Sure, no objection, in fact I should/could probably update my new code to use it as well. > Grant, Benjamin would you be happy for me to resubmit this series > which provides the of_pci_range_parser which will be used by the > separate implementations of pci_process_bridge_OF_ranges in > PowerPC/Microblaze? Sure, in fact feel free to update my new code if you have more bandwidth than I do, it should hit Linus tree soon hopefully unless he objects to me having a second pull request this merge window... > Benjamin are you able to still use of_pci_range_parser in your > 'Support per-aperture memory offset' patch? I see no reason why not. I just haven't looked into it much, I admit, being bogged down with a pile of new HW bringup in the lab etc... Cheers, Ben.