Re: [PATCH V3 14/21] x86, ia64: Include acpi_pci_{add|remove}_bus to the default pcibios_{add|remove}_bus implementation.

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 01/14/2016 05:33 PM, Lorenzo Pieralisi wrote:
On Wed, Jan 13, 2016 at 02:21:00PM +0100, Tomasz Nowicki wrote:
Arches in subject are the only one who use pcibios_{add|remove}_bus hooks
"x86 and ia64 are the only arches that implement..."

and implement it in the same way. Moreover ARM64 is going to do the same.
"implement them"

So it seams that acpi_pci_{add|remove}_bus is generic enough to be default
s/seams/seems

option for pcibios_{add|remove}_bus hooks. Also, it is always safe to run
acpi_pci_{add|remove}_bus as they have empty stubs for !ACPI case and
return if ACPI has been switched off in run time.

After all we can remove x86 and ia64 pcibios_{add|remove}_bus
implementation.
I think you should rephrase the commit log and fix the typos.
yes, I will.
Signed-off-by: Tomasz Nowicki <tn@xxxxxxxxxxxx>
---
  arch/ia64/pci/pci.c   | 10 ----------
  arch/x86/pci/common.c | 10 ----------
  drivers/pci/probe.c   |  3 +++
  3 files changed, 3 insertions(+), 20 deletions(-)
Patch seems fine to me, I do not see why moving the acpi calls into
generic code would create issues (and why it has not been done before)
as you mentioned in the commit log, so:

Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@xxxxxxx>
Thanks,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-pci" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [DMA Engine]     [Linux Coverity]     [Linux USB]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Greybus]

  Powered by Linux