Commit 5eb8deb4 authored by Sergey Shtylyov's avatar Sergey Shtylyov Committed by Damien Le Moal

ata: libata-core: get rid of *else* branches in ata_id_n_sectors()

Using *else* after *return* doesn't make much sense -- getting rid of such
*else* branches reduces the indentation levels and thus reduces # of LoC...
Signed-off-by: default avatarSergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: default avatarDamien Le Moal <damien.lemoal@opensource.wdc.com>
parent 79ad6a56
...@@ -1103,18 +1103,16 @@ static u64 ata_id_n_sectors(const u16 *id) ...@@ -1103,18 +1103,16 @@ static u64 ata_id_n_sectors(const u16 *id)
if (ata_id_has_lba(id)) { if (ata_id_has_lba(id)) {
if (ata_id_has_lba48(id)) if (ata_id_has_lba48(id))
return ata_id_u64(id, ATA_ID_LBA_CAPACITY_2); return ata_id_u64(id, ATA_ID_LBA_CAPACITY_2);
else
return ata_id_u32(id, ATA_ID_LBA_CAPACITY); return ata_id_u32(id, ATA_ID_LBA_CAPACITY);
} else { }
if (ata_id_current_chs_valid(id)) if (ata_id_current_chs_valid(id))
return (u32)id[ATA_ID_CUR_CYLS] * return (u32)id[ATA_ID_CUR_CYLS] * (u32)id[ATA_ID_CUR_HEADS] *
(u32)id[ATA_ID_CUR_HEADS] *
(u32)id[ATA_ID_CUR_SECTORS]; (u32)id[ATA_ID_CUR_SECTORS];
else
return (u32)id[ATA_ID_CYLS] * return (u32)id[ATA_ID_CYLS] * (u32)id[ATA_ID_HEADS] *
(u32)id[ATA_ID_HEADS] *
(u32)id[ATA_ID_SECTORS]; (u32)id[ATA_ID_SECTORS];
}
} }
u64 ata_tf_to_lba48(const struct ata_taskfile *tf) u64 ata_tf_to_lba48(const struct ata_taskfile *tf)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment