public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Jean Tourrilhes <jt@bougret.hpl.hp.com>
To: Linux kernel mailing list <linux-kernel@vger.kernel.org>,
	William Lee Irwin III <wli@holomorphy.com>
Subject: Re: [3/12] remove irda usage of isa_virt_to_bus()
Date: Mon, 14 Jun 2004 18:43:44 -0700	[thread overview]
Message-ID: <20040615014344.GA17657@bougret.hpl.hp.com> (raw)

William Lee Irwin III wrote :
> 
>  * Removed uses of isa_virt_to_bus
> This resolves Debian BTS #218878.
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=218878
> 
> 	From: Paavo Hartikainen <pahartik@sci.fi>
> 	To: Debian Bug Tracking System <submit@bugs.debian.org>
> 	Subject: IrDA modules fail to load
> 	Message-Id: <E1AGYn5-0000LT-00@mufasa.vip.fi>
> 
> When trying to "modprobe irda irtty", it fails with following message:
>   FATAL: Error inserting irda (/lib/modules/2.6.0-test9/kernel/net/irda/irda.ko): \
> Unknown symbol in module, or unknown parameter (see dmesg) And in "dmesg" I see \
> these:  irda: Unknown symbol isa_virt_to_bus
>   irda: Unknown symbol isa_virt_to_bus

	Could you please send this directly to me. I hate scrubbing
large patches from the mailing list archive.
	Note that before even thinking of pushing this patch in the
kernel, we need to perform testing with the hardware on i386 and
potentially on ARM. The author only tried with irtty that doesn't use
this function, so that's not a valid test at all. Finding people test
those changes is going to be tough, as usual.
	I'm also wondering about the validity of those changes, but
that's another matter I need to go through. During 2.5.X, some people
assured me that using isa_virt_to_bus was safe on all platform with an
ISA bus...

	Thanks...

	Jean


             reply	other threads:[~2004-06-15  1:43 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-06-15  1:43 Jean Tourrilhes [this message]
2004-06-15  9:12 ` [3/12] remove irda usage of isa_virt_to_bus() William Lee Irwin III
2004-06-15 17:01   ` Jean Tourrilhes
2004-06-15 17:35     ` William Lee Irwin III
  -- strict thread matches above, loose matches on Subject: below --
2004-06-14  0:31 [0/12] Debian bugfixes William Lee Irwin III
2004-06-14  0:33 ` [1/12] don't dereference netdev->name before register_netdev() William Lee Irwin III
2004-06-14  0:34   ` [2/12] lower priority of "too many keys" msg in atkbd.c William Lee Irwin III
2004-06-14  0:36     ` [3/12] remove irda usage of isa_virt_to_bus() William Lee Irwin III
2004-06-14  8:11       ` Christoph Hellwig

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=20040615014344.GA17657@bougret.hpl.hp.com \
    --to=jt@bougret.hpl.hp.com \
    --cc=jt@hpl.hp.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=wli@holomorphy.com \
    /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