system/freebsd-src
system/freebsd-src
1
0
Fork 0
mirror of https://github.com/freebsd/freebsd-src synced 2024-09-06 09:10:28 +00:00
Code Issues Packages Projects Releases Wiki Activity

Add ready polling after PHY reset on VIA SATA controllers.

Browse source 
According to PR there are cases of controller hang if soft reset is
sent before device report ready status after the hard reset.

I don't think this patch is perfect, but it was reported as working
by the submitter, and I have neither the old hardware nor interest to
test some improved version, so just done some style cleaning.

PR:		183294
Submitted by:	alexandre.martins@netasq.com
MFC after:	1 month
This commit is contained in:
Alexander Motin 2018-05-23 15:22:58 +00:00
parent 0cdd15f5a4
commit af66b67ab2
Notes: svn2git 2020-12-20 02:59:44 +00:00 
svn path=/head/; revision=334099
1 changed files with 19 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
sys/dev/ata/chipsets/ata-via.c
View file

@ -449,12 +449,29 @@ static void
ata_via_sata_reset(device_t dev)
{
struct ata_channel *ch = device_get_softc(dev);
int devs;
int devs, count;
uint8_t status;
if (ch->unit == 0) {
devs = ata_sata_phy_reset(dev, 0, 0);
DELAY(10000);
count = 0;
do {
ATA_IDX_OUTB(ch, ATA_DRIVE, ATA_D_IBM | ATA_D_LBA |
ATA_DEV(ATA_MASTER));
DELAY(1000);
status = ATA_IDX_INB(ch, ATA_STATUS);
count++;
} while (status & ATA_S_BUSY && count < 100);
devs += ata_sata_phy_reset(dev, 1, 0);
count = 0;
do {
ATA_IDX_OUTB(ch, ATA_DRIVE, ATA_D_IBM | ATA_D_LBA |
ATA_DEV(ATA_SLAVE));
DELAY(1000);
status = ATA_IDX_INB(ch, ATA_STATUS);
count++;
} while (status & ATA_S_BUSY && count < 100);
} else
devs = 1;
if (devs)