Date: Mon, 12 May 2008 10:00:13 +0400 From: "Andrey V. Elsukov" <bu7cher@yandex.ru> To: Andrey Chernov <ache@nagual.pp.ru>, sos@freebsd.org, grehan@FreeBSD.ORG, current@freebsd.org Subject: Re: atadev->param.model problem or bug with Marvell 88SE6811 chip Message-ID: <4827DCED.1000205@yandex.ru> In-Reply-To: <20080511004607.GA67182@nagual.pp.ru> References: <20080511004607.GA67182@nagual.pp.ru>
next in thread | previous in thread | raw e-mail | index | archive | help
[-- Attachment #1 --]
Andrey Chernov wrote:
> I have device detection problem with device attached to
> atapci0: <Marvell ATA controller> port 0xec00-0xec07,0xe880-0xe883,0xe800-0xe807,0xe480-0xe483,0xe400-0xe40f mem 0xfebffc00-0xfebfffff irq 16 at device 0.0 on pci3
> (ASUS P5E3 motherboard).
> It seems that this controller not fills atadev->param.model as current
> ata-all.c code expects:
Can you try this patch and show verbose dmesg with patched kernel?
--
WBR, Andrey V. Elsukov
[-- Attachment #2 --]
Index: src/sys/dev/ata/ata-all.c
===================================================================
RCS file: /ncvs/src/sys/dev/ata/ata-all.c,v
retrieving revision 1.287
diff -u -b -p -r1.287 ata-all.c
--- src/sys/dev/ata/ata-all.c 20 Apr 2008 17:45:32 -0000 1.287
+++ src/sys/dev/ata/ata-all.c 12 May 2008 05:56:33 -0000
@@ -605,6 +605,11 @@ ata_getparam(struct ata_device *atadev,
ata_free_request(request);
}
+ if (!error && bootverbose)
+ device_printf(ch->dev, "IDENTIFY DEVICE data is %s, word2 is 0x%04x\n",
+ (atadev->config & (1 << 2)) ? "incomplete":"complete",
+ atadev->reserved2);
+
if (!error && (isprint(atadev->param.model[0]) ||
isprint(atadev->param.model[1]))) {
struct ata_params *atacap = &atadev->param;
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4827DCED.1000205>