Patchwork Question about new_resource()

login
register
about
Submitter Sven Schnelle
Date 2011-01-27 17:38:55
Message ID <87sjweqma8.fsf@begreifnix.stackframe.org>
Download mbox | patch
Permalink /patch/2568/
State Superseded
Headers show

Comments

Sven Schnelle - 2011-01-27 17:38:55
Hi List,

while  adding support for the Lenovo PMH7 Embedded Controller,
i was wondering about the index parameter for new_resource().
From looking at the coreboot source, it looks like it is an
arbitrary number, depending on the device type for which the
resource is allocated. What would be the corect number to use
for an EC? Or should we add a new index for such devices? 

I'm attaching the pmh7 diff to this mail, i have chosen 0x77
as a temporary number.

Thanks,

Sven

Patch

Index: ec/lenovo/Kconfig
===================================================================
--- ec/lenovo/Kconfig	(revision 0)
+++ ec/lenovo/Kconfig	(revision 0)
@@ -0,0 +1 @@ 
+source src/ec/lenovo/pmh7/Kconfig
Index: ec/lenovo/Makefile.inc
===================================================================
--- ec/lenovo/Makefile.inc	(revision 0)
+++ ec/lenovo/Makefile.inc	(revision 0)
@@ -0,0 +1 @@ 
+subdirs-$(CONFIG_EC_LENOVO_PMH7) += pmh7
Index: ec/lenovo/pmh7/Kconfig
===================================================================
--- ec/lenovo/pmh7/Kconfig	(revision 0)
+++ ec/lenovo/pmh7/Kconfig	(revision 0)
@@ -0,0 +1,2 @@ 
+config EC_LENOVO_PMH7
+	bool
Index: ec/lenovo/pmh7/pmh7.h
===================================================================
--- ec/lenovo/pmh7/pmh7.h	(revision 0)
+++ ec/lenovo/pmh7/pmh7.h	(revision 0)
@@ -0,0 +1,13 @@ 
+#ifndef EC_LENOVO_PMH7_H
+#define EC_LENOVO_PMH7_H
+
+#define EC_LENOVO_PMH7_BASE 0x15e0
+#define EC_LENOVO_PMH7_ADDR (EC_LENOVO_PMH7_BASE + 0x0c)
+#define EC_LENOVO_PMH7_DATA (EC_LENOVO_PMH7_BASE + 0x0e)
+
+void pmh7_register_set_bit(int reg, int bit);
+void pmh7_register_clear_bit(int reg, int bit);
+char pmh7_register_read(int reg);
+void pmh7_register_write(int reg, int val);
+
+#endif
Index: ec/lenovo/pmh7/Makefile.inc
===================================================================
--- ec/lenovo/pmh7/Makefile.inc	(revision 0)
+++ ec/lenovo/pmh7/Makefile.inc	(revision 0)
@@ -0,0 +1 @@ 
+driver-y += pmh7.c
Index: ec/lenovo/pmh7/pmh7.c
===================================================================
--- ec/lenovo/pmh7/pmh7.c	(revision 0)
+++ ec/lenovo/pmh7/pmh7.c	(revision 0)
@@ -0,0 +1,56 @@ 
+#include <arch/io.h>
+#include <console/console.h>
+#include <device/device.h>
+#include <device/pnp.h>
+#include <stdlib.h>
+
+#include "pmh7.h"
+
+void pmh7_register_set_bit(int reg, int bit)
+{
+	char val;
+
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	val = inb(EC_LENOVO_PMH7_DATA);
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	outb(val | (1 << bit), EC_LENOVO_PMH7_DATA);
+}
+
+void pmh7_register_clear_bit(int reg, int bit)
+{
+	char val;
+
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	val = inb(EC_LENOVO_PMH7_DATA);
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	outb(val &= ~(1 << bit), EC_LENOVO_PMH7_DATA);
+}
+
+char pmh7_register_read(int reg)
+{
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	return inb(EC_LENOVO_PMH7_DATA);
+}
+
+void pmh7_register_write(int reg, int val)
+{
+	outb(reg, EC_LENOVO_PMH7_ADDR);
+	outb(val, EC_LENOVO_PMH7_DATA);
+}
+
+static void enable_dev(device_t dev)
+{
+	struct resource *resource;
+	resource = new_resource(dev, 0x77);
+	resource->flags = IORESOURCE_IO | IORESOURCE_FIXED;
+	resource->base = EC_LENOVO_PMH7_BASE;
+	resource->size = 16;
+	resource->align = 5;
+	resource->gran = 5;
+
+}
+
+struct chip_operations ec_lenovo_pmh7_ops = {
+	CHIP_NAME("Lenovo Power Management Hardware Hub 7")
+	.enable_dev = enable_dev,
+};