[coreboot-gerrit] Change in coreboot[master]: autoport: Add support for Haswell-LynxPoint platform

[Iru Cai (Code Review)]

Attention is currently required from: Máté Kukri, Maxim Polyakov, Tristan Corrick, Angel Pons, Arthur Heymans.
Iru Cai has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30890 )

Change subject: autoport: Add support for Haswell-LynxPoint platform
......................................................................

Patch Set 33:

(4 comments)

File util/autoport/lynxpoint.go:

https://review.coreboot.org/c/coreboot/+/30890/comment/c2e8d251_d0db8ae7 
PS1, Line 272: RestoreDIRRoute(sb, "D31IR", uint16(inteltool.RCBA[0x3140]))
             : 	RestoreDIRRoute(sb, "D29IR", uint16(inteltool.RCBA[0x3144]))
             : 	RestoreDIRRoute(sb, "D28IR", uint16(inteltool.RCBA[0x3146]))
             : 	RestoreDIRRoute(sb, "D27IR", uint16(inteltool.RCBA[0x3148]))
             : 	RestoreDIRRoute(sb, "D26IR", uint16(inteltool.RCBA[0x314c]))
             : 	RestoreDIRRoute(sb, "D25IR", uint16(inteltool.RCBA[0x3150]))
             : 	RestoreDIRRoute(sb, "D22IR", uint16(inteltool.RCBA[0x315c]))
             : 	RestoreDIRRoute(sb, "D20IR", uint16(inteltool.RCBA[0x3160]))
> > DxxIR and DxxIP are only useful to support legacy interrupt routing. I'm […]

Removed.

File util/autoport/lynxpoint.go:

https://review.coreboot.org/c/coreboot/+/30890/comment/f077b1dc_6c0afe06 
PS32, Line 199: 				lvl = "LOW"
> +1

Done

https://review.coreboot.org/c/coreboot/+/30890/comment/4141ed9d_26568c21 
PS32, Line 201: 				lvl = "HIGH"
> fmt. […]

Done

https://review.coreboot.org/c/coreboot/+/30890/comment/bf791e19_c48f024a 
PS32, Line 536: !isULT
> I'd flip the if/else branches to avoid the negation in the condition

Done