[coreboot-gerrit] Change in ...coreboot[master]: sb/intel/i82801gx: Autodisable functions based on devicetree

[HAOUAS Elyes (Code Review)]

HAOUAS Elyes has posted comments on this change. ( https://review.coreboot.org/c/coreboot/+/30244 )

Change subject: sb/intel/i82801gx: Autodisable functions based on devicetree
......................................................................


Patch Set 7:

I've enabled by mistake the device 00:1b.0, as a result, the system hang on (copy of the end of console log):

SMI_STS:
GPE0_STS: GPIO15 GPIO14 GPIO12 GPIO7
ALT_GP_SMI_STS: GPI14 GPI12 GPI7 GPI5 GPI4 GPI3 GPI2 GPI1 GPI0
TCO_STS:
Locking SMM.
CPU_CLUSTER: 0 init finished in 306948 usecs
PCI: 00:1b.0 init ...
Azalia: codec type: Azalia
Azalia: base = e0200000
Azalia: codec_mask = 01
Azalia: Initializing codec #0
PCI: 00:1b.0 init finished in 14634 usecs
PCI: 00:1c.0 init ...
Initializing ICH7 PCIe bridge.
PCI: 00:1c.0 init finished in 4805 usecs
PCI: 00:1c.1 init ...
Initializing ICH7 PCIe bridge.
PCI: 00:1c.1 init finished in 4806 usecs
PCI: 00:1d.0 init ...
UHCI: Setting up controller.. done.
PCI: 00:1d.0 init finished in 5235 usecs
PCI: 00:1d.1 init ...
UHCI: Setting up controller.. done.
PCI: 00:1d.1 init finished in 5235 usecs
PCI: 00:1d.2 init ...
UHCI: Setting up controller.. done.
PCI: 00:1d.2 init finished in 5237 usecs
PCI: 00:1d.3 init ...
UHCI: Setting up controller.. done.
PCI: 00:1d.3 init finished in 5236 usecs
PCI: 00:1d.7 init ...
EHCI: Setting up controller.. done.
PCI: 00:1d.7 init finished in 5244 usecs
PCI: 00:1e.0 init ...
PCI: 00:1e.0 init finished in 2015 usecs
PCI: 00:1e.2 init ...
Initializing AC'97 Audio.
No primary codec. Disabling AC'97 Audio.
PCI: 00:1e.2 init finished in 608002 usecs
PCI: 00:1e.3 init ...
▒▒▒▒▒▒▒


-- 
To view, visit https://review.coreboot.org/c/coreboot/+/30244
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-Change-Id: I83576599538a02d295fe00b35826f98d8c97d1cf
Gerrit-Change-Number: 30244
Gerrit-PatchSet: 7
Gerrit-Owner: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Alexander Couzens <lynxis at fe80.eu>
Gerrit-Reviewer: Angel Pons <th3fanbus at gmail.com>
Gerrit-Reviewer: Arthur Heymans <arthur at aheymans.xyz>
Gerrit-Reviewer: Damien Zammit <damien at zamaudio.com>
Gerrit-Reviewer: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
Gerrit-CC: HAOUAS Elyes <ehaouas at noos.fr>
Gerrit-Comment-Date: Sun, 16 Dec 2018 12:50:32 +0000
Gerrit-HasComments: No
Gerrit-Has-Labels: No
Gerrit-MessageType: comment
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181216/d80e6244/attachment.html>