diff options
| author | Dario Binacchi <dariobin@libero.it> | 2021-01-24 19:13:10 +0100 |
|---|---|---|
| committer | Andre Przywara <andre.przywara@arm.com> | 2021-01-31 23:55:56 +0000 |
| commit | a38bb0d0377dd47dfc9270fb46112bca263e8885 (patch) | |
| tree | 72a47bb1d653cf273b3754bef24061b7a062774a /drivers | |
| parent | 36ff6f00fb381fa906f563b153bdf533081d7513 (diff) | |
| download | u-boot-a38bb0d0377dd47dfc9270fb46112bca263e8885.tar.gz | |
ata: sunxi: fix debug messages
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>
Diffstat (limited to 'drivers')
| -rw-r--r-- | drivers/ata/ahci_sunxi.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/drivers/ata/ahci_sunxi.c b/drivers/ata/ahci_sunxi.c index 58e345d998..94a3379c53 100644 --- a/drivers/ata/ahci_sunxi.c +++ b/drivers/ata/ahci_sunxi.c @@ -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; } |