* Re: [Devel] [PATCH v2] iasl: Update to IORT SMMUv3 disassembling
@ 2017-06-22 15:05 Lorenzo Pieralisi
0 siblings, 0 replies; 3+ messages in thread
From: Lorenzo Pieralisi @ 2017-06-22 15:05 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 3671 bytes --]
Hi Ganapatrao,
On Tue, Jun 20, 2017 at 08:04:13AM +0530, Ganapatrao Kulkarni wrote:
> Hi Lv, Robert,
>
> if you don't have any comments on this patch, can i send you github
> pull request?
I can't find the Linux ACPICA version of this patch in -next so I
suspect it will miss the merge window (and that it is still not ACPICA
upstream), there is not much I can do with the IORT kernel patch if the
ACPICA header is not updated in turn.
Lorenzo
> On Fri, Jun 16, 2017 at 9:03 AM, Ganapatrao Kulkarni
> <ganapatrao.kulkarni(a)cavium.com> wrote:
> > ARM IORT specification has provision to define Proximity domain
> > in SMMUv3 IORT table. Adding required changes to decode
> > Proximity domain of SMMUv3 IORT table.
> >
> > Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni(a)cavium.com>
> >
> > v2:
> > - updated to decode reserved1 and reserved2 in IORT SMMUv3 table
> > as suggested from Hanjun Guo.
> > - Minor updates to variable name and description.
> >
> > v1: first patch
> > ---
> > source/common/dmtbinfo.c | 4 ++++
> > source/include/actbl2.h | 4 ++++
> > 2 files changed, 8 insertions(+)
> >
> > diff --git a/source/common/dmtbinfo.c b/source/common/dmtbinfo.c
> > index 692763f..56dd553 100644
> > --- a/source/common/dmtbinfo.c
> > +++ b/source/common/dmtbinfo.c
> > @@ -1838,6 +1838,7 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Flags), "Flags (decoded below)", 0},
> > {ACPI_DMT_FLAG0, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "COHACC Override", 0},
> > {ACPI_DMT_FLAG1, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "HTTU Override", 0},
> > + {ACPI_DMT_FLAG3, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "Proximity Domain Valid", 0},
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Reserved), "Reserved", 0},
> > {ACPI_DMT_UINT64, ACPI_IORT4_OFFSET (VatosAddress), "VATOS Address", 0},
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Model), "Model", 0},
> > @@ -1845,6 +1846,9 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (PriGsiv), "PRI GSIV", 0},
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (GerrGsiv), "GERR GSIV", 0},
> > {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (SyncGsiv), "Sync GSIV", 0},
> > + {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Pxm), "Proximity Domain", 0},
> > + {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Reserved1), "Reserved", 0},
> > + {ACPI_DMT_UINT16, ACPI_IORT4_OFFSET (Reserved2), "Reserved", 0},
> > ACPI_DMT_TERMINATOR
> > };
> >
> > diff --git a/source/include/actbl2.h b/source/include/actbl2.h
> > index 80495d8..8029a5a 100644
> > --- a/source/include/actbl2.h
> > +++ b/source/include/actbl2.h
> > @@ -1041,6 +1041,9 @@ typedef struct acpi_iort_smmu_v3
> > UINT32 PriGsiv;
> > UINT32 GerrGsiv;
> > UINT32 SyncGsiv;
> > + UINT8 Pxm;
> > + UINT8 Reserved1;
> > + UINT16 Reserved2;
> >
> > } ACPI_IORT_SMMU_V3;
> >
> > @@ -1048,6 +1051,7 @@ typedef struct acpi_iort_smmu_v3
> >
> > #define ACPI_IORT_SMMU_V3_COHACC_OVERRIDE (1)
> > #define ACPI_IORT_SMMU_V3_HTTU_OVERRIDE (1<<1)
> > +#define ACPI_IORT_SMMU_V3_PXM_VALID (1<<3)
> >
> >
> > /*******************************************************************************
> > --
> > 1.8.1.4
> >
>
> thanks
> Ganapat
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [Devel] [PATCH v2] iasl: Update to IORT SMMUv3 disassembling
@ 2017-06-20 2:34 Ganapatrao Kulkarni
0 siblings, 0 replies; 3+ messages in thread
From: Ganapatrao Kulkarni @ 2017-06-20 2:34 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 3173 bytes --]
Hi Lv, Robert,
if you don't have any comments on this patch, can i send you github
pull request?
On Fri, Jun 16, 2017 at 9:03 AM, Ganapatrao Kulkarni
<ganapatrao.kulkarni(a)cavium.com> wrote:
> ARM IORT specification has provision to define Proximity domain
> in SMMUv3 IORT table. Adding required changes to decode
> Proximity domain of SMMUv3 IORT table.
>
> Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni(a)cavium.com>
>
> v2:
> - updated to decode reserved1 and reserved2 in IORT SMMUv3 table
> as suggested from Hanjun Guo.
> - Minor updates to variable name and description.
>
> v1: first patch
> ---
> source/common/dmtbinfo.c | 4 ++++
> source/include/actbl2.h | 4 ++++
> 2 files changed, 8 insertions(+)
>
> diff --git a/source/common/dmtbinfo.c b/source/common/dmtbinfo.c
> index 692763f..56dd553 100644
> --- a/source/common/dmtbinfo.c
> +++ b/source/common/dmtbinfo.c
> @@ -1838,6 +1838,7 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Flags), "Flags (decoded below)", 0},
> {ACPI_DMT_FLAG0, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "COHACC Override", 0},
> {ACPI_DMT_FLAG1, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "HTTU Override", 0},
> + {ACPI_DMT_FLAG3, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "Proximity Domain Valid", 0},
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Reserved), "Reserved", 0},
> {ACPI_DMT_UINT64, ACPI_IORT4_OFFSET (VatosAddress), "VATOS Address", 0},
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Model), "Model", 0},
> @@ -1845,6 +1846,9 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (PriGsiv), "PRI GSIV", 0},
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (GerrGsiv), "GERR GSIV", 0},
> {ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (SyncGsiv), "Sync GSIV", 0},
> + {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Pxm), "Proximity Domain", 0},
> + {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Reserved1), "Reserved", 0},
> + {ACPI_DMT_UINT16, ACPI_IORT4_OFFSET (Reserved2), "Reserved", 0},
> ACPI_DMT_TERMINATOR
> };
>
> diff --git a/source/include/actbl2.h b/source/include/actbl2.h
> index 80495d8..8029a5a 100644
> --- a/source/include/actbl2.h
> +++ b/source/include/actbl2.h
> @@ -1041,6 +1041,9 @@ typedef struct acpi_iort_smmu_v3
> UINT32 PriGsiv;
> UINT32 GerrGsiv;
> UINT32 SyncGsiv;
> + UINT8 Pxm;
> + UINT8 Reserved1;
> + UINT16 Reserved2;
>
> } ACPI_IORT_SMMU_V3;
>
> @@ -1048,6 +1051,7 @@ typedef struct acpi_iort_smmu_v3
>
> #define ACPI_IORT_SMMU_V3_COHACC_OVERRIDE (1)
> #define ACPI_IORT_SMMU_V3_HTTU_OVERRIDE (1<<1)
> +#define ACPI_IORT_SMMU_V3_PXM_VALID (1<<3)
>
>
> /*******************************************************************************
> --
> 1.8.1.4
>
thanks
Ganapat
^ permalink raw reply [flat|nested] 3+ messages in thread
* [Devel] [PATCH v2] iasl: Update to IORT SMMUv3 disassembling
@ 2017-06-16 3:33 Ganapatrao Kulkarni
0 siblings, 0 replies; 3+ messages in thread
From: Ganapatrao Kulkarni @ 2017-06-16 3:33 UTC (permalink / raw)
To: devel
[-- Attachment #1: Type: text/plain, Size: 2843 bytes --]
ARM IORT specification has provision to define Proximity domain
in SMMUv3 IORT table. Adding required changes to decode
Proximity domain of SMMUv3 IORT table.
Signed-off-by: Ganapatrao Kulkarni <ganapatrao.kulkarni(a)cavium.com>
v2:
- updated to decode reserved1 and reserved2 in IORT SMMUv3 table
as suggested from Hanjun Guo.
- Minor updates to variable name and description.
v1: first patch
---
source/common/dmtbinfo.c | 4 ++++
source/include/actbl2.h | 4 ++++
2 files changed, 8 insertions(+)
diff --git a/source/common/dmtbinfo.c b/source/common/dmtbinfo.c
index 692763f..56dd553 100644
--- a/source/common/dmtbinfo.c
+++ b/source/common/dmtbinfo.c
@@ -1838,6 +1838,7 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Flags), "Flags (decoded below)", 0},
{ACPI_DMT_FLAG0, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "COHACC Override", 0},
{ACPI_DMT_FLAG1, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "HTTU Override", 0},
+ {ACPI_DMT_FLAG3, ACPI_IORT4_FLAG_OFFSET (Flags, 0), "Proximity Domain Valid", 0},
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Reserved), "Reserved", 0},
{ACPI_DMT_UINT64, ACPI_IORT4_OFFSET (VatosAddress), "VATOS Address", 0},
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (Model), "Model", 0},
@@ -1845,6 +1846,9 @@ ACPI_DMTABLE_INFO AcpiDmTableInfoIort4[] =
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (PriGsiv), "PRI GSIV", 0},
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (GerrGsiv), "GERR GSIV", 0},
{ACPI_DMT_UINT32, ACPI_IORT4_OFFSET (SyncGsiv), "Sync GSIV", 0},
+ {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Pxm), "Proximity Domain", 0},
+ {ACPI_DMT_UINT8, ACPI_IORT4_OFFSET (Reserved1), "Reserved", 0},
+ {ACPI_DMT_UINT16, ACPI_IORT4_OFFSET (Reserved2), "Reserved", 0},
ACPI_DMT_TERMINATOR
};
diff --git a/source/include/actbl2.h b/source/include/actbl2.h
index 80495d8..8029a5a 100644
--- a/source/include/actbl2.h
+++ b/source/include/actbl2.h
@@ -1041,6 +1041,9 @@ typedef struct acpi_iort_smmu_v3
UINT32 PriGsiv;
UINT32 GerrGsiv;
UINT32 SyncGsiv;
+ UINT8 Pxm;
+ UINT8 Reserved1;
+ UINT16 Reserved2;
} ACPI_IORT_SMMU_V3;
@@ -1048,6 +1051,7 @@ typedef struct acpi_iort_smmu_v3
#define ACPI_IORT_SMMU_V3_COHACC_OVERRIDE (1)
#define ACPI_IORT_SMMU_V3_HTTU_OVERRIDE (1<<1)
+#define ACPI_IORT_SMMU_V3_PXM_VALID (1<<3)
/*******************************************************************************
--
1.8.1.4
^ permalink raw reply related [flat|nested] 3+ messages in thread
end of thread, other threads:[~2017-06-22 15:05 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-06-22 15:05 [Devel] [PATCH v2] iasl: Update to IORT SMMUv3 disassembling Lorenzo Pieralisi
-- strict thread matches above, loose matches on Subject: below --
2017-06-20 2:34 Ganapatrao Kulkarni
2017-06-16 3:33 Ganapatrao Kulkarni
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.