ata: sunxi: fix debug messages
authorDario Binacchi <dariobin@libero.it>
Sun, 24 Jan 2021 18:13:10 +0000 (19:13 +0100)
committerAndre Przywara <andre.przywara@arm.com>
Sun, 31 Jan 2021 23:55:56 +0000 (23:55 +0000)
It is useless and misleading to print the ret variable that is not set
by the dev_read_addr routine. Also, move the '\n' character after the
round bracket that contains the error code.

Signed-off-by: Dario Binacchi <dariobin@libero.it>
Reviewed-by: Simon Glass <sjg@chromium.org>
Signed-off-by: Andre Przywara <andre.przywara@arm.com>
drivers/ata/ahci_sunxi.c

index 58e345d..94a3379 100644 (file)
@@ -80,18 +80,18 @@ static int sunxi_sata_probe(struct udevice *dev)
 
        base = dev_read_addr(dev);
        if (base == FDT_ADDR_T_NONE) {
-               debug("%s: Failed to find address (err=%d\n)", __func__, ret);
+               debug("%s: Failed to find address\n", __func__);
                return -EINVAL;
        }
        reg = (u8 *)base;
        ret = sunxi_ahci_phy_init(reg);
        if (ret) {
-               debug("%s: Failed to init phy (err=%d\n)", __func__, ret);
+               debug("%s: Failed to init phy (err=%d)\n", __func__, ret);
                return ret;
        }
        ret = ahci_probe_scsi(dev, base);
        if (ret) {
-               debug("%s: Failed to probe (err=%d\n)", __func__, ret);
+               debug("%s: Failed to probe (err=%d)\n", __func__, ret);
                return ret;
        }
 
@@ -105,7 +105,7 @@ static int sunxi_sata_bind(struct udevice *dev)
 
        ret = ahci_bind_scsi(dev, &scsi_dev);
        if (ret) {
-               debug("%s: Failed to bind (err=%d\n)", __func__, ret);
+               debug("%s: Failed to bind (err=%d)\n", __func__, ret);
                return ret;
        }