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