Patchwork [2/2] acpi: acpidump: Add support for ACPI 5.0 PCCT

login
register
mail settings
Submitter Colin King
Date Sept. 18, 2012, 1:02 p.m.
Message ID <1347973329-18245-3-git-send-email-colin.king@canonical.com>
Download mbox | patch
Permalink /patch/184708/
State Accepted
Headers show

Comments

Colin King - Sept. 18, 2012, 1:02 p.m.
From: Colin Ian King <colin.king@canonical.com>

ACPI 5.0 Adds the Platform Communications Channel Table (see
section 14 of the ACPI 5.0 specification).  So dump it out in
acpidump.

Signed-off-by: Colin Ian King <colin.king@canonical.com>
---
 src/acpi/acpidump/acpidump.c |   54 ++++++++++++++++++++++++++++++++++++++++++
 1 file changed, 54 insertions(+)
Alex Hung - Sept. 20, 2012, 6:44 a.m.
On 09/18/2012 09:02 PM, Colin King wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> ACPI 5.0 Adds the Platform Communications Channel Table (see
> section 14 of the ACPI 5.0 specification).  So dump it out in
> acpidump.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>   src/acpi/acpidump/acpidump.c |   54 ++++++++++++++++++++++++++++++++++++++++++
>   1 file changed, 54 insertions(+)
>
> diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c
> index 31a3417..bd75da6 100644
> --- a/src/acpi/acpidump/acpidump.c
> +++ b/src/acpi/acpidump/acpidump.c
> @@ -1620,6 +1620,59 @@ static void acpidump_rasf(fwts_framework *fw, fwts_acpi_table_info *table)
>   	acpi_dump_raw_table(fw, table);
>   }
>
> +/*
> + *  acpidump_pcct()
> + *	dump RASF, see 14 ACPI PCCT (Platform Communications Channel)
> + *	of version 5.0 ACPI spec.
> + */
> +static void acpidump_pcct(fwts_framework *fw, fwts_acpi_table_info *table)
> +{
> +	uint8_t *data = (uint8_t *)table->data;
> +	size_t length = table->length;
> +	uint8_t *ptr  = data;
> +
> +	static fwts_acpidump_field pcct_fields[] = {
> +		FIELD_UINT ("Flags",	fwts_acpi_table_pcct, flags),
> +		FIELD_UINTS("Reserved", fwts_acpi_table_pcct, reserved),
> +		FIELD_END
> +	};
> +
> +	static fwts_acpidump_field type0_fields[] = {
> +		FIELD_UINTS("Reserved",		fwts_acpi_table_pcct_subspace_type_0, reserved),
> +		FIELD_UINT ("Base Address",	fwts_acpi_table_pcct_subspace_type_0, base_address),
> +		FIELD_UINT ("Length",		fwts_acpi_table_pcct_subspace_type_0, length),
> +		FIELD_GAS  ("Doorbell Register",fwts_acpi_table_pcct_subspace_type_0, doorbell_register),
> +		FIELD_UINT ("Doorbell Preserve",fwts_acpi_table_pcct_subspace_type_0, doorbell_preserve),
> +		FIELD_UINT ("Doorbell Write",	fwts_acpi_table_pcct_subspace_type_0, doorbell_write),
> +		FIELD_UINT ("Nominal Latency",	fwts_acpi_table_pcct_subspace_type_0, nominal_latency),
> +		FIELD_UINT ("Max. Access Rate",	fwts_acpi_table_pcct_subspace_type_0, max_periodic_access_rate),
> +		FIELD_UINT ("Min. Turnaround Time", fwts_acpi_table_pcct_subspace_type_0, min_request_turnaround_time),
> +		FIELD_END
> +	};
> +
> +	acpi_dump_table_fields(fw, data, pcct_fields, length, length);
> +
> +	ptr += sizeof(fwts_acpi_table_pcct);
> +
> +	/* Now scan through the array of subspace structures */
> +	while (ptr < data + length) {
> +		fwts_acpi_table_pcct_subspace_header *header =
> +			(fwts_acpi_table_pcct_subspace_header *)ptr;
> +
> +		/* Currently just type 0 is supported */
> +		switch (header->type) {
> +		case 0:
> +			fwts_log_info_verbatum(fw,
> +				"General Communications Subspace Structure (type 0):");
> +			__acpi_dump_table_fields(fw, ptr, type0_fields, ptr - data);
> +			
> +			break;
> +		default:
> +			break;
> +		}
> +		ptr += header->length;
> +	}
> +}
>
>   typedef struct {
>   	char *name;
> @@ -1654,6 +1707,7 @@ static acpidump_table_vec table_vec[] = {
>   	{ "MCFG", 	acpidump_mcfg, 	1 },
>   	{ "MPST",	acpidump_mpst,  1 },
>   	{ "MSCT", 	acpidump_msct, 	1 },
> +	{ "PCCT",	acpidump_pcct,	1 },
>   	{ "PSDT", 	acpidump_amlcode, 1 },
>   	{ "RASF",	acpidump_rasf,	1 },
>   	{ "RSDT", 	acpidump_rsdt, 	1 },
>
Acked-by: Alex Hung <alex.hung@canonical.com>
Keng-Yu Lin - Sept. 20, 2012, 8:29 a.m.
On Tue, Sep 18, 2012 at 9:02 PM, Colin King <colin.king@canonical.com> wrote:
> From: Colin Ian King <colin.king@canonical.com>
>
> ACPI 5.0 Adds the Platform Communications Channel Table (see
> section 14 of the ACPI 5.0 specification).  So dump it out in
> acpidump.
>
> Signed-off-by: Colin Ian King <colin.king@canonical.com>
> ---
>  src/acpi/acpidump/acpidump.c |   54 ++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 54 insertions(+)
>
> diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c
> index 31a3417..bd75da6 100644
> --- a/src/acpi/acpidump/acpidump.c
> +++ b/src/acpi/acpidump/acpidump.c
> @@ -1620,6 +1620,59 @@ static void acpidump_rasf(fwts_framework *fw, fwts_acpi_table_info *table)
>         acpi_dump_raw_table(fw, table);
>  }
>
> +/*
> + *  acpidump_pcct()
> + *     dump RASF, see 14 ACPI PCCT (Platform Communications Channel)
> + *     of version 5.0 ACPI spec.
> + */
> +static void acpidump_pcct(fwts_framework *fw, fwts_acpi_table_info *table)
> +{
> +       uint8_t *data = (uint8_t *)table->data;
> +       size_t length = table->length;
> +       uint8_t *ptr  = data;
> +
> +       static fwts_acpidump_field pcct_fields[] = {
> +               FIELD_UINT ("Flags",    fwts_acpi_table_pcct, flags),
> +               FIELD_UINTS("Reserved", fwts_acpi_table_pcct, reserved),
> +               FIELD_END
> +       };
> +
> +       static fwts_acpidump_field type0_fields[] = {
> +               FIELD_UINTS("Reserved",         fwts_acpi_table_pcct_subspace_type_0, reserved),
> +               FIELD_UINT ("Base Address",     fwts_acpi_table_pcct_subspace_type_0, base_address),
> +               FIELD_UINT ("Length",           fwts_acpi_table_pcct_subspace_type_0, length),
> +               FIELD_GAS  ("Doorbell Register",fwts_acpi_table_pcct_subspace_type_0, doorbell_register),
> +               FIELD_UINT ("Doorbell Preserve",fwts_acpi_table_pcct_subspace_type_0, doorbell_preserve),
> +               FIELD_UINT ("Doorbell Write",   fwts_acpi_table_pcct_subspace_type_0, doorbell_write),
> +               FIELD_UINT ("Nominal Latency",  fwts_acpi_table_pcct_subspace_type_0, nominal_latency),
> +               FIELD_UINT ("Max. Access Rate", fwts_acpi_table_pcct_subspace_type_0, max_periodic_access_rate),
> +               FIELD_UINT ("Min. Turnaround Time", fwts_acpi_table_pcct_subspace_type_0, min_request_turnaround_time),
> +               FIELD_END
> +       };
> +
> +       acpi_dump_table_fields(fw, data, pcct_fields, length, length);
> +
> +       ptr += sizeof(fwts_acpi_table_pcct);
> +
> +       /* Now scan through the array of subspace structures */
> +       while (ptr < data + length) {
> +               fwts_acpi_table_pcct_subspace_header *header =
> +                       (fwts_acpi_table_pcct_subspace_header *)ptr;
> +
> +               /* Currently just type 0 is supported */
> +               switch (header->type) {
> +               case 0:
> +                       fwts_log_info_verbatum(fw,
> +                               "General Communications Subspace Structure (type 0):");
> +                       __acpi_dump_table_fields(fw, ptr, type0_fields, ptr - data);
> +
> +                       break;
> +               default:
> +                       break;
> +               }
> +               ptr += header->length;
> +       }
> +}
>
>  typedef struct {
>         char *name;
> @@ -1654,6 +1707,7 @@ static acpidump_table_vec table_vec[] = {
>         { "MCFG",       acpidump_mcfg,  1 },
>         { "MPST",       acpidump_mpst,  1 },
>         { "MSCT",       acpidump_msct,  1 },
> +       { "PCCT",       acpidump_pcct,  1 },
>         { "PSDT",       acpidump_amlcode, 1 },
>         { "RASF",       acpidump_rasf,  1 },
>         { "RSDT",       acpidump_rsdt,  1 },
> --
> 1.7.10.4
>
Acked-by: Keng-Yu Lin <kengyu@canonical.com>

Patch

diff --git a/src/acpi/acpidump/acpidump.c b/src/acpi/acpidump/acpidump.c
index 31a3417..bd75da6 100644
--- a/src/acpi/acpidump/acpidump.c
+++ b/src/acpi/acpidump/acpidump.c
@@ -1620,6 +1620,59 @@  static void acpidump_rasf(fwts_framework *fw, fwts_acpi_table_info *table)
 	acpi_dump_raw_table(fw, table);
 }
 
+/*
+ *  acpidump_pcct()
+ *	dump RASF, see 14 ACPI PCCT (Platform Communications Channel)
+ *	of version 5.0 ACPI spec.
+ */
+static void acpidump_pcct(fwts_framework *fw, fwts_acpi_table_info *table)
+{
+	uint8_t *data = (uint8_t *)table->data;
+	size_t length = table->length;
+	uint8_t *ptr  = data;
+
+	static fwts_acpidump_field pcct_fields[] = {
+		FIELD_UINT ("Flags",	fwts_acpi_table_pcct, flags),
+		FIELD_UINTS("Reserved", fwts_acpi_table_pcct, reserved),
+		FIELD_END
+	};
+
+	static fwts_acpidump_field type0_fields[] = {
+		FIELD_UINTS("Reserved",		fwts_acpi_table_pcct_subspace_type_0, reserved),
+		FIELD_UINT ("Base Address",	fwts_acpi_table_pcct_subspace_type_0, base_address),
+		FIELD_UINT ("Length",		fwts_acpi_table_pcct_subspace_type_0, length),
+		FIELD_GAS  ("Doorbell Register",fwts_acpi_table_pcct_subspace_type_0, doorbell_register),
+		FIELD_UINT ("Doorbell Preserve",fwts_acpi_table_pcct_subspace_type_0, doorbell_preserve),
+		FIELD_UINT ("Doorbell Write",	fwts_acpi_table_pcct_subspace_type_0, doorbell_write),
+		FIELD_UINT ("Nominal Latency",	fwts_acpi_table_pcct_subspace_type_0, nominal_latency),
+		FIELD_UINT ("Max. Access Rate",	fwts_acpi_table_pcct_subspace_type_0, max_periodic_access_rate),
+		FIELD_UINT ("Min. Turnaround Time", fwts_acpi_table_pcct_subspace_type_0, min_request_turnaround_time),
+		FIELD_END
+	};
+
+	acpi_dump_table_fields(fw, data, pcct_fields, length, length);
+
+	ptr += sizeof(fwts_acpi_table_pcct);
+
+	/* Now scan through the array of subspace structures */
+	while (ptr < data + length) {
+		fwts_acpi_table_pcct_subspace_header *header =
+			(fwts_acpi_table_pcct_subspace_header *)ptr;
+
+		/* Currently just type 0 is supported */
+		switch (header->type) {
+		case 0: 
+			fwts_log_info_verbatum(fw,
+				"General Communications Subspace Structure (type 0):");
+			__acpi_dump_table_fields(fw, ptr, type0_fields, ptr - data);
+			
+			break;
+		default:
+			break;
+		}
+		ptr += header->length;
+	}
+}
 
 typedef struct {
 	char *name;
@@ -1654,6 +1707,7 @@  static acpidump_table_vec table_vec[] = {
 	{ "MCFG", 	acpidump_mcfg, 	1 },
 	{ "MPST",	acpidump_mpst,  1 },
 	{ "MSCT", 	acpidump_msct, 	1 },
+	{ "PCCT",	acpidump_pcct,	1 },
 	{ "PSDT", 	acpidump_amlcode, 1 },
 	{ "RASF",	acpidump_rasf,	1 },
 	{ "RSDT", 	acpidump_rsdt, 	1 },