[coreboot-gerrit] Change in coreboot[master]: soc/intel/{skylake,common}/acpi/dptf/thermal.asl: Prevent iasl remarks

[Wim Vervoorn (Code Review)]

Wim Vervoorn has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/38428 )

Change subject: soc/intel/{skylake,common}/acpi/dptf/thermal.asl: Prevent iasl remarks
......................................................................

Patch Set 5:

(2 comments)

https://review.coreboot.org/c/coreboot/+/38428/1/src/soc/intel/skylake/acpi/... 
File src/soc/intel/skylake/acpi/dptf/thermal.asl:

https://review.coreboot.org/c/coreboot/+/38428/1/src/soc/intel/skylake/acpi/... 
PS1, Line 96: 		Store( Arg0, Local0)
> Done

Done

https://review.coreboot.org/c/coreboot/+/38428/4/src/soc/intel/skylake/acpi/... 
File src/soc/intel/skylake/acpi/dptf/thermal.asl:

https://review.coreboot.org/c/coreboot/+/38428/4/src/soc/intel/skylake/acpi/... 
PS4, Line 89: #ifndef EC_ENABLE_MULTIPLE_DPTF_PROFILES
            : 	/* Prevent iasl remarks about unused parameters */
            : 	Store(Arg0, Local0)
            : #endif
            : 	Store(Arg1, Local0)
> If I add this as well, it works. I will create an updated patch. […]

Done