Re: VT6410 IDE and WDC_NO_IDS

[Manuel Bouyer <bouyer%antioche.eu.org@localhost>]

On Sun, Dec 25, 2011 at 09:35:40PM +0100, Frank Wille wrote:
> In my effort to add Iomega Storcenter G2 support to sandpoint I
> experimented with Takahiro Kambe's patch for the VIA VT6410 IDE RAID
> controller (not using the RAID part here):
>   http://mail-index.netbsd.org/port-i386/2007/03/30/0000.html
> 
> The VT6410 freezes the kernel with an interrupt storm during wdcprobe().
> But after some time I found out that defining the option WDC_NO_IDS
> fixes that.
> 
> As far as I understand the problem, we cannot enable interrupts during
> drive probing, because there is something wrong (with the chip or with
> our driver?). But I am not an IDE expert.
> 
> I found no such hack in the Linux and FreeBSD sources, and not even
> in the Storcenter GPL patches, which Iomega released, so I see no
> other solution. Unfortunately there are (as usual) no datasheets for
> the VT6410 available either.
> 
> 
> So I tend to accept WDC_NO_IDS as a solution, but I don't like it as a
> config file option. In the GENERIC sandpoint config it would also enable
> this hack for all the other (innocent) IDE chips.
> 
> Now I would like to turn this option into a device property, so it
> can be enabled on demand.
> 
> AFAICS WDC_NO_IDS was introduced for evbmips/LOONGSON (workaround
> CS5536+JMH330), which I would have to change too.
> 
> Would that be ok for everybody? Or is there a better solution?

I'm not sure a device property would work, unless you can force it
with a config option: I found the JMH330 connected to a CS5536 in
the fuloong, but you can connect a JMH330 to any PATA controller
(and there's no way to detect this at run time). 

-- 
Manuel Bouyer <bouyer%antioche.eu.org@localhost>
     NetBSD: 26 ans d'experience feront toujours la difference
--