ACK: [PATCH] acpi: pmtt: fix typo from pmmt to pmtt

ivanhu ivan.hu at canonical.com
Wed Aug 2 03:13:35 UTC 2017



On 07/20/2017 04:05 AM, Alex Hung wrote:
> Signed-off-by: Alex Hung <alex.hung at canonical.com>
> ---
>   src/acpi/pmtt/pmtt.c | 24 ++++++++++++------------
>   1 file changed, 12 insertions(+), 12 deletions(-)
> 
> diff --git a/src/acpi/pmtt/pmtt.c b/src/acpi/pmtt/pmtt.c
> index 9bf139a..d8a6108 100644
> --- a/src/acpi/pmtt/pmtt.c
> +++ b/src/acpi/pmtt/pmtt.c
> @@ -39,7 +39,7 @@ static int pmtt_init(fwts_framework *fw)
>   	return FWTS_OK;
>   }
>   
> -static void pmmt_subtable_header_test(fwts_framework *fw, fwts_acpi_table_pmtt_header *entry, bool *passed)
> +static void pmtt_subtable_header_test(fwts_framework *fw, fwts_acpi_table_pmtt_header *entry, bool *passed)
>   {
>   	fwts_log_info_verbatim(fw, "  PMTT Subtable:");
>   	fwts_log_info_verbatim(fw, "    Type:                           0x%2.2" PRIx8, entry->type);
> @@ -80,9 +80,9 @@ static void pmmt_subtable_header_test(fwts_framework *fw, fwts_acpi_table_pmtt_h
>   	}
>   }
>   
> -static void pmmt_physical_component_test(fwts_framework *fw, fwts_acpi_table_pmtt_physical_component *entry, bool *passed)
> +static void pmtt_physical_component_test(fwts_framework *fw, fwts_acpi_table_pmtt_physical_component *entry, bool *passed)
>   {
> -	pmmt_subtable_header_test(fw, &entry->header, passed);
> +	pmtt_subtable_header_test(fw, &entry->header, passed);
>   	fwts_log_info_verbatim(fw, "    Physical Component Identifier:  0x%4.4" PRIx16, entry->component_id);
>   	fwts_log_info_verbatim(fw, "    Reserved:                       0x%4.4" PRIx16, entry->reserved);
>   	fwts_log_info_verbatim(fw, "    Size of DIMM:                   0x%8.8" PRIx32, entry->memory_size);
> @@ -105,13 +105,13 @@ static void pmmt_physical_component_test(fwts_framework *fw, fwts_acpi_table_pmt
>   	}
>   }
>   
> -static void pmmt_controller_test(fwts_framework *fw, fwts_acpi_table_pmtt_controller *entry, bool *passed)
> +static void pmtt_controller_test(fwts_framework *fw, fwts_acpi_table_pmtt_controller *entry, bool *passed)
>   {
>   	fwts_acpi_table_pmtt_header *header;
>   	uint32_t offset = 0;
>   	size_t i;
>   
> -	pmmt_subtable_header_test(fw, &entry->header, passed);
> +	pmtt_subtable_header_test(fw, &entry->header, passed);
>   	fwts_log_info_verbatim(fw, "    Read Latency:                   0x%8.8" PRIx32, entry->read_latency);
>   	fwts_log_info_verbatim(fw, "    Write latency:                  0x%8.8" PRIx32, entry->write_latency);
>   	fwts_log_info_verbatim(fw, "    Read Bandwidth:                 0x%8.8" PRIx32, entry->read_bandwidth);
> @@ -149,7 +149,7 @@ static void pmmt_controller_test(fwts_framework *fw, fwts_acpi_table_pmtt_contro
>   	header = (fwts_acpi_table_pmtt_header *) (((char *) entry) + offset);
>   	while (offset < entry->header.length) {
>   		if (header->type == FWTS_ACPI_PMTT_TYPE_DIMM) {
> -			pmmt_physical_component_test(fw, (fwts_acpi_table_pmtt_physical_component *) header, passed);
> +			pmtt_physical_component_test(fw, (fwts_acpi_table_pmtt_physical_component *) header, passed);
>   		} else {
>   			*passed = false;
>   			fwts_failed(fw, LOG_LEVEL_HIGH,
> @@ -163,12 +163,12 @@ static void pmmt_controller_test(fwts_framework *fw, fwts_acpi_table_pmtt_contro
>   	}
>   }
>   
> -static void pmmt_socket_test(fwts_framework *fw, fwts_acpi_table_pmtt_socket *entry, bool *passed)
> +static void pmtt_socket_test(fwts_framework *fw, fwts_acpi_table_pmtt_socket *entry, bool *passed)
>   {
>   	fwts_acpi_table_pmtt_header *header;
>   	uint32_t offset;
>   Acked-by: Ivan Hu <ivan.hu at canonical.com>
> -	pmmt_subtable_header_test(fw, &entry->header, passed);
> +	pmtt_subtable_header_test(fw, &entry->header, passed);
>   	fwts_log_info_verbatim(fw, "    Socket Identifier:              0x%4.4" PRIx16, entry->socket_id);
>   	fwts_log_info_verbatim(fw, "    Reserved:                       0x%4.4" PRIx16, entry->reserved);
>   
> @@ -184,7 +184,7 @@ static void pmmt_socket_test(fwts_framework *fw, fwts_acpi_table_pmtt_socket *en
>   	header = (fwts_acpi_table_pmtt_header *) (((char *) entry) + offset);
>   	while (offset < entry->header.length) {
>   		if (header->type == FWTS_ACPI_PMTT_TYPE_CONTROLLER) {
> -			pmmt_controller_test(fw, (fwts_acpi_table_pmtt_controller *) header, passed);
> +			pmtt_controller_test(fw, (fwts_acpi_table_pmtt_controller *) header, passed);
>   		} else {
>   			*passed = false;
>   			fwts_failed(fw, LOG_LEVEL_HIGH,
> @@ -230,13 +230,13 @@ static int pmtt_test1(fwts_framework *fw)
>   
>   		switch(entry->type) {
>   			case FWTS_ACPI_PMTT_TYPE_SOCKET:
> -				pmmt_socket_test(fw, (fwts_acpi_table_pmtt_socket *) entry, &passed);
> +				pmtt_socket_test(fw, (fwts_acpi_table_pmtt_socket *) entry, &passed);
>   				break;
>   			case FWTS_ACPI_PMTT_TYPE_CONTROLLER:
> -				pmmt_controller_test(fw, (fwts_acpi_table_pmtt_controller *) entry, &passed);
> +				pmtt_controller_test(fw, (fwts_acpi_table_pmtt_controller *) entry, &passed);
>   				break;
>   			case FWTS_ACPI_PMTT_TYPE_DIMM:
> -				pmmt_physical_component_test(fw, (fwts_acpi_table_pmtt_physical_component *) entry, &passed);
> +				pmtt_physical_component_test(fw, (fwts_acpi_table_pmtt_physical_component *) entry, &passed);
>   				break;
>   			default:
>   				passed = false;
> 

Acked-by: Ivan Hu <ivan.hu at canonical.com>



More information about the fwts-devel mailing list