This function is only defined by one driver and is empty. Move it into
the SCSI implementation itself. We could remove it, but it should be
useful for debugging.

Signed-off-by: Simon Glass <s...@chromium.org>
---

 common/scsi.c                | 5 +++++
 drivers/block/ahci.c         | 5 -----
 drivers/block/sandbox_scsi.c | 4 ----
 include/scsi.h               | 1 -
 4 files changed, 5 insertions(+), 10 deletions(-)

diff --git a/common/scsi.c b/common/scsi.c
index 4896fb9350..6175e50764 100644
--- a/common/scsi.c
+++ b/common/scsi.c
@@ -48,6 +48,11 @@ static struct blk_desc 
scsi_dev_desc[CONFIG_SYS_SCSI_MAX_DEVICE];
 #define SCSI_MAX_READ_BLK 0xFFFF
 #define SCSI_LBA48_READ        0xFFFFFFF
 
+static void scsi_print_error(ccb *pccb)
+{
+       /* Dummy function that could print an error for debugging */
+}
+
 #ifdef CONFIG_SYS_64BIT_LBA
 void scsi_setup_read16(ccb *pccb, lbaint_t start, unsigned long blocks)
 {
diff --git a/drivers/block/ahci.c b/drivers/block/ahci.c
index 3fa14a76b8..f4744718a8 100644
--- a/drivers/block/ahci.c
+++ b/drivers/block/ahci.c
@@ -1092,8 +1092,3 @@ __weak void scsi_bus_reset(void)
 {
        /*Not implement*/
 }
-
-void scsi_print_error(ccb * pccb)
-{
-       /*The ahci error info can be read in the ahci driver*/
-}
diff --git a/drivers/block/sandbox_scsi.c b/drivers/block/sandbox_scsi.c
index ad961bd225..f4004a350c 100644
--- a/drivers/block/sandbox_scsi.c
+++ b/drivers/block/sandbox_scsi.c
@@ -23,7 +23,3 @@ int scsi_exec(ccb *pccb)
 {
        return 0;
 }
-
-void scsi_print_error(ccb *pccb)
-{
-}
diff --git a/include/scsi.h b/include/scsi.h
index 190dacd0f2..621d9382fc 100644
--- a/include/scsi.h
+++ b/include/scsi.h
@@ -163,7 +163,6 @@ typedef struct SCSI_cmd_block{
  * decleration of functions which have to reside in the LowLevel Part Driver
  */
 
-void scsi_print_error(ccb *pccb);
 int scsi_exec(ccb *pccb);
 void scsi_bus_reset(void);
 #if !defined(CONFIG_DM_SCSI)
-- 
2.13.0.506.g27d5fe0cd-goog

_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to