<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
  <meta content="text/html; charset=ISO-8859-1"
 http-equiv="Content-Type">
</head>
<body bgcolor="#ffffff" text="#000000">
Hi Myles,<br>
<br>
I like the patch, but it looks like you're wiping calls to adt7463
initialization on the s2881 completely with no replacement. is that on
purpose?<br>
<br>
Stefan<br>
<br>
On 6/16/10 10:50 PM, Myles Watson wrote:
<blockquote
 cite="mid:AANLkTingtsR4xtaPMENTl3bk5tOWM1hMqR2t9w8Ep3vR@mail.gmail.com"
 type="cite">
  <div class="moz-text-plain" wrap="true" graphical-quote="true"
 style="font-family: -moz-fixed; font-size: 13px;" lang="x-western">
  <pre wrap="">Index: svn/src/mainboard/tyan/s2881/mainboard.c
===================================================================
--- svn.orig/src/mainboard/tyan/s2881/mainboard.c
+++ svn/src/mainboard/tyan/s2881/mainboard.c
@@ -98,56 +98,6 @@ static void adt7463_init(device_t dev)
        printk(BIOS_DEBUG, "ADT7463 properly initialized\n");
 }
 
-static void dummy_noop(device_t dummy)
-{
-}
-
-static struct device_operations dummy_operations = {
-       .read_resources         = dummy_noop,
-       .set_resources          = dummy_noop,
-       .enable_resources       = dummy_noop,
-       .init                   = adt7463_init,
-};
-
-static unsigned int scan_root_bus(device_t root, unsigned int max)
-{
-       struct device_path path;
-       device_t dummy;
-
-       max = root_dev_scan_bus(root, max);
-
-       printk(BIOS_DEBUG, "scan_root_bus ok\n");
-
-       /* The following is a little silly. We need a hook into the boot
-        * process <b class="moz-txt-star"><span class="moz-txt-tag">*</span>after<span
 class="moz-txt-tag">*</span></b> the ADT7463 device has been initialized. So we
-        * create this dummy device, and we put the ADT7463 S2881 specific
-        * settings in its init function, which gets called
-        * as the last device to be initialized.
-        */
-
-       path.type = DEVICE_PATH_PNP;
-       path.pnp.port = 0;
-       path.pnp.device = 0;
-       dummy = alloc_dev(root->link_list, &path);
-       dummy->ops = &dummy_operations;
-
-       return max;
-}
-
-static struct device_operations mainboard_operations = {
-       .read_resources         = root_dev_read_resources,
-       .set_resources          = root_dev_set_resources,
-       .enable_resources       = root_dev_enable_resources,
-       .init                   = root_dev_init,
-       .scan_bus               = scan_root_bus,
-};
-
-static void enable_dev(struct device *dev)
-{
-       dev->ops = &mainboard_operations;
-}
-
 struct chip_operations mainboard_ops = {
        CHIP_NAME("Tyan S2881 Mainboard")
-       .enable_dev = enable_dev,
 };
  </pre>
  </div>
  <div class="moz-text-plain" wrap="true" graphical-quote="true"
 style="font-family: -moz-fixed; font-size: 13px;" lang="x-western">
  <pre wrap="">
  </pre>
  </div>
</blockquote>
<br>
</body>
</html>