Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1311466

Re: [PATCH V3 19/21] pci, acpi: Support for ACPI based generic PCI host controller init

From Tomasz Nowicki <tn@semihalf.com>
Newsgroups linux.kernel
Subject Re: [PATCH V3 19/21] pci, acpi: Support for ACPI based generic PCI host controller init
Date 2016-01-18 11:40 +0100
Message-ID <qSfo6-7P5-27@gated-at.bofh.it> (permalink)
References <qQtES-hT-3@gated-at.bofh.it> <qQtET-hT-43@gated-at.bofh.it> <qSes3-78G-43@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


On 18.01.2016 10:25, liudongdong (C) wrote:
> I see here is different from V2 patch,
> in V2 patch,  0x0000022004000000 is cpu addr.
> but in V3 patch, 0x0000022004000000 is pci addr.
> which one is right ?

You are right, I look back at v2 and see bug there. But v3 fixes it, for 
MEM & IO we always need to calculate this way:
CPU address defined as: cpu_addr = res->start + entry->offset;
PCI address: pci_addr = res->start;

>
> 0x0000022004000000  is the value of AddressMinimum.
> AddressMinimum evaluates to a 64-bit integer that specifies the lowest
> possible base address of the Memory range
> QWordMemory ( // 64-bit BAR Windows
>      ResourceProducer, PosDecode,
>      MinFixed, MaxFixed,
>      Cacheable, ReadWrite,
>      0x000000000000000, // Granularity
>      0x0000022004000000, // Min Base Address
>      0x000002200fffffff, // Max Base Address
>      0x0000021ff9000000, // Translate
>      0x000000000c000000 // Length
> )

For your case:
cpu_addr = 0x0000022004000000 + 0x0000021ff9000000:
pci_addr = 0x0000022004000000;

Regards,
Tomasz

Back to linux.kernel | Previous | NextPrevious in thread | Find similar | Unroll thread


Thread

Re: [PATCH V3 19/21] pci, acpi: Support for ACPI based generic PCI  host controller init "liudongdong (C)" <liudongdong3@huawei.com> - 2016-01-18 10:40 +0100
  Re: [PATCH V3 19/21] pci, acpi: Support for ACPI based generic PCI  host controller init Tomasz Nowicki <tn@semihalf.com> - 2016-01-18 11:40 +0100

csiph-web