18 Aug
2019
18 Aug
'19
10:37 p.m.
This patchset started off with the aim of building the PCI-based machine
device trees downwards from the root node to avoid switching either the active
package and/or current instance at various points during the tree construction.
The basis for this was to try and remove all named nodes (those which are
placed at a given path) and instead build the hierarchy in order by using
new-device and finish-device as per the IEEE-1275 specification. This was
realised by creating a new BIND_NODE_METHODS() macro which is similar to
REGISTER_NAMED_NODE() except that it doesn't alter either the active package
or the current instance, and then working through the device tree from the
root converting all the devices for the PCI-based machines to use it.
Once this conversion has been completed for all devices beneath the PCI node,
it becomes possible to remove both the REGISTER_NAMED_NODE() and
REGISTER_NAMED_NODE_PHANDLE() macros to prevent future patches from accidentally
changing the tree construction order.
Next the x86, PPC and SPARC64 device tree roots were altered to set the active
package and current instance, allowing the remaining places where the PCI code
alters either of them to be removed.
At this point instance variables were working correctly, and I noticed that
there were several PCI devices which made use of C initialisation functions and/or
properties for initial configuration. These can now be replaced with instance
variables to keep everything simple and consistent.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
v3:
- Rebase onto master
- Fix up USB debug string for keyboard reported by Zoltan
- Fix for creating devices that don't have an entry in the PCI database
- Fix for MacOS 9 not initialising network reported by Howard
v2:
- Fix up NVRAM initialisation so that -prom-env 'vga-ndrv?=false' works again
Mark Cave-Ayland (30):
libopenbios: introduce BIND_NODE_METHODS() macro
pc_kbd: convert to use BIND_NODE_METHODS() macro
pc_serial: convert to use BIND_NODE_METHODS() macro
floppy: convert to use BIND_NODE_METHODS() macro
ide: convert to use BIND_NODE_METHODS() macro
adb: convert to use BIND_NODE_METHODS() macro
escc: convert to use BIND_NODE_METHODS() macro
cuda: convert to use BIND_NODE_METHODS() macro
macio: convert to use BIND_NODE_METHODS() macro
pmu: convert to use BIND_NODE_METHODS() macro
usbhid: convert to use BIND_NODE_METHODS() macro
nvram: convert to use BIND_NODE_METHODS() macro
lsi: don't change active package when setting device alias
lsi: convert to use BIND_NODE_METHODS() macro
virtio: convert to use BIND_NODE_METHODS() macro
pci: convert to use BIND_NODE_METHODS() macro
pci: remove ob_pci_initialize() and ob_pci_empty_node
libopenbios: remove REGISTER_NAMED_NODE and
REGISTER_NAMED_NODE_PHANDLE macros
nvram: ensure that NVRAM configuration is separate from NVRAM node
creation
ppc: move New World uninorth and nvram device node creation to the
root device
ppc: set active package and current instance to root device node
before probe
SPARC64: set active package and current instance to root device node
before probe
x86: set active package and current instance to root device node
before probe
pci: remove explicit find-device from PCI devices
pci: remove explicit setting of my-self from PCI devices
pci: call set-args before configuring PCI device nodes
pc_serial: remove separate init word
pc_kbd: use instance value to initialise C instance parameter
lsi: use instance value to hold sd_private_t pointer
virtio: use instance value to initialise C instance parameter
arch/ppc/qemu/init.c | 18 +++-
arch/sparc64/openbios.c | 6 ++
arch/x86/openbios.c | 9 +-
drivers/adb_bus.c | 57 +++++++-----
drivers/adb_kbd.c | 25 ++++--
drivers/adb_mouse.c | 31 ++++---
drivers/cuda.c | 158 ++++++++++++++++++++-------------
drivers/escc.c | 113 ++++++++++++++---------
drivers/floppy.c | 68 +++++++-------
drivers/ide.c | 123 +++++++++++++++----------
drivers/lsi.c | 65 +++++++-------
drivers/macio.c | 118 ++++++++++++------------
drivers/macio.h | 1 -
drivers/pc_kbd.c | 48 +++++-----
drivers/pc_serial.c | 32 +++----
drivers/pci.c | 132 +++++++++++++--------------
drivers/pmu.c | 152 +++++++++++++++++++++----------
drivers/usbhid.c | 17 ++--
drivers/virtio.c | 56 +++---------
include/drivers/drivers.h | 2 +
include/libopenbios/bindings.h | 19 ++--
include/packages/nvram.h | 4 +-
libopenbios/bindings.c | 10 +++
packages/nvram.c | 35 ++++++--
24 files changed, 755 insertions(+), 544 deletions(-)
--
2.20.1
18 Aug
18 Aug
10:37 p.m.
New subject: [PATCH v3 01/30] libopenbios: introduce BIND_NODE_METHODS() macro
This macro is designed to bind C functions into Forth words during creation
of the device tree, but only into an existing node and without attempting
to create any missing parent nodes.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
include/libopenbios/bindings.h | 8 ++++++++
libopenbios/bindings.c | 10 ++++++++++
2 files changed, 18 insertions(+)
diff --git a/include/libopenbios/bindings.h b/include/libopenbios/bindings.h
index 4ec9789..6360a4f 100644
--- a/include/libopenbios/bindings.h
+++ b/include/libopenbios/bindings.h
@@ -127,6 +127,11 @@ typedef struct {
paths, 1, name##_m, sizeof(name##_m)/sizeof(method_t)); \
} while(0)
+#define BIND_NODE_METHODS(ph, name) do { \
+ bind_node_methods(ph, name##_flags_, name##_size_, \
+ name##_m, sizeof(name##_m)/sizeof(method_t)); \
+} while(0)
+
#define DECLARE_UNNAMED_NODE( name, flags, size ) \
static const int name##_flags_ = flags; \
static const int name##_size_ = size;
@@ -144,6 +149,9 @@ static const method_t name##_m[]
name##_m, sizeof(name##_m)/sizeof(method_t) ); \
} while(0)
+extern void
+bind_node_methods(phandle_t ph, int flags, int size, const method_t *methods, int nmet);
+
extern void bind_node( int flags, int size, const char * const *paths, int npaths,
const method_t *methods, int nmethods );
diff --git a/libopenbios/bindings.c b/libopenbios/bindings.c
index 926944d..9b4308b 100644
--- a/libopenbios/bindings.c
+++ b/libopenbios/bindings.c
@@ -484,6 +484,16 @@ add_methods( int flags, int size, const method_t *methods, int nmet
)
make_openable(0);
}
+void
+bind_node_methods(phandle_t ph, int flags, int size, const method_t *methods, int nmet)
+{
+ phandle_t save_ph = get_cur_dev();
+
+ activate_dev(ph);
+ add_methods(flags, size, methods, nmet);
+ activate_dev( save_ph );
+}
+
void
bind_node( int flags, int size, const char * const *paths, int npaths,
const method_t *methods, int nmet )
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 02/30] pc_kbd: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pc_kbd.c | 25 ++++++++++++++-----------
1 file changed, 14 insertions(+), 11 deletions(-)
diff --git a/drivers/pc_kbd.c b/drivers/pc_kbd.c
index 3b74f8c..ed6b82c 100644
--- a/drivers/pc_kbd.c
+++ b/drivers/pc_kbd.c
@@ -194,7 +194,7 @@ pc_kbd_open(unsigned long *address)
RET ( -1 );
}
-DECLARE_UNNAMED_NODE(pc_kbd, INSTALL_OPEN, sizeof(unsigned long));
+DECLARE_UNNAMED_NODE(pc_kbd, 0, sizeof(unsigned long));
NODE_METHODS(pc_kbd) = {
{ "open", pc_kbd_open },
@@ -219,7 +219,10 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
push_str("8042");
fword("device-name");
-
+
+ /* Make openable */
+ fword("is-open");
+
PUSH((base + offset) >> 32);
fword("encode-int");
PUSH((base + offset) & 0xffffffff);
@@ -259,15 +262,9 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
push_str("interrupts");
fword("property");
-
- fword("finish-device");
-
- /* Keyboard */
- snprintf(nodebuff, sizeof(nodebuff), "%s/8042/%s", path, kdev_name);
- REGISTER_NAMED_NODE(pc_kbd, nodebuff);
- push_str(nodebuff);
- fword("find-device");
+ /* Keyboard */
+ fword("new-device");
push_str(kdev_name);
fword("device-name");
@@ -289,7 +286,11 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
fword("encode-int");
push_str("address");
fword("property");
-
+
+ BIND_NODE_METHODS(get_cur_dev(), pc_kbd);
+ fword("finish-device");
+
+ snprintf(nodebuff, sizeof(nodebuff), "%s/8042/%s", path, kdev_name);
chosen = find_dev("/chosen");
push_str(nodebuff);
fword("open-dev");
@@ -331,4 +332,6 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
fword("finish-device");
}
+
+ fword("finish-device");
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 03/30] pc_serial: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pc_serial.c | 22 ++++++++++++++--------
1 file changed, 14 insertions(+), 8 deletions(-)
diff --git a/drivers/pc_serial.c b/drivers/pc_serial.c
index a638e1f..0d42689 100644
--- a/drivers/pc_serial.c
+++ b/drivers/pc_serial.c
@@ -151,7 +151,7 @@ pc_serial_init(unsigned long *address)
*address = POP();
}
-DECLARE_UNNAMED_NODE(pc_serial, INSTALL_OPEN, sizeof(unsigned long));
+DECLARE_UNNAMED_NODE(pc_serial, 0, sizeof(unsigned long));
NODE_METHODS(pc_serial) = {
{ "init", pc_serial_init },
@@ -168,15 +168,13 @@ ob_pc_serial_init(const char *path, const char *dev_name, uint64_t
base,
phandle_t aliases;
char nodebuff[128];
- snprintf(nodebuff, sizeof(nodebuff), "%s/%s", path, dev_name);
- REGISTER_NAMED_NODE(pc_serial, nodebuff);
-
- push_str(nodebuff);
+ push_str(path);
fword("find-device");
- PUSH(offset);
- PUSH(find_package_method("init", get_cur_dev()));
- fword("execute");
+ fword("new-device");
+
+ push_str(dev_name);
+ fword("device-name");
push_str("serial");
fword("device-type");
@@ -203,6 +201,14 @@ ob_pc_serial_init(const char *path, const char *dev_name, uint64_t
base,
set_int_property(get_cur_dev(), "interrupts", 1);
#endif
+ BIND_NODE_METHODS(get_cur_dev(), pc_serial);
+
+ PUSH(offset);
+ feval("['] init execute");
+
+ fword("finish-device");
+
aliases = find_dev("/aliases");
+ snprintf(nodebuff, sizeof(nodebuff), "%s/%s", path, dev_name);
set_property(aliases, "ttya", nodebuff, strlen(nodebuff) + 1);
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 04/30] floppy: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/floppy.c | 71 +++++++++++++++++++++++++++---------------------
1 file changed, 40 insertions(+), 31 deletions(-)
diff --git a/drivers/floppy.c b/drivers/floppy.c
index 329987b..f34811a 100644
--- a/drivers/floppy.c
+++ b/drivers/floppy.c
@@ -9,7 +9,7 @@
#include "timer.h"
/* DECLARE data structures for the nodes. */
-DECLARE_UNNAMED_NODE( ob_floppy, INSTALL_OPEN, 2*sizeof(int) );
+DECLARE_UNNAMED_NODE( ob_floppy, 0, 2*sizeof(int) );
#ifdef CONFIG_DEBUG_FLOPPY
#define printk_info printk
@@ -1077,22 +1077,6 @@ static void floppy_reset(void)
fdc_state.in_sync = 1;
}
-static void
-ob_floppy_initialize(const char *path)
-{
- int props[3];
- phandle_t ph = find_dev(path);
-
- set_property(ph, "device_type", "block",
sizeof("block"));
-
- // Set dummy reg properties
- props[0] = __cpu_to_be32(0); props[1] = __cpu_to_be32(0); props[2] =
__cpu_to_be32(0);
- set_property(ph, "reg", (char *)&props, 3*sizeof(int));
-
- fword("is-deblocker");
-}
-
-
static void
ob_floppy_open(int *idx)
{
@@ -1159,21 +1143,46 @@ NODE_METHODS(ob_floppy) = {
int ob_floppy_init(const char *path, const char *dev_name,
unsigned long io_base, unsigned long mmio_base)
{
- char nodebuff[128];
- phandle_t aliases;
+ char nodebuff[128];
+ phandle_t aliases;
- snprintf(nodebuff, sizeof(nodebuff), "%s/%s", path, dev_name);
- if (!mmio_base) {
- REGISTER_NAMED_NODE(ob_floppy, nodebuff);
- ob_floppy_initialize(nodebuff);
- } else {
- // Already in tree and mapped
- REGISTER_NODE_METHODS(ob_floppy, nodebuff);
- }
- floppy_init(io_base, mmio_base);
+ push_str(path);
+ fword("find-device");
- aliases = find_dev("/aliases");
- set_property(aliases, "floppy", nodebuff, strlen(nodebuff) + 1);
+ fword("new-device");
- return 0;
+ push_str(dev_name);
+ fword("device-name");
+ push_str("block");
+ fword("device-type");
+
+ if (!mmio_base) {
+ BIND_NODE_METHODS(get_cur_dev(), ob_floppy);
+
+ PUSH(0);
+ fword("encode-int");
+ PUSH(0);
+ fword("encode-int");
+ fword("encode+");
+ PUSH(0);
+ fword("encode-int");
+ fword("encode+");
+
+ push_str("reg");
+ fword("property");
+
+ fword("is-deblocker");
+ } else {
+ // Already in tree and mapped
+ BIND_NODE_METHODS(get_cur_dev(), ob_floppy);
+ }
+ floppy_init(io_base, mmio_base);
+
+ fword("finish-device");
+
+ aliases = find_dev("/aliases");
+ snprintf(nodebuff, sizeof(nodebuff), "%s/%s", path, dev_name);
+ set_property(aliases, "floppy", nodebuff, strlen(nodebuff) + 1);
+
+ return 0;
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 05/30] ide: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/ide.c | 129 +++++++++++++++++++++++++++++++-------------------
1 file changed, 80 insertions(+), 49 deletions(-)
diff --git a/drivers/ide.c b/drivers/ide.c
index 519ad46..18a27c6 100644
--- a/drivers/ide.c
+++ b/drivers/ide.c
@@ -32,8 +32,8 @@ do { printk("IDE - %s: " fmt, __func__ , ##args); } while (0)
#endif
/* DECLARE data structures for the nodes. */
-DECLARE_UNNAMED_NODE( ob_ide, INSTALL_OPEN, sizeof(struct ide_drive*) );
-DECLARE_UNNAMED_NODE( ob_ide_ctrl, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE( ob_ide, 0, sizeof(struct ide_drive*) );
+DECLARE_UNNAMED_NODE( ob_ide_ctrl, 0, sizeof(int));
/*
* define to 2 for the standard 2 channels only
@@ -1224,26 +1224,6 @@ ob_ide_block_size(int *idx)
PUSH(drive->bs);
}
-static void
-ob_ide_initialize(int *idx)
-{
- int props[3];
- phandle_t ph=get_cur_dev();
-
- push_str("block");
- fword("device-type");
-
- // Set dummy reg properties
-
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
-
- props[0] = __cpu_to_be32(0); props[1] = __cpu_to_be32(0); props[2] = __cpu_to_be32(0);
- set_property(ph, "reg", (char *)&props, 3*sizeof(int));
-
- fword("is-deblocker");
-}
-
static void
ob_ide_open(int *idx)
{
@@ -1318,7 +1298,6 @@ ob_ide_dma_sync(int *idx)
}
NODE_METHODS(ob_ide) = {
- { NULL, ob_ide_initialize },
{ "open", ob_ide_open },
{ "close", ob_ide_close },
{ "read-blocks", ob_ide_read_blocks },
@@ -1332,26 +1311,25 @@ NODE_METHODS(ob_ide) = {
};
static void
-ob_ide_ctrl_initialize(int *idx)
+ob_ide_ctrl_decodeunit(int *idx)
{
- phandle_t ph=get_cur_dev();
-
- /* set device type */
- push_str(DEV_TYPE);
- fword("device-type");
+ fword("parse-hex");
+}
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
+static void
+ob_ide_ctrl_open(int *idx)
+{
+ RET(-1);
}
static void
-ob_ide_ctrl_decodeunit(int *idx)
+ob_ide_ctrl_close(int *idx)
{
- fword("parse-hex");
}
NODE_METHODS(ob_ide_ctrl) = {
- { NULL, ob_ide_ctrl_initialize },
+ { "open", ob_ide_ctrl_open },
+ { "close", ob_ide_ctrl_close },
{ "decode-unit", ob_ide_ctrl_decodeunit },
{ "dma-alloc", ob_ide_dma_alloc },
{ "dma-free", ob_ide_dma_free },
@@ -1414,6 +1392,9 @@ int ob_ide_init(const char *path, uint32_t io_port0, uint32_t
ctl_port0,
io_ports[1] = io_port1;
ctl_ports[1] = ctl_port1;
+ push_str(path);
+ fword("find-device");
+
for (i = 0; i < IDE_NUM_CHANNELS; i++, current_channel++) {
chan = malloc(sizeof(struct ide_channel));
@@ -1457,11 +1438,12 @@ int ob_ide_init(const char *path, uint32_t io_port0, uint32_t
ctl_port0,
ob_ide_identify_drives(chan);
- snprintf(nodebuff, sizeof(nodebuff), "%s/" DEV_NAME, path);
- REGISTER_NAMED_NODE_PHANDLE(ob_ide_ctrl, nodebuff, dnode);
-
+ fword("new-device");
+ dnode = get_cur_dev();
chan->ph = dnode;
+ BIND_NODE_METHODS(get_cur_dev(), ob_ide_ctrl);
+
#if !defined(CONFIG_PPC) && !defined(CONFIG_SPARC64)
props[0]=14; props[1]=0;
set_property(dnode, "interrupts",
@@ -1473,6 +1455,15 @@ int ob_ide_init(const char *path, uint32_t io_port0, uint32_t
ctl_port0,
props[2] = 0;
set_property(dnode, "reg", (char *)&props, 3*sizeof(props[0]));
+ set_int_property(dnode, "#address-cells", 1);
+ set_int_property(dnode, "#size-cells", 0);
+
+ push_str(DEV_NAME);
+ fword("device-name");
+
+ push_str(DEV_TYPE);
+ fword("device-type");
+
IDE_DPRINTF(DEV_NAME": [io ports 0x%x-0x%x,0x%x]\n",
current_channel, chan->io_regs[0],
chan->io_regs[0] + 7, chan->io_regs[8]);
@@ -1500,20 +1491,34 @@ int ob_ide_init(const char *path, uint32_t io_port0, uint32_t
ctl_port0,
media = "disk";
break;
}
+
IDE_DPRINTF("%s]: %s\n", media, drive->model);
- snprintf(nodebuff, sizeof(nodebuff), "%s/%s",
- get_path_from_ph(dnode), media);
- REGISTER_NAMED_NODE_PHANDLE(ob_ide, nodebuff, dnode);
+
+ fword("new-device");
+ dnode = get_cur_dev();
set_int_property(dnode, "reg", j);
+ push_str(media);
+ fword("device-name");
- /* create aliases */
+ push_str("block");
+ fword("device-type");
+
+ BIND_NODE_METHODS(dnode, ob_ide);
+ fword("is-deblocker");
+ fword("finish-device");
+
+ /* create aliases */
+ snprintf(nodebuff, sizeof(nodebuff), "%s",
+ get_path_from_ph(dnode));
set_ide_alias(nodebuff);
if (drive->media == ide_media_cdrom)
set_cd_alias(nodebuff);
if (drive->media == ide_media_disk)
set_hd_alias(nodebuff);
}
+
+ fword("finish-device");
}
return 0;
@@ -1583,6 +1588,9 @@ int macio_ide_init(const char *path, uint32_t addr, int
nb_channels)
* Also see comments in pci.c:ob_pci_host_set_interrupt_map() */
current_channel = 3;
+ push_str(path);
+ fword("find-device");
+
for (i = 0; i < nb_channels; i++) {
chan = malloc(sizeof(struct ide_channel));
@@ -1622,12 +1630,20 @@ int macio_ide_init(const char *path, uint32_t addr, int
nb_channels)
ob_ide_identify_drives(chan);
- snprintf(nodebuff, sizeof(nodebuff), "%s/" DEV_NAME "-%d", path,
- current_channel);
- REGISTER_NAMED_NODE_PHANDLE(ob_ide_ctrl, nodebuff, dnode);
-
+ fword("new-device");
+ dnode = get_cur_dev();
chan->ph = dnode;
+ snprintf(nodebuff, sizeof(nodebuff), DEV_NAME "-%d", current_channel);
+ push_str(nodebuff);
+ fword("device-name");
+
+ push_str(DEV_TYPE);
+ fword("device-type");
+
+ set_int_property(dnode, "#address-cells", 1);
+ set_int_property(dnode, "#size-cells", 0);
+
set_property(dnode, "compatible", (is_oldworld() ?
"heathrow-ata" : "keylargo-ata"), 13);
@@ -1695,6 +1711,8 @@ int macio_ide_init(const char *path, uint32_t addr, int
nb_channels)
IDE_DPRINTF(DEV_NAME": [io ports 0x%lx]\n",
current_channel, chan->mmio);
+ BIND_NODE_METHODS(dnode, ob_ide_ctrl);
+
for (j = 0; j < 2; j++) {
struct ide_drive *drive = &chan->drives[j];
const char *media = "UNKNOWN";
@@ -1719,19 +1737,32 @@ int macio_ide_init(const char *path, uint32_t addr, int
nb_channels)
break;
}
IDE_DPRINTF("%s]: %s\n", media, drive->model);
- snprintf(nodebuff, sizeof(nodebuff), "%s/%s",
- get_path_from_ph(dnode), media);
- REGISTER_NAMED_NODE_PHANDLE(ob_ide, nodebuff, dnode);
+
+ fword("new-device");
+ dnode = get_cur_dev();
set_int_property(dnode, "reg", j);
+ push_str(media);
+ fword("device-name");
- /* create aliases */
+ push_str("block");
+ fword("device-type");
+
+ BIND_NODE_METHODS(dnode, ob_ide);
+ fword("is-deblocker");
+
+ fword("finish-device");
+ /* create aliases */
+ snprintf(nodebuff, sizeof(nodebuff), "%s",
+ get_path_from_ph(dnode));
set_ide_alias(nodebuff);
if (drive->media == ide_media_cdrom)
set_cd_alias(nodebuff);
if (drive->media == ide_media_disk)
set_hd_alias(nodebuff);
}
+
+ fword("finish-device");
}
return 0;
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 06/30] adb: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/adb_bus.c | 60 +++++++++++++++++++++++++++++----------------
drivers/adb_kbd.c | 28 ++++++++++++++-------
drivers/adb_mouse.c | 34 +++++++++++++++++--------
3 files changed, 82 insertions(+), 40 deletions(-)
diff --git a/drivers/adb_bus.c b/drivers/adb_bus.c
index 88a3a2d..c505322 100644
--- a/drivers/adb_bus.c
+++ b/drivers/adb_bus.c
@@ -28,25 +28,7 @@
#include "adb_kbd.h"
#include "adb_mouse.h"
-DECLARE_UNNAMED_NODE( adb, INSTALL_OPEN, sizeof(int));
-
-static void
-adb_initialize (int *idx)
-{
- phandle_t ph=get_cur_dev();
-
- push_str("adb");
- fword("device-type");
-
- if (has_pmu()) {
- set_property(ph, "compatible", "pmu-99", 7);
- } else {
- set_property(ph, "compatible", "adb", 4);
- }
-
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
-}
+DECLARE_UNNAMED_NODE( adb, 0, sizeof(int));
static void
adb_open(int *idx)
@@ -60,7 +42,6 @@ adb_close(int *idx)
}
NODE_METHODS( adb ) = {
- { NULL, adb_initialize },
{ "open", adb_open },
{ "close", adb_close },
};
@@ -95,8 +76,43 @@ int adb_bus_init (char *path, adb_bus_t *bus)
int reloc = 0, next_free = 7;
int keep;
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("adb");
+ fword("device-name");
+
+ push_str("adb");
+ fword("device-type");
+
+ if (has_pmu()) {
+ push_str("pmu-99");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+ } else {
+ push_str("adb");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+ }
+
+ PUSH(1);
+ fword("encode-int");
+ push_str("#address-cells");
+ fword("property");
+
+ PUSH(0);
+ fword("encode-int");
+ push_str("#size-cells");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), adb);
+
snprintf(buf, sizeof(buf), "%s/adb", path);
- REGISTER_NAMED_NODE( adb, buf);
+
/* Reset the bus */
// ADB_DPRINTF("\n");
adb_reset(bus);
@@ -195,6 +211,8 @@ int adb_bus_init (char *path, adb_bus_t *bus)
}
}
+ fword("finish-device");
+
return 0;
}
diff --git a/drivers/adb_kbd.c b/drivers/adb_kbd.c
index be6099a..df53fe0 100644
--- a/drivers/adb_kbd.c
+++ b/drivers/adb_kbd.c
@@ -28,7 +28,7 @@
#include "adb_bus.h"
#include "adb_kbd.h"
-DECLARE_UNNAMED_NODE( keyboard, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE( keyboard, 0, sizeof(int));
static void
keyboard_open(int *idx)
@@ -523,8 +523,7 @@ static int adb_kbd_read (void *private)
void *adb_kbd_new (char *path, void *private)
{
char buf[64];
- int props[1];
- phandle_t ph, aliases;
+ phandle_t aliases;
adb_kbd_t *kbd;
adb_dev_t *dev = private;
kbd = (adb_kbd_t*)malloc(sizeof(adb_kbd_t));
@@ -553,16 +552,27 @@ void *adb_kbd_new (char *path, void *private)
my_adb_dev = dev;
}
- snprintf(buf, sizeof(buf), "%s/keyboard", path);
- REGISTER_NAMED_NODE( keyboard, buf);
+ push_str(path);
+ fword("find-device");
- ph = find_dev(buf);
+ fword("new-device");
- set_property(ph, "device_type", "keyboard", 9);
- props[0] = __cpu_to_be32(dev->addr);
- set_property(ph, "reg", (char *)&props, sizeof(props));
+ push_str("keyboard");
+ fword("device-name");
+
+ push_str("keyboard");
+ fword("device-type");
+
+ PUSH(dev->addr);
+ fword("encode-int");
+ push_str("reg");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), keyboard);
+ fword("finish-device");
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/keyboard", path);
set_property(aliases, "adb-keyboard", buf, strlen(buf) + 1);
return kbd;
diff --git a/drivers/adb_mouse.c b/drivers/adb_mouse.c
index b94a344..6ee512f 100644
--- a/drivers/adb_mouse.c
+++ b/drivers/adb_mouse.c
@@ -27,7 +27,7 @@
#include "adb_bus.h"
#include "adb_mouse.h"
-DECLARE_UNNAMED_NODE( mouse, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE( mouse, 0, sizeof(int));
static void
mouse_open(int *idx)
@@ -48,20 +48,34 @@ NODE_METHODS( mouse ) = {
void adb_mouse_new (char *path, void *private)
{
char buf[64];
- int props[1];
- phandle_t ph, aliases;
+ phandle_t aliases;
adb_dev_t *dev = private;
- snprintf(buf, sizeof(buf), "%s/mouse", path);
- REGISTER_NAMED_NODE( mouse, buf);
+ push_str(path);
+ fword("find-device");
- ph = find_dev(buf);
+ fword("new-device");
- set_property(ph, "device_type", "mouse", 6);
- props[0] = __cpu_to_be32(dev->addr);
- set_property(ph, "reg", (char *)&props, sizeof(props));
- set_int_property(ph, "#buttons", 3);
+ push_str("mouse");
+ fword("device-name");
+
+ push_str("mouse");
+ fword("device-type");
+
+ PUSH(dev->addr);
+ fword("encode-int");
+ push_str("reg");
+ fword("property");
+
+ PUSH(3);
+ fword("encode-int");
+ push_str("#buttons");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), mouse);
+ fword("finish-device");
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/mouse", path);
set_property(aliases, "adb-mouse", buf, strlen(buf) + 1);
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 07/30] escc: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/escc.c | 114 +++++++++++++++++++++++++++++++------------------
1 file changed, 72 insertions(+), 42 deletions(-)
diff --git a/drivers/escc.c b/drivers/escc.c
index 4ea8308..199d571 100644
--- a/drivers/escc.c
+++ b/drivers/escc.c
@@ -131,7 +131,7 @@ void serial_cls(void)
/* ( addr len -- actual ) */
static void
-escc_read(ucell *address)
+escc_port_read(ucell *address)
{
char *addr;
int len;
@@ -152,7 +152,7 @@ escc_read(ucell *address)
/* ( addr len -- actual ) */
static void
-escc_write(ucell *address)
+escc_port_write(ucell *address)
{
unsigned char *addr;
int i, len;
@@ -167,14 +167,15 @@ escc_write(ucell *address)
}
static void
-escc_close(void)
+escc_port_close(void)
{
}
+#ifdef CONFIG_DRIVER_ESCC_SUN
static void
-escc_open(ucell *address)
+escc_port_open(ucell *address)
{
-#ifdef CONFIG_DRIVER_ESCC_SUN
+
int len;
phandle_t ph;
unsigned long *prop;
@@ -193,19 +194,46 @@ escc_open(ucell *address)
//printk("escc_open: address %lx, args %s\n", *address, args);
free(args);
}
+
+ RET ( -1 );
+}
+
#else
+
+static void
+escc_port_open(ucell *address)
+{
*address = (unsigned long)escc_serial_dev; // XXX
-#endif
- RET ( -1 );
+ RET(-1);
}
-DECLARE_UNNAMED_NODE(escc, INSTALL_OPEN, sizeof(ucell));
+static void
+escc_open(int *idx)
+{
+ RET(-1);
+}
+
+static void
+escc_close(int *idx)
+{
+}
+
+DECLARE_UNNAMED_NODE(escc, 0, sizeof(int *));
NODE_METHODS(escc) = {
- { "open", escc_open },
- { "close", escc_close },
- { "read", escc_read },
- { "write", escc_write },
+ { "open", escc_open },
+ { "close", escc_close },
+};
+
+#endif
+
+DECLARE_UNNAMED_NODE(escc_port, 0, sizeof(ucell));
+
+NODE_METHODS(escc_port) = {
+ { "open", escc_port_open },
+ { "close", escc_port_close },
+ { "read", escc_port_read },
+ { "write", escc_port_write },
};
#ifdef CONFIG_DRIVER_ESCC_SUN
@@ -305,11 +333,11 @@ escc_read_keyboard(void)
}
}
-DECLARE_UNNAMED_NODE(escc_keyboard, INSTALL_OPEN, sizeof(ucell));
+DECLARE_UNNAMED_NODE(escc_keyboard, 0, sizeof(ucell));
NODE_METHODS(escc_keyboard) = {
- { "open", escc_open },
- { "close", escc_close },
+ { "open", escc_port_open },
+ { "close", escc_port_close },
{ "read", escc_read_keyboard },
};
@@ -352,19 +380,20 @@ ob_zs_init(phys_addr_t base, uint64_t offset, int intr, int slave,
int keyboard)
push_str("port-b-ignore-cd");
fword("property");
+ if (keyboard) {
+ BIND_NODE_METHODS(get_cur_dev(), escc_keyboard);
+ } else {
+ BIND_NODE_METHODS(get_cur_dev(), escc_port);
+ }
+
fword("finish-device");
- snprintf(nodebuff, sizeof(nodebuff), "/obio/zs@0,%x",
- (int)offset & 0xffffffff);
+ aliases = find_dev("/aliases");
if (keyboard) {
- REGISTER_NODE_METHODS(escc_keyboard, nodebuff);
-
- aliases = find_dev("/aliases");
+ snprintf(nodebuff, sizeof(nodebuff), "/obio/zs@0,%x",
+ (int)offset & 0xffffffff);
set_property(aliases, "keyboard", nodebuff, strlen(nodebuff) + 1);
} else {
- REGISTER_NODE_METHODS(escc, nodebuff);
-
- aliases = find_dev("/aliases");
snprintf(nodebuff, sizeof(nodebuff), "/obio/zs@0,%x:a",
(int)offset & 0xffffffff);
set_property(aliases, "ttya", nodebuff, strlen(nodebuff) + 1);
@@ -421,11 +450,16 @@ escc_add_channel(const char *path, const char *node, phys_addr_t
addr,
/* add device */
- snprintf(buf, sizeof(buf), "%s/ch-%s", path, node);
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
- REGISTER_NAMED_NODE(escc, buf);
+ snprintf(buf, sizeof(buf), "ch-%s", node);
+ push_str(buf);
+ fword("device-name");
- activate_device(buf);
+ BIND_NODE_METHODS(get_cur_dev(), escc_port);
/* add aliases */
@@ -440,7 +474,7 @@ escc_add_channel(const char *path, const char *node, phys_addr_t
addr,
}
/* add properties */
- dnode = find_dev(buf);
+ dnode = get_cur_dev();
set_property(dnode, "device_type", "serial",
strlen("serial") + 1);
@@ -484,7 +518,7 @@ escc_add_channel(const char *path, const char *node, phys_addr_t
addr,
set_int_property(dnode, "slot-names", 0);
- device_end();
+ fword("finish-device");
uart_init_line((unsigned char*)addr + offset + reg_offsets[legacy][index][0],
CONFIG_SERIAL_SPEED);
@@ -504,10 +538,7 @@ escc_init(const char *path, phys_addr_t addr)
push_str("escc");
fword("device-name");
- snprintf(buf, sizeof(buf), "%s/escc", path);
-
- dnode = find_dev(buf);
-
+ dnode = get_cur_dev();
set_int_property(dnode, "#address-cells", 1);
props[0] = __cpu_to_be32(IO_ESCC_OFFSET);
props[1] = __cpu_to_be32(IO_ESCC_SIZE);
@@ -517,25 +548,22 @@ escc_init(const char *path, phys_addr_t addr)
set_property(dnode, "compatible", "escc\0CHRP,es0", 14);
set_property(dnode, "ranges", "", 0);
- fword("finish-device");
-
+ snprintf(buf, sizeof(buf), "%s/escc", path);
escc_add_channel(buf, "a", addr, 2);
escc_add_channel(buf, "b", addr, 3);
+ BIND_NODE_METHODS(dnode, escc);
+ fword("finish-device");
+
escc_serial_dev = (unsigned char *)addr + IO_ESCC_OFFSET +
(CONFIG_SERIAL_PORT ? 0 : 0x20);
- push_str(path);
- fword("find-device");
fword("new-device");
push_str("escc-legacy");
fword("device-name");
- snprintf(buf, sizeof(buf), "%s/escc-legacy", path);
-
- dnode = find_dev(buf);
-
+ dnode = get_cur_dev();
set_int_property(dnode, "#address-cells", 1);
props[0] = __cpu_to_be32(IO_ESCC_LEGACY_OFFSET);
props[1] = __cpu_to_be32(IO_ESCC_LEGACY_SIZE);
@@ -545,9 +573,11 @@ escc_init(const char *path, phys_addr_t addr)
set_property(dnode, "compatible", "chrp,es1", 9);
set_property(dnode, "ranges", "", 0);
- fword("finish-device");
-
+ snprintf(buf, sizeof(buf), "%s/escc-legacy", path);
escc_add_channel(buf, "a", addr, 4);
escc_add_channel(buf, "b", addr, 5);
+
+ BIND_NODE_METHODS(dnode, escc);
+ fword("finish-device");
}
#endif
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 08/30] cuda: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/cuda.c | 161 ++++++++++++++++++++++++++++++-------------------
1 file changed, 100 insertions(+), 61 deletions(-)
diff --git a/drivers/cuda.c b/drivers/cuda.c
index d079d1e..d88edf8 100644
--- a/drivers/cuda.c
+++ b/drivers/cuda.c
@@ -170,7 +170,7 @@ static int cuda_adb_req (void *host, const uint8_t *snd_buf, int len,
}
-DECLARE_UNNAMED_NODE(ob_cuda, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE(ob_cuda, 0, sizeof(int));
static cuda_t *main_cuda;
@@ -192,43 +192,6 @@ ppc32_poweroff(void)
cuda_request(main_cuda, CUDA_PACKET, cmdbuf, sizeof(cmdbuf), obuf);
}
-static void
-ob_cuda_initialize (int *idx)
-{
- phandle_t ph=get_cur_dev();
- int props[2];
-
- push_str("via-cuda");
- fword("device-type");
-
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
-
- set_property(ph, "compatible", "cuda", 5);
-
- props[0] = __cpu_to_be32(IO_CUDA_OFFSET);
- props[1] = __cpu_to_be32(IO_CUDA_SIZE);
-
- set_property(ph, "reg", (char *)&props, sizeof(props));
-
- /* on newworld machines the cuda is on interrupt 0x19 */
-
- props[0] = 0x19;
- props[1] = 0;
- NEWWORLD(set_property(ph, "interrupts", (char *)props, sizeof(props)));
- NEWWORLD(set_int_property(ph, "#interrupt-cells", 2));
-
- /* we emulate an oldworld hardware, so we must use
- * non-standard oldworld property (needed by linux 2.6.18)
- */
-
- OLDWORLD(set_int_property(ph, "AAPL,interrupts", 0x12));
-
- bind_func("ppc32-reset-all", ppc32_reset_all);
- push_str("' ppc32-reset-all to reset-all");
- fword("eval");
-}
-
static void
ob_cuda_open(int *idx)
{
@@ -241,12 +204,11 @@ ob_cuda_close(int *idx)
}
NODE_METHODS(ob_cuda) = {
- { NULL, ob_cuda_initialize },
{ "open", ob_cuda_open },
{ "close", ob_cuda_close },
};
-DECLARE_UNNAMED_NODE(rtc, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE(rtc, 0, sizeof(int));
static void
rtc_open(int *idx)
@@ -368,41 +330,67 @@ NODE_METHODS(rtc) = {
static void
rtc_init(char *path)
{
- phandle_t ph, aliases;
+ phandle_t aliases;
char buf[128];
- snprintf(buf, sizeof(buf), "%s/rtc", path);
- REGISTER_NAMED_NODE(rtc, buf);
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("rtc");
+ fword("device-name");
+
+ push_str("rtc");
+ fword("device-type");
+
+ push_str("rtc");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
- ph = find_dev(buf);
- set_property(ph, "device_type", "rtc", 4);
- set_property(ph, "compatible", "rtc", 4);
+ BIND_NODE_METHODS(get_cur_dev(), rtc);
+ fword("finish-device");
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/rtc", path);
set_property(aliases, "rtc", buf, strlen(buf) + 1);
-
}
static void
powermgt_init(char *path)
{
- phandle_t ph;
- char buf[128];
+ push_str(path);
+ fword("find-device");
- snprintf(buf, sizeof(buf), "%s/power-mgt", path);
- REGISTER_NAMED_NODE(rtc, buf);
+ fword("new-device");
- ph = find_dev(buf);
- set_property(ph, "device_type", "power-mgt", 10);
- set_property(ph, "mgt-kind", "min-consumption-pwm-led",
strlen("min-consumption-pwm-led") + 1);
- set_property(ph, "compatible", "cuda", strlen("cuda") +
1);
+ push_str("power-mgt");
+ fword("device-name");
+
+ push_str("power-mgt");
+ fword("device-type");
+
+ push_str("min-consumption-pwm-led");
+ fword("encode-string");
+ push_str("mgt-kind");
+ fword("property");
+
+ push_str("cuda");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), rtc);
+ fword("finish-device");
}
cuda_t *cuda_init (const char *path, phys_addr_t base)
{
cuda_t *cuda;
char buf[64];
- phandle_t aliases;
+ phandle_t ph, aliases;
+ int props[2];
base += IO_CUDA_OFFSET;
CUDA_DPRINTF(" base=" FMT_plx "\n", base);
@@ -410,10 +398,57 @@ cuda_t *cuda_init (const char *path, phys_addr_t base)
if (cuda == NULL)
return NULL;
- snprintf(buf, sizeof(buf), "%s/via-cuda", path);
- REGISTER_NAMED_NODE(ob_cuda, buf);
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("via-cuda");
+ fword("device-name");
+
+ push_str("via-cuda");
+ fword("device-type");
+
+ push_str("cuda");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ PUSH(1);
+ fword("encode-int");
+ push_str("#address-cells");
+ fword("property");
+
+ PUSH(0);
+ fword("encode-int");
+ push_str("#size-cells");
+ fword("property");
+
+ PUSH(IO_CUDA_OFFSET);
+ fword("encode-int");
+ PUSH(IO_CUDA_SIZE);
+ fword("encode-int");
+ fword("encode+");
+ push_str("reg");
+ fword("property");
+
+ ph = get_cur_dev();
+
+ /* on newworld machines the cuda is on interrupt 0x19 */
+ props[0] = 0x19;
+ props[1] = 0;
+ NEWWORLD(set_property(ph, "interrupts", (char *)props, sizeof(props)));
+ NEWWORLD(set_int_property(ph, "#interrupt-cells", 2));
+
+ /* we emulate an oldworld hardware, so we must use
+ * non-standard oldworld property (needed by linux 2.6.18)
+ */
+ OLDWORLD(set_int_property(ph, "AAPL,interrupts", 0x12));
+
+ BIND_NODE_METHODS(get_cur_dev(), ob_cuda);
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/via-cuda", path);
set_property(aliases, "via-cuda", buf, strlen(buf) + 1);
cuda->base = base;
@@ -430,10 +465,14 @@ cuda_t *cuda_init (const char *path, phys_addr_t base)
rtc_init(buf);
powermgt_init(buf);
- main_cuda = cuda;
+ main_cuda = cuda;
+
+ fword("finish-device");
- device_end();
- bind_func("poweroff", ppc32_poweroff);
+ bind_func("ppc32-power-off", ppc32_poweroff);
+ feval("['] ppc32-power-off to power-off");
+ bind_func("ppc32-reset-all", ppc32_reset_all);
+ feval("['] ppc32-reset-all to reset-all");
return cuda;
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 09/30] macio: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/macio.c | 111 ++++++++++++++++++++++++++----------------------
1 file changed, 61 insertions(+), 50 deletions(-)
diff --git a/drivers/macio.c b/drivers/macio.c
index be6f9e4..eb2af04 100644
--- a/drivers/macio.c
+++ b/drivers/macio.c
@@ -189,7 +189,7 @@ openpic_init(const char *path, phys_addr_t addr)
fword("finish-device");
}
-DECLARE_NODE(ob_macio, INSTALL_OPEN, sizeof(int), "Tmac-io");
+DECLARE_UNNAMED_NODE(ob_macio, 0, sizeof(int));
/* ( str len -- addr ) */
@@ -284,20 +284,41 @@ ob_unin_init(void)
fword("finish-device");
}
-static void ob_macio_gpio_initialize (int *idx)
+static void macio_gpio_init(const char *path)
{
- phandle_t ph=get_cur_dev();
- int props[2];
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("gpio");
+ fword("device-name");
push_str("gpio");
fword("device-type");
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
- set_property(ph, "compatible", "mac-io-gpio", 12);
- props[0] = __cpu_to_be32(0x50);
- props[1] = __cpu_to_be32(0x30);
- set_property(ph, "reg", (char *)&props, sizeof(props));
+ PUSH(1);
+ fword("encode-int");
+ push_str("#address-cells");
+ fword("property");
+
+ PUSH(0);
+ fword("encode-int");
+ push_str("#size-cells");
+ fword("property");
+
+ push_str("mac-io-gpio");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ PUSH(0x50);
+ fword("encode-int");
+ PUSH(0x30);
+ fword("encode-int");
+ fword("encode+");
+ push_str("reg");
+ fword("property");
/* Build the extint-gpio1 for the PMU */
fword("new-device");
@@ -339,57 +360,47 @@ static void ob_macio_gpio_initialize (int *idx)
push_str("interrupts");
fword("property");
fword("finish-device");
-}
-DECLARE_UNNAMED_NODE(ob_macio_gpio, 0, 0);
-
-NODE_METHODS(ob_macio_gpio) = {
- { NULL, ob_macio_gpio_initialize }
-};
-
-static void macio_gpio_init(const char *path)
-{
- char buf[64];
-
- snprintf(buf, sizeof(buf), "%s/gpio", path);
- REGISTER_NAMED_NODE(ob_macio_gpio, buf);
+ fword("finish-device");
}
void
ob_macio_heathrow_init(const char *path, phys_addr_t addr)
{
- phandle_t aliases;
+ phandle_t aliases;
- REGISTER_NODE(ob_macio);
- aliases = find_dev("/aliases");
- set_property(aliases, "mac-io", path, strlen(path) + 1);
+ BIND_NODE_METHODS(get_cur_dev(), ob_macio);
+
+ cuda_init(path, addr);
+ macio_nvram_init(path, addr);
+ escc_init(path, addr);
+ macio_ide_init(path, addr, 2);
- cuda_init(path, addr);
- macio_nvram_init(path, addr);
- escc_init(path, addr);
- macio_ide_init(path, addr, 2);
+ aliases = find_dev("/aliases");
+ set_property(aliases, "mac-io", path, strlen(path) + 1);
}
void
ob_macio_keylargo_init(const char *path, phys_addr_t addr)
{
- phandle_t aliases;
-
- REGISTER_NODE(ob_macio);
- aliases = find_dev("/aliases");
- set_property(aliases, "mac-io", path, strlen(path) + 1);
-
- if (has_pmu()) {
- macio_gpio_init(path);
- pmu_init(path, addr);
- } else {
- cuda_init(path, addr);
- }
-
- /* The NewWorld NVRAM is not located in the MacIO device */
- macio_nvram_init("", 0);
- escc_init(path, addr);
- macio_ide_init(path, addr, 2);
- openpic_init(path, addr);
- ob_unin_init();
+ phandle_t aliases;
+
+ BIND_NODE_METHODS(get_cur_dev(), ob_macio);
+
+ if (has_pmu()) {
+ macio_gpio_init(path);
+ pmu_init(path, addr);
+ } else {
+ cuda_init(path, addr);
+ }
+
+ /* The NewWorld NVRAM is not located in the MacIO device */
+ macio_nvram_init("", 0);
+ escc_init(path, addr);
+ macio_ide_init(path, addr, 2);
+ openpic_init(path, addr);
+ ob_unin_init();
+
+ aliases = find_dev("/aliases");
+ set_property(aliases, "mac-io", path, strlen(path) + 1);
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 10/30] pmu: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pmu.c | 155 ++++++++++++++++++++++++++++++++++----------------
1 file changed, 107 insertions(+), 48 deletions(-)
diff --git a/drivers/pmu.c b/drivers/pmu.c
index 9198846..38fa1e8 100644
--- a/drivers/pmu.c
+++ b/drivers/pmu.c
@@ -348,7 +348,7 @@ static int pmu_adb_req(void *host, const uint8_t *snd_buf, int len,
}
#endif
-DECLARE_UNNAMED_NODE(ob_pmu, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE(ob_pmu, 0, sizeof(int));
static pmu_t *main_pmu;
@@ -364,32 +364,6 @@ static void pmu_poweroff(void)
pmu_request(main_pmu, PMU_SHUTDOWN, 4, params, NULL, NULL);
}
-static void ob_pmu_initialize (int *idx)
-{
- phandle_t ph=get_cur_dev();
- int props[2];
-
- push_str("via-pmu");
- fword("device-type");
-
- set_int_property(ph, "#address-cells", 1);
- set_int_property(ph, "#size-cells", 0);
- set_property(ph, "compatible", "pmu", 4);
-
- props[0] = __cpu_to_be32(IO_PMU_OFFSET);
- props[1] = __cpu_to_be32(IO_PMU_SIZE);
- set_property(ph, "reg", (char *)&props, sizeof(props));
-
- /* On newworld machines the PMU is on interrupt 0x19 */
- props[0] = 0x19;
- props[1] = 1;
- set_property(ph, "interrupts", (char *)props, sizeof(props));
- set_int_property(ph, "pmu-version", 0xd0330c);
-
- bind_func("pmu-reset-all", pmu_reset_all);
- feval("['] pmu-reset-all to reset-all");
-}
-
static void ob_pmu_open(int *idx)
{
RET(-1);
@@ -400,18 +374,21 @@ static void ob_pmu_close(int *idx)
}
NODE_METHODS(ob_pmu) = {
- { NULL, ob_pmu_initialize },
{ "open", ob_pmu_open },
{ "close", ob_pmu_close },
};
-DECLARE_UNNAMED_NODE(rtc, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE(rtc, 0, sizeof(int));
static void rtc_open(int *idx)
{
RET(-1);
}
+static void rtc_close(int *idx)
+{
+}
+
/*
* get-time ( -- second minute hour day month year )
*
@@ -516,31 +493,64 @@ static void rtc_set_time(int *idx)
NODE_METHODS(rtc) = {
{ "open", rtc_open },
+ { "close", rtc_close },
{ "get-time", rtc_get_time },
{ "set-time", rtc_set_time },
};
static void rtc_init(char *path)
{
- phandle_t ph, aliases;
+ phandle_t aliases;
char buf[128];
- snprintf(buf, sizeof(buf), "%s/rtc", path);
- REGISTER_NAMED_NODE(rtc, buf);
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
- ph = find_dev(buf);
- set_property(ph, "device_type", "rtc", 4);
- set_property(ph, "compatible", "rtc,via-pmu", 12);
+ push_str("rtc");
+ fword("device-name");
+
+ push_str("rtc");
+ fword("device-type");
+
+ push_str("rtc,via-pmu");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), rtc);
+ fword("finish-device");
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/rtc", path);
set_property(aliases, "rtc", buf, strlen(buf) + 1);
- device_end();
}
static void powermgt_init(char *path)
{
phandle_t ph;
- char buf[128];
+
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("power-mgt");
+ fword("device-name");
+
+ push_str("power-mgt");
+ fword("device-type");
+
+ push_str("via-pmu-99");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ push_str("extint-gpio1");
+ fword("encode-string");
+ push_str("registry-name");
+ fword("property");
/* This is a bunch of magic "Feature" bits for which we only have
* partial definitions from Darwin. These are taken from a
@@ -574,15 +584,12 @@ static void powermgt_init(char *path)
0x46, 0x00, 0x02, 0x78,
0x78, 0x3c, 0x00 };
- snprintf(buf, sizeof(buf), "%s/power-mgt", path);
- REGISTER_NAMED_NODE(rtc, buf); // XXX ?
+ ph = get_cur_dev();
+ BIND_NODE_METHODS(ph, rtc);
- ph = find_dev(buf);
- set_property(ph, "device_type", "power-mgt", 10);
- set_property(ph, "compatible", "via-pmu-99", 11);
- set_property(ph, "registry-name", "extint-gpio1", 13);
set_property(ph, "prim-info", prim, sizeof(prim));
- device_end();
+
+ fword("finish-device");
}
pmu_t *pmu_init(const char *path, phys_addr_t base)
@@ -599,10 +606,58 @@ pmu_t *pmu_init(const char *path, phys_addr_t base)
return NULL;
}
- snprintf(buf, sizeof(buf), "%s/via-pmu", path);
- REGISTER_NAMED_NODE(ob_pmu, buf);
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
+ push_str("via-pmu");
+ fword("device-name");
+
+ push_str("via-pmu");
+ fword("device-type");
+
+ push_str("pmu");
+ fword("encode-string");
+ push_str("compatible");
+ fword("property");
+
+ PUSH(1);
+ fword("encode-int");
+ push_str("#address-cells");
+ fword("property");
+
+ PUSH(0);
+ fword("encode-int");
+ push_str("#size-cells");
+ fword("property");
+
+ PUSH(IO_PMU_OFFSET);
+ fword("encode-int");
+ PUSH(IO_PMU_SIZE);
+ fword("encode-int");
+ fword("encode+");
+ push_str("reg");
+ fword("property");
+
+ /* On newworld machines the PMU is on interrupt 0x19 */
+ PUSH(0x19);
+ fword("encode-int");
+ PUSH(1);
+ fword("encode-int");
+ fword("encode+");
+ push_str("interrupts");
+ fword("property");
+
+ PUSH(0xd0330c);
+ fword("encode-int");
+ push_str("pmu-version");
+ fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), ob_pmu);
aliases = find_dev("/aliases");
+ snprintf(buf, sizeof(buf), "%s/via-pmu", path);
set_property(aliases, "via-pmu", buf, strlen(buf) + 1);
pmu->base = base;
@@ -618,8 +673,12 @@ pmu_t *pmu_init(const char *path, phys_addr_t base)
main_pmu = pmu;
- device_end();
- bind_func("poweroff", pmu_poweroff);
+ fword("finish-device");
+
+ bind_func("pmu-power-off", pmu_poweroff);
+ feval("['] pmu-power-off to power-off");
+ bind_func("pmu-reset-all", pmu_reset_all);
+ feval("['] pmu-reset-all to reset-all");
return pmu;
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 11/30] usbhid: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/usbhid.c | 20 ++++++++++++++------
1 file changed, 14 insertions(+), 6 deletions(-)
diff --git a/drivers/usbhid.c b/drivers/usbhid.c
index 1c13113..a847b6e 100644
--- a/drivers/usbhid.c
+++ b/drivers/usbhid.c
@@ -39,7 +39,7 @@
#include "drivers/usb.h"
#include "usb.h"
-DECLARE_UNNAMED_NODE(usb_kbd, INSTALL_OPEN, sizeof(int));
+DECLARE_UNNAMED_NODE(usb_kbd, 0, sizeof(int));
static void
keyboard_open(int *idx)
@@ -564,16 +564,24 @@ void ob_usb_hid_add_keyboard(const char *path)
char name[128];
phandle_t aliases;
- snprintf(name, sizeof(name), "%s/keyboard", path);
- usb_debug("Found keyboard at %s\n", name);
- REGISTER_NAMED_NODE(usb_kbd, name);
-
- push_str(name);
+ push_str(path);
fword("find-device");
+ fword("new-device");
+
+ push_str("keyboard");
+ fword("device-name");
+
push_str("keyboard");
fword("device-type");
+ snprintf(name, sizeof(name), "%s/keyboard", path);
+ usb_debug("Found keyboard at %s\n", name);
+
+ BIND_NODE_METHODS(get_cur_dev(), usb_kbd);
+
+ fword("finish-device");
+
aliases = find_dev("/aliases");
set_property(aliases, "keyboard", name, strlen(name) + 1);
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 12/30] nvram: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/macio.c | 8 ++++----
include/packages/nvram.h | 4 +++-
packages/nvram.c | 32 ++++++++++++++++++++++++++++++--
3 files changed, 37 insertions(+), 7 deletions(-)
diff --git a/drivers/macio.c b/drivers/macio.c
index eb2af04..5a94252 100644
--- a/drivers/macio.c
+++ b/drivers/macio.c
@@ -91,9 +91,8 @@ void macio_nvram_init(const char *path, phys_addr_t addr)
nvram_size = macio_nvram_size();
nvram = (char*)addr + nvram_offset;
- snprintf(buf, sizeof(buf), "%s/nvram", path);
- nvram_init(buf);
- dnode = find_dev(buf);
+ snprintf(buf, sizeof(buf), "%s", path);
+ dnode = nvram_init(buf);
set_int_property(dnode, "#bytes", arch_nvram_size() );
props[0] = __cpu_to_be32(nvram_offset);
props[1] = __cpu_to_be32(nvram_size);
@@ -102,6 +101,7 @@ void macio_nvram_init(const char *path, phys_addr_t addr)
NEWWORLD(set_property(dnode, "compatible", "nvram,flash", 12));
chosen = find_dev("/chosen");
+ snprintf(buf, sizeof(buf), "%s", get_path_from_ph(dnode));
push_str(buf);
fword("open-dev");
set_int_property(chosen, "nvram", POP());
@@ -395,7 +395,7 @@ ob_macio_keylargo_init(const char *path, phys_addr_t addr)
}
/* The NewWorld NVRAM is not located in the MacIO device */
- macio_nvram_init("", 0);
+ macio_nvram_init("/", 0);
escc_init(path, addr);
macio_ide_init(path, addr, 2);
openpic_init(path, addr);
diff --git a/include/packages/nvram.h b/include/packages/nvram.h
index ba1b38b..7803814 100644
--- a/include/packages/nvram.h
+++ b/include/packages/nvram.h
@@ -17,8 +17,10 @@
#ifndef _H_NVRAM_PACKAGE
#define _H_NVRAM_PACKAGE
+#include "kernel/stack.h"
+
extern void nvconf_init( void );
-extern void nvram_init( const char *path );
+extern phandle_t nvram_init( const char *path );
extern void update_nvram( void );
#endif /* _H_NVRAM_PACKAGE */
diff --git a/packages/nvram.c b/packages/nvram.c
index e895fed..6fd39b1 100644
--- a/packages/nvram.c
+++ b/packages/nvram.c
@@ -291,7 +291,20 @@ nvram_size( __attribute__((unused)) nvram_ibuf_t *nd )
PUSH( nvram.size );
}
+static void
+nvram_open( __attribute__((unused)) nvram_ibuf_t *nd )
+{
+ RET(-1);
+}
+
+static void
+nvram_close( __attribute__((unused)) nvram_ibuf_t *nd )
+{
+}
+
NODE_METHODS( nvram ) = {
+ { "open", (void*)nvram_open },
+ { "close", (void*)nvram_close },
{ "size", (void*)nvram_size },
{ "read", (void*)nvram_read },
{ "write", (void*)nvram_write },
@@ -299,10 +312,25 @@ NODE_METHODS( nvram ) = {
};
-void
+phandle_t
nvram_init( const char *path )
{
+ phandle_t ph;
+
+ push_str(path);
+ fword("find-device");
+
+ fword("new-device");
+
nvconf_init();
- REGISTER_NAMED_NODE( nvram, path );
+ ph = get_cur_dev();
+
+ push_str("nvram");
+ fword("device-name");
+
+ BIND_NODE_METHODS(get_cur_dev(), nvram);
+ fword("finish-device");
+
+ return ph;
}
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 13/30] lsi: don't change active package when setting device alias
When generating the DT based upon the active package and current instance, we
cannot change either when setting the alias. Instead look up the phandle and
then set the properties directly.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/lsi.c | 11 +++++------
1 file changed, 5 insertions(+), 6 deletions(-)
diff --git a/drivers/lsi.c b/drivers/lsi.c
index 1f3f038..37e4b9d 100644
--- a/drivers/lsi.c
+++ b/drivers/lsi.c
@@ -624,13 +624,12 @@ NODE_METHODS(ob_lsi) = {
static void
add_alias(const char *device, const char *alias)
{
+ phandle_t aliases;
+
DPRINTF("add_alias dev \"%s\" = alias \"%s\"\n",
device, alias);
- push_str("/aliases");
- fword("find-device");
- push_str(device);
- fword("encode-string");
- push_str(alias);
- fword("property");
+
+ aliases = find_dev("/aliases");
+ set_property(aliases, alias, device, strlen(device) + 1);
}
int
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 14/30] lsi: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/lsi.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)
diff --git a/drivers/lsi.c b/drivers/lsi.c
index 37e4b9d..bc4893a 100644
--- a/drivers/lsi.c
+++ b/drivers/lsi.c
@@ -642,7 +642,7 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
int i;
ucell addr;
- REGISTER_NODE_METHODS(ob_lsi, path);
+ BIND_NODE_METHODS(ph, ob_lsi);
lsi = malloc(sizeof(lsi_private_t));
if (!lsi) {
@@ -744,10 +744,13 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
fword("encode+");
push_str("reg");
fword("property");
+
+ BIND_NODE_METHODS(get_cur_dev(), ob_sd);
fword("finish-device");
+
snprintf(nodebuff, sizeof(nodebuff), "%s/sd@%d",
get_path_from_ph(ph), id);
- REGISTER_NODE_METHODS(ob_sd, nodebuff);
+
if (lsi->sd[id].media == TYPE_ROM) {
counter_ptr = &cdcount;
} else {
--
2.20.1
10:37 p.m.
New subject: [PATCH v3 15/30] virtio: convert to use BIND_NODE_METHODS() macro
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/virtio.c | 6 ++----
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/virtio.c b/drivers/virtio.c
index 171db5a..6008ae9 100644
--- a/drivers/virtio.c
+++ b/drivers/virtio.c
@@ -503,12 +503,11 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
ucell addr;
VDev *vdev, **_vdev;
- REGISTER_NODE_METHODS(ob_virtio, path);
-
/* Open ob_virtio */
fword("my-self");
push_str(path);
feval("open-dev to my-self");
+ BIND_NODE_METHODS(get_cur_dev(), ob_virtio);
ph = find_ih_method("vdev", my_self());
PUSH(ph);
@@ -546,10 +545,9 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
push_str("_vdev");
fword("property");
+ BIND_NODE_METHODS(get_cur_dev(), ob_virtio_disk);
fword("finish-device");
snprintf(buf, sizeof(buf), "%s/disk", path);
- REGISTER_NODE_METHODS(ob_virtio_disk, buf);
-
set_virtio_alias(buf, idx);
}
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 16/30] pci: convert to use BIND_NODE_METHODS() macro
Moving the setup of PCI devices to within a new-device...finish-device sequence
as part of the conversion further enables us to move the PCI host bridge
configuration logic from ob_pci_init() to ob_configure_pci_device() where it
belongs.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pci.c | 96 +++++++++++++++++++++++----------------------------
1 file changed, 43 insertions(+), 53 deletions(-)
diff --git a/drivers/pci.c b/drivers/pci.c
index 60e3948..623f649 100644
--- a/drivers/pci.c
+++ b/drivers/pci.c
@@ -528,7 +528,9 @@ static void pci_set_bus_range(const pci_config_t *config)
set_property(dev, "bus-range", (char *)props, 2 * sizeof(props[0]));
}
-static void pci_host_set_reg(phandle_t phandle)
+static void ob_pci_reload_device_path(phandle_t phandle, pci_config_t *config);
+
+static void pci_host_set_reg(phandle_t phandle, pci_config_t *config)
{
phandle_t dev = phandle;
@@ -544,6 +546,8 @@ static void pci_host_set_reg(phandle_t phandle)
set_property(dev, "reg", (char *)props, ncells * sizeof(props[0]));
+ ob_pci_reload_device_path(dev, config);
+
#if defined(CONFIG_DEBUG_PCI)
dump_reg_property("pci_host_set_reg", 4, props);
#endif
@@ -579,8 +583,6 @@ static void pci_host_set_ranges(const pci_config_t *config)
int host_config_cb(const pci_config_t *config)
{
- //XXX this overrides "reg" property
- pci_host_set_reg(get_cur_dev());
pci_host_set_ranges(config);
return 0;
@@ -1265,6 +1267,7 @@ static void ob_pci_add_properties(phandle_t phandle,
uint8_t rev;
uint8_t class_prog;
uint32_t class_code;
+ char path[256];
vendor_id = pci_config_read16(addr, PCI_VENDOR_ID);
device_id = pci_config_read16(addr, PCI_DEVICE_ID);
@@ -1272,24 +1275,22 @@ static void ob_pci_add_properties(phandle_t phandle,
class_prog = pci_config_read8(addr, PCI_CLASS_PROG);
class_code = pci_config_read16(addr, PCI_CLASS_DEVICE);
+ /* Default path if we don't match anything */
+ snprintf(path, sizeof(path), "pci%x,%x", vendor_id, device_id);
+
if (pci_dev) {
/**/
if (pci_dev->name) {
push_str(pci_dev->name);
- fword("encode-string");
- push_str("name");
- fword("property");
+ fword("device-name");
} else {
- char path[256];
- snprintf(path, sizeof(path),
- "pci%x,%x", vendor_id, device_id);
push_str(path);
- fword("encode-string");
- push_str("name");
- fword("property");
+ fword("device-name");
}
} else {
PCI_DPRINTF("*** missing pci_dev\n");
+ push_str(path);
+ fword("device-name");
}
/* create properties as described in 2.5 */
@@ -1335,15 +1336,11 @@ static void ob_pci_add_properties(phandle_t phandle,
if (pci_dev) {
if (pci_dev->type) {
push_str(pci_dev->type);
- fword("encode-string");
- push_str("device_type");
- fword("property");
+ fword("device-type");
}
if (pci_dev->model) {
push_str(pci_dev->model);
- fword("encode-string");
- push_str("model");
- fword("property");
+ fword("model");
}
if (pci_dev->compat)
set_property(dev, "compatible",
@@ -1765,7 +1762,7 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
uint8_t class, subclass, iface;
int num_bars, rom_bar;
- phandle_t phandle = 0;
+ phandle_t phandle = 0, t;
int is_host_bridge = 0;
if (!ob_pci_read_identification(bus, devnum, fn, &vid, &did, &class,
&subclass)) {
@@ -1789,54 +1786,52 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
}
}
- /* stop adding host bridge accessible from it's primary bus
- PCI host bridge is to be added by host code
- */
if (class == PCI_BASE_CLASS_BRIDGE &&
subclass == PCI_SUBCLASS_BRIDGE_HOST) {
is_host_bridge = 1;
}
+ /* stop adding host bridge accessible from it's primary bus
+ PCI host bridge is to be added by host code
+ */
if (is_host_bridge) {
/* reuse device tree node */
PCI_DPRINTF("host bridge found - ");
- snprintf(config.path, sizeof(config.path),
- "%s", parent_path);
+
+ t = find_dev(parent_path);
+ if (get_property(t, "vendor-id", NULL)) {
+ PCI_DPRINTF("host bridge already configured\n");
+ return 0;
+ }
} else if (pci_dev == NULL || pci_dev->name == NULL) {
snprintf(config.path, sizeof(config.path),
"%s/pci%x,%x", parent_path, vid, did);
- }
- else {
+ } else {
snprintf(config.path, sizeof(config.path),
"%s/%s", parent_path, pci_dev->name);
}
+ fword("new-device");
+
PCI_DPRINTF("%s - ", config.path);
config.dev = addr & 0x00FFFFFF;
+ phandle = get_cur_dev();
+
switch (class) {
case PCI_BASE_CLASS_BRIDGE:
if (subclass != PCI_SUBCLASS_BRIDGE_HOST) {
- REGISTER_NAMED_NODE_PHANDLE(ob_pci_bridge_node, config.path, phandle);
+ BIND_NODE_METHODS(phandle, ob_pci_bridge_node);
+ } else {
+ BIND_NODE_METHODS(phandle, ob_pci_bus_node);
}
break;
default:
- REGISTER_NAMED_NODE_PHANDLE(ob_pci_empty_node, config.path, phandle);
+ BIND_NODE_METHODS(phandle, ob_pci_empty_node);
break;
}
- if (is_host_bridge) {
- phandle = find_dev(config.path);
-
- if (get_property(phandle, "vendor-id", NULL)) {
- PCI_DPRINTF("host bridge already configured\n");
- return 0;
- }
- }
-
- activate_dev(phandle);
-
if (htype & PCI_HEADER_TYPE_BRIDGE) {
num_bars = 2;
rom_bar = PCI_ROM_ADDRESS1;
@@ -1852,6 +1847,8 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
if (!is_host_bridge) {
pci_set_reg(phandle, &config, num_bars);
+ } else {
+ pci_host_set_reg(phandle, &config);
}
/* call device-specific configuration callback */
@@ -1862,12 +1859,9 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
/* if devices haven't supplied open/close words then supply them with simple
defaults */
if (!find_package_method("open", phandle) &&
!find_package_method("close", phandle)) {
- REGISTER_NODE_METHODS(ob_pci_simple_node, config.path);
+ BIND_NODE_METHODS(phandle, ob_pci_simple_node);
}
- /* device is configured so we may move it out of scope */
- device_end();
-
/* scan bus behind bridge device */
//if (htype & PCI_HEADER_TYPE_BRIDGE && class == PCI_BASE_CLASS_BRIDGE)
{
if ( class == PCI_BASE_CLASS_BRIDGE &&
@@ -1882,6 +1876,8 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
ob_configure_pci_bridge(addr, bus_num, mem_base, io_base, bus, &config);
}
+ fword("finish-device");
+
return phandle;
}
@@ -2103,7 +2099,8 @@ int ob_pci_init(void)
PCI_DPRINTF("Initializing PCI host bridge...\n");
- activate_device("/");
+ push_str("/");
+ fword("find-device");
/* Find all PCI bridges */
@@ -2130,16 +2127,9 @@ int ob_pci_init(void)
/* create root node for host PCI bridge */
/* configure */
- snprintf(config.path, sizeof(config.path), "/pci");
-
- REGISTER_NAMED_NODE_PHANDLE(ob_pci_bus_node, config.path, phandle_host);
-
- pci_host_set_reg(phandle_host);
-
- /* update device path after changing "reg" property */
- ob_pci_reload_device_path(phandle_host, &config);
+ strncpy(config.path, "", sizeof(config.path));
- ob_configure_pci_device(config.path, &bus, &mem_base, &io_base,
+ phandle_host = ob_configure_pci_device(config.path, &bus, &mem_base,
&io_base,
bus, devnum, fn, 0);
/* we expect single host PCI bridge
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 17/30] pci: remove ob_pci_initialize() and ob_pci_empty_node
The ob_pci_initialize() function can be removed as it is empty, whilst
ob_pci_empty_node is no longer required since the existing fallback code
will always apply the ob_pci_simple_node bindings if no callback is
configured.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pci.c | 15 ---------------
1 file changed, 15 deletions(-)
diff --git a/drivers/pci.c b/drivers/pci.c
index 623f649..d497109 100644
--- a/drivers/pci.c
+++ b/drivers/pci.c
@@ -54,7 +54,6 @@
DECLARE_UNNAMED_NODE( ob_pci_bus_node, INSTALL_OPEN, 2*sizeof(int) );
DECLARE_UNNAMED_NODE( ob_pci_bridge_node, INSTALL_OPEN, 2*sizeof(int) );
DECLARE_UNNAMED_NODE( ob_pci_simple_node, 0, 2*sizeof(int) );
-DECLARE_UNNAMED_NODE( ob_pci_empty_node, 0, 2*sizeof(int) );
const pci_arch_t *arch;
@@ -189,11 +188,6 @@ ob_pci_close(int *idx)
{
}
-static void
-ob_pci_initialize(int *idx)
-{
-}
-
/* ( str len -- phys.lo phys.mid phys.hi ) */
static void
@@ -462,7 +456,6 @@ ob_pci_dma_sync(int *idx)
}
NODE_METHODS(ob_pci_bus_node) = {
- { NULL, ob_pci_initialize },
{ "open", ob_pci_open },
{ "close", ob_pci_close },
{ "decode-unit", ob_pci_decode_unit },
@@ -485,7 +478,6 @@ ob_pci_bridge_map_in(int *idx)
}
NODE_METHODS(ob_pci_bridge_node) = {
- { NULL, ob_pci_initialize },
{ "open", ob_pci_open },
{ "close", ob_pci_close },
{ "decode-unit", ob_pci_decode_unit },
@@ -503,10 +495,6 @@ NODE_METHODS(ob_pci_simple_node) = {
{ "close", ob_pci_close },
};
-NODE_METHODS(ob_pci_empty_node) = {
- { NULL, ob_pci_initialize }
-};
-
static void pci_set_bus_range(const pci_config_t *config)
{
phandle_t dev = find_dev(config->path);
@@ -1827,9 +1815,6 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
BIND_NODE_METHODS(phandle, ob_pci_bus_node);
}
break;
- default:
- BIND_NODE_METHODS(phandle, ob_pci_empty_node);
- break;
}
if (htype & PCI_HEADER_TYPE_BRIDGE) {
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 18/30] libopenbios: remove REGISTER_NAMED_NODE and REGISTER_NAMED_NODE_PHANDLE macros
Now that we've removed all references to these binding macros they can be removed
completely.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
include/libopenbios/bindings.h | 11 -----------
1 file changed, 11 deletions(-)
diff --git a/include/libopenbios/bindings.h b/include/libopenbios/bindings.h
index 6360a4f..6b0302d 100644
--- a/include/libopenbios/bindings.h
+++ b/include/libopenbios/bindings.h
@@ -108,17 +108,6 @@ typedef struct {
void *func;
} method_t;
-#define REGISTER_NAMED_NODE( name, path ) do { \
- bind_new_node( name##_flags_, name##_size_, \
- path, name##_m, sizeof(name##_m)/sizeof(method_t)); \
- } while(0)
-
-#define REGISTER_NAMED_NODE_PHANDLE( name, path, phandle ) do { \
- phandle = \
- bind_new_node( name##_flags_, name##_size_, \
- path, name##_m, sizeof(name##_m)/sizeof(method_t)); \
- } while(0)
-
#define REGISTER_NODE_METHODS( name, path ) do { \
const char *paths[1]; \
\
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 19/30] nvram: ensure that NVRAM configuration is separate from NVRAM node creation
As part of this separation we can also move "update-nvram" directly to the
NVRAM node bindings.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/macio.c | 1 +
packages/nvram.c | 5 +----
2 files changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/macio.c b/drivers/macio.c
index 5a94252..33f29f1 100644
--- a/drivers/macio.c
+++ b/drivers/macio.c
@@ -91,6 +91,7 @@ void macio_nvram_init(const char *path, phys_addr_t addr)
nvram_size = macio_nvram_size();
nvram = (char*)addr + nvram_offset;
+ nvconf_init();
snprintf(buf, sizeof(buf), "%s", path);
dnode = nvram_init(buf);
set_int_property(dnode, "#bytes", arch_nvram_size() );
diff --git a/packages/nvram.c b/packages/nvram.c
index 6fd39b1..a402f00 100644
--- a/packages/nvram.c
+++ b/packages/nvram.c
@@ -186,8 +186,6 @@ nvconf_init( void )
nvram.data = malloc( nvram.size );
arch_nvram_get( nvram.data );
- bind_func( "update-nvram", update_nvram );
-
for( ;; ) {
nvpart_t *p = NULL;
int err;
@@ -309,6 +307,7 @@ NODE_METHODS( nvram ) = {
{ "read", (void*)nvram_read },
{ "write", (void*)nvram_write },
{ "seek", (void*)nvram_seek },
+ { "update-nvram", (void*)update_nvram },
};
@@ -322,8 +321,6 @@ nvram_init( const char *path )
fword("new-device");
- nvconf_init();
-
ph = get_cur_dev();
push_str("nvram");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 20/30] ppc: move New World uninorth and nvram device node creation to the root device
Whilst the NVRAM device node exists under the mac-io device node for Old World
Macs, both itself and the Uninorth device node exist under the root device node
on New World Macs.
Move creation of both device nodes to arch_of_init() for New World machines to
enable the active package and current instance to be set correctly during device
tree construction.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
arch/ppc/qemu/init.c | 12 +++++++++++-
drivers/macio.c | 5 +----
drivers/macio.h | 1 -
drivers/pci.c | 1 -
include/drivers/drivers.h | 2 ++
5 files changed, 14 insertions(+), 7 deletions(-)
diff --git a/arch/ppc/qemu/init.c b/arch/ppc/qemu/init.c
index f72728c..9056975 100644
--- a/arch/ppc/qemu/init.c
+++ b/arch/ppc/qemu/init.c
@@ -859,7 +859,17 @@ arch_of_init(void)
feval("['] ppc-dma-sync to (dma-sync)");
#ifdef CONFIG_DRIVER_PCI
- ob_pci_init();
+ switch (machine_id) {
+ case ARCH_MAC99:
+ case ARCH_MAC99_U3:
+ /* The NewWorld NVRAM is not located in the MacIO device */
+ macio_nvram_init("/", 0);
+ ob_pci_init();
+ ob_unin_init();
+ break;
+ default:
+ ob_pci_init();
+ }
#endif
printk("\n");
diff --git a/drivers/macio.c b/drivers/macio.c
index 33f29f1..62a1e74 100644
--- a/drivers/macio.c
+++ b/drivers/macio.c
@@ -262,7 +262,7 @@ NODE_METHODS(ob_macio) = {
{ "dma-sync", ob_macio_dma_sync },
};
-static void
+void
ob_unin_init(void)
{
phandle_t dnode;
@@ -395,12 +395,9 @@ ob_macio_keylargo_init(const char *path, phys_addr_t addr)
cuda_init(path, addr);
}
- /* The NewWorld NVRAM is not located in the MacIO device */
- macio_nvram_init("/", 0);
escc_init(path, addr);
macio_ide_init(path, addr, 2);
openpic_init(path, addr);
- ob_unin_init();
aliases = find_dev("/aliases");
set_property(aliases, "mac-io", path, strlen(path) + 1);
diff --git a/drivers/macio.h b/drivers/macio.h
index 925b881..36cc4bf 100644
--- a/drivers/macio.h
+++ b/drivers/macio.h
@@ -2,4 +2,3 @@ extern phandle_t pic_handle;
void ob_macio_heathrow_init(const char *path, phys_addr_t addr);
void ob_macio_keylargo_init(const char *path, phys_addr_t addr);
-void macio_nvram_init(const char *path, phys_addr_t addr);
diff --git a/drivers/pci.c b/drivers/pci.c
index d497109..c2da62f 100644
--- a/drivers/pci.c
+++ b/drivers/pci.c
@@ -31,7 +31,6 @@
#include "pci.h"
#include "pci_database.h"
#ifdef CONFIG_DRIVER_MACIO
-#include "cuda.h"
#include "macio.h"
#endif
#ifdef CONFIG_DRIVER_USB
diff --git a/include/drivers/drivers.h b/include/drivers/drivers.h
index e568f28..3d6fa12 100644
--- a/include/drivers/drivers.h
+++ b/include/drivers/drivers.h
@@ -149,6 +149,8 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
int macio_get_nvram_size(void);
void macio_nvram_put(char *buf);
void macio_nvram_get(char *buf);
+void macio_nvram_init(const char *path, phys_addr_t addr);
+void ob_unin_init(void);
/* drivers/timer.c */
void setup_timers(void);
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 21/30] ppc: set active package and current instance to root device node before probe
Now that all PCI devices have been converted to new-device...finish-device we can
set both the active package and current instance to the root device node before
probe, giving all child devices a correct active package and instance chain during
creation.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
arch/ppc/qemu/init.c | 6 ++++++
1 file changed, 6 insertions(+)
diff --git a/arch/ppc/qemu/init.c b/arch/ppc/qemu/init.c
index 9056975..45cd77e 100644
--- a/arch/ppc/qemu/init.c
+++ b/arch/ppc/qemu/init.c
@@ -859,6 +859,10 @@ arch_of_init(void)
feval("['] ppc-dma-sync to (dma-sync)");
#ifdef CONFIG_DRIVER_PCI
+ push_str("/");
+ fword("find-device");
+ feval("\" /\" open-dev to my-self");
+
switch (machine_id) {
case ARCH_MAC99:
case ARCH_MAC99_U3:
@@ -870,6 +874,8 @@ arch_of_init(void)
default:
ob_pci_init();
}
+
+ feval("0 to my-self");
#endif
printk("\n");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 22/30] SPARC64: set active package and current instance to root device node before probe
Now that all PCI devices have been converted to new-device...finish-device we can
set both the active package and current instance to the root device node before
probe, giving all child devices a correct active package and instance chain during
creation.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
arch/sparc64/openbios.c | 6 ++++++
drivers/ide.c | 3 ---
2 files changed, 6 insertions(+), 3 deletions(-)
diff --git a/arch/sparc64/openbios.c b/arch/sparc64/openbios.c
index e2f7e59..e9f4726 100644
--- a/arch/sparc64/openbios.c
+++ b/arch/sparc64/openbios.c
@@ -821,11 +821,17 @@ arch_init( void )
feval("['] sparc64-dma-sync to (dma-sync)");
#ifdef CONFIG_DRIVER_PCI
+ push_str("/");
+ fword("find-device");
+ feval("\" /\" open-dev to my-self");
+
ob_pci_init();
/* Set TAS register to match the virtual-dma properties
set during sabre configure */
sparc64_set_tas_register(PBM_PCI_TARGET_AS_CD_ENABLE);
+
+ feval("0 to my-self");
#endif
nvconf_init();
device_end();
diff --git a/drivers/ide.c b/drivers/ide.c
index 18a27c6..36e3b89 100644
--- a/drivers/ide.c
+++ b/drivers/ide.c
@@ -1392,9 +1392,6 @@ int ob_ide_init(const char *path, uint32_t io_port0, uint32_t
ctl_port0,
io_ports[1] = io_port1;
ctl_ports[1] = ctl_port1;
- push_str(path);
- fword("find-device");
-
for (i = 0; i < IDE_NUM_CHANNELS; i++, current_channel++) {
chan = malloc(sizeof(struct ide_channel));
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 23/30] x86: set active package and current instance to root device node before probe
Now that all PCI devices have been converted to new-device...finish-device we can
set both the active package and current instance to the root device node before
probe, giving all child devices a correct active package and instance chain during
creation.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
arch/x86/openbios.c | 9 ++++++++-
1 file changed, 8 insertions(+), 1 deletion(-)
diff --git a/arch/x86/openbios.c b/arch/x86/openbios.c
index 8a48a95..1274d2b 100644
--- a/arch/x86/openbios.c
+++ b/arch/x86/openbios.c
@@ -56,8 +56,15 @@ arch_init( void )
openbios_init();
modules_init();
#ifdef CONFIG_DRIVER_PCI
- arch = &default_pci_host;
+ arch = &default_pci_host;
+
+ push_str("/");
+ fword("find-device");
+ feval("\" /\" open-dev to my-self");
+
ob_pci_init();
+
+ feval("0 to my-self");
#endif
#ifdef CONFIG_DRIVER_IDE
setup_timers();
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 24/30] pci: remove explicit find-device from PCI devices
Since the correct active package is now being set during probe, there is no
need to explicitly find the parent node before creating the device.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/adb_bus.c | 3 ---
drivers/adb_kbd.c | 3 ---
drivers/adb_mouse.c | 3 ---
drivers/cuda.c | 3 ---
drivers/escc.c | 5 -----
drivers/floppy.c | 3 ---
drivers/ide.c | 3 ---
drivers/macio.c | 7 -------
drivers/pc_kbd.c | 7 -------
drivers/pc_serial.c | 3 ---
drivers/pci.c | 5 -----
drivers/pmu.c | 3 ---
drivers/usbhid.c | 3 ---
13 files changed, 51 deletions(-)
diff --git a/drivers/adb_bus.c b/drivers/adb_bus.c
index c505322..1aa442a 100644
--- a/drivers/adb_bus.c
+++ b/drivers/adb_bus.c
@@ -76,9 +76,6 @@ int adb_bus_init (char *path, adb_bus_t *bus)
int reloc = 0, next_free = 7;
int keep;
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("adb");
diff --git a/drivers/adb_kbd.c b/drivers/adb_kbd.c
index df53fe0..dec8366 100644
--- a/drivers/adb_kbd.c
+++ b/drivers/adb_kbd.c
@@ -552,9 +552,6 @@ void *adb_kbd_new (char *path, void *private)
my_adb_dev = dev;
}
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("keyboard");
diff --git a/drivers/adb_mouse.c b/drivers/adb_mouse.c
index 6ee512f..38eabd5 100644
--- a/drivers/adb_mouse.c
+++ b/drivers/adb_mouse.c
@@ -51,9 +51,6 @@ void adb_mouse_new (char *path, void *private)
phandle_t aliases;
adb_dev_t *dev = private;
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("mouse");
diff --git a/drivers/cuda.c b/drivers/cuda.c
index d88edf8..56d73e1 100644
--- a/drivers/cuda.c
+++ b/drivers/cuda.c
@@ -398,9 +398,6 @@ cuda_t *cuda_init (const char *path, phys_addr_t base)
if (cuda == NULL)
return NULL;
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("via-cuda");
diff --git a/drivers/escc.c b/drivers/escc.c
index 199d571..392625e 100644
--- a/drivers/escc.c
+++ b/drivers/escc.c
@@ -450,9 +450,6 @@ escc_add_channel(const char *path, const char *node, phys_addr_t
addr,
/* add device */
- push_str(path);
- fword("find-device");
-
fword("new-device");
snprintf(buf, sizeof(buf), "ch-%s", node);
@@ -531,8 +528,6 @@ escc_init(const char *path, phys_addr_t addr)
int props[2];
phandle_t dnode;
- push_str(path);
- fword("find-device");
fword("new-device");
push_str("escc");
diff --git a/drivers/floppy.c b/drivers/floppy.c
index f34811a..a3dff1f 100644
--- a/drivers/floppy.c
+++ b/drivers/floppy.c
@@ -1146,9 +1146,6 @@ int ob_floppy_init(const char *path, const char *dev_name,
char nodebuff[128];
phandle_t aliases;
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str(dev_name);
diff --git a/drivers/ide.c b/drivers/ide.c
index 36e3b89..e56cb4c 100644
--- a/drivers/ide.c
+++ b/drivers/ide.c
@@ -1585,9 +1585,6 @@ int macio_ide_init(const char *path, uint32_t addr, int
nb_channels)
* Also see comments in pci.c:ob_pci_host_set_interrupt_map() */
current_channel = 3;
- push_str(path);
- fword("find-device");
-
for (i = 0; i < nb_channels; i++) {
chan = malloc(sizeof(struct ide_channel));
diff --git a/drivers/macio.c b/drivers/macio.c
index 62a1e74..496bab1 100644
--- a/drivers/macio.c
+++ b/drivers/macio.c
@@ -168,8 +168,6 @@ openpic_init(const char *path, phys_addr_t addr)
int props[2];
char buf[128];
- push_str(path);
- fword("find-device");
fword("new-device");
push_str("interrupt-controller");
fword("device-name");
@@ -268,8 +266,6 @@ ob_unin_init(void)
phandle_t dnode;
int props[2];
- push_str("/");
- fword("find-device");
fword("new-device");
push_str("uni-n");
fword("device-name");
@@ -287,9 +283,6 @@ ob_unin_init(void)
static void macio_gpio_init(const char *path)
{
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("gpio");
diff --git a/drivers/pc_kbd.c b/drivers/pc_kbd.c
index ed6b82c..dfb73bf 100644
--- a/drivers/pc_kbd.c
+++ b/drivers/pc_kbd.c
@@ -208,9 +208,6 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
{
phandle_t chosen, aliases;
char nodebuff[128];
-
- push_str(path);
- fword("find-device");
fword("new-device");
@@ -303,10 +300,6 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
/* Mouse (optional) */
if (mdev_name != NULL) {
- snprintf(nodebuff, sizeof(nodebuff), "%s/8042", path);
- push_str(nodebuff);
- fword("find-device");
-
fword("new-device");
push_str(mdev_name);
diff --git a/drivers/pc_serial.c b/drivers/pc_serial.c
index 0d42689..c39afb4 100644
--- a/drivers/pc_serial.c
+++ b/drivers/pc_serial.c
@@ -168,9 +168,6 @@ ob_pc_serial_init(const char *path, const char *dev_name, uint64_t
base,
phandle_t aliases;
char nodebuff[128];
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str(dev_name);
diff --git a/drivers/pci.c b/drivers/pci.c
index c2da62f..4fba12a 100644
--- a/drivers/pci.c
+++ b/drivers/pci.c
@@ -2083,9 +2083,6 @@ int ob_pci_init(void)
PCI_DPRINTF("Initializing PCI host bridge...\n");
- push_str("/");
- fword("find-device");
-
/* Find all PCI bridges */
mem_base = arch->pci_mem_base;
@@ -2128,7 +2125,5 @@ int ob_pci_init(void)
intc = ob_pci_host_set_interrupt_map(phandle_host);
ob_pci_bus_set_interrupt_map(phandle_host, intc, ob_pci_host_bus_interrupt);
- device_end();
-
return 0;
}
diff --git a/drivers/pmu.c b/drivers/pmu.c
index 38fa1e8..f74a30b 100644
--- a/drivers/pmu.c
+++ b/drivers/pmu.c
@@ -606,9 +606,6 @@ pmu_t *pmu_init(const char *path, phys_addr_t base)
return NULL;
}
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("via-pmu");
diff --git a/drivers/usbhid.c b/drivers/usbhid.c
index a847b6e..b8e1548 100644
--- a/drivers/usbhid.c
+++ b/drivers/usbhid.c
@@ -564,9 +564,6 @@ void ob_usb_hid_add_keyboard(const char *path)
char name[128];
phandle_t aliases;
- push_str(path);
- fword("find-device");
-
fword("new-device");
push_str("keyboard");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 25/30] pci: remove explicit setting of my-self from PCI devices
Since the correct current instance is now being set during probe, there is no
need to explicitly set it whilst creating the device.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/lsi.c | 5 -----
drivers/virtio.c | 4 ----
2 files changed, 9 deletions(-)
diff --git a/drivers/lsi.c b/drivers/lsi.c
index bc4893a..a325547 100644
--- a/drivers/lsi.c
+++ b/drivers/lsi.c
@@ -653,10 +653,6 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
global_lsi = lsi;
/* Buffer for commands */
- fword("my-self");
- push_str(path);
- feval("open-dev to my-self");
-
PUSH(0x1000);
feval("dma-alloc");
addr = POP();
@@ -695,7 +691,6 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
set_int_property(ph, "#address-cells", 1);
set_int_property(ph, "#size-cells", 0);
- feval("to my-self");
/* Initialise SCRIPTS */
lsi->mmio = (uint8_t *)(uint32_t)mmio;
diff --git a/drivers/virtio.c b/drivers/virtio.c
index 6008ae9..4f9f5ee 100644
--- a/drivers/virtio.c
+++ b/drivers/virtio.c
@@ -504,9 +504,6 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
VDev *vdev, **_vdev;
/* Open ob_virtio */
- fword("my-self");
- push_str(path);
- feval("open-dev to my-self");
BIND_NODE_METHODS(get_cur_dev(), ob_virtio);
ph = find_ih_method("vdev", my_self());
@@ -532,7 +529,6 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
vdev->ring_area = cell2pointer(addr);
*_vdev = vdev;
- feval("to my-self");
fword("new-device");
push_str("disk");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 26/30] pci: call set-args before configuring PCI device nodes
This is to ensure that the my-space and my-address words are set correctly
for any PCI devices that wish to execute FCode.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pci.c | 21 +++++++++++++++++++++
1 file changed, 21 insertions(+)
diff --git a/drivers/pci.c b/drivers/pci.c
index 4fba12a..9d501d3 100644
--- a/drivers/pci.c
+++ b/drivers/pci.c
@@ -1748,6 +1748,8 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
const pci_dev_t *pci_dev;
uint8_t class, subclass, iface;
int num_bars, rom_bar;
+ uint32_t props[3];
+ char *unit_str;
phandle_t phandle = 0, t;
int is_host_bridge = 0;
@@ -1804,6 +1806,25 @@ static phandle_t ob_configure_pci_device(const char* parent_path,
config.dev = addr & 0x00FFFFFF;
+ if (!is_host_bridge) {
+ /* Get encoded address for set-args */
+ pci_encode_phys_addr(props, 0, CONFIGURATION_SPACE, config.dev, 0, 0);
+ PUSH(props[2]);
+ PUSH(props[1]);
+ PUSH(props[0]);
+ call_parent_method("encode-unit");
+
+ unit_str = pop_fstr_copy();
+
+ /* Call set-args to set up my-space and my-address */
+ PUSH(0);
+ PUSH(0);
+ push_str(unit_str);
+ fword("set-args");
+
+ free(unit_str);
+ }
+
phandle = get_cur_dev();
switch (class) {
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 27/30] pc_serial: remove separate init word
Instead use an instance value and initialise the C instance parameter within
the open word.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pc_serial.c | 13 +++++--------
1 file changed, 5 insertions(+), 8 deletions(-)
diff --git a/drivers/pc_serial.c b/drivers/pc_serial.c
index c39afb4..f67383d 100644
--- a/drivers/pc_serial.c
+++ b/drivers/pc_serial.c
@@ -142,19 +142,16 @@ pc_serial_close(void)
static void
pc_serial_open(unsigned long *address)
{
- RET ( -1 );
-}
-
-static void
-pc_serial_init(unsigned long *address)
-{
+ PUSH(find_ih_method("address", my_self()));
+ fword("execute");
*address = POP();
+
+ RET ( -1 );
}
DECLARE_UNNAMED_NODE(pc_serial, 0, sizeof(unsigned long));
NODE_METHODS(pc_serial) = {
- { "init", pc_serial_init },
{ "open", pc_serial_open },
{ "close", pc_serial_close },
{ "read", pc_serial_read },
@@ -201,7 +198,7 @@ ob_pc_serial_init(const char *path, const char *dev_name, uint64_t
base,
BIND_NODE_METHODS(get_cur_dev(), pc_serial);
PUSH(offset);
- feval("['] init execute");
+ feval("value address");
fword("finish-device");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 28/30] pc_kbd: use instance value to initialise C instance parameter
This simplifies the open word by avoiding having to locate and read the value
of the address property.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/pc_kbd.c | 16 +++++++---------
1 file changed, 7 insertions(+), 9 deletions(-)
diff --git a/drivers/pc_kbd.c b/drivers/pc_kbd.c
index dfb73bf..a98e57e 100644
--- a/drivers/pc_kbd.c
+++ b/drivers/pc_kbd.c
@@ -181,15 +181,9 @@ pc_kbd_close(void)
static void
pc_kbd_open(unsigned long *address)
{
- int len;
- phandle_t ph;
- unsigned long *prop;
-
- fword("my-self");
- fword("ihandle>phandle");
- ph = (phandle_t)POP();
- prop = (unsigned long *)get_property(ph, "address", &len);
- *address = *prop;
+ PUSH(find_ih_method("address", my_self()));
+ fword("execute");
+ *address = POP();
RET ( -1 );
}
@@ -285,6 +279,10 @@ ob_pc_kbd_init(const char *path, const char *kdev_name, const char
*mdev_name,
fword("property");
BIND_NODE_METHODS(get_cur_dev(), pc_kbd);
+
+ PUSH(offset);
+ feval("value address");
+
fword("finish-device");
snprintf(nodebuff, sizeof(nodebuff), "%s/8042/%s", path, kdev_name);
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 29/30] lsi: use instance value to hold sd_private_t pointer
It is also possible to remove the global_lsi variable by adding a pointer to
the parent lsi_private_t instance within sd_private_t.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/lsi.c | 42 +++++++++++++++++++++++++-----------------
1 file changed, 25 insertions(+), 17 deletions(-)
diff --git a/drivers/lsi.c b/drivers/lsi.c
index a325547..51a541c 100644
--- a/drivers/lsi.c
+++ b/drivers/lsi.c
@@ -18,7 +18,11 @@
#include "drivers/drivers.h"
#include "scsi.h"
-typedef struct sd_private {
+typedef struct sd_private sd_private_t;
+typedef struct lsi_table lsi_table_t;
+typedef struct lsi_private lsi_private_t;
+
+struct sd_private {
unsigned int bs;
const char *media_str[2];
uint32_t sectors;
@@ -26,9 +30,10 @@ typedef struct sd_private {
uint8_t id;
uint8_t present;
char model[40];
-} sd_private_t;
+ lsi_private_t *lsi;
+};
-typedef struct lsi_table {
+struct lsi_table {
uint32_t id;
uint32_t id_addr;
uint32_t msg_out_len;
@@ -41,9 +46,9 @@ typedef struct lsi_table {
uint32_t status_ptr;
uint32_t msg_in_len;
uint32_t msg_in_ptr;
-} lsi_table_t;
+};
-typedef struct lsi_private {
+struct lsi_private {
volatile uint8_t *mmio;
uint32_t *scripts;
uint32_t *scripts_iova;
@@ -52,9 +57,7 @@ typedef struct lsi_private {
volatile uint8_t *buffer;
volatile uint8_t *buffer_iova;
sd_private_t sd[8];
-} lsi_private_t;
-
-static lsi_private_t *global_lsi;
+};
#ifdef CONFIG_DEBUG_LSI
#define DPRINTF(fmt, args...) \
@@ -504,6 +507,7 @@ ob_sd_read_blocks(sd_private_t **sd)
ucell blk = POP();
char *dest = (char*)POP();
int pos, spb, sect_offset;
+ lsi_private_t *lsi = (*sd)->lsi;
DPRINTF("ob_sd_read_blocks id %d %lx block=%d n=%d\n", (*sd)->id,
(unsigned long)dest, blk, n );
@@ -516,12 +520,12 @@ ob_sd_read_blocks(sd_private_t **sd)
sect_offset = blk / spb;
pos = (blk - sect_offset * spb) * 512;
- if (ob_sd_read_sector(global_lsi, *sd, sect_offset)) {
+ if (ob_sd_read_sector(lsi, *sd, sect_offset)) {
DPRINTF("ob_sd_read_blocks: error\n");
RET(0);
}
while (n && pos < spb * 512) {
- memcpy(dest, (uint8_t *)&global_lsi->buffer[LSI_TABLE_DATA_OFFSET] +
pos, 512);
+ memcpy(dest, (uint8_t *)&lsi->buffer[LSI_TABLE_DATA_OFFSET] + pos,
512);
pos += 512;
dest += 512;
n--;
@@ -540,12 +544,12 @@ ob_sd_block_size(__attribute__((unused))sd_private_t **sd)
static void
ob_sd_open(__attribute__((unused))sd_private_t **sd)
{
- int ret = 1, id;
+ int ret = 1;
phandle_t ph;
- fword("my-unit");
- id = POP();
- *sd = &global_lsi->sd[id];
+ PUSH(find_ih_method("sd-private", my_self()));
+ fword("execute");
+ *sd = cell2pointer(POP());
#ifdef CONFIG_DEBUG_LSI
{
@@ -553,7 +557,7 @@ ob_sd_open(__attribute__((unused))sd_private_t **sd)
fword("my-args");
args = pop_fstr_copy();
- DPRINTF("opening drive %d args %s\n", id, args);
+ DPRINTF("opening drive args %s\n", args);
free(args);
}
#endif
@@ -650,8 +654,6 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
return -1;
}
- global_lsi = lsi;
-
/* Buffer for commands */
PUSH(0x1000);
feval("dma-alloc");
@@ -726,6 +728,9 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
for (id = 0; id < 8; id++) {
if (!lsi->sd[id].present)
continue;
+
+ lsi->sd[id].lsi = lsi;
+
fword("new-device");
push_str("sd");
fword("device-name");
@@ -740,6 +745,9 @@ ob_lsi_init(const char *path, uint64_t mmio, uint64_t ram)
push_str("reg");
fword("property");
+ PUSH(pointer2cell(&lsi->sd[id]));
+ feval("value sd-private");
+
BIND_NODE_METHODS(get_cur_dev(), ob_sd);
fword("finish-device");
--
2.20.1
10:38 p.m.
New subject: [PATCH v3 30/30] virtio: use instance value to initialise C instance parameter
This simplifies the open word by avoiding having to locate and read the value
of the _vdev property.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
---
drivers/virtio.c | 46 +++++++++++-----------------------------------
1 file changed, 11 insertions(+), 35 deletions(-)
diff --git a/drivers/virtio.c b/drivers/virtio.c
index 4f9f5ee..ff7ed6f 100644
--- a/drivers/virtio.c
+++ b/drivers/virtio.c
@@ -337,9 +337,14 @@ ob_virtio_configure_device(VDev *vdev)
static void
ob_virtio_disk_open(VDev **_vdev)
{
- VDev *vdev = *_vdev;
+ VDev *vdev;
phandle_t ph;
+ PUSH(find_ih_method("vdev", my_self()));
+ fword("execute");
+ *_vdev = cell2pointer(POP());
+ vdev = *_vdev;
+
vdev->pos = 0;
if (!vdev->configured) {
@@ -409,24 +414,9 @@ static void set_virtio_alias(const char *path, int idx)
set_property(aliases, name, path, strlen(path) + 1);
}
-static void
-ob_virtio_disk_initialize(VDev **_vdev)
-{
- phandle_t ph = get_cur_dev();
- VDev *vdev;
- int len;
-
- vdev = cell2pointer(get_int_property(ph, "_vdev", &len));
- push_str("_vdev");
- feval("delete-property");
-
- *_vdev = vdev;
-}
-
DECLARE_UNNAMED_NODE(ob_virtio_disk, 0, sizeof(VDev *));
NODE_METHODS(ob_virtio_disk) = {
- { NULL, ob_virtio_disk_initialize },
{ "open", ob_virtio_disk_open },
{ "close", ob_virtio_disk_close },
{ "seek", ob_virtio_disk_seek },
@@ -445,12 +435,6 @@ ob_virtio_close(VDev **_vdev)
return;
}
-static void
-ob_virtio_vdev(VDev **_vdev)
-{
- PUSH(pointer2cell(_vdev));
-}
-
static void
ob_virtio_dma_alloc(__attribute__((unused)) VDev **_vdev)
{
@@ -486,7 +470,6 @@ DECLARE_UNNAMED_NODE(ob_virtio, 0, sizeof(VDev *));
NODE_METHODS(ob_virtio) = {
{ "open", ob_virtio_open },
{ "close", ob_virtio_close },
- { "vdev", ob_virtio_vdev },
{ "dma-alloc", ob_virtio_dma_alloc },
{ "dma-free", ob_virtio_dma_free },
{ "dma-map-in", ob_virtio_dma_map_in },
@@ -499,18 +482,12 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
int idx)
{
char buf[256];
- phandle_t ph;
ucell addr;
- VDev *vdev, **_vdev;
+ VDev *vdev;
/* Open ob_virtio */
BIND_NODE_METHODS(get_cur_dev(), ob_virtio);
- ph = find_ih_method("vdev", my_self());
- PUSH(ph);
- fword("execute");
- _vdev = cell2pointer(POP());
-
vdev = malloc(sizeof(VDev));
vdev->common_cfg = common_cfg;
vdev->device_cfg = device_cfg;
@@ -518,6 +495,9 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
vdev->notify_mult = notify_mult;
vdev->configured = 0;
+ PUSH(pointer2cell(vdev));
+ feval("value vdev");
+
PUSH(sizeof(VRing) * VIRTIO_MAX_VQS);
feval("dma-alloc");
addr = POP();
@@ -528,8 +508,6 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
addr = POP();
vdev->ring_area = cell2pointer(addr);
- *_vdev = vdev;
-
fword("new-device");
push_str("disk");
fword("device-name");
@@ -537,9 +515,7 @@ void ob_virtio_init(const char *path, const char *dev_name, uint64_t
common_cfg,
fword("device-type");
PUSH(pointer2cell(vdev));
- fword("encode-int");
- push_str("_vdev");
- fword("property");
+ feval("value vdev");
BIND_NODE_METHODS(get_cur_dev(), ob_virtio_disk);
fword("finish-device");
--
2.20.1
22 Aug
22 Aug
10:59 p.m.
On 18/08/2019 21:37, Mark Cave-Ayland wrote:
This patchset started off with the aim of building the
PCI-based machine
device trees downwards from the root node to avoid switching either the active
package and/or current instance at various points during the tree construction.
The basis for this was to try and remove all named nodes (those which are
placed at a given path) and instead build the hierarchy in order by using
new-device and finish-device as per the IEEE-1275 specification. This was
realised by creating a new BIND_NODE_METHODS() macro which is similar to
REGISTER_NAMED_NODE() except that it doesn't alter either the active package
or the current instance, and then working through the device tree from the
root converting all the devices for the PCI-based machines to use it.
Once this conversion has been completed for all devices beneath the PCI node,
it becomes possible to remove both the REGISTER_NAMED_NODE() and
REGISTER_NAMED_NODE_PHANDLE() macros to prevent future patches from accidentally
changing the tree construction order.
Next the x86, PPC and SPARC64 device tree roots were altered to set the active
package and current instance, allowing the remaining places where the PCI code
alters either of them to be removed.
At this point instance variables were working correctly, and I noticed that
there were several PCI devices which made use of C initialisation functions and/or
properties for initial configuration. These can now be replaced with instance
variables to keep everything simple and consistent.
Signed-off-by: Mark Cave-Ayland <mark.cave-ayland(a)ilande.co.uk>
v3:
- Rebase onto master
- Fix up USB debug string for keyboard reported by Zoltan
- Fix for creating devices that don't have an entry in the PCI database
- Fix for MacOS 9 not initialising network reported by Howard
v2:
- Fix up NVRAM initialisation so that -prom-env 'vga-ndrv?=false' works again
Mark Cave-Ayland (30):
libopenbios: introduce BIND_NODE_METHODS() macro
pc_kbd: convert to use BIND_NODE_METHODS() macro
pc_serial: convert to use BIND_NODE_METHODS() macro
floppy: convert to use BIND_NODE_METHODS() macro
ide: convert to use BIND_NODE_METHODS() macro
adb: convert to use BIND_NODE_METHODS() macro
escc: convert to use BIND_NODE_METHODS() macro
cuda: convert to use BIND_NODE_METHODS() macro
macio: convert to use BIND_NODE_METHODS() macro
pmu: convert to use BIND_NODE_METHODS() macro
usbhid: convert to use BIND_NODE_METHODS() macro
nvram: convert to use BIND_NODE_METHODS() macro
lsi: don't change active package when setting device alias
lsi: convert to use BIND_NODE_METHODS() macro
virtio: convert to use BIND_NODE_METHODS() macro
pci: convert to use BIND_NODE_METHODS() macro
pci: remove ob_pci_initialize() and ob_pci_empty_node
libopenbios: remove REGISTER_NAMED_NODE and
REGISTER_NAMED_NODE_PHANDLE macros
nvram: ensure that NVRAM configuration is separate from NVRAM node
creation
ppc: move New World uninorth and nvram device node creation to the
root device
ppc: set active package and current instance to root device node
before probe
SPARC64: set active package and current instance to root device node
before probe
x86: set active package and current instance to root device node
before probe
pci: remove explicit find-device from PCI devices
pci: remove explicit setting of my-self from PCI devices
pci: call set-args before configuring PCI device nodes
pc_serial: remove separate init word
pc_kbd: use instance value to initialise C instance parameter
lsi: use instance value to hold sd_private_t pointer
virtio: use instance value to initialise C instance parameter
arch/ppc/qemu/init.c | 18 +++-
arch/sparc64/openbios.c | 6 ++
arch/x86/openbios.c | 9 +-
drivers/adb_bus.c | 57 +++++++-----
drivers/adb_kbd.c | 25 ++++--
drivers/adb_mouse.c | 31 ++++---
drivers/cuda.c | 158 ++++++++++++++++++++-------------
drivers/escc.c | 113 ++++++++++++++---------
drivers/floppy.c | 68 +++++++-------
drivers/ide.c | 123 +++++++++++++++----------
drivers/lsi.c | 65 +++++++-------
drivers/macio.c | 118 ++++++++++++------------
drivers/macio.h | 1 -
drivers/pc_kbd.c | 48 +++++-----
drivers/pc_serial.c | 32 +++----
drivers/pci.c | 132 +++++++++++++--------------
drivers/pmu.c | 152 +++++++++++++++++++++----------
drivers/usbhid.c | 17 ++--
drivers/virtio.c | 56 +++---------
include/drivers/drivers.h | 2 +
include/libopenbios/bindings.h | 19 ++--
include/packages/nvram.h | 4 +-
libopenbios/bindings.c | 10 +++
packages/nvram.c | 35 ++++++--
24 files changed, 755 insertions(+), 544 deletions(-)
I've had no bug reports since posting v3 last week, so I've now pushed this to
master. Thanks to everyone for helping test the changes.
ATB,
Mark.
23 Aug
23 Aug
12:16 a.m.
On Thu, 22 Aug 2019, Mark Cave-Ayland wrote:
I've had no bug reports since posting v3 last
week, so I've now pushed this to
master. Thanks to everyone for helping test the changes.
Thank you. Now only my pci-map-in patch is needed for ATI FCode ROM and I
have sent another patch before to improve printing of assigned-addresses
for what I've just sent a ping to remind you.
Regards,
BALATON Zoltan
561
days inactive
565
days old
32 comments
2 participants
participants (2)
-
BALATON Zoltan -
Mark Cave-Ayland