Date: Fri, 02 Mar 2012 12:45:37 +0200 From: Alexander Motin <mav@FreeBSD.org> To: Jia-Shiun Li <jiashiun@gmail.com> Cc: freebsd-current@freebsd.org Subject: Re: FreeBSD: Marvell 88SX61xx should be 88SE61xx Message-ID: <4F50A4D1.8050001@FreeBSD.org> In-Reply-To: <CAHNYxxPG2u%2BbnuVg-sS-Trt4PySUGwOzbhbk35Ar5fd_V7UdQQ@mail.gmail.com> References: <AANLkTi=_bh3JO%2BwkpgpsOzpcxnSJ_v2hndjUSOyhXS%2Be@mail.gmail.com> <CAHNYxxPp5FY_XkBZgY=C=tRi1BMQHxH7o1WjY1QwVquxWP4pVQ@mail.gmail.com> <CAHNYxxPG2u%2BbnuVg-sS-Trt4PySUGwOzbhbk35Ar5fd_V7UdQQ@mail.gmail.com>
next in thread | previous in thread | raw e-mail | index | archive | help
On 02/25/12 15:55, Jia-Shiun Li wrote: > Maybe some spam filters ate my mails so I am replying to current@. > Could anyone help to commit it? Sorry, I had vacation. Committed. > On Thu, Feb 23, 2012 at 8:57 PM, Jia-Shiun Li<jiashiun@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@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
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?4F50A4D1.8050001>