Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 14 Apr 2010 20:44:19 +0300
From:      Alexander Motin <mav@FreeBSD.org>
To:        Marius Strobl <marius@alchemy.franken.de>
Cc:        svn-src-head@freebsd.org, svn-src-all@freebsd.org, src-committers@freebsd.org
Subject:   Re: svn commit: r206604 - head/sys/dev/ata/chipsets
Message-ID:  <4BC5FEF3.2090801@FreeBSD.org>
In-Reply-To: <20100414170952.GA23301@alchemy.franken.de>
References:  <201004141529.o3EFTWd9011506@svn.freebsd.org> <20100414170952.GA23301@alchemy.franken.de>

next in thread | previous in thread | raw e-mail | index | archive | help
Marius Strobl wrote:
> On Wed, Apr 14, 2010 at 03:29:32PM +0000, Alexander Motin wrote:
>> Author: mav
>> Date: Wed Apr 14 15:29:32 2010
>> New Revision: 206604
>> URL: http://svn.freebsd.org/changeset/base/206604
>>
>> Log:
>>   For early ALI chips do not announce I/O sizes that require unsupported
>>   48bit DMA commands.
> 
> Wouldn't it make more sense to limit the I/O size based ATA_NO_48BIT_DMA
> in a central place rather than letting each chipset driver deal with it?

This driver is the only place where this flag is set now. And looking
that 48bit is almost mandatory now, unlikely it will be ever needed again.

-- 
Alexander Motin



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4BC5FEF3.2090801>