[coreboot] New patch to review for coreboot: 9dcc23d w83627hf: add method to retrive wake event source register to ASL include
Christoph Grenz
christophg+cb at grenz-bonn.de
Wed Oct 19 15:22:06 CEST 2011
Hello Paul,
Am Mittwoch, 19. Oktober 2011, um 10:08:07 schrieb Paul Menzel:
> Dear Christoph,
>
> [...]
> > Add a method WAKS to devtree.asl which returns the wake-up status
> > register
>
> Should it be »wake-up *source* register«? The data sheet [1] calls it
> »CRE3 (Read only) Keyboard/Mouse Wake-Up Status register« but source
> seems to be a better description as you also used it in the code.
> I send this message because I got confused by all the different names
> for the same register, if I am not mistaken.
Yes, I mixed my own naming (as *source* register really fits better) and the
one from the data sheet. I think I'll change it everywhere to "wake-up source
register".
>
> > to simplify retrieving the wake source e.g. in \_WAK.
> >
> > Change-Id: Ia258f8fc9ff79b18391c55464da73863889e2255
> > Signed-off-by: Christoph Grenz <christophg+cb at grenz-bonn.de>
> >
> > ---
> >
> [...]
>
> Otherwise I do not know ASL, but I guess you tested that, so it looks
> good to me.
Yeah, I copy-pasted it to my mainboards original DSDT and iasl, acpiexec and
the linux kernel were happy with it.
> Thanks,
>
> Paul
>
>
> [1] http://www.itox.com/pages/support/wdt/W83627HF.pdf
More information about the coreboot
mailing list