HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/43283 )
Change subject: src: Remove redundant use of ACPI offset(0) ......................................................................
Patch Set 3:
(9 comments)
Thank you
https://review.coreboot.org/c/coreboot/+/43283/3/src/ec/roda/it8518/acpi/ec.... File src/ec/roda/it8518/acpi/ec.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/ec/roda/it8518/acpi/ec.... PS3, Line 17: Offset(0x00), // [Configuration Space 0x0]
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/mainboard/lippert/front... File src/mainboard/lippert/frontrunner-af/dsdt.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/mainboard/lippert/front... PS3, Line 137: */
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/mainboard/lippert/touca... File src/mainboard/lippert/toucan-af/dsdt.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/mainboard/lippert/touca... PS3, Line 137: */
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/soc/intel/cannonlake/ac... File src/soc/intel/cannonlake/acpi/scs.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/soc/intel/cannonlake/ac... PS3, Line 26: */
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/amd/cimx/sb... File src/southbridge/amd/cimx/sb800/acpi/fch.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/amd/cimx/sb... PS3, Line 216: */
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/bd82x... File src/southbridge/intel/bd82x6x/acpi/pch.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/bd82x... PS3, Line 66: Select
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/i8280... File src/southbridge/intel/i82801ix/acpi/ich9.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/i8280... PS3, Line 35: Select
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/i8280... File src/southbridge/intel/i82801jx/acpi/ich10.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/southbridge/intel/i8280... PS3, Line 36: Select
Here you lose the comment as well
Done
https://review.coreboot.org/c/coreboot/+/43283/3/src/superio/fintek/f81803a/... File src/superio/fintek/f81803a/acpi/superio.asl:
https://review.coreboot.org/c/coreboot/+/43283/3/src/superio/fintek/f81803a/... PS3, Line 237: */
Here you lose the comment as well
Done