[coreboot] [patch] ck804 ACPI PIC/APIC interrupt logic
Jonathan A. Kollasch
jakllsch at kollasch.net
Wed May 25 18:59:54 CEST 2011
On Mon, May 23, 2011 at 09:53:49PM +0200, Stefan Reinauer wrote:
> * Jonathan A. Kollasch <jakllsch at kollasch.net> [110523 21:28]:
> > Add ACPI automatic PIC/APIC interrupt routing logic for ck804.
> >
> > Signed-off-by: Jonathan Kollasch <jakllsch at kollasch.net>
>
> > Index: src/southbridge/nvidia/ck804/acpi/ck804.asl
> > ===================================================================
> > --- src/southbridge/nvidia/ck804/acpi/ck804.asl (revision 0)
> > +++ src/southbridge/nvidia/ck804/acpi/ck804.asl (revision 0)
> > @@ -0,0 +1,918 @@
> > +/* For _CRS */
> > +Method (ILM0, 1, Serialized) {
>
> For Methods the { should be on a new line.
>
> > + If (LEqual (Arg0, 8)) {
> white spaces look messed up
No, that was intentional, i wanted the LEqual to line up,
though then I didn't pad the single character numbers ...
>
> > + Return (20)
> > + } ElseIf (LEqual (Arg0, 13)) {
> > + Return (21)
> > + } ElseIf (LEqual (Arg0, 2)) {
> > + Return (22)
> > + } ElseIf (LEqual (Arg0, 1)) {
> > + Return (23)
> > + } Else {
> > + Return (0)
> > + }
> > +}
> > +/* For _SRS */
> > +Method (MLI0, 1, Serialized) {
> > + If (LEqual (Arg0, 20)) {
> > + Return (8)
>
> ditto
too much indentation inside?
>
> > +/* For _CRS */
> > +Method (ILM1, 1, Serialized) {
> > + If (LEqual (Arg0, 8)) {
> > + Return (16)
>
> ... and here
>
> > +/* For _SRS */
> > +Method (MLI1, 1, Serialized) {
> > + If (LEqual (Arg0, 16)) {
> > + Return (8)
>
> .. here too.
>
> > +Device (APIB) {
> > + Name (_HID, EISAID ("PNP0C0F"))
>
> Can you call these LNKA/LNKB... we used those names in all other targets
> afaict.
This is intentional, I need a link Device for APIC and PIC for each
device.
Jonathan Kollasch
More information about the coreboot
mailing list