[U-Boot] [PATCH v2] arm/ls1021a: Add sata support on qds and twr board
York Sun
yorksun at freescale.com
Tue Aug 18 21:04:48 CEST 2015
On 08/17/2015 02:42 AM, Yuantian.Tang at freescale.com wrote:
> From: Tang Yuantian <Yuantian.Tang at freescale.com>
>
> Freescale ARM-based Layerscape LS102xA contain a SATA controller
> which comply with the serial ATA 3.0 specification and the
> AHCI 1.3 specification.
> This patch adds SATA feature on ls1021aqds and ls1021atwr boards.
>
> Signed-off-by: Tang Yuantian <Yuantian.Tang at freescale.com>
> ---
> v2:
> - rebase to latest git tree
> - use micro SATA_ECC_REG_ADDR instead of hard coding
>
> arch/arm/include/asm/arch-ls102xa/config.h | 15 ++++++
> arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h | 24 +++++++++
> board/freescale/ls1021aqds/ls1021aqds.c | 61 +++++++++++++++++++++++
> board/freescale/ls1021atwr/ls1021atwr.c | 61 +++++++++++++++++++++++
> 4 files changed, 161 insertions(+)
>
> diff --git a/arch/arm/include/asm/arch-ls102xa/config.h b/arch/arm/include/asm/arch-ls102xa/config.h
> index c55cdef..a4a5d84 100644
> --- a/arch/arm/include/asm/arch-ls102xa/config.h
> +++ b/arch/arm/include/asm/arch-ls102xa/config.h
> @@ -79,6 +79,21 @@
> #define CONFIG_SYS_PCIE2_PHYS_ADDR (CONFIG_SYS_PCIE2_PHYS_BASE + \
> CONFIG_SYS_PCIE2_VIRT_ADDR)
>
> +/* SATA */
> +#define AHCI_BASE_ADDR (CONFIG_SYS_IMMR + 0x02200000)
> +#define CONFIG_BOARD_LATE_INIT
> +#define CONFIG_CMD_SCSI
> +#define CONFIG_LIBATA
> +#define CONFIG_SCSI_AHCI
> +#define CONFIG_SCSI_AHCI_PLAT
> +#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)
> +#define CONFIG_CMD_FAT
> +#define CONFIG_DOS_PARTITION
> +#define CONFIG_SYS_FSL_ERRATUM_A008407
> +
> #ifdef CONFIG_DDR_SPD
> #define CONFIG_SYS_FSL_DDR_BE
> #define CONFIG_VERY_BIG_RAM
> diff --git a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> index d34044a..211fe1d 100644
> --- a/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> +++ b/arch/arm/include/asm/arch-ls102xa/immap_ls102xa.h
> @@ -397,4 +397,28 @@ struct ccsr_cci400 {
> u8 res_e004[0x10000 - 0xe004];
> };
>
> +/* AHCI (sata) register map */
> +struct ccsr_ahci {
> + u32 res1[0xa4/4]; /* 0x0 - 0xa4 */
> + u32 pcfg; /* port config */
> + u32 ppcfg; /* port phy1 config */
> + u32 pp2c; /* port phy2 config */
> + u32 pp3c; /* port phy3 config */
> + u32 pp4c; /* port phy4 config */
> + u32 pp5c; /* port phy5 config */
> + u32 paxic; /* port AXI config */
> + u32 axicc; /* AXI cache control */
> + u32 axipc; /* AXI PROT control */
> + u32 ptc; /* port Trans Config */
> + u32 pts; /* port Trans Status */
> + u32 plc; /* port link config */
> + u32 plc1; /* port link config1 */
> + u32 plc2; /* port link config2 */
> + u32 pls; /* port link status */
> + u32 pls1; /* port link status1 */
> + u32 pcmdc; /* port CMD config */
> + u32 ppcs; /* port phy control status */
> + u32 pberr; /* port 0/1 BIST error */
> + u32 cmds; /* port 0/1 CMD status error */
> +};
> #endif /* __ASM_ARCH_LS102XA_IMMAP_H_ */
> diff --git a/board/freescale/ls1021aqds/ls1021aqds.c b/board/freescale/ls1021aqds/ls1021aqds.c
> index d6ef6ba..1b26ed3 100644
> --- a/board/freescale/ls1021aqds/ls1021aqds.c
> +++ b/board/freescale/ls1021aqds/ls1021aqds.c
> @@ -18,6 +18,8 @@
> #include <fsl_ifc.h>
> #include <fsl_sec.h>
> #include <spl.h>
> +#include <ahci.h>
> +#include <scsi.h>
>
> #include "../common/sleep.h"
> #include "../common/qixis.h"
> @@ -54,6 +56,52 @@ enum {
> GE1_CLK125,
> };
>
> +static void ls1021a_sata_init(void)
> +{
> + struct ccsr_ahci __iomem *ccsr_ahci = (void *)AHCI_BASE_ADDR;
> +
> +#ifdef CONFIG_SYS_FSL_ERRATUM_A008407
> +#define SATA_ECC_REG_ADDR 0x20220520
> + unsigned int __iomem *ecc_reg = (void *)SATA_ECC_REG_ADDR;
> +#endif
> +
> + out_le32(&ccsr_ahci->ppcfg, 0xa003fffe);
> + out_le32(&ccsr_ahci->pp2c, 0x28183411);
> + out_le32(&ccsr_ahci->pp3c, 0x0e081004);
> + out_le32(&ccsr_ahci->pp4c, 0x00480811);
> + out_le32(&ccsr_ahci->pp5c, 0x192c96a4);
> + out_le32(&ccsr_ahci->ptc, 0x08000025);
What are these numbers?
> +
> +#ifdef CONFIG_SYS_FSL_ERRATUM_A008407
> + out_le32(ecc_reg, 0x00020000);
> +#endif
Same here.
> +}
> +
> +#ifdef CONFIG_SCSI_AHCI_PLAT
> +static int ls1021a_sata_start(void)
> +{
> + struct ccsr_gur *gur = (struct ccsr_gur *)CONFIG_SYS_FSL_GUTS_ADDR;
> + u32 cfg;
> + int rc = -1;
> +
> + cfg = in_be32(&gur->rcwsr[4]) & RCWSR4_SRDS1_PRTCL_MASK;
> + cfg >>= RCWSR4_SRDS1_PRTCL_SHIFT;
> +
> + if (cfg != 0x30 && cfg != 0x70) {
> + printf("SATA disabled: serdes protocol doesn't support\n");
> + return rc;
> + }
> +
> + rc = ahci_init((void __iomem *)AHCI_BASE_ADDR);
> + if (rc)
> + return rc;
> +
> + scsi_scan(0);
> +
> + return 0;
> +}
> +#endif
> +
> #ifdef CONFIG_LS102XA_NS_ACCESS
> static struct csu_ns_dev ns_dev[] = {
> { CSU_CSLX_PCIE2_IO, CSU_ALL_RW },
> @@ -327,6 +375,8 @@ int board_early_init_f(void)
> fsl_dp_disable_console();
> #endif
>
> + ls1021a_sata_init();
Is it OK to run this init regardless SerDes protocol?
> +
> return 0;
> }
>
> @@ -388,6 +438,17 @@ void board_init_f(ulong dummy)
> }
> #endif
>
> +#ifdef CONFIG_BOARD_LATE_INIT
> +int board_late_init(void)
> +{
> +#ifdef CONFIG_SCSI_AHCI_PLAT
> + ls1021a_sata_start();
> +#endif
> +
> + return 0;
> +}
> +#endif
> +
> void config_etseccm_source(int etsec_gtx_125_mux)
> {
> struct ccsr_scfg *scfg = (struct ccsr_scfg *)CONFIG_SYS_FSL_SCFG_ADDR;
> diff --git a/board/freescale/ls1021atwr/ls1021atwr.c b/board/freescale/ls1021atwr/ls1021atwr.c
> index b7458a9..6a964c3 100644
> --- a/board/freescale/ls1021atwr/ls1021atwr.c
> +++ b/board/freescale/ls1021atwr/ls1021atwr.c
> @@ -22,6 +22,8 @@
> #include <tsec.h>
> #include <fsl_sec.h>
> #include <spl.h>
> +#include <ahci.h>
> +#include <scsi.h>
> #include "../common/sleep.h"
> #ifdef CONFIG_U_QE
> #include "../../../drivers/qe/qe.h"
> @@ -173,6 +175,52 @@ struct cpld_data {
> u8 rev2; /* Reserved */
> };
>
> +static void ls1021a_sata_init(void)
> +{
> + struct ccsr_ahci __iomem *ccsr_ahci = (void *)AHCI_BASE_ADDR;
> +
> +#ifdef CONFIG_SYS_FSL_ERRATUM_A008407
Looks like ERRATUM A008407 is a SoC erratum. The workaround shouldn't be in
board file.
York
More information about the U-Boot
mailing list