mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Adam J. Richter" <adam@yggdrasil.com>
To: bkz@linux-ide.org
Cc: linux-kernel@vger.kernel.org
Subject: Patch(?): linux-2.4.0-test11-pre4/drivers/sound/yss225.c compile failure
Date: Mon, 13 Nov 2000 14:44:34 -0800	[thread overview]
Message-ID: <200011132244.OAA03269@baldur.yggdrasil.com> (raw)




	linux-2.4.0-test11-pre4/drivers/sound/yss225.c uses __initdata
but does not include <linux/init.h>, so it could not compile.  I have
attached below.

	Note that I am a bit uncertain about the correctness of
the __initdata prefix here in the first place.  Is yss225 a PCI
device?  If so, a kernel that supports PCI hot plugging should
be prepared to support the possibility of a hot pluggable yss225
card being inserted after the module has already been initialized.
Even if no CardBus or CompactPCI version of yss225 hardware exists
yet, it will require less maintenance for PCI drivers to be prepared
for this possibility from the outset (besides, is it possible to have a
hot pluggable PCI bridge card that bridges to a regular PCI bus?).

	So, if yss225 is a PCI device, the declaration should use
__devinitdata.  On the other hand, if it is ISA only, then __initdata
should be correct.

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."

--- linux-2.4.0-test11-pre4/drivers/sound/yss225.c	Mon Nov 13 13:36:50 2000
+++ linux/drivers/sound/yss225.c	Mon Nov 13 09:11:02 2000
@@ -1,3 +1,4 @@
+#include <linux/init.h>
 unsigned char page_zero[] __initdata = {
 0x01, 0x7c, 0x00, 0x1e, 0x00, 0x00, 0x00, 0x00, 0x00, 0xf5, 0x00,
 0x11, 0x00, 0x20, 0x00, 0x32, 0x00, 0x40, 0x00, 0x13, 0x00, 0x00,
-
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-13 23:15 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2000-11-13 22:44 Adam J. Richter [this message]
2000-11-14 22:56 ` Bartlomiej Zolnierkiewicz
2000-11-14 23:01   ` Patch(?): linux-2.4.0-test11-pre4/drivers/sound/yss225.c compilefailure Jeff Garzik
2000-11-14 23:17     ` Bartlomiej Zolnierkiewicz

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=200011132244.OAA03269@baldur.yggdrasil.com \
    --to=adam@yggdrasil.com \
    --cc=bkz@linux-ide.org \
    --cc=linux-kernel@vger.kernel.org \
    /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®