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