FreeBSD: Marvell 88SX61xx should be 88SE61xx

Jia-Shiun Li jiashiun at gmail.com
Sat Feb 25 13:55:50 UTC 2012


Hi all,

Maybe some spam filters ate my mails so I am replying to current at .
Could anyone help to commit it?

Regards,

Jia-Shiun

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?
>>
>> Regards,
>>
>> Jia-Shiun.


More information about the freebsd-current mailing list