linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Adam Belay <ambx1@neo.rr.com>
To: Linus Torvalds <torvalds@osdl.org>
Cc: Misha Nasledov <misha@nasledov.com>,
	Andrew Morton <akpm@osdl.org>,
	Russell King <rmk@arm.linux.org.uk>,
	linux-kernel@vger.kernel.org
Subject: Re: 2.5/2.6 PCMCIA Issues
Date: Wed, 6 Aug 2003 11:42:25 +0000	[thread overview]
Message-ID: <20030806114225.GI13275@neo.rr.com> (raw)
In-Reply-To: <200308060559.h765xhI05860@mail.osdl.org>

On Tue, Aug 05, 2003 at 10:59:43PM -0700, OSDL wrote:
> Misha Nasledov wrote:
> > 
> > I am attaching the dmesg output after booting 2.6.0-test2; this does
> > not include the insertion of the Orinoco card as the console freezes
> > immediately after the event. I inspected my logs after a reboot and
> > there were no messages whatsoever regarding the event of the insertion
> > of the Orinoco card.
> 
> Can you try with PnP and the i82365 support _disabled_. I find this sequence
> very suspicious:
> 
>         Intel PCIC probe: PNP <6>pnp: Device 00:17 activated.
>         invalid resources ?
>         pnp: Device 00:17 disabled.
>         not found.

Here's a quick fix that will at least correct the i82365 probing.  It will
be interesting to see what effect it has on these other problems.  Note
that this driver needs some work with check_region etc.

[PCMCIA] Fix PnP Probing in i82365.c
pnp_x_valid returns 1 if valid.  Therefore we should be using !pnp_port_valid.
Also cleans up some formatting issues.

--- a/drivers/pcmcia/i82365.c	2003-07-27 17:08:32.000000000 +0000
+++ b/drivers/pcmcia/i82365.c	2003-08-06 11:18:21.000000000 +0000
@@ -795,16 +795,14 @@

 	    if (pnp_device_attach(dev) < 0)
 	    	continue;
-
-	    printk("PNP ");
-
+
 	    if (pnp_activate_dev(dev) < 0) {
 		printk("activate failed\n");
 		pnp_device_detach(dev);
 		break;
 	    }

-	    if (pnp_port_valid(dev, 0)) {
+	    if (!pnp_port_valid(dev, 0)) {
 		printk("invalid resources ?\n");
 		pnp_device_detach(dev);
 		break;


> 
> and I bet it messes up some of the register state that the yenta probe had
> just set up.
> 
> You should try with just CONFIG_YENTA - the 82365 stuff is for the old
> 16-bit only controllers.
> 
>                 Linus

Thanks,
Adam

  parent reply	other threads:[~2003-08-06 16:15 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2003-08-04 23:22 2.5/2.6 PCMCIA Issues Misha Nasledov
2003-08-05 13:44 ` Russell King
2003-08-06  4:56   ` Misha Nasledov
2003-08-06  5:59     ` OSDL
2003-08-06  6:42       ` Andrew Morton
2003-08-06  7:11         ` Linus Torvalds
2003-08-06  9:33           ` Russell King
2003-08-06 11:23             ` Russell King
2003-08-06 15:33               ` Linus Torvalds
2003-08-07 11:29           ` Alan Cox
2003-08-06  6:42       ` Misha Nasledov
2003-08-06  7:06       ` Misha Nasledov
2003-08-06 11:42       ` Adam Belay [this message]
2003-08-06 16:26         ` Russell King
2003-08-06 20:34         ` Andrew Morton
2003-08-06 20:53           ` Russell King
     [not found] <20031013161010.GH1623@ee.oulu.fi>
2003-10-13 16:15 ` Misha Nasledov

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=20030806114225.GI13275@neo.rr.com \
    --to=ambx1@neo.rr.com \
    --cc=akpm@osdl.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=misha@nasledov.com \
    --cc=rmk@arm.linux.org.uk \
    --cc=torvalds@osdl.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;
as well as URLs for NNTP newsgroup(s).