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

[Myles Watson]

> 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