Issue #480 has been reported by Paul Menzel.


----------------------------------------

Bug #480: amd/common/acpi/gpio_bank_lib.asl: IASL remarks: Creation of named 
objects within a method is highly inefficient, use globals or method local 
variables instead

https://ticket.coreboot.org/issues/480



* Author: Paul Menzel

* Status: New

* Priority: Normal

* Target version: none

* Start date: 2023-04-10

* Affected versions: master

----------------------------------------

Building AMD boards using AMD common code like *google/kahlee (Careena)* or 
*google/myst*, IASL show the remarks below:



```

$ git log --oneline --no-decorate -1

f2e8865d76 soc/amd/common/blk/pcie: Program LTR max latencies

$ make menuconfig

[…]

$ make -j4

Intel ACPI Component Architecture

ASL+ Optimizing Compiler/Disassembler version 20200925

Copyright (c) 2000 - 2020 Intel Corporation



dsdt.asl     49:  Name(BUFF, Buffer(Local0) {})

Remark   2173 -          ^ Creation of named objects within a method is highly 
inefficient, use globals or method local variables instead (\_SB.S2BF)



dsdt.asl    117:  OperationRegion (GPDW, SystemMemory, Local0, 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.GPRD)



dsdt.asl    126:  OperationRegion (GPDW, SystemMemory, Local0, 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.GPWR)



dsdt.asl    134:  OperationRegion (GPDW, SystemMemory, GPAD (Arg0), 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.STXS)



dsdt.asl    143:  OperationRegion (GPDW, SystemMemory, GPAD (Arg0), 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.CTXS)



dsdt.asl    152:  OperationRegion (GPDW, SystemMemory, GPAD (Arg0), 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.GRXS)



dsdt.asl    162:  OperationRegion (GPDW, SystemMemory, GPAD (Arg0), 4)

Remark   2173 -                      ^ Creation of named objects within a 
method is highly inefficient, use globals or method local variables instead 
(\_SB.GTXS)



ASL Input:     dsdt.asl -   33891 bytes   1278 keywords    901 source lines

AML Output:    dsdt.aml -   13412 bytes    909 opcodes     369 named objects



Compilation successful. 0 Errors, 0 Warnings, 7 Remarks, 295 Optimizations, 46 
Constants Folded

    IASL       3150 warning types were ignored!

    IASL       build/dsdt.aml disassembled correctly.

```



Moved to common in [commit 251d305e73f7 (soc/amd/stoneyridge: Move GPIO support 
to common)](https://review.coreboot.org/c/coreboot/+/32651).







-- 

You have received this notification because you have either subscribed to it, 
or are involved in it.

To change your notification preferences, please click here: 
https://ticket.coreboot.org/my/account

_______________________________________________
coreboot mailing list -- [email protected]
To unsubscribe send an email to [email protected]

Reply via email to