Re: atadev->param.model problem or bug with Marvell 88SE6811 chip

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

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
_at__at_ -605,6 +605,11 _at__at_ 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;