Since my hard drive literally screamed at me today, I figured it is time
to send my current raw v3 diff to the list. Some parts of it are already
acked, some are just reminders something is wrong.
Carl-Daniel
Index: include/device/device.h
===================================================================
--- include/device/device.h (Revision 498)
+++ include/device/device.h (Arbeitskopie)
@@ -202,7 +202,7 @@
struct resource resource[MAX_RESOURCES];
unsigned int resources;
- /* link are (down sream) buses attached to the device, usually a leaf
+ /* link are (downstream) buses attached to the device, usually a leaf
* device with no children have 0 buses attached and a bridge has 1 bus
*/
struct bus link[MAX_LINKS];
Index: mainboard/adl/msm800sev/stage1.c
===================================================================
--- mainboard/adl/msm800sev/stage1.c (Revision 498)
+++ mainboard/adl/msm800sev/stage1.c (Arbeitskopie)
@@ -33,7 +33,8 @@
#include <southbridge/amd/cs5536/cs5536.h>
#include <superio/winbond/w83627hf/w83627hf.h>
-#define SERIAL_DEV 0x30
+#define SERIAL_DEV W83627HF_SP1
+#define SERIAL_IOBASE 0x3f8
void hardware_stage1(void)
{
@@ -49,6 +50,6 @@
* for cs5536
*/
cs5536_disable_internal_uart();
- w83627hf_enable_serial(0x2e, 0x30, 0x3f8);
+ w83627hf_enable_serial(0x2e, SERIAL_DEV, SERIAL_IOBASE);
printk(BIOS_DEBUG, "Done %s\n", __FUNCTION__);
}
Index: device/device.c
===================================================================
--- device/device.c (Revision 498)
+++ device/device.c (Arbeitskopie)
@@ -260,8 +260,10 @@
for (curdev = bus->children; curdev; curdev = curdev->sibling) {
unsigned int links;
int i;
- printk(BIOS_SPEW, "%s: %s(%s) have_resources %d enabled %d\n",
+ printk(BIOS_SPEW,
+ "%s: %s(%s) dtsname %s have_resources %d enabled %d\n",
__func__, bus->dev->dtsname, dev_path(bus->dev),
+ curdev->dtsname,
curdev->have_resources, curdev->enabled);
if (curdev->have_resources) {
continue;
Index: lib/lar.c
===================================================================
--- lib/lar.c (Revision 498)
+++ lib/lar.c (Arbeitskopie)
@@ -90,7 +90,7 @@
* Unsigned pointer arithmetic that wraps to zero can be messy.
*/
for (walk = archive->start;
- (walk < (char *)(archive->start + archive->len - sizeof(struct
lar_header))) &&
+ (walk <= (char *)(archive->start + archive->len - sizeof(struct
lar_header))) &&
(walk >= (char *)archive->start); walk += 16) {
if (strncmp(walk, MAGIC, 8) != 0)
continue;
Index: lib/stage2.c
===================================================================
--- lib/stage2.c (Revision 498)
+++ lib/stage2.c (Arbeitskopie)
@@ -31,8 +31,9 @@
/**
* Main function of the DRAM part of LinuxBIOS.
*
- * LinuxBIOS is divided into pre-DRAM part and DRAM part. The phases before
- * this part are phase 0 and phase 1. This part contains phases x through y.
+ * LinuxBIOS is divided into pre-DRAM part and DRAM part. The stages before
+ * this part are stage 0 and stage 1. This part contains stage 2, which
+ * consists of phases 1 through 6.
*
* Device Enumeration: in the dev_enumerate() phase.
*
@@ -53,6 +54,7 @@
post_code(0x20);
+ /* TODO: Explain why we use printk here although it is impossible */
printk(BIOS_NOTICE, console_test);
dev_init();
--
linuxbios mailing list
[email protected]
http://www.linuxbios.org/mailman/listinfo/linuxbios