All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] ACPI: Add AEST in ACPI Table Definitions
@ 2021-10-26  7:52 Shuuichirou Ishii
  2021-11-04  7:07   ` ishii.shuuichir
  0 siblings, 1 reply; 8+ messages in thread
From: Shuuichirou Ishii @ 2021-10-26  7:52 UTC (permalink / raw)
  To: rjw, lenb, linux-acpi, linux-kernel, robert.moore, erik.kaneda,
	rafael.j.wysocki, devel
  Cc: ishii.shuuichir

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


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions
  2021-10-26  7:52 [PATCH] ACPI: Add AEST in ACPI Table Definitions Shuuichirou Ishii
@ 2021-11-04  7:07   ` ishii.shuuichir
  0 siblings, 0 replies; 8+ messages in thread
From: ishii.shuuichir @ 2021-11-04  7:07 UTC (permalink / raw)
  To: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, 'lorenzo.pieralisi@arm.com',
	'guohanjun@huawei.com', 'sudeep.holla@arm.com',
	'linux-arm-kernel@lists.infradead.org'
  Cc: ishii.shuuichir

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


_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions
@ 2021-11-04  7:07   ` ishii.shuuichir
  0 siblings, 0 replies; 8+ messages in thread
From: ishii.shuuichir @ 2021-11-04  7:07 UTC (permalink / raw)
  To: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, 'lorenzo.pieralisi@arm.com',
	'guohanjun@huawei.com', 'sudeep.holla@arm.com',
	'linux-arm-kernel@lists.infradead.org'
  Cc: ishii.shuuichir

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


^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] ACPI: Add AEST in ACPI Table Definitions
  2021-11-04  7:07   ` ishii.shuuichir
  (?)
@ 2021-11-04 11:31     ` Rafael J. Wysocki
  -1 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2021-11-04 11:31 UTC (permalink / raw)
  To: ishii.shuuichir
  Cc: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, lorenzo.pieralisi, guohanjun,
	sudeep.holla, linux-arm-kernel

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
>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [PATCH] ACPI: Add AEST in ACPI Table Definitions
@ 2021-11-04 11:31     ` Rafael J. Wysocki
  0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2021-11-04 11:31 UTC (permalink / raw)
  To: ishii.shuuichir
  Cc: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, lorenzo.pieralisi, guohanjun,
	sudeep.holla, linux-arm-kernel

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
>

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [Devel] Re: [PATCH] ACPI: Add AEST in ACPI Table Definitions
@ 2021-11-04 11:31     ` Rafael J. Wysocki
  0 siblings, 0 replies; 8+ messages in thread
From: Rafael J. Wysocki @ 2021-11-04 11:31 UTC (permalink / raw)
  To: devel

[-- Attachment #1: Type: text/plain, Size: 2557 bytes --]

On Thu, Nov 4, 2021 at 8:14 AM ishii.shuuichir(a)fujitsu.com
<ishii.shuuichir(a)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(a)fujitsu.com>
> > Sent: Tuesday, October 26, 2021 4:53 PM
> > To: rjw(a)rjwysocki.net; lenb(a)kernel.org; linux-acpi(a)vger.kernel.org;
> > linux-kernel(a)vger.kernel.org; robert.moore(a)intel.com; erik.kaneda(a)intel.com;
> > rafael.j.wysocki(a)intel.com; devel(a)acpica.org
> > Cc: Ishii, Shuuichirou/石井 周一郎 <ishii.shuuichir(a)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(a)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
>

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions
  2021-11-04 11:31     ` Rafael J. Wysocki
@ 2021-11-04 23:22       ` ishii.shuuichir
  -1 siblings, 0 replies; 8+ messages in thread
From: ishii.shuuichir @ 2021-11-04 23:22 UTC (permalink / raw)
  To: 'Rafael J. Wysocki'
  Cc: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, lorenzo.pieralisi, guohanjun,
	sudeep.holla, linux-arm-kernel

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
> >
_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

^ permalink raw reply	[flat|nested] 8+ messages in thread

* RE: [PATCH] ACPI: Add AEST in ACPI Table Definitions
@ 2021-11-04 23:22       ` ishii.shuuichir
  0 siblings, 0 replies; 8+ messages in thread
From: ishii.shuuichir @ 2021-11-04 23:22 UTC (permalink / raw)
  To: 'Rafael J. Wysocki'
  Cc: rjw, lenb, linux-acpi, linux-kernel, robert.moore,
	rafael.j.wysocki, devel, lorenzo.pieralisi, guohanjun,
	sudeep.holla, linux-arm-kernel

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
> >

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2021-11-04 23:29 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-10-26  7:52 [PATCH] ACPI: Add AEST in ACPI Table Definitions Shuuichirou Ishii
2021-11-04  7:07 ` ishii.shuuichir
2021-11-04  7:07   ` ishii.shuuichir
2021-11-04 11:31   ` Rafael J. Wysocki
2021-11-04 11:31     ` [Devel] " Rafael J. Wysocki
2021-11-04 11:31     ` Rafael J. Wysocki
2021-11-04 23:22     ` ishii.shuuichir
2021-11-04 23:22       ` ishii.shuuichir

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.