[PATCH] fix UDMA settings in SB700

[Rudolf Marek]

Hello,

Following patch enables UDMA on ALL IDE devices. The current code enables it 
only for primary master, which causes my DVD drive to fail under windows install
and even after hard reset in linux (DMA seems lockup).

The fix should not have any influence for Linux because the IDE driver will
correctly reprogram this bit.

This brings some attention to IDE code, to what PIO/(U)DMA  should we setup? Is 
there a sane way to assume that all IDE devices are capable of PIO0 and UDMA0?

Maybe Seabios should do some basic setup because it can talk to discs...

Signed-off-by: Rudolf Marek

Thanks,
Rudolf

[Uwe Hermann]

On Thu, Oct 07, 2010 at 12:39:12AM +0200, Rudolf Marek wrote:
> Hello,
>
> Following patch enables UDMA on ALL IDE devices. The current code
> enables it only for primary master, which causes my DVD drive to
> fail under windows install
> and even after hard reset in linux (DMA seems lockup).
>
> The fix should not have any influence for Linux because the IDE driver will
> correctly reprogram this bit.
>
> This brings some attention to IDE code, to what PIO/(U)DMA  should
> we setup? Is there a sane way to assume that all IDE devices are
> capable of PIO0 and UDMA0?
>
> Maybe Seabios should do some basic setup because it can talk to discs...
>
> Signed-off-by: Rudolf Marek

Acked-by: Uwe Hermann

Uwe.

[Patrick Georgi]

Am 10.10.2010 21:06, schrieb Uwe Hermann:
> > This brings some attention to IDE code, to what PIO/(U)DMA  should
> > we setup? Is there a sane way to assume that all IDE devices are
> > capable of PIO0 and UDMA0?
> >
> > Maybe Seabios should do some basic setup because it can talk to discs...

There's some chipset specific code necessary on i945 (which doesn't do
it completely correct at this time yet), so coreboot would be the right
place, not seabios.

This involves a couple of ATA roundtrips, but well, that's life. The ATA
code probably belongs in src/lib, for the benefit of all chipsets,
should they want to fully evaluate the ATA configuration.

Patrick