svn commit: r347930 - head/sys/dev/pci

[Ruslan Bukin]

On Fri, May 17, 2019 at 05:05:17PM +0000, Emmanuel Vadot wrote:
> Author: manu
> Date: Fri May 17 17:05:16 2019
> New Revision: 347930
> URL: https://svnweb.freebsd.org/changeset/base/347930
> 
> Log:
>   pci: ecam: Correctly parse memory and IO region
>   
>   When activating a resource do not compare the resource id to the adress.
>   Treat IO region as MEMORY region too.
>   
>   Submitted by:	Tuan Phan <tphan at amperecomputing.com> (Original Version)
>   Sponsored by:	Ampere Computing, LLC
>   Differential Revision:	https://reviews.freebsd.org/D20214
> 
> Modified:
>   head/sys/dev/pci/pci_host_generic.c
>   head/sys/dev/pci/pci_host_generic_acpi.c
> 
> Modified: head/sys/dev/pci/pci_host_generic_acpi.c
> ==============================================================================
> --- head/sys/dev/pci/pci_host_generic_acpi.c	Fri May 17 17:04:01 2019	(r347929)
> +++ head/sys/dev/pci/pci_host_generic_acpi.c	Fri May 17 17:05:16 2019	(r347930)
> @@ -293,7 +291,7 @@ pci_host_generic_acpi_attach(device_t dev)
>  			continue; /* empty range element */
>  		if (sc->base.ranges[tuple].flags & FLAG_MEM) {
>  			error = rman_manage_region(&sc->base.mem_rman,
> -			   phys_base, phys_base + size - 1);
> +			   pci_base, pci_base + size - 1);
>  		} else if (sc->base.ranges[tuple].flags & FLAG_IO) {
>  			error = rman_manage_region(&sc->base.io_rman,
>  			   pci_base + PCI_IO_WINDOW_OFFSET,
> 

Don't we need to make a similar change to pci_host_generic_fdt.c ?

Ruslan