Re: svn commit: r359499 - head/sys/dev/ahci
Hi.
On 01.04.2020 00:28, Ravi Pokala wrote:
> -Original Message-
> From: on behalf of Alexander Motin
>
> Date: 2020-03-31, Tuesday at 20:19
> To: , ,
>
> Subject: svn commit: r359499 - head/sys/dev/ahci
>
> Author: mav
> Date: Wed Apr 1 03:19:42 2020
> New Revision: 359499
> URL: https://svnweb.freebsd.org/changeset/base/359499
>
> Log:
> Add ID for JMicron JMB582/JMB585 AHCI controller.
>
> JMB582 has 2 6Gbps SATA ports and PCIe 3.0 x1.
> JMB585 has 5 6Gbps SATA ports and PCIe 3.0 x2.
>
> Both chips support AHCI v1.31, Port Multiplier with FBS and 8 MSI
> vectors.
>
> Hi Alexander,
>
> The second line of diff seems unrelated...?
It is related, because the check for shared PCI function for SATA and
PATA is no longer applicable to the new generation of HBAs, implementing
only AHCI. And without adding more strict check the driver does not
attach when it should.
>
> MFC after: 2 weeks
>
> Modified:
> head/sys/dev/ahci/ahci_pci.c
>
> Modified: head/sys/dev/ahci/ahci_pci.c
>
> ==
> --- head/sys/dev/ahci/ahci_pci.c Wed Apr 1 02:13:01 2020
> (r359498)
> +++ head/sys/dev/ahci/ahci_pci.c Wed Apr 1 03:19:42 2020
> (r359499)
> @@ -247,6 +247,7 @@ static const struct {
> {0x2365197b, 0x00, "JMicron JMB365",AHCI_Q_NOFORCE},
> {0x2366197b, 0x00, "JMicron JMB366",AHCI_Q_NOFORCE},
> {0x2368197b, 0x00, "JMicron JMB368",AHCI_Q_NOFORCE},
> + {0x0585197b, 0x00, "JMicron JMB58x",0},
> {0x61ab, 0x00, "Marvell 88SE6111", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
> AHCI_Q_1CH | AHCI_Q_EDGEIS},
> {0x612111ab, 0x00, "Marvell 88SE6121", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
> @@ -399,6 +400,7 @@ ahci_probe(device_t dev)
>!(ahci_ids[i].quirks & AHCI_Q_NOFORCE {
> /* Do not attach JMicrons with single PCI function. */
> if (pci_get_vendor(dev) == 0x197b &&
> + (ahci_ids[i].quirks & AHCI_Q_NOFORCE) &&
> (pci_read_config(dev, 0xdf, 1) & 0x40) == 0)
> return (ENXIO);
> snprintf(buf, sizeof(buf), "%s AHCI SATA controller",
>
>
>
--
Alexander Motin
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
Re: svn commit: r359499 - head/sys/dev/ahci
-Original Message-
From: on behalf of Alexander Motin
Date: 2020-03-31, Tuesday at 20:19
To: , ,
Subject: svn commit: r359499 - head/sys/dev/ahci
Author: mav
Date: Wed Apr 1 03:19:42 2020
New Revision: 359499
URL: https://svnweb.freebsd.org/changeset/base/359499
Log:
Add ID for JMicron JMB582/JMB585 AHCI controller.
JMB582 has 2 6Gbps SATA ports and PCIe 3.0 x1.
JMB585 has 5 6Gbps SATA ports and PCIe 3.0 x2.
Both chips support AHCI v1.31, Port Multiplier with FBS and 8 MSI vectors.
Hi Alexander,
The second line of diff seems unrelated...?
Thanks,
Ravi (rpokala@)
MFC after:2 weeks
Modified:
head/sys/dev/ahci/ahci_pci.c
Modified: head/sys/dev/ahci/ahci_pci.c
==
--- head/sys/dev/ahci/ahci_pci.cWed Apr 1 02:13:01 2020
(r359498)
+++ head/sys/dev/ahci/ahci_pci.cWed Apr 1 03:19:42 2020
(r359499)
@@ -247,6 +247,7 @@ static const struct {
{0x2365197b, 0x00, "JMicron JMB365",AHCI_Q_NOFORCE},
{0x2366197b, 0x00, "JMicron JMB366",AHCI_Q_NOFORCE},
{0x2368197b, 0x00, "JMicron JMB368",AHCI_Q_NOFORCE},
+ {0x0585197b, 0x00, "JMicron JMB58x",0},
{0x61ab, 0x00, "Marvell 88SE6111", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
AHCI_Q_1CH | AHCI_Q_EDGEIS},
{0x612111ab, 0x00, "Marvell 88SE6121", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
@@ -399,6 +400,7 @@ ahci_probe(device_t dev)
!(ahci_ids[i].quirks & AHCI_Q_NOFORCE {
/* Do not attach JMicrons with single PCI function. */
if (pci_get_vendor(dev) == 0x197b &&
+ (ahci_ids[i].quirks & AHCI_Q_NOFORCE) &&
(pci_read_config(dev, 0xdf, 1) & 0x40) == 0)
return (ENXIO);
snprintf(buf, sizeof(buf), "%s AHCI SATA controller",
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
svn commit: r359499 - head/sys/dev/ahci
Author: mav
Date: Wed Apr 1 03:19:42 2020
New Revision: 359499
URL: https://svnweb.freebsd.org/changeset/base/359499
Log:
Add ID for JMicron JMB582/JMB585 AHCI controller.
JMB582 has 2 6Gbps SATA ports and PCIe 3.0 x1.
JMB585 has 5 6Gbps SATA ports and PCIe 3.0 x2.
Both chips support AHCI v1.31, Port Multiplier with FBS and 8 MSI vectors.
MFC after:2 weeks
Modified:
head/sys/dev/ahci/ahci_pci.c
Modified: head/sys/dev/ahci/ahci_pci.c
==
--- head/sys/dev/ahci/ahci_pci.cWed Apr 1 02:13:01 2020
(r359498)
+++ head/sys/dev/ahci/ahci_pci.cWed Apr 1 03:19:42 2020
(r359499)
@@ -247,6 +247,7 @@ static const struct {
{0x2365197b, 0x00, "JMicron JMB365",AHCI_Q_NOFORCE},
{0x2366197b, 0x00, "JMicron JMB366",AHCI_Q_NOFORCE},
{0x2368197b, 0x00, "JMicron JMB368",AHCI_Q_NOFORCE},
+ {0x0585197b, 0x00, "JMicron JMB58x",0},
{0x61ab, 0x00, "Marvell 88SE6111", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
AHCI_Q_1CH | AHCI_Q_EDGEIS},
{0x612111ab, 0x00, "Marvell 88SE6121", AHCI_Q_NOFORCE | AHCI_Q_NOPMP |
@@ -399,6 +400,7 @@ ahci_probe(device_t dev)
!(ahci_ids[i].quirks & AHCI_Q_NOFORCE {
/* Do not attach JMicrons with single PCI function. */
if (pci_get_vendor(dev) == 0x197b &&
+ (ahci_ids[i].quirks & AHCI_Q_NOFORCE) &&
(pci_read_config(dev, 0xdf, 1) & 0x40) == 0)
return (ENXIO);
snprintf(buf, sizeof(buf), "%s AHCI SATA controller",
___
svn-src-all@freebsd.org mailing list
https://lists.freebsd.org/mailman/listinfo/svn-src-all
To unsubscribe, send any mail to "svn-src-all-unsubscr...@freebsd.org"
