[U-Boot] [PATCH 1/2] scsi: add PCIe to sata converter Marvell chip 88SE9170
Add to the command scsi device list.
Signed-off-by: Po Liu po.liu@nxp.com --- cmd/scsi.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/cmd/scsi.c b/cmd/scsi.c index 8991125..c85fefc 100644 --- a/cmd/scsi.c +++ b/cmd/scsi.c @@ -31,6 +31,11 @@ #define SCSI_VEND_ID 0x10b9 #define SCSI_DEV_ID 0x5288
+#elif defined CONFIG_SATA_88SE9170 + +#define SCSI_VEND_ID 0x1b4b +#define SCSI_DEV_ID 0x9170 + #elif !defined(CONFIG_SCSI_AHCI_PLAT) #error no scsi device defined #endif
On 04/13/2016 04:09 AM, Po Liu wrote:
Add to the command scsi device list.
Signed-off-by: Po Liu po.liu@nxp.com
cmd/scsi.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/cmd/scsi.c b/cmd/scsi.c index 8991125..c85fefc 100644 --- a/cmd/scsi.c +++ b/cmd/scsi.c @@ -31,6 +31,11 @@ #define SCSI_VEND_ID 0x10b9 #define SCSI_DEV_ID 0x5288
+#elif defined CONFIG_SATA_88SE9170
+#define SCSI_VEND_ID 0x1b4b +#define SCSI_DEV_ID 0x9170
#elif !defined(CONFIG_SCSI_AHCI_PLAT) #error no scsi device defined #endif
Po,
Would it be a better idea to define CONFIG_SCSI_DEV_LIST in your board header file? You can merge your two patches into one.
York
-----Original Message----- From: York Sun [mailto:york.sun@nxp.com] Sent: Tuesday, May 17, 2016 12:03 AM To: Po Liu Cc: u-boot@lists.denx.de; Simon Glass; Bin Meng; osp@andrep.de; mark.langsdorf@gmail.com Subject: Re: [PATCH 1/2] scsi: add PCIe to sata converter Marvell chip 88SE9170
On 04/13/2016 04:09 AM, Po Liu wrote:
Add to the command scsi device list.
Signed-off-by: Po Liu po.liu@nxp.com
cmd/scsi.c | 5 +++++ 1 file changed, 5 insertions(+)
diff --git a/cmd/scsi.c b/cmd/scsi.c index 8991125..c85fefc 100644 --- a/cmd/scsi.c +++ b/cmd/scsi.c @@ -31,6 +31,11 @@ #define SCSI_VEND_ID 0x10b9 #define SCSI_DEV_ID 0x5288
+#elif defined CONFIG_SATA_88SE9170
+#define SCSI_VEND_ID 0x1b4b +#define SCSI_DEV_ID 0x9170
#elif !defined(CONFIG_SCSI_AHCI_PLAT) #error no scsi device defined #endif
Po,
Would it be a better idea to define CONFIG_SCSI_DEV_LIST in your board header file? You can merge your two patches into one.
Ok, Understand.
York
Add scsi command and enable supporting the pcie2sata chip 88SE9170 from Marvell.
Signed-off-by: Po Liu po.liu@nxp.com --- merge to one patch only.
include/configs/ls1043ardb.h | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+)
diff --git a/include/configs/ls1043ardb.h b/include/configs/ls1043ardb.h index 6d35be2..e299545 100644 --- a/include/configs/ls1043ardb.h +++ b/include/configs/ls1043ardb.h @@ -285,6 +285,27 @@ #define CONFIG_USB_STORAGE #endif
+/* SATA */ +#define CONFIG_LIBATA +#define CONFIG_SCSI_AHCI +#define CONFIG_CMD_SCSI +#ifndef CONFIG_CMD_FAT +#define CONFIG_CMD_FAT +#endif +#ifndef CONFIG_CMD_EXT2 +#define CONFIG_CMD_EXT2 +#endif +#define CONFIG_DOS_PARTITION +#define CONFIG_BOARD_LATE_INIT +#define CONFIG_SYS_SCSI_MAX_SCSI_ID 2 +#define CONFIG_SYS_SCSI_MAX_LUN 2 +#define CONFIG_SYS_SCSI_MAX_DEVICE (CONFIG_SYS_SCSI_MAX_SCSI_ID * \ + CONFIG_SYS_SCSI_MAX_LUN) +#define SCSI_VEND_ID 0x1b4b +#define SCSI_DEV_ID 0x9170 +#define CONFIG_SCSI_DEV_LIST {SCSI_VEND_ID, SCSI_DEV_ID} +#define CONFIG_PCI + #include <asm/fsl_secure_boot.h>
#endif /* __LS1043ARDB_H__ */
On 05/17/2016 07:19 PM, Po Liu wrote:
Add scsi command and enable supporting the pcie2sata chip 88SE9170 from Marvell.
Signed-off-by: Po Liu po.liu@nxp.com
merge to one patch only.
include/configs/ls1043ardb.h | 21 +++++++++++++++++++++ 1 file changed, 21 insertions(+)
Minor adjustment to commit subject and message. Applied to u-boot-fsl-qoriq master, awaiting upstream.
Thanks.
York
participants (2)
-
Po Liu -
York Sun