diff mbox series

Patch: Make ACPI subsystem provide CEDT table

Message ID a20337bb-9e83-e387-7d10-a2843f7286b3@boldcoder.com
State New, archived
Headers show
Series Patch: Make ACPI subsystem provide CEDT table | expand

Commit Message

Robert Kiraly Jan. 27, 2022, 10:15 a.m. UTC
This is a simple ACPI patch. I'm submitting a revised version to 
linux-acpi per Rafael Wysocki and linux-cxl per Dan Williams.

Thunderbird has added CRs so "patch -p1 -l" is needed.

==== start of patch ====
Subject: Make ACPI subsystem provide CEDT table
Signed-off-by: Robert Kiraly <me@boldcoder.com>

This is a simple patch to the ACPI subsystem. The patch adds ACPI
_SIG_CEDT to table_sigs[] in  "drivers/acpi/tables.c".

Presently,  CXL code isn't able to make use of the  CEDT table at
the initrd stage. Subsequent to the change, the CEDT table can be
used.

If  a developer  plans to use the CEDT table to  program  the CXL
ports,  this change is needed.  Otherwise,  it isn't  expected to
make a difference.

I certify  compliance with  Developer’s Certificate of Origin 1.1
as listed in:
https://www.kernel.org/doc/html/v4.10/process/submitting-patches.
html

==== end of patch ====

Comments

Rafael J. Wysocki Feb. 4, 2022, 6:45 p.m. UTC | #1
On Thu, Jan 27, 2022 at 11:15 AM Robert Kiraly <me@boldcoder.com> wrote:
>
> This is a simple ACPI patch. I'm submitting a revised version to
> linux-acpi per Rafael Wysocki and linux-cxl per Dan Williams.
>
> Thunderbird has added CRs so "patch -p1 -l" is needed.
>
> ==== start of patch ====
> Subject: Make ACPI subsystem provide CEDT table
> Signed-off-by: Robert Kiraly <me@boldcoder.com>
>
> This is a simple patch to the ACPI subsystem. The patch adds ACPI
> _SIG_CEDT to table_sigs[] in  "drivers/acpi/tables.c".
>
> Presently,  CXL code isn't able to make use of the  CEDT table at
> the initrd stage. Subsequent to the change, the CEDT table can be
> used.
>
> If  a developer  plans to use the CEDT table to  program  the CXL
> ports,  this change is needed.  Otherwise,  it isn't  expected to
> make a difference.
>
> I certify  compliance with  Developer’s Certificate of Origin 1.1
> as listed in:
> https://www.kernel.org/doc/html/v4.10/process/submitting-patches.
> html
>
> --- linux-5.16.old/drivers/acpi/tables.c
> +++ linux-5.16/drivers/acpi/tables.c
> @@ -500,7 +500,7 @@
>         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_CEDT, ACPI_SIG_NHLT };
>
>   #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)
>
> ==== end of patch ====

I've queued up this change for 5.18, but I needed to rebase the patch
and I changed its subject and changelog.

Thanks!
Robert Kiraly Feb. 4, 2022, 7:34 p.m. UTC | #2
On 02/04/2022 10:45AM, Rafael J. Wysocki wrote:
 >
 > I've queued up this change for 5.18, but I needed to rebase the patch
 > and I changed its subject and changelog.
 >
 > Thanks!

Thank you for your time and patience in discussion of issues.
diff mbox series

Patch

--- linux-5.16.old/drivers/acpi/tables.c
+++ linux-5.16/drivers/acpi/tables.c
@@ -500,7 +500,7 @@ 
  	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_CEDT, ACPI_SIG_NHLT };

  #define ACPI_HEADER_SIZE sizeof(struct acpi_table_header)