[U-Boot] [PATCH v4 01/15] imx6: Add imx6_src_get_boot_mode

Stefano Babic
2017-02-26T11:31:58Z

On 24/02/2017 11:15, Jagan Teki wrote:
> For i.MX6, the bootmode determine code is part of spl_boot_device,
> but there is might be a possibility for other part the code need to
> check the desired boot mode for adding new functionalities like
> modeboot env variable, or changing boot order etc.
>
> So introduced imx6_src_get_boot_mode which actually reading the
> boot mode register for desired modes.
>
> More cleanup will be add in future patches.
>
> Cc: Stefano Babic <sbabic@denx.de>
> Cc: Tim Harvey <tharvey@gateworks.com>
> Cc: Matteo Lisi <matteo.lisi@engicam.com>
> Cc: Michael Trimarchi <michael@amarulasolutions.com>
> Signed-off-by: Jagan Teki <jagan@openedev.com>
> ---
> arch/arm/imx-common/init.c | 12 ++++++++++++
> arch/arm/imx-common/spl.c | 4 ++--
> arch/arm/include/asm/imx-common/sys_proto.h | 14 ++++++++++++++
> 3 files changed, 28 insertions(+), 2 deletions(-)
>
> diff --git a/arch/arm/imx-common/init.c b/arch/arm/imx-common/init.c
> index e5dbd93..036ebb2 100644
> --- a/arch/arm/imx-common/init.c
> +++ b/arch/arm/imx-common/init.c
> @@ -115,3 +115,15 @@ void boot_mode_apply(unsigned cfg_val)
> writel(reg, &psrc->gpr10);
> }
> #endif
> +
> +#if defined(CONFIG_MX6)
> +u32 imx6_src_get_boot_mode(void)
> +{
> + struct src *psrc = (struct src *)SRC_BASE_ADDR;
> +
> + if (imx6_is_bmode_from_gpr9())
> + return readl(&psrc->gpr9);
> + else
> + return readl(&psrc->sbmr1);
> +}
> +#endif
> diff --git a/arch/arm/imx-common/spl.c b/arch/arm/imx-common/spl.c
> index 46325ec..6c20f28 100644
> --- a/arch/arm/imx-common/spl.c
> +++ b/arch/arm/imx-common/spl.c
> @@ -10,6 +10,7 @@
> #include <common.h>
> #include <asm/io.h>
> #include <asm/arch/imx-regs.h>
> +#include <asm/arch/sys_proto.h>
> #include <asm/spl.h>
> #include <spl.h>
> #include <asm/imx-common/hab.h>
> @@ -19,9 +20,8 @@
> u32 spl_boot_device(void)
> {
> struct src *psrc = (struct src *)SRC_BASE_ADDR;
> - unsigned int gpr10_boot = readl(&psrc->gpr10) & (1 << 28);
> - unsigned reg = gpr10_boot ? readl(&psrc->gpr9) : readl(&psrc->sbmr1);
> unsigned int bmode = readl(&psrc->sbmr2);
> + u32 reg = imx6_src_get_boot_mode();
>
> /*
> * Check for BMODE if serial downloader is enabled
> diff --git a/arch/arm/include/asm/imx-common/sys_proto.h b/arch/arm/include/asm/imx-common/sys_proto.h
> index 539d34b..99e3869 100644
> --- a/arch/arm/include/asm/imx-common/sys_proto.h
> +++ b/arch/arm/include/asm/imx-common/sys_proto.h
> @@ -8,6 +8,7 @@
> #ifndef _SYS_PROTO_H_
> #define _SYS_PROTO_H_
>
> +#include <asm/io.h>
> #include <asm/imx-common/regs-common.h>
> #include <common.h>
> #include "../arch-imx/cpu.h"
> @@ -38,6 +39,19 @@
> #define is_mx6ull() (is_cpu_type(MXC_CPU_MX6ULL))
> #define is_mx6sll() (is_cpu_type(MXC_CPU_MX6SLL))
>
> +#ifdef CONFIG_MX6
> +#define IMX6_SRC_GPR10_BMODE BIT(28)
> +
> +static inline u8 imx6_is_bmode_from_gpr9(void)
> +{
> + struct src *psrc = (struct src *)SRC_BASE_ADDR;
> +
> + return readl(&psrc->gpr10) & IMX6_SRC_GPR10_BMODE;
> +}
> +
> +u32 imx6_src_get_boot_mode(void);
> +#endif /* CONFIG_MX6 */
> +
> u32 get_nr_cpus(void);
> u32 get_cpu_rev(void);
> u32 get_cpu_speed_grade_hz(void);
>

Thanks !

Reviewed-by: Stefano Babic <sbabic@denx.de>

Best regards,
Stefano Babic

--
Meet DENX at the Embedded World Trade Show
14 Mar - 16 Mar 2017, Nuremberg Trade Fair Centre, Hall 4, Booth 581
--
=====================================================================
DENX Software Engineering GmbH, Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-53 Fax: +49-8142-66989-80 Email: sbabic at denx.de
=====================================================================