On Tue, 16 Apr 2013 11:30:06 +0100, Andrew Murray <andrew.mur...@arm.com> 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 <andrew.mur...@arm.com> > > Signed-off-by: Liviu Dudau <liviu.du...@arm.com> > > Reviewed-by: Rob Herring <rob.herr...@calxeda.com> > > Tested-by: Thomas Petazzoni <thomas.petazz...@free-electrons.com> > > Tested-by: Linus Walleij <linus.wall...@linaro.org> > > Acked-by: Michal Simek <mon...@monstr.eu> > > --- > > 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. g. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev