thank you! Marc
what is the scheme of of the pci scanning?
when a new pci device is detected, all the pci device are re-scanned?
for example,bridge 00:14.4 is scanned two times. there is a vga device
connected to this bridge and the vga is allocted different bus number
in the scanning. so I don't think it is resonable.
Thank youi!
On Thu, Sep 27, 2012 at 1:15 AM, Marc Jones <marcj303(a)gmail.com> wrote:
> On Wed, Sep 26, 2012 at 4:14 AM, 王思源 <wangsiyuanbuaa(a)gmail.com> wrote:
>> it seems that the pci devices are scanned two rounds in our mainboard.
>> first round: bus 0 -> bridge 00:14.4 -> bus 1 (this is the vga device
>> under the bridge 00:14.4)
>> second round: bus 0 -> bridge 00:02.0 -> bridge 00:04.0 -> bridge
>> 00:05.0. this bridges are all pcie bridges.
>>
>> I want to know whether the two rounds scanning style is right. if it
>> is designed to scan two rounds, why?
>> any reply is helpful to me.
>> thank you!
>>
> Yes, there are multiple steps in device tress and pci resource
> discovery and allocation.
>
>> WANG Siyuan
>>
>> --
>> coreboot mailing list: coreboot(a)coreboot.org
>> http://www.coreboot.org/mailman/listinfo/coreboot
>
>
>
> --
> http://se-eng.com
--
王思源
Guys,
It's been quite a while since I worked with any coreboot tools.
I see that a lot has changed...
I am posting a trivial FreeBSD-related patch here, but I am ready to follow the
proper way if you give me a kick towards it :-)
Thank you!
diff --git a/util/superiotool/Makefile b/util/superiotool/Makefile
index 41625d3..2388f4e 100644
--- a/util/superiotool/Makefile
+++ b/util/superiotool/Makefile
@@ -41,7 +41,7 @@ LIBS = -framework IOKit -framework DirectHW -lpci -lz
endif
ifeq ($(OS_ARCH), FreeBSD)
CFLAGS = -O2 -Wall -Werror -Wstrict-prototypes -Wundef -Wstrict-aliasing \
- -Werror-implicit-function-declaration -ansi $(SVNDEF) \
+ -Werror-implicit-function-declaration -ansi $(VERSION) \
-I/usr/local/include
LDFLAGS += -L/usr/local/lib
LIBS = -lz
--
Andriy Gapon
the following patch was just integrated into master:
commit a8649ba43bcaef355b75ef9e11048342fa03ce71
Author: Patrick Georgi <patrick.georgi(a)secunet.com>
Date: Mon Sep 24 20:06:27 2012 +0200
libpayload: Extend CMOS access library
libpayload already contained a number of functions for convenient
access to CMOS configuration. Add functions to support iteration
over available enum fields.
Change-Id: If95f45d7223d2e19c42f1d8680c12d23f6890a01
Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com>
Build-Tested: build bot (Jenkins) at Tue Sep 25 12:19:35 2012, giving +1
Reviewed-By: Stefan Reinauer <stefan.reinauer(a)coreboot.org> at Wed Sep 26 00:20:51 2012, giving +2
See http://review.coreboot.org/1538 for details.
-gerrit
Anton Kochkov (anton.kochkov(a)gmail.com) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1541
-gerrit
commit 83a039fde2ec1516d0b38971b1e9db1605f38745
Author: Anton Kochkov <anton.kochkov(a)gmail.com>
Date: Wed Sep 26 22:31:04 2012 +0400
libpayload: Set 8bits per char for serial port
Previously we assume that hardware using 8 bits
per char by default, but on Asrock A53 Pro
this is not true (7 bit per char by default).
Forcing use 8n1 now.
Change-Id: Ib701725d2ec6dacd7862016b2045270956b27029
Signed-off-by: Anton Kochkov <anton.kochkov(a)gmail.com>
---
payloads/libpayload/drivers/serial.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/payloads/libpayload/drivers/serial.c b/payloads/libpayload/drivers/serial.c
index c6804d2..0f79b52 100644
--- a/payloads/libpayload/drivers/serial.c
+++ b/payloads/libpayload/drivers/serial.c
@@ -40,8 +40,6 @@ static void serial_io_hardware_init(int port, int speed, int word_bits, int pari
{
unsigned char reg;
- /* We will assume 8n1 for now. Does anyone use anything else these days? */
-
/* Disable interrupts. */
outb(0, port + 0x01);
@@ -56,8 +54,9 @@ static void serial_io_hardware_init(int port, int speed, int word_bits, int pari
outb(DIVISOR(speed) & 0xFF, port);
outb(DIVISOR(speed) >> 8 & 0xFF, port + 1);
- /* Restore the previous value of the divisor. */
- outb(reg & ~0x80, port + 0x03);
+ /* Restore the previous value of the divisor.
+ * And set 8 bits per character */
+ outb((reg & ~0x80) | 3, port + 0x03);
}
static void serial_mem_hardware_init(int port, int speed, int word_bits, int parity, int stop_bits)
@@ -80,8 +79,9 @@ static void serial_mem_hardware_init(int port, int speed, int word_bits, int par
writeb(DIVISOR(speed) & 0xFF, MEMBASE);
writeb(DIVISOR(speed) >> 8 & 0xFF, MEMBASE + 1);
- /* Restore the previous value of the divisor. */
- writeb(reg & ~0x80, MEMBASE + 0x03);
+ /* Restore the previous value of the divisor.
+ * And set 8 bits per character */
+ writeb((reg & ~0x80) | 3, MEMBASE + 0x03);
}
#endif
Aladyshev Konstantin (kostr(a)list.ru) just uploaded a new patch set to gerrit, which you can find at http://review.coreboot.org/1540
-gerrit
commit 18ae0cc9515987ce5725185f6b7f99fd0d9a3a2b
Author: Kostr <aladyshev(a)nicevt.ru>
Date: Wed Sep 26 22:11:20 2012 +0400
Mainboard: Fix IO-HUB link number in Dinar mainboard
According to file "northbridge.c" in family 15h code
IO-HUB should be placed on link_lsit[0] in devicetree.cb.
This hack in "northbridge.c" was made to satisfy both f10 and f15 cpu's.
Change-Id: I4754235bd38239460347b0dc4a82cd4e58ae7cd0
Signed-off-by: Kostr <aladyshev(a)nicevt.ru>
---
src/mainboard/amd/dinar/devicetree.cb | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/src/mainboard/amd/dinar/devicetree.cb b/src/mainboard/amd/dinar/devicetree.cb
index 92fe521..3211a2f 100644
--- a/src/mainboard/amd/dinar/devicetree.cb
+++ b/src/mainboard/amd/dinar/devicetree.cb
@@ -26,8 +26,7 @@ chip northbridge/amd/agesa/family15/root_complex
device pci_domain 0 on
subsystemid 0x1022 0x1705 inherit
chip northbridge/amd/agesa/family15 # CPU side of HT root complex
- device pci 18.0 on end # Link 0
- device pci 18.0 on # Link 1, IO-HUB on socket0 link 2(internal Node0 Link 1)
+ device pci 18.0 on # Put IO-HUB at link_num 0, Instead of HT Link topology to satisfy both f10 and f15 CPUs
chip northbridge/amd/cimx/rd890 # North Bridge PCI side of HT Root complex
device pci 0.0 on end # HT Root Complex
device pci 0.1 off end # CLKCONFIG
On Wed, Sep 26, 2012 at 4:14 AM, 王思源 <wangsiyuanbuaa(a)gmail.com> wrote:
> it seems that the pci devices are scanned two rounds in our mainboard.
> first round: bus 0 -> bridge 00:14.4 -> bus 1 (this is the vga device
> under the bridge 00:14.4)
> second round: bus 0 -> bridge 00:02.0 -> bridge 00:04.0 -> bridge
> 00:05.0. this bridges are all pcie bridges.
>
> I want to know whether the two rounds scanning style is right. if it
> is designed to scan two rounds, why?
> any reply is helpful to me.
> thank you!
>
Yes, there are multiple steps in device tress and pci resource
discovery and allocation.
> WANG Siyuan
>
> --
> coreboot mailing list: coreboot(a)coreboot.org
> http://www.coreboot.org/mailman/listinfo/coreboot
--
http://se-eng.com
the following patch was just integrated into master:
commit f7870767f5a5b82ec2eeb14afc0e7d7740bf4652
Author: Patrick Georgi <patrick.georgi(a)secunet.com>
Date: Mon Sep 24 20:56:13 2012 +0200
HAVE_HIGH_TABLES is gone
... but no-one told intel/sch.
Change-Id: I68eaae6910bd6fc579c35b5bc038b9597cd1b3e7
Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com>
Build-Tested: build bot (Jenkins) at Tue Sep 25 12:06:47 2012, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Tue Sep 25 16:09:19 2012, giving +2
See http://review.coreboot.org/1537 for details.
-gerrit
the following patch was just integrated into master:
commit 1fc1ca9986ad93f30eb82d23e8024a04398ea17f
Author: Patrick Georgi <patrick.georgi(a)secunet.com>
Date: Mon Sep 24 20:52:42 2012 +0200
abuild: abort if payload.sh failed
With this chance it becomes practical to have payload.sh build/update
the payload, and abort abuild if something bad happened.
Change-Id: Iee25de2e8b62153c477b8e5d32e097b59797523c
Signed-off-by: Patrick Georgi <patrick.georgi(a)secunet.com>
Build-Tested: build bot (Jenkins) at Tue Sep 25 11:53:51 2012, giving +1
Reviewed-By: Ronald G. Minnich <rminnich(a)gmail.com> at Tue Sep 25 16:08:53 2012, giving +2
See http://review.coreboot.org/1536 for details.
-gerrit