On 27/05/20 7:01 pm, Jan Kiszka wrote:
On 27.05.20 14:23, [email protected] wrote:
From: Nikhil Devshatwar <[email protected]>
PCI kernel framework requires that all the instances in the
device tree either specify the PCIe domain or none does.
Currently, Jailhouse dynamic overlay describes the PCI domain
which causes problems because root cell DTS does not describe
this.
Fix this by not specifying the domain in root cell config.
Also, fix the size of the PCI target region to match with the
correct description in TRM.
Please rather patch your DTs to use the domain ID (upstream). This here
means that you will get a new domain ID on every Jailhouse start.
+Kishon,
Can you please let me know why are we not using hard coded domains for
PCIe device for J721e?
Regards,
Nikhil D
Jan
Signed-off-by: Nikhil Devshatwar <[email protected]>
---
configs/arm64/k3-j721e-evm.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/configs/arm64/k3-j721e-evm.c b/configs/arm64/k3-j721e-evm.c
index 578caf02..dbf0ca7c 100644
--- a/configs/arm64/k3-j721e-evm.c
+++ b/configs/arm64/k3-j721e-evm.c
@@ -44,7 +44,7 @@ struct {
.pci_mmconfig_base = 0x76000000,
.pci_mmconfig_end_bus = 0,
.pci_is_virtual = 1,
- .pci_domain = 3,
+ .pci_domain = -1,
.arm = {
.gic_version = 3,
.gicd_base = 0x01800000,
@@ -173,7 +173,7 @@ struct {
/* PCIe Core */ {
.phys_start = 0x0d000000,
.virt_start = 0x0d000000,
- .size = 0x01000000,
+ .size = 0x02000000,
.flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
JAILHOUSE_MEM_IO,
},
@@ -398,7 +398,6 @@ struct {
.pci_devices = {
/* 0003:00:01.0 */ {
.type = JAILHOUSE_PCI_TYPE_IVSHMEM,
- .domain = 3,
.bdf = 1 << 3,
.bar_mask = JAILHOUSE_IVSHMEM_BAR_MASK_INTX,
.shmem_regions_start = 0,
--
You received this message because you are subscribed to the Google Groups
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
To view this discussion on the web visit
https://groups.google.com/d/msgid/jailhouse-dev/c57f24ac-80ed-92ba-ec9f-69a53908076b%40ti.com.
