> On 11 Jun 2018, at 10:08, Carlo Caione <ca...@caione.org> wrote: > > From: Carlo Caione <ca...@endlessm.com> > > The GRF_SOC_CON0.grf_force_jtag bit is automatically set at boot and it > is preventing the SDMMC to work correctly. Disable the JTAG function on > the assumption that a working SD has higher priority over JTAG. > > Signed-off-by: Carlo Caione <ca...@endlessm.com> > Reviewed-by: Simon Glass <s...@chromium.org>
Reviewed-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> Acked-by: Philipp Tomsich <philipp.toms...@theobroma-systems.com> See below for a nitpick ... > --- > arch/arm/mach-rockchip/rk3288-board.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/arch/arm/mach-rockchip/rk3288-board.c > b/arch/arm/mach-rockchip/rk3288-board.c > index 0365793009..bf24d8e074 100644 > --- a/arch/arm/mach-rockchip/rk3288-board.c > +++ b/arch/arm/mach-rockchip/rk3288-board.c > @@ -307,6 +307,7 @@ U_BOOT_CMD( > "" > ); > > +#define GRF_SOC_CON0 0xff770244 > #define GRF_SOC_CON2 0xff77024c Could you convert these to ‘const uintptr_t GRF_SOC_CON0 = …’ ? The compiler will generate the same code for a const as if it’s a define, but we’ll have full type-safety. > > int board_early_init_f(void) > @@ -339,5 +340,11 @@ int board_early_init_f(void) > } > rk_setreg(GRF_SOC_CON2, 1 << 0); > > + /* > + * Disable JTAG on sdmmc0 IO. The SDMMC won't work until this bit is > + * cleared > + */ > + rk_clrreg(GRF_SOC_CON0, 1 << 12); > + > return 0; > } > -- > 2.17.1 > _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot