From: Stefan Richter <stefanr@s5r6.in-berlin.de>
To: linux1394-devel@lists.sourceforge.net, linux-kernel@vger.kernel.org
Cc: Jesse Barnes <jbarnes@virtuousgeek.org>,
Andrew Morton <akpm@osdl.org>,
rob@janerob.com
Subject: Re: ohci1394 unhandled interrupts bug in 2.6.14-rc2
Date: Mon, 17 Oct 2005 20:50:56 +0200 [thread overview]
Message-ID: <4353F290.10009@s5r6.in-berlin.de> (raw)
In-Reply-To: <200510170930.33530.jbarnes@virtuousgeek.org>
Jesse Barnes wrote:
>>Jesse, what DMI_PRODUCT_NAME matches your laptop?
>
> I'll have to check when I get home, is the relevant info from the "System
> Information" section of the dmidecode output?
AFAICT yes. Although I wonder if there is a better suited part of the
DMI table to look at. From what Rob posted, we could simply match
{DMI_SYS_VENDOR, "TOSHIBA"} && {DMI_PRODUCT_VERSION, "PS5"} which
probably catches all Satellite 5xxx's. I hope this isn't too general.
I also wonder how 1394 CardBus cards would react on this patch in
affected Toshibas...
| + toshiba = dmi_check_system(extra_init_dmi_table);
| + if (toshiba) {
| + PRINT_G(KERN_INFO, "Toshiba %s detected, enabling extra
initialization code",
| + dmi_get_system_info(DMI_PRODUCT_NAME));
| + dev->current_state = 4;
| + pci_read_config_word(dev, PCI_CACHE_LINE_SIZE, &toshiba_pcls);
| + }
...
| if (pci_enable_device(dev))
| FAIL(-ENXIO, "Failed to enable OHCI hardware");
...
| + if (toshiba) {
| + mdelay(10);
| + pci_write_config_word(dev, PCI_CACHE_LINE_SIZE, toshiba_pcls);
| + pci_write_config_word(dev, PCI_INTERRUPT_LINE, dev->irq);
| + pci_write_config_dword(dev, PCI_BASE_ADDRESS_0,
pci_resource_start(dev, 0));
| + pci_write_config_dword(dev, PCI_BASE_ADDRESS_1,
pci_resource_start(dev, 1));
| + }
...
| pci_set_master(dev);
--
Stefan Richter
-=====-=-=-= =-=- =---=
http://arcgraph.de/sr/
next prev parent reply other threads:[~2005-10-17 18:53 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2005-10-15 18:55 jbarnes
2005-10-15 19:39 ` Stefan Richter
2005-10-15 20:29 ` Jesse Barnes
2005-10-15 20:40 ` new PCI quirk for Toshiba Satellite? Jesse Barnes
2005-10-20 0:06 ` Greg KH
2005-10-20 18:32 ` Stefan Richter
2005-10-21 18:38 ` Jesse Barnes
2005-10-21 20:13 ` Stefan Richter
2005-10-24 17:45 ` Jesse Barnes
2005-10-24 18:07 ` Jesse Barnes
2005-10-24 18:21 ` Stefan Richter
2005-10-24 21:09 ` Ivan Kokshaysky
2005-10-15 21:02 ` ohci1394 unhandled interrupts bug in 2.6.14-rc2 Stefan Richter
2005-10-15 21:59 ` Jesse Barnes
2005-10-17 7:55 ` Andrew Morton
2005-10-17 9:35 ` Stefan Richter
2005-10-17 9:42 ` Andrew Morton
2005-10-17 10:03 ` Stefan Richter
2005-10-17 16:30 ` Jesse Barnes
2005-10-17 18:50 ` Stefan Richter [this message]
2005-10-19 17:54 ` Jesse Barnes
2005-10-17 12:48 ` rob
2005-10-17 15:58 ` Stefan Richter
2005-10-18 5:32 ` rob
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=4353F290.10009@s5r6.in-berlin.de \
--to=stefanr@s5r6.in-berlin.de \
--cc=akpm@osdl.org \
--cc=jbarnes@virtuousgeek.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux1394-devel@lists.sourceforge.net \
--cc=rob@janerob.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
Powered by JetHome