[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Message-ID: <d91565e9ac1c59bd2bb22b06256f5fefcb43a973.1723793768.git.mchehab+huawei@kernel.org>
Date: Fri, 16 Aug 2024 09:37:44 +0200
From: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
To:
Cc: Jonathan Cameron <Jonathan.Cameron@...wei.com>,
Shiju Jose <shiju.jose@...wei.com>,
Mauro Carvalho Chehab <mchehab+huawei@...nel.org>,
"Michael S. Tsirkin" <mst@...hat.com>,
Ani Sinha <anisinha@...hat.com>,
Dongjiu Geng <gengdongjiu1@...il.com>,
Igor Mammedov <imammedo@...hat.com>,
linux-kernel@...r.kernel.org,
qemu-arm@...gnu.org,
qemu-devel@...gnu.org
Subject: [PATCH v8 12/13] acpi/ghes: cleanup generic error data logic
Remove comments that are obvious.
No functional changes.
Signed-off-by: Mauro Carvalho Chehab <mchehab+huawei@...nel.org>
---
hw/acpi/ghes.c | 38 +++++++++++++++-----------------------
1 file changed, 15 insertions(+), 23 deletions(-)
diff --git a/hw/acpi/ghes.c b/hw/acpi/ghes.c
index 4f7b6c5ad2b6..a822a5eafaa0 100644
--- a/hw/acpi/ghes.c
+++ b/hw/acpi/ghes.c
@@ -130,34 +130,28 @@ static void build_ghes_hw_error_notification(GArray *table, const uint8_t type)
* ACPI 6.1: 18.3.2.7.1 Generic Error Data
*/
static void acpi_ghes_generic_error_data(GArray *table,
- const uint8_t *section_type, uint32_t error_severity,
- uint8_t validation_bits, uint8_t flags,
- uint32_t error_data_length, QemuUUID fru_id,
- uint64_t time_stamp)
+ const uint8_t *section_type,
+ uint32_t error_severity,
+ uint8_t validation_bits,
+ uint8_t flags,
+ uint32_t error_data_length,
+ QemuUUID fru_id,
+ uint64_t time_stamp)
{
const uint8_t fru_text[20] = {0};
- /* Section Type */
g_array_append_vals(table, section_type, 16);
-
- /* Error Severity */
build_append_int_noprefix(table, error_severity, 4);
+
/* Revision */
build_append_int_noprefix(table, 0x300, 2);
- /* Validation Bits */
+
build_append_int_noprefix(table, validation_bits, 1);
- /* Flags */
build_append_int_noprefix(table, flags, 1);
- /* Error Data Length */
build_append_int_noprefix(table, error_data_length, 4);
- /* FRU Id */
g_array_append_vals(table, fru_id.data, ARRAY_SIZE(fru_id.data));
-
- /* FRU Text */
g_array_append_vals(table, fru_text, sizeof(fru_text));
-
- /* Timestamp */
build_append_int_noprefix(table, time_stamp, 8);
}
@@ -165,19 +159,17 @@ static void acpi_ghes_generic_error_data(GArray *table,
* Generic Error Status Block
* ACPI 6.1: 18.3.2.7.1 Generic Error Data
*/
-static void acpi_ghes_generic_error_status(GArray *table, uint32_t block_status,
- uint32_t raw_data_offset, uint32_t raw_data_length,
- uint32_t data_length, uint32_t error_severity)
+static void acpi_ghes_generic_error_status(GArray *table,
+ uint32_t block_status,
+ uint32_t raw_data_offset,
+ uint32_t raw_data_length,
+ uint32_t data_length,
+ uint32_t error_severity)
{
- /* Block Status */
build_append_int_noprefix(table, block_status, 4);
- /* Raw Data Offset */
build_append_int_noprefix(table, raw_data_offset, 4);
- /* Raw Data Length */
build_append_int_noprefix(table, raw_data_length, 4);
- /* Data Length */
build_append_int_noprefix(table, data_length, 4);
- /* Error Severity */
build_append_int_noprefix(table, error_severity, 4);
}
--
2.46.0
Powered by blists - more mailing lists