Эх сурвалжийг харах

scsi: Drop scsi_print_error()

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 <sjg@chromium.org>
Reviewed-by: Bin Meng <bmeng.cn@gmail.com>
Simon Glass 8 жил өмнө
parent
commit
a6fb185c70

+ 5 - 0
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)
 {

+ 0 - 5
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*/
-}

+ 0 - 4
drivers/block/sandbox_scsi.c

@@ -23,7 +23,3 @@ int scsi_exec(ccb *pccb)
 {
 	return 0;
 }
-
-void scsi_print_error(ccb *pccb)
-{
-}

+ 0 - 1
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)