ata: libata-sff: make ata_devchk() return 'bool'
authorSergey Shtylyov <s.shtylyov@omp.ru>
Wed, 9 Feb 2022 21:46:28 +0000 (00:46 +0300)
committerDamien Le Moal <damien.lemoal@opensource.wdc.com>
Sat, 19 Feb 2022 02:18:43 +0000 (11:18 +0900)
ata_devchk() returns 1 if a device is present, 0 if not -- the 'bool' type
clearly fits better here than 'unsigned int'...

Signed-off-by: Sergey Shtylyov <s.shtylyov@omp.ru>
Signed-off-by: Damien Le Moal <damien.lemoal@opensource.wdc.com>
drivers/ata/libata-sff.c

index c9dbfb3641fcf70b090b66dd4f6e45d10a565f71..546b1f73ede51734eea53076c677b483fd213311 100644 (file)
@@ -1751,10 +1751,13 @@ EXPORT_SYMBOL_GPL(ata_sff_prereset);
  *     correctly storing and echoing back the
  *     ATA shadow register contents.
  *
+ *     RETURN:
+ *     true if device is present, false if not.
+ *
  *     LOCKING:
  *     caller.
  */
-static unsigned int ata_devchk(struct ata_port *ap, unsigned int device)
+static bool ata_devchk(struct ata_port *ap, unsigned int device)
 {
        struct ata_ioports *ioaddr = &ap->ioaddr;
        u8 nsect, lbal;
@@ -1774,9 +1777,9 @@ static unsigned int ata_devchk(struct ata_port *ap, unsigned int device)
        lbal = ioread8(ioaddr->lbal_addr);
 
        if ((nsect == 0x55) && (lbal == 0xaa))
-               return 1;       /* we found a device */
+               return true;    /* we found a device */
 
-       return 0;               /* nothing found */
+       return false;           /* nothing found */
 }
 
 /**