FreeBSD: Marvell 88SX61xx should be 88SE61xx

Alexander Motin mav at FreeBSD.org
Fri Mar 2 10:45:42 UTC 2012


On 02/25/12 15:55, Jia-Shiun Li wrote:
> Maybe some spam filters ate my mails so I am replying to current at .
> Could anyone help to commit it?

Sorry, I had vacation. Committed.

> On Thu, Feb 23, 2012 at 8:57 PM, Jia-Shiun Li<jiashiun at gmail.com>  wrote:
>> Hi Alexander,
>>
>> I've submitted a PR for this:
>>
>> http://www.freebsd.org/cgi/query-pr.cgi?pr=165271
>>
>> could you help to review the patch attached to the PR and check it in?
>> It only changes strings, not any functionality.
>>
>> Regards,
>> Jia-Shiun.
>>
>> On Tue, Mar 29, 2011 at 1:57 AM, Jia-Shiun Li<jiashiun at gmail.com>  wrote:
>>> Hi Alexander,
>>>
>>> I am Jia-Shiun Li, writing to you for the Marvell SATA device names in
>>> FreeBSD drivers.
>>>
>>> I noticed the boot message for my onboard 88SE6121:
>>>
>>> atapci0:<Marvell 88SX6121 UDMA133 controller>  port
>>> 0xec00-0xec07,0xe880-0xe883,0xe800-0xe807,0xe480-0xe483,0xe400-0xe40f mem
>>> 0xfeaffc00-0xfeafffff irq 16 at device 0.0 on pci2
>>> ahci0:<Marvell 88SX6121 AHCI SATA controller>  on atapci0
>>>
>>> according to Marvell their names should be 88SE[69]1xx instead of
>>> 88SX[69]1xx:
>>> http://www.marvell.com/products/storage/storage_system_solutions/sata_controllers_pc_consumer/
>>>
>>> a simple way to tell is that mvs(4) devices are 88SX and ahci(4) devices are
>>> 88SE.
>>>
>>> I made a patch out of this, basically just text replacement. It does not
>>> affect functionality, but I think it would be better to be identified
>>> correctly anyway. Would you mind help committing the changes?

-- 
Alexander Motin


More information about the freebsd-current mailing list