From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Thu, 18 Apr 2013 09:06:55 -0400 From: Jason Cooper To: Grant Likely Subject: Re: [PATCH v7 1/3] of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC Message-ID: <20130418130655.GH27197@titan.lakedaemon.net> References: <1366107508-12672-1-git-send-email-Andrew.Murray@arm.com> <1366107508-12672-2-git-send-email-Andrew.Murray@arm.com> <20130416103005.GB12726@arm.com> <20130417160015.777453E2B73@localhost> <20130417161036.GB27197@titan.lakedaemon.net> <20130417162223.GC27197@titan.lakedaemon.net> <20130418124832.D2C733E118C@localhost> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii In-Reply-To: <20130418124832.D2C733E118C@localhost> 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 , Gregory CLEMENT , "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" , "arnd@arndb.de" , "devicetree-discuss@lists.ozlabs.org" , "rob.herring@calxeda.com" , "kgene.kim@samsung.com" , "bh elgaas@g oogle.com" , "linux-arm-kernel@lists.infradead.org" , "thomas.petazzoni@free-electrons.com" , "monstr@monstr.eu" , "linux-kernel@vger.kernel.org" , Andrew Lunn , "suren.reddy@samsung.com" , Olof Johansson , Andrew Murray , "linuxppc-dev@lists.ozlabs.org" List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , On Thu, Apr 18, 2013 at 01:48:32PM +0100, Grant Likely wrote: > On Wed, 17 Apr 2013 12:22:23 -0400, Jason Cooper wrote: > > On Wed, Apr 17, 2013 at 05:17:48PM +0100, Grant Likely wrote: > > > On Wed, Apr 17, 2013 at 5:10 PM, Jason Cooper wrote: > > > > On Wed, Apr 17, 2013 at 05:00:15PM +0100, Grant Likely wrote: > > > >> On Tue, 16 Apr 2013 11:30:06 +0100, Andrew Murray wrote: > > > >> > On Tue, Apr 16, 2013 at 11:18:26AM +0100, Andrew Murray wrote: > > > >> > > The pci_process_bridge_OF_ranges function, used to parse the "ranges" > > > >> > > property of a PCI host device, is found in both Microblaze and PowerPC > > > >> > > architectures. These implementations are nearly identical. This patch > > > >> > > moves this common code to a common place. > > > >> > > > > > >> > > Signed-off-by: Andrew Murray > > > >> > > Signed-off-by: Liviu Dudau > > > >> > > Reviewed-by: Rob Herring > > > >> > > Tested-by: Thomas Petazzoni > > > >> > > Tested-by: Linus Walleij > > > >> > > Acked-by: Michal Simek > > > >> > > --- > > > >> > > arch/microblaze/include/asm/pci-bridge.h | 5 +- > > > >> > > arch/microblaze/pci/pci-common.c | 192 ---------------------------- > > > >> > > arch/powerpc/include/asm/pci-bridge.h | 5 +- > > > >> > > arch/powerpc/kernel/pci-common.c | 192 ---------------------------- > > > >> > > > > >> > Is there anyone on linuxppc-dev/linux-mips that can help test this patchset? > > > >> > > > > >> > I've tested that it builds on powerpc with a variety of configs (some which > > > >> > include fsl_pci.c implementation). Though I don't have hardware to verify that > > > >> > it works. > > > >> > > > > >> > I haven't tested this builds or runs on MIPS. > > > >> > > > > >> > You shouldn't see any difference in behaviour or new warnings and PCI devices > > > >> > should continue to operate as before. > > > >> > > > >> I've got through a line-by-line comparison between powerpc, microblaze, > > > >> and then new code. The differences are purely cosmetic, so I have > > > >> absolutely no concerns about this patch. I've applied it to my tree. > > > > > > > > oops. Due to the number of dependencies the mvebu-pcie series has (this > > > > being one of them, we (arm-soc/mvebu) asked if we could take this > > > > through our tree. Rob Herring agreed to this several days ago. Is this > > > > a problem for you? > > > > > > > > It would truly (dogs and cats living together) upset the apple cart for > > > > us at this stage to pipe these through a different tree... > > > > > > Not a problem at all. I'll drop it. > > > > Great! Thanks. > > You can add my Acked-by: Grant Likely to the > first patch. I've not reviewed out the second or third patches yet. > > None of this appears to be in linux-next yet. I've boot tested on > PowerPC, but that isn't the same as an ack by the PowerPC maintainers. > It is probably too late for the whole now since we're after -rc7. > However, if you ask me to, I have absolutely no problem putting the > first patch into my tree for v3.10 merging. I went over the patch > line-by-line and am convinced that it is functionally identical. > > Let me know if you need me to do this. Thanks for the offer, Olof just pulled the branch last night (my PRs were a little late trying to sort all this out), so it should be in shortly. FYI: 65ee348 of/pci: Unify pci_process_bridge_OF_ranges from Microblaze and PowerPC is in arm-soc/mvebu/drivers as well as arm-soc/next/drivers (and arm-soc/for-next if you want to test merging everything...) thx, Jason.