On 10/31/2017 02:55 AM, Ashish Kumar wrote: > Signed-off-by: Ashish Kumar <ashish.ku...@nxp.com> > Signed-off-by: Amrita Kumari <amrita.kum...@nxp.com> > --- > v2: Rebase to top > v3: Consolidate defines in common file > > include/configs/ls1088a_common.h | 11 +++++++++++ > 1 file changed, 11 insertions(+) > > diff --git a/include/configs/ls1088a_common.h > b/include/configs/ls1088a_common.h > index fa058f7..9d59472 100644 > --- a/include/configs/ls1088a_common.h > +++ b/include/configs/ls1088a_common.h > @@ -148,6 +148,17 @@ unsigned long long get_qixis_addr(void); > /* Miscellaneous configurable options */ > #define CONFIG_SYS_LOAD_ADDR (CONFIG_SYS_DDR_SDRAM_BASE + 0x10000000) > > +/* SATA */ > +#define CONFIG_LIBATA > +#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) > + > /* Physical Memory Map */ > #define CONFIG_CHIP_SELECTS_PER_CTRL 4 > >
I see you use "imply SCSI" in Kconfig, wouldn't it be better to put #ifdef to guard above macros so it may be easier to turn off SATA by menuconfig and/or defconfig if needed? York _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot
