Angel Pons has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/45703 )
Change subject: mb/packardbell/ms2290/acpi: Convert 'battery.asl' to ASL 2.0 syntax ......................................................................
Patch Set 6:
(1 comment)
https://review.coreboot.org/c/coreboot/+/45703/6/src/mainboard/packardbell/m... File src/mainboard/packardbell/ms2290/acpi/battery.asl:
https://review.coreboot.org/c/coreboot/+/45703/6/src/mainboard/packardbell/m... PS6, Line 57: /* : * FIXME: in old code "Divide (Local2, 20, Local0, Index(Arg0, 5))" : * the remainder "Local0" is not used. : * So "Local0" is removed from converted code below to ASL 2.0 : */ Fix what? Delete the comment?
Just remove `Local0` in a preceding commit, and explain why it is not used in the commit message.