diff options
author | Tejun Heo <htejun@gmail.com> | 2006-03-31 08:48:52 -0500 |
---|---|---|
committer | Jeff Garzik <jeff@garzik.org> | 2006-03-31 10:14:42 -0500 |
commit | be9a50c8524b53003e5fa32f072945772ffd13a5 (patch) | |
tree | a82555012a718f030f6be95fbec5336ff9b76cca | |
parent | 8e0e694a3a48212bfe29a9ad3cd592bf68dfec81 (diff) |
[PATCH] libata: fix ata_xfer_tbl termination
ata_xfer_tbl is terminated by entry with -1 as ->shift. However,
->shift was unsigned int making the termination condition bogus. This
patch converts ->shift and ->bits to int.
Signed-off-by: Tejun Heo <htejun@gmail.com>
Signed-off-by: Jeff Garzik <jeff@garzik.org>
-rw-r--r-- | drivers/scsi/libata-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/libata-core.c b/drivers/scsi/libata-core.c index 63ae23b5c078..e63c1ff1e102 100644 --- a/drivers/scsi/libata-core.c +++ b/drivers/scsi/libata-core.c | |||
@@ -278,7 +278,7 @@ static void ata_unpack_xfermask(unsigned int xfer_mask, | |||
278 | } | 278 | } |
279 | 279 | ||
280 | static const struct ata_xfer_ent { | 280 | static const struct ata_xfer_ent { |
281 | unsigned int shift, bits; | 281 | int shift, bits; |
282 | u8 base; | 282 | u8 base; |
283 | } ata_xfer_tbl[] = { | 283 | } ata_xfer_tbl[] = { |
284 | { ATA_SHIFT_PIO, ATA_BITS_PIO, XFER_PIO_0 }, | 284 | { ATA_SHIFT_PIO, ATA_BITS_PIO, XFER_PIO_0 }, |