Re: [PATCH v10 2/5] ACPI: Add CXL protocol error defines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Ben Cheatham wrote:
> Add CXL protocol error defines to include/actbl1.h.
> 
> Signed-off-by: Ben Cheatham <Benjamin.Cheatham@xxxxxxx>
> ---
> 
> I made a pull request for this support in the ACPICA project which has

Hmm, it has been a few months with no movement and nothing in v6.8.

> been accepted (link below), so this patch is temporary and I expect it
> to be dropped once the kernel updates from ACPICA.

Ben, not sure what is happening with the ACPICA process right now, but
since the ACPI_EINJ driver is the only consumer of these defines I would
say that in the spirit of the policy in "include/linux/pci_ids.h":

"Do not add new entries to this file unless the definitions
 are shared between multiple drivers."

...go ahead and defines these locally in einj.c.

#ifndef ACPI_EINJ_CXL_CACHE_CORRECTABLE
#define ACPI_EINJ_CXL_CACHE_CORRECTABLE     (1<<12)
#define ACPI_EINJ_CXL_CACHE_UNCORRECTABLE   (1<<13)
#define ACPI_EINJ_CXL_CACHE_FATAL           (1<<14)
#define ACPI_EINJ_CXL_MEM_CORRECTABLE       (1<<15)
#define ACPI_EINJ_CXL_MEM_UNCORRECTABLE     (1<<16)
#define ACPI_EINJ_CXL_MEM_FATAL             (1<<17)
#endif

Then you can delete them later if they ever get duplicated in actbl1.h.




[Index of Archives]     [Linux IBM ACPI]     [Linux Power Management]     [Linux Kernel]     [Linux Laptop]     [Kernel Newbies]     [Share Photos]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Samba]     [Video 4 Linux]     [Device Mapper]     [Linux Resources]
  Powered by Linux