[coreboot] [PATCH 2/2] AMD boards to use new PowerNow infrastructure

Myles Watson mylesgw at gmail.com
Thu Feb 19 04:47:39 CET 2009


> The following differences exist (shortened):
> --- old
> +++ ruik
> -    Scope (_PR)
> +    Scope (\_PR)
>      {
> -        Processor (CPU0, 0x00, 0x00000808, 0x06)
> +        Processor (\_PR.CPU0, 0x00, 0x00000808, 0x06)
>          {
>              Name (_PCT, Package (0x02)
>              {
...
> If you are sure that your code can handle multicore/multiprocessor
> constellations and if you can explain that additional backslash before
> _PR, the patch is

My understanding is that the backslash makes the scoping explicit.  It puts
it in the "root".  So as long as _PR was not inside another scope, \_PR and
_PR should be equivalent.  Similarly, \_PR.CPU0 is the same as CPU0 defined
in \_PR.

The scoping seems to make a difference in the size of the generated AML
code, but besides that shouldn't matter too much.

Thanks,
Myles 





More information about the coreboot mailing list