Skip site navigation (1)Skip section navigation (2)
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
This is a multi-part message in MIME format.
--------------070307030306000202090406
Content-Type: text/plain; charset=KOI8-R; format=flowed
Content-Transfer-Encoding: 7bit

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

--------------070307030306000202090406
Content-Type: text/plain;
 name="ata.diff.txt"
Content-Transfer-Encoding: 7bit
Content-Disposition: inline;
 filename="ata.diff.txt"

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;

--------------070307030306000202090406--



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4827DCED.1000205>