| Message ID | 20171026092804.4834-1-andy.tang@nxp.com |
|---|---|
| State | Not Applicable |
| Delegated to: | York Sun |
| Headers | show |
| Series | [U-Boot,1/2] armv8: ls1088a: Enable sata on ls1088a | expand |
On 10/26/2017 02:46 AM, andy.tang@nxp.com wrote: > From: Yuantian Tang <andy.tang@nxp.com> > > Enable sata feature on ls1088a platforms > > Signed-off-by: Tang Yuantian <andy.tang@nxp.com> > --- > include/configs/ls1088aqds.h | 12 ++++++++++++ > include/configs/ls1088ardb.h | 13 +++++++++++++ > 2 files changed, 25 insertions(+) > > diff --git a/include/configs/ls1088aqds.h b/include/configs/ls1088aqds.h > index 71d0e4e7d1..c1087edb38 100644 > --- a/include/configs/ls1088aqds.h > +++ b/include/configs/ls1088aqds.h > @@ -293,6 +293,18 @@ unsigned long get_board_ddr_clk(void); > #define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS 3 > #define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS 5 > > +/* SATA */ How about check CONFIG_SCSI here, and use Kconfig to select SCSI? > +#define CONFIG_LIBATA > +#define CONFIG_SCSI > +#define CONFIG_SCSI_AHCI > +#define CONFIG_SCSI_AHCI_PLAT > + > +#define CONFIG_SYS_SATA1 AHCI_BASE_ADDR1 > + > +#define CONFIG_SYS_SCSI_MAX_SCSI_ID 1 > +#define CONFIG_SYS_SCSI_MAX_LUN 1 > +#define CONFIG_SYS_SCSI_MAX_DEVICE (CONFIG_SYS_SCSI_MAX_SCSI_ID * \ > + CONFIG_SYS_SCSI_MAX_LUN) > /* QSPI device */ > #if defined(CONFIG_QSPI_BOOT) > #define CONFIG_FSL_QSPI > diff --git a/include/configs/ls1088ardb.h b/include/configs/ls1088ardb.h > index 39f1345f97..8587234dcd 100644 > --- a/include/configs/ls1088ardb.h > +++ b/include/configs/ls1088ardb.h > @@ -232,6 +232,19 @@ > #define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS 3 > #define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS 5 > > +/* SATA */ > +#define CONFIG_LIBATA > +#define CONFIG_SCSI > +#define CONFIG_SCSI_AHCI > +#define CONFIG_SCSI_AHCI_PLAT > + > +#define CONFIG_SYS_SATA1 AHCI_BASE_ADDR1 > + > +#define CONFIG_SYS_SCSI_MAX_SCSI_ID 1 > +#define CONFIG_SYS_SCSI_MAX_LUN 1 > +#define CONFIG_SYS_SCSI_MAX_DEVICE (CONFIG_SYS_SCSI_MAX_SCSI_ID * \ > + CONFIG_SYS_SCSI_MAX_LUN) > + Aren't the above two sections are identical? Do they fit well in the common header file? York
diff --git a/include/configs/ls1088aqds.h b/include/configs/ls1088aqds.h index 71d0e4e7d1..c1087edb38 100644 --- a/include/configs/ls1088aqds.h +++ b/include/configs/ls1088aqds.h @@ -293,6 +293,18 @@ unsigned long get_board_ddr_clk(void); #define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS 3 #define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS 5 +/* SATA */ +#define CONFIG_LIBATA +#define CONFIG_SCSI +#define CONFIG_SCSI_AHCI +#define CONFIG_SCSI_AHCI_PLAT + +#define CONFIG_SYS_SATA1 AHCI_BASE_ADDR1 + +#define CONFIG_SYS_SCSI_MAX_SCSI_ID 1 +#define CONFIG_SYS_SCSI_MAX_LUN 1 +#define CONFIG_SYS_SCSI_MAX_DEVICE (CONFIG_SYS_SCSI_MAX_SCSI_ID * \ + CONFIG_SYS_SCSI_MAX_LUN) /* QSPI device */ #if defined(CONFIG_QSPI_BOOT) #define CONFIG_FSL_QSPI diff --git a/include/configs/ls1088ardb.h b/include/configs/ls1088ardb.h index 39f1345f97..8587234dcd 100644 --- a/include/configs/ls1088ardb.h +++ b/include/configs/ls1088ardb.h @@ -232,6 +232,19 @@ #define CONFIG_SYS_EEPROM_PAGE_WRITE_BITS 3 #define CONFIG_SYS_EEPROM_PAGE_WRITE_DELAY_MS 5 +/* SATA */ +#define CONFIG_LIBATA +#define CONFIG_SCSI +#define CONFIG_SCSI_AHCI +#define CONFIG_SCSI_AHCI_PLAT + +#define CONFIG_SYS_SATA1 AHCI_BASE_ADDR1 + +#define CONFIG_SYS_SCSI_MAX_SCSI_ID 1 +#define CONFIG_SYS_SCSI_MAX_LUN 1 +#define CONFIG_SYS_SCSI_MAX_DEVICE (CONFIG_SYS_SCSI_MAX_SCSI_ID * \ + CONFIG_SYS_SCSI_MAX_LUN) + /* QSPI device */ #if defined(CONFIG_QSPI_BOOT) #define CONFIG_FSL_QSPI