mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Adam J. Richter" <adam@yggdrasil.com>
To: davem@redhat.com, jgarzik@mandrakesoft.com
Cc: linux-kernel@vger.kernel.org, willy@meta-x.org, wtarreau@yahoo.fr
Subject: Re: sunhme.c patch for new PCI interface (UNTESTED)
Date: Fri, 17 Nov 2000 14:15:03 -0800	[thread overview]
Message-ID: <200011172215.OAA06687@adam.yggdrasil.com> (raw)

Jeff Garzik writes:

>Are you aware of any hotplug sunhme hardware?  If no, don't change it to
>__devinit...

	Can I have a hot plug PCI bridge card that connects to
a regular PCI backplane (perhaps as some kind of CardBus docking
station card)?  If so, all PCI drivers should use __dev{init,exit}{,data}.

	The other excellent points that you raise apply equally to
the driver before and after my patch (although my patch made it
more clear that the struct netdevice parameters previously being
passed around were always null).  It is important to realize this
becase, as of yesterday, Dave had said that he was not going to
apply the new style PCI changes at this point, but had integrated
the MODULE_DEVICE_TABLE changes.  So, Dave: you should look at
the points that Jeff raised, even if you are not integrating
the rest of my new style PCI patch.

Adam J. Richter     __     ______________   4880 Stevens Creek Blvd, Suite 104
adam@yggdrasil.com     \ /                  San Jose, California 95129-1034
+1 408 261-6630         | g g d r a s i l   United States of America
fax +1 408 261-6631      "Free Software For The Rest Of Us."
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
Please read the FAQ at http://www.tux.org/lkml/

             reply	other threads:[~2000-11-17 22:46 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-11-17 22:15 Adam J. Richter [this message]
2000-11-17 22:40 ` Jeff Garzik
2000-11-19  6:50 ` Albert D. Cahalan
  -- strict thread matches above, loose matches on Subject: below --
2000-11-19  7:54 Adam J. Richter
2000-11-18  2:14 Adam J. Richter
2000-11-18  1:44 Adam J. Richter
2000-11-16 11:10 Adam J. Richter
2000-11-16 11:07 ` David S. Miller
2000-11-16  8:33 Adam J. Richter
2000-11-16  8:45 ` David S. Miller
2000-11-16 12:13   ` Christoph Hellwig
2000-11-16 12:22     ` David S. Miller
2000-11-17  6:51       ` Zach Brown
2000-11-17 21:42 ` Jeff Garzik

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=200011172215.OAA06687@adam.yggdrasil.com \
    --to=adam@yggdrasil.com \
    --cc=davem@redhat.com \
    --cc=jgarzik@mandrakesoft.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=willy@meta-x.org \
    --cc=wtarreau@yahoo.fr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®