[U-Boot] [PATCH v2 4/7] scsi: Move pccb buffer initalization directly to scsi_detect_dev
Michal Simek
michal.simek at xilinx.com
Fri Nov 25 16:00:17 CET 2016
pccb is pointer to temporary buffer which is used only for sending
command. Make it local as is done in scsi_read/scsi_write.
Signed-off-by: Michal Simek <michal.simek at xilinx.com>
Reviewed-by: Simon Glass <sjg at chromium.org>
---
Changes in v2: None
common/scsi.c | 12 ++++++------
1 file changed, 6 insertions(+), 6 deletions(-)
diff --git a/common/scsi.c b/common/scsi.c
index e8f102a8a4b5..5a925bafda28 100644
--- a/common/scsi.c
+++ b/common/scsi.c
@@ -480,10 +480,11 @@ static void scsi_init_dev_desc(struct blk_desc *dev_desc, int devnum)
#endif
}
+
/**
* scsi_detect_dev - Detect scsi device
*
- * @pccb: pointer to temporary SCSI command block
+ * @target: target id
* @dev_desc: block device description
*
* The scsi_detect_dev detects and fills a dev_desc structure when the device is
@@ -491,12 +492,14 @@ static void scsi_init_dev_desc(struct blk_desc *dev_desc, int devnum)
*
* Return: 0 on success, error value otherwise
*/
-static int scsi_detect_dev(ccb *pccb, struct blk_desc *dev_desc)
+static int scsi_detect_dev(int target, struct blk_desc *dev_desc)
{
unsigned char perq, modi;
lbaint_t capacity;
unsigned long blksz;
+ ccb *pccb = (ccb *)&tempccb;
+ pccb->target = target;
pccb->lun = dev_desc->lun;
pccb->pdata = (unsigned char *)&tempbuff;
pccb->datalen = 512;
@@ -561,7 +564,6 @@ void scsi_scan(int mode)
{
unsigned char i, lun;
int ret;
- ccb *pccb = (ccb *)&tempccb;
if (mode == 1)
printf("scanning bus for devices...\n");
@@ -570,11 +572,9 @@ void scsi_scan(int mode)
scsi_max_devs = 0;
for (i = 0; i < CONFIG_SYS_SCSI_MAX_SCSI_ID; i++) {
- pccb->target = i;
for (lun = 0; lun < CONFIG_SYS_SCSI_MAX_LUN; lun++) {
scsi_dev_desc[scsi_max_devs].lun = lun;
- ret = scsi_detect_dev(pccb,
- &scsi_dev_desc[scsi_max_devs]);
+ ret = scsi_detect_dev(i, &scsi_dev_desc[scsi_max_devs]);
if (ret)
continue;
--
1.9.1
More information about the U-Boot
mailing list