Thread (5 messages) 5 messages, 2 authors, 2021-11-29

RE: [PATCH v2] acpi/tables: Add AEST in ACPI Table Definition

From: ishii.shuuichir@fujitsu.com <hidden>
Date: 2021-11-28 23:56:22
Also in: linux-acpi, lkml
Subsystem: acpi, the rest · Maintainers: "Rafael J. Wysocki", Linus Torvalds

Hi, Hanjun.

Thank you for your comment.
We apologize for the basic mistake.

We will try to fix it as follows, is it OK?

---
Subject: [PATCH v3] acpi/tables: Add AEST in ACPI Table Definition

When We added AEST using the Upgrading ACPI tables via initrd function,
the kernel could not recognize the AEST, so added the AEST table to
the list to enable the table upgrade function.

Signed-off-by: Shuuichirou Ishii <redacted>
---
v3 : Changed the location of the change log so that it does not
     disappear when the patch is applied.

v2 : The reason for committing in the commit log was not clear,
     so it was pointed out and corrected.

 drivers/acpi/tables.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c
index 71419eb16e09..2699bf7e21ab 100644
--- a/drivers/acpi/tables.c
+++ b/drivers/acpi/tables.c
@@ -500,7 +500,7 @@ static const char table_sigs[][ACPI_NAMESEG_SIZE] __initconst = {
    ACPI_SIG_WDDT, ACPI_SIG_WDRT, ACPI_SIG_DSDT, ACPI_SIG_FADT,
    ACPI_SIG_PSDT, ACPI_SIG_RSDT, ACPI_SIG_XSDT, ACPI_SIG_SSDT,
    ACPI_SIG_IORT, ACPI_SIG_NFIT, ACPI_SIG_HMAT, ACPI_SIG_PPTT,
-   ACPI_SIG_NHLT };
+   ACPI_SIG_NHLT, ACPI_SIG_AEST };

 #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)

--
Best regards,
Shuuichrou.
-----Original Message-----
From: Hanjun Guo <guohanjun@huawei.com>
Sent: Saturday, November 27, 2021 12:01 PM
To: Ishii, Shuuichirou/石井 周一郎 <redacted>;
rafael@kernel.org; lenb@kernel.org; linux-acpi@vger.kernel.org;
linux-kernel@vger.kernel.org; lorenzo.pieralisi@arm.com;
sudeep.holla@arm.com
Cc: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH v2] acpi/tables: Add AEST in ACPI Table Definition

On 2021/11/25 16:32, Shuuichirou Ishii wrote:
quoted
When We added AEST using the Upgrading ACPI tables via initrd
function, the kernel could not recognize the AEST, so added the AEST
table to the list to enable the table upgrade function.

Change log:
v2 : The reason for committing in the commit log was not clear,
      so it was pointed out and corrected.
Please remove the change log in the commit message.
quoted
Signed-off-by: Shuuichirou Ishii <redacted>
---
You can put the change log here then it will be removed when applying this patch.

Thanks
Hanjun
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help