Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-23 Thread Palmer Dabbelt
On Thu, 08 Jun 2017 01:35:29 PDT (-0700), Arnd Bergmann wrote: > On Thu, Jun 8, 2017 at 10:12 AM, Christoph Hellwig wrote: >> On Wed, Jun 07, 2017 at 09:19:49AM +0200, Geert Uytterhoeven wrote: >>> CC pci folks >> >> Ok, replying with pci folks in Cc then :) >> >> Weak symbols have (rightly) gotte

Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-23 Thread Palmer Dabbelt
On Wed, 07 Jun 2017 01:01:57 PDT (-0700), Arnd Bergmann wrote: > On Wed, Jun 7, 2017 at 9:19 AM, Geert Uytterhoeven > wrote: >> CC pci folks >> >> On Wed, Jun 7, 2017 at 12:59 AM, Palmer Dabbelt wrote: >>> While upstreaming the RISC-V port, it was pointed out that multiple >>> architectures (arc

Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-08 Thread Arnd Bergmann
On Thu, Jun 8, 2017 at 10:12 AM, Christoph Hellwig wrote: > On Wed, Jun 07, 2017 at 09:19:49AM +0200, Geert Uytterhoeven wrote: >> CC pci folks > > Ok, replying with pci folks in Cc then :) > > Weak symbols have (rightly) gotten a bad reputation, so maybe > we should approach this without them. A

Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-08 Thread Christoph Hellwig
On Wed, Jun 07, 2017 at 09:19:49AM +0200, Geert Uytterhoeven wrote: > CC pci folks Ok, replying with pci folks in Cc then :) Weak symbols have (rightly) gotten a bad reputation, so maybe we should approach this without them. It seems we have a large number of emptry pcibios_fixup_bus calls alrea

Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-07 Thread Arnd Bergmann
On Wed, Jun 7, 2017 at 9:19 AM, Geert Uytterhoeven wrote: > CC pci folks > > On Wed, Jun 7, 2017 at 12:59 AM, Palmer Dabbelt wrote: >> While upstreaming the RISC-V port, it was pointed out that multiple >> architectures (arc, arm64, cris, microblaze, sh, tile) have copied the >> mostly empty vers

Re: [PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-07 Thread Geert Uytterhoeven
CC pci folks On Wed, Jun 7, 2017 at 12:59 AM, Palmer Dabbelt wrote: > While upstreaming the RISC-V port, it was pointed out that multiple > architectures (arc, arm64, cris, microblaze, sh, tile) have copied the > mostly empty versions of at least one of these functions. This defines > weakly bou

[PATCH 06/17] pci: Add generic pcibios_{fixup_bus,align_resource}

2017-06-06 Thread Palmer Dabbelt
While upstreaming the RISC-V port, it was pointed out that multiple architectures (arc, arm64, cris, microblaze, sh, tile) have copied the mostly empty versions of at least one of these functions. This defines weakly bound versions of the common functions so other architetures can use them. Signe