diff options
author | Tejun Heo <htejun@gmail.com> | 2008-04-07 09:47:20 -0400 |
---|---|---|
committer | Jeff Garzik <jgarzik@redhat.com> | 2008-04-17 15:44:23 -0400 |
commit | 5958e3025fd9d97429163e074d9cfa3848f51f28 (patch) | |
tree | f20492dc35fc2651452c871fbc812bd4f8379b40 /drivers/ata/libata-pmp.c | |
parent | ac371987a81c61c2efbd6931245cdcaf43baad89 (diff) |
libata: move PMP SCR access failure during reset to ata_eh_reset()
If PMP fan-out reset fails and SCR isn't accessible, PMP should be
reset. This used to be tested by sata_pmp_std_hardreset() and
communicated to EH by -ERESTART. However, this logic is generic and
doesn't really have much to do with specific hardreset implementation.
This patch moves SCR access failure detection logic to ata_eh_reset()
where it belongs. As this makes sata_pmp_std_hardreset() identical to
sata_std_hardreset(), the function is killed and replaced with the
standard method.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Diffstat (limited to 'drivers/ata/libata-pmp.c')
-rw-r--r-- | drivers/ata/libata-pmp.c | 35 |
1 files changed, 0 insertions, 35 deletions
diff --git a/drivers/ata/libata-pmp.c b/drivers/ata/libata-pmp.c index 2f8a9577c26d..9c998611b644 100644 --- a/drivers/ata/libata-pmp.c +++ b/drivers/ata/libata-pmp.c | |||
@@ -176,41 +176,6 @@ int sata_pmp_scr_write(struct ata_link *link, int reg, u32 val) | |||
176 | } | 176 | } |
177 | 177 | ||
178 | /** | 178 | /** |
179 | * sata_pmp_std_hardreset - standard hardreset method for PMP link | ||
180 | * @link: link to be reset | ||
181 | * @class: resulting class of attached device | ||
182 | * @deadline: deadline jiffies for the operation | ||
183 | * | ||
184 | * Hardreset PMP port @link. Note that this function doesn't | ||
185 | * wait for BSY clearance. There simply isn't a generic way to | ||
186 | * wait the event. Instead, this function return -EAGAIN thus | ||
187 | * telling libata-EH to followup with softreset. | ||
188 | * | ||
189 | * LOCKING: | ||
190 | * Kernel thread context (may sleep) | ||
191 | * | ||
192 | * RETURNS: | ||
193 | * 0 on success, -errno otherwise. | ||
194 | */ | ||
195 | int sata_pmp_std_hardreset(struct ata_link *link, unsigned int *class, | ||
196 | unsigned long deadline) | ||
197 | { | ||
198 | u32 tmp; | ||
199 | int rc; | ||
200 | |||
201 | DPRINTK("ENTER\n"); | ||
202 | |||
203 | rc = sata_std_hardreset(link, class, deadline); | ||
204 | |||
205 | /* if SCR isn't accessible, we need to reset the PMP */ | ||
206 | if (rc && rc != -EAGAIN && sata_scr_read(link, SCR_STATUS, &tmp)) | ||
207 | rc = -ERESTART; | ||
208 | |||
209 | DPRINTK("EXIT, rc=%d\n", rc); | ||
210 | return rc; | ||
211 | } | ||
212 | |||
213 | /** | ||
214 | * sata_pmp_read_gscr - read GSCR block of SATA PMP | 179 | * sata_pmp_read_gscr - read GSCR block of SATA PMP |
215 | * @dev: PMP device | 180 | * @dev: PMP device |
216 | * @gscr: buffer to read GSCR block into | 181 | * @gscr: buffer to read GSCR block into |