lkml.org 
[lkml]   [2013]   [Sep]   [28]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH 1/1] AHCI: disabled FBS prior to issuing software reset
From
hi, Tejun
i had tested the patch with Marvell 88se9235. And driver can find disk
if FBS disabled, or can't find disk.
Please see chapter 9.3.9 and 9.3.8 of AHCI spec. Thanks.


2013/9/28 Tejun Heo <tj@kernel.org>:
> Hello,
>
> On Sat, Sep 28, 2013 at 07:13:36PM +0800, Xiangliang Yu wrote:
>> If device is attached to port multiplier, the detection process
>> look like this:
>> ahci_hardreset(link, class, deadline)
>> if (class == ATA_DEV_PMP) {
>> sata_pmp_attach(dev) /* will enable FBS */
>> sata_pmp_init_links(ap, nr_ports);
>> ata_for_each_link(link, ap, EDGE) {
>> sata_std_hardreset(link, class, deadline);
>> if (link_is_online)
>> ahci_softreset(link, class, deadline);
>> }
>> }
>> But, according to chapter 9.3.9 in AHCI spec: Prior to issuing software reset,
>> software shall clear PxCMD.ST to '0' and then clear PxFBS.EN to '0'.
>
> How was this tested? Do you observe any behavior difference? At this
> point, we are not using SRST on PMP ports anyway, so I can't see how
> this would make any difference.
>
> Thanks.
>
> --
> tejun


\
 
 \ /
  Last update: 2013-09-28 15:21    [W:0.083 / U:2.100 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site