Attachment 302271 Details for Bug 442260 – patch to make hal support of_platform bus

[patch] patch to make hal support of_platform bus

hal-0.5.11-of_platform.patch (text/plain), 3.98 KB, created by David Woodhouse on 2008-04-13 15:37:04 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Woodhouse

Created: 2008-04-13 15:37:04 UTC

Size: 3.98 KB

patch

obsolete

>diff --git a/doc/spec/hal-spec-properties.xml b/doc/spec/hal-spec-properties.xml
>index 1fd72b6..afad7c6 100644
>--- a/doc/spec/hal-spec-properties.xml
>+++ b/doc/spec/hal-spec-properties.xml
>@@ -2993,6 +2993,40 @@ org.freedesktop.Hal.Device.Volume.method_signatures = {'ssas', 'as', 'as'}
>         </tgroup>
>       </informaltable>
>     </sect2>
>+
>+    <sect2 id="device-properties-of_platform">
>+      <title>
>+        of_platform namespace
>+      </title>
>+      <para>
>+        Devices on the virtual 'of_platform' bus are represented
>+        by device objects where <literal>info.subsystem</literal>
>+        equals <literal>of_platform</literal>. The following
>+        properties are available for such device objects.
>+      </para>
>+      <informaltable>
>+        <tgroup cols="2">
>+          <thead>
>+            <row>
>+              <entry>Key (type)</entry>
>+              <entry>Values</entry>
>+              <entry>Mandatory</entry>
>+              <entry>Description</entry>
>+            </row>
>+          </thead>
>+          <tbody>
>+            <row>
>+              <entry>
>+                <literal>of_platform.id</literal> (string)
>+              </entry>
>+              <entry>example: f0003000.ethernet</entry>
>+              <entry>Yes</entry>
>+              <entry>Device identification</entry>
>+            </row>
>+          </tbody>
>+        </tgroup>
>+      </informaltable>
>+    </sect2>
>   </sect1>
> 
>   <sect1 id="properties-functional">
>diff --git a/hald/linux/device.c b/hald/linux/device.c
>index 1026b2a..e4aae0c 100644
>--- a/hald/linux/device.c
>+++ b/hald/linux/device.c
>@@ -3610,6 +3610,52 @@ virtio_compute_udi (HalDevice *d)
> /*--------------------------------------------------------------------------------------------------------------*/
> 
> static HalDevice *
>+of_platform_add (const gchar *sysfs_path, const gchar *device_file, HalDevice *parent_dev,
>+		const gchar *sysfs_path_in_devices)
>+{
>+	HalDevice *d;
>+	const gchar *dev_id;
>+	gchar buf[64];
>+
>+	d = hal_device_new ();
>+	hal_device_property_set_string (d, "linux.sysfs_path", sysfs_path);
>+	hal_device_property_set_string (d, "info.subsystem", "of_platform");
>+	hal_device_property_set_string (d, "info.bus", "of_platform");
>+	if (parent_dev != NULL) {
>+		hal_device_property_set_string (d, "info.parent", hal_device_get_udi (parent_dev));
>+	} else {
>+		hal_device_property_set_string (d, "info.parent", "/org/freedesktop/Hal/devices/computer");
>+	}
>+
>+	hal_util_set_driver (d, "info.linux.driver", sysfs_path);
>+
>+	dev_id = hal_util_get_last_element (sysfs_path);
>+
>+	hal_device_property_set_string (d, "of_platform.id", dev_id);
>+
>+	g_snprintf (buf, sizeof (buf), "OpenFirmware Platform Device (%s)", hal_device_property_get_string (d, "of_platform.id"));
>+	hal_device_property_set_string (d, "info.product", buf);
>+
>+	return d;
>+}
>+
>+static gboolean
>+of_platform_compute_udi (HalDevice *d)
>+{
>+	gchar udi[256];
>+
>+	hal_util_compute_udi (hald_get_gdl (), udi, sizeof (udi),
>+			      "/org/freedesktop/Hal/devices/of_platform_%s",
>+			      hal_device_property_get_string (d, "of_platform.id"));
>+	hal_device_set_udi (d, udi);
>+	hal_device_property_set_string (d, "info.udi", udi);
>+
>+	return TRUE;
>+}
>+
>+/*--------------------------------------------------------------------------------------------------------------*/
>+
>+static HalDevice *
> pseudo_add (const gchar *sysfs_path, const gchar *device_file, HalDevice *parent_dev, const gchar *parent_path)
> {
> 	HalDevice *d;
>@@ -3977,6 +4023,14 @@ static DevHandler dev_handler_virtio =
> 	.remove      = dev_remove
> };
> 
>+static DevHandler dev_handler_of_platform =
>+{
>+	.subsystem   = "of_platform",
>+	.add         = of_platform_add,
>+	.compute_udi = of_platform_compute_udi,
>+	.remove      = dev_remove
>+};
>+
> /* SCSI debug, to test thousends of fake devices */
> static DevHandler dev_handler_pseudo = {
> 	.subsystem   = "pseudo",
>@@ -4024,7 +4078,8 @@ static DevHandler *dev_handlers[] = {
> 	&dev_handler_drm,
> 	&dev_handler_ps3_system_bus,
> 	&dev_handler_virtio,
> 	&dev_handler_vio,
>+	&dev_handler_of_platform,
> 	NULL
> };
>

Actions: View | Diff

Attachments on bug 442260: 302271