mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: "Randy.Dunlap" <rddunlap@osdl.org>
To: "Cory Bell" <cory.bell@usa.net>
Cc: john@deater.net, linux-kernel@vger.kernel.org
Subject: Re: IRQ Routing Problem on ALi Chipset Laptop (HP Pavilion N5425)
Date: Thu, 6 Dec 2001 17:03:35 -0800	[thread overview]
Message-ID: <20011206170335.03856f7b.rddunlap@osdl.org> (raw)
In-Reply-To: <1007685691.6675.1.camel@localhost.localdomain>
In-Reply-To: <Pine.LNX.4.33.0112060938340.32381-100000@pianoman.cluster.toy> <1007685691.6675.1.camel@localhost.localdomain>

On 06 Dec 2001 16:41:29 -0800
"Cory Bell" <cory.bell@usa.net> wrote:

| On Thu, 2001-12-06 at 07:11, John Clemens wrote:
| > You are absolutely correct :) I did the same thing a few weeks ago (when i
| > was really working on it), and traced the lspci -vvxxx output and
| > interpreted everything linux was saying about it.  I was looking at it
| > from the acpect of maybe just changing the PCI router in config space as
| > well as the PCI irq from user space without requiring kernel changes at
| > all.  The reason why I didn't try that was because i chickened out and
| > didn't know wether changing the PIRQ table woudl a) work or b) permanently
| > screw up my machine.  This may still be the "correct way" however...
| 
| Well, the *actual* PIRQ table is supposed to be static, according to the
| spec. I don't see the $PIR signature anywhere in the ROM, so it may be
| generated on boot. As for changing the IRQ router PCI config space, the
| last patch is doing that already - r->set is just calling pirq_ali_set,
| which fiddles the bit in question.
| 
...
| 
| > Actually, i think the BIOS might "adjust" the pIRQ table at boot to match
| > it's view of the world.  I don't know.
| 
| I looked in the ROM file came with my latest BIOS update. I don't see
| the $PIR signature in there anywhere, so it may be generated, but
| static.

Hi-

Did your search for "$PIR" or "RIP$" ?
It is suppsed to be the latter (little-endian).

~Randy

| > I would really appreciate comments from someone who'se had more experience
| > than us with pIRQ problems...
| 
| That makes two of us.
| 
| > I guess the question is where to we proceed from here.  Our "best option"
| > may be to, at DMI scan time, recognise our laptops and change both PCI
| > config space and the routing table to point to irq 11.  And then we just
| > have to be brave enough to try it.  PCI config spae I don't mind mucking
| > with... internal chipset registers on the ISA bridge, that scares me
| > without proper documentation.  Maybe we should ask ALi for it?
| 
| Possibilities:
| 1) pirq_ali_get nastiness (above)
| 2) previous patch nastiness
| 3) DMI & 1
| 4) DMI & 2
| 5) ?
| 
| What do you think? Option 1 & 3 bother me a little, given that I find it
| conceptually dirty to write to registers from a function intended to
| read from registers. Still, is IS more separated from the generic code,
| and it's all in one place.
|  
| WRT DMI: Would we just create an is_broken_hp_pavilion_bios variable,
| similar to the is_sony_vaio_laptop variable? Declare it extern in
| dmi_scan.c and declare it int pci-irq.c or the other way around? Would
| it have to be a global variable?
| 
| I'm not really a C coder (I don't even play one on TV), I just code by
| example...and I learn fairly quickly.
| 
| I'll call ALi USA tomorrow morning. I used their "tech support page",
| but got no response. Supposedly, Darlene Brown (in the San Jose, USA)
| office is the one to speak to (I spoke to a secretary today). The
| secretary told me they don't generally release datasheets to
| individuals, but maybe I can get ahold of one, who knows. I also emailed
| Dan Hollis, who posted looking for a datasheet a while ago, but he said
| he never got one.

  parent reply	other threads:[~2001-12-07  1:09 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2001-12-05  5:16 Cory Bell
2001-12-05  5:58 ` John Clemens
2001-12-05  8:40   ` Cory Bell
2001-12-05 16:41     ` John Clemens
2001-12-06  7:00       ` Cory Bell
2001-12-06 15:11         ` John Clemens
2001-12-07  0:41           ` Cory Bell
2001-12-07  1:27             ` Cory Bell
2001-12-07 16:24               ` rddunlap
2001-12-07 17:35               ` Patrick Mochel
2001-12-07 21:23                 ` Cory Bell
2001-12-07 22:32                   ` Cory Bell
2001-12-08 22:38                   ` Pavel Machek
2001-12-08 23:04                   ` Pavel Machek
2001-12-09 20:02                   ` Cory Bell
2001-12-10 10:49                     ` Pavel Machek
2001-12-19 21:53                     ` Pavel Machek
2001-12-20 21:48                       ` Kai Germaschewski
2001-12-20 22:40                         ` Pavel Machek
2001-12-20 23:58                           ` Kai Germaschewski
2001-12-07 21:48                 ` Daniel Gryniewicz
2001-12-07 20:33             ` Pavel Machek
2001-12-09  5:37             ` Cory Bell
2001-12-09 13:13               ` Pavel Machek
2001-12-10 15:26                 ` John Clemens
2001-12-10 16:04                   ` Pavel Machek
2001-12-10 17:53                     ` John Clemens
2001-12-11  1:53                       ` Cory Bell
2001-12-11  9:45                         ` Pavel Machek
2001-12-10 18:32                   ` Pavel Machek
2001-12-10 20:56                   ` Cory Bell
2001-12-10 11:03               ` Pavel Machek
2001-12-10 16:01               ` Pavel Machek
2001-12-10 21:24               ` Cory Bell
2001-12-11 10:02                 ` Pavel Machek
2001-12-11 10:14               ` Pavel Machek
2001-12-11 15:19               ` Cory Bell
2001-12-11 15:36                 ` Pavel Machek
2001-12-11 17:08                 ` Cory Bell
2001-12-07  1:03           ` Randy.Dunlap [this message]
2001-12-05 16:54     ` Daniel Gryniewicz
2001-12-05 18:48       ` James Cassidy
2001-12-05 19:43       ` Daniel Gryniewicz
2001-12-07 18:13 Grover, Andrew

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=20011206170335.03856f7b.rddunlap@osdl.org \
    --to=rddunlap@osdl.org \
    --cc=cory.bell@usa.net \
    --cc=john@deater.net \
    --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®