On Thu, Mar 09, 2023 at 03:11:56PM +0800, Shaoqin Huang wrote: ... > > +#define ACPI_TABLE_HEADER_DEF /* ACPI common table header */ \ > > + u32 signature; /* ACPI signature (4 ASCII characters) */ \ > > + u32 length; /* Length of table, in bytes, including header */ \ > > + u8 revision; /* ACPI Specification minor version # */ \ > > + u8 checksum; /* To make sum of entire table == 0 */ \ > > + u8 oem_id [6]; /* OEM identification */ \ > ^ > > + u8 oem_table_id [8]; /* OEM table identification */ \ > ^ > > + u32 oem_revision; /* OEM revision number */ \ > > + u8 asl_compiler_id [4]; /* ASL compiler vendor ID */ \ > ^ > nit: These space should also be deleted. > Yes, but the fixup belongs in the next patch that does manual style changes. Thanks, drew