| Message ID | 20211026075257.3785036-1-ishii.shuuichir@fujitsu.com (mailing list archive) |
|---|---|
| State | Superseded, archived |
| Headers | show |
| Series | ACPI: Add AEST in ACPI Table Definitions | expand |
ping? P.S. We should have added the maintainer of ACPI FOR ARM64 (ACPI/arm64) first, but since AEST is an arm-spec ACPI table, added the concerned persons as new e-mail addresses. > -----Original Message----- > From: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > Sent: Tuesday, October 26, 2021 4:53 PM > To: rjw@rjwysocki.net; lenb@kernel.org; linux-acpi@vger.kernel.org; > linux-kernel@vger.kernel.org; robert.moore@intel.com; erik.kaneda@intel.com; > rafael.j.wysocki@intel.com; devel@acpica.org > Cc: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com> > Subject: [PATCH] ACPI: Add AEST in ACPI Table Definitions > > When We added AEST using the Upgrading ACPI tables via initrd function, the > kernel could not recognize the AEST, so We added AEST the ACPI table definition. > > Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > --- > drivers/acpi/tables.c | 2 +- > include/acpi/actbl2.h | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c index > f9383736fa0f..ab0fb4c33e07 100644 > --- a/drivers/acpi/tables.c > +++ b/drivers/acpi/tables.c > @@ -499,7 +499,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) > > diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index > a47b32a5cbde..b586e40d4b86 100644 > --- a/include/acpi/actbl2.h > +++ b/include/acpi/actbl2.h > @@ -48,6 +48,7 @@ > #define ACPI_SIG_SDEV "SDEV" /* Secure Devices table */ > #define ACPI_SIG_NHLT "NHLT" /* Non-HDAudio Link Table > */ > #define ACPI_SIG_SVKL "SVKL" /* Storage Volume Key > Location Table */ > +#define ACPI_SIG_AEST "AEST" /* Arm Error Source Table */ > > /* > * All tables must be byte-packed to match the ACPI specification, since > -- > 2.27.0
On Thu, Nov 4, 2021 at 8:14 AM ishii.shuuichir@fujitsu.com <ishii.shuuichir@fujitsu.com> wrote: > > ping? > > P.S. > We should have added the maintainer of ACPI FOR ARM64 (ACPI/arm64) first, > but since AEST is an arm-spec ACPI table, added the concerned persons > as new e-mail addresses. Please resend the patch, then, with all of the requisite addresses present in the CC list. > > -----Original Message----- > > From: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > > Sent: Tuesday, October 26, 2021 4:53 PM > > To: rjw@rjwysocki.net; lenb@kernel.org; linux-acpi@vger.kernel.org; > > linux-kernel@vger.kernel.org; robert.moore@intel.com; erik.kaneda@intel.com; > > rafael.j.wysocki@intel.com; devel@acpica.org > > Cc: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com> > > Subject: [PATCH] ACPI: Add AEST in ACPI Table Definitions > > > > When We added AEST using the Upgrading ACPI tables via initrd function, the > > kernel could not recognize the AEST, so We added AEST the ACPI table definition. > > > > Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > > --- > > drivers/acpi/tables.c | 2 +- > > include/acpi/actbl2.h | 1 + > > 2 files changed, 2 insertions(+), 1 deletion(-) > > > > diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c index > > f9383736fa0f..ab0fb4c33e07 100644 > > --- a/drivers/acpi/tables.c > > +++ b/drivers/acpi/tables.c > > @@ -499,7 +499,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) > > > > diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index > > a47b32a5cbde..b586e40d4b86 100644 > > --- a/include/acpi/actbl2.h > > +++ b/include/acpi/actbl2.h > > @@ -48,6 +48,7 @@ > > #define ACPI_SIG_SDEV "SDEV" /* Secure Devices table */ > > #define ACPI_SIG_NHLT "NHLT" /* Non-HDAudio Link Table > > */ > > #define ACPI_SIG_SVKL "SVKL" /* Storage Volume Key > > Location Table */ > > +#define ACPI_SIG_AEST "AEST" /* Arm Error Source Table */ > > > > /* > > * All tables must be byte-packed to match the ACPI specification, since > > -- > > 2.27.0 >
Hi, Rafael. Thank you for your comments. We will respond as you suggested. Best regards, Shuuichirou. > -----Original Message----- > From: Rafael J. Wysocki <rafael@kernel.org> > Sent: Thursday, November 4, 2021 8:31 PM > To: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com> > Cc: rjw@rjwysocki.net; lenb@kernel.org; linux-acpi@vger.kernel.org; > linux-kernel@vger.kernel.org; robert.moore@intel.com; > rafael.j.wysocki@intel.com; devel@acpica.org; lorenzo.pieralisi@arm.com; > guohanjun@huawei.com; sudeep.holla@arm.com; > linux-arm-kernel@lists.infradead.org > Subject: Re: [PATCH] ACPI: Add AEST in ACPI Table Definitions > > On Thu, Nov 4, 2021 at 8:14 AM ishii.shuuichir@fujitsu.com > <ishii.shuuichir@fujitsu.com> wrote: > > > > ping? > > > > P.S. > > We should have added the maintainer of ACPI FOR ARM64 (ACPI/arm64) > > first, but since AEST is an arm-spec ACPI table, added the concerned > > persons as new e-mail addresses. > > Please resend the patch, then, with all of the requisite addresses present in the > CC list. > > > > -----Original Message----- > > > From: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > > > Sent: Tuesday, October 26, 2021 4:53 PM > > > To: rjw@rjwysocki.net; lenb@kernel.org; linux-acpi@vger.kernel.org; > > > linux-kernel@vger.kernel.org; robert.moore@intel.com; > > > erik.kaneda@intel.com; rafael.j.wysocki@intel.com; devel@acpica.org > > > Cc: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir@fujitsu.com> > > > Subject: [PATCH] ACPI: Add AEST in ACPI Table Definitions > > > > > > When We added AEST using the Upgrading ACPI tables via initrd > > > function, the kernel could not recognize the AEST, so We added AEST the > ACPI table definition. > > > > > > Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> > > > --- > > > drivers/acpi/tables.c | 2 +- > > > include/acpi/actbl2.h | 1 + > > > 2 files changed, 2 insertions(+), 1 deletion(-) > > > > > > diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c index > > > f9383736fa0f..ab0fb4c33e07 100644 > > > --- a/drivers/acpi/tables.c > > > +++ b/drivers/acpi/tables.c > > > @@ -499,7 +499,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) > > > > > > diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index > > > a47b32a5cbde..b586e40d4b86 100644 > > > --- a/include/acpi/actbl2.h > > > +++ b/include/acpi/actbl2.h > > > @@ -48,6 +48,7 @@ > > > #define ACPI_SIG_SDEV "SDEV" /* Secure Devices table > */ > > > #define ACPI_SIG_NHLT "NHLT" /* Non-HDAudio Link > Table > > > */ > > > #define ACPI_SIG_SVKL "SVKL" /* Storage Volume Key > > > Location Table */ > > > +#define ACPI_SIG_AEST "AEST" /* Arm Error Source Table */ > > > > > > /* > > > * All tables must be byte-packed to match the ACPI specification, > > > since > > > -- > > > 2.27.0 > >
diff --git a/drivers/acpi/tables.c b/drivers/acpi/tables.c index f9383736fa0f..ab0fb4c33e07 100644 --- a/drivers/acpi/tables.c +++ b/drivers/acpi/tables.c @@ -499,7 +499,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) diff --git a/include/acpi/actbl2.h b/include/acpi/actbl2.h index a47b32a5cbde..b586e40d4b86 100644 --- a/include/acpi/actbl2.h +++ b/include/acpi/actbl2.h @@ -48,6 +48,7 @@ #define ACPI_SIG_SDEV "SDEV" /* Secure Devices table */ #define ACPI_SIG_NHLT "NHLT" /* Non-HDAudio Link Table */ #define ACPI_SIG_SVKL "SVKL" /* Storage Volume Key Location Table */ +#define ACPI_SIG_AEST "AEST" /* Arm Error Source Table */ /* * All tables must be byte-packed to match the ACPI specification, since
When We added AEST using the Upgrading ACPI tables via initrd function, the kernel could not recognize the AEST, so We added AEST the ACPI table definition. Signed-off-by: Shuuichirou Ishii <ishii.shuuichir@fujitsu.com> --- drivers/acpi/tables.c | 2 +- include/acpi/actbl2.h | 1 + 2 files changed, 2 insertions(+), 1 deletion(-)