[U-Boot] [PATCH 1/2] arm: socfpga: remove re-added ad-hoc reset code
Simon Goldschmidt
simon.k.r.goldschmidt at gmail.com
Mon May 13 19:14:20 UTC 2019
Unfortunately, there's 2/2 (which removes this unused function from s10,
too) missing here since patman won't let me send the mail due to "Ang"
(from Ang, Chee Hong <chee.hong.ang at intel.com>) not being a well-formed
email address...
I'll work on that and re-send...
Regards,
Simon
Am 13.05.2019 um 21:09 schrieb Simon Goldschmidt:
> commit c5de2b7eae68 ("arm: socfpga: implement proper peripheral reset")
> has removed the call to 'reset_deassert_peripherals_handoff()' from
> socfpga gen5 SPL since the reset driver now handles resets. However,
> commit c1d4b464c8b8 ("ARM: socfpga: Disable bridges in SPL unless booting from FPGA")
> has re-added this ad-hoc reset code, so that all peripherals were now
> again enabled instead of letting the drivers enable them by request.
>
> While at it, remove this function for gen5 as it should not be used.
>
> Fixes: commit c1d4b464c8b8 ("ARM: socfpga: Disable bridges in SPL unless booting from FPGA")
> Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
> ---
>
> arch/arm/mach-socfpga/include/mach/reset_manager_gen5.h | 1 -
> arch/arm/mach-socfpga/reset_manager_gen5.c | 8 --------
> arch/arm/mach-socfpga/spl_gen5.c | 3 +--
> 3 files changed, 1 insertion(+), 11 deletions(-)
>
> diff --git a/arch/arm/mach-socfpga/include/mach/reset_manager_gen5.h b/arch/arm/mach-socfpga/include/mach/reset_manager_gen5.h
> index 5e490d182e..f4dcb14623 100644
> --- a/arch/arm/mach-socfpga/include/mach/reset_manager_gen5.h
> +++ b/arch/arm/mach-socfpga/include/mach/reset_manager_gen5.h
> @@ -8,7 +8,6 @@
>
> #include <dt-bindings/reset/altr,rst-mgr.h>
>
> -void reset_deassert_peripherals_handoff(void);
> void socfpga_bridges_set_handoff_regs(bool h2f, bool lwh2f, bool f2h);
> void socfpga_bridges_reset(int enable);
>
> diff --git a/arch/arm/mach-socfpga/reset_manager_gen5.c b/arch/arm/mach-socfpga/reset_manager_gen5.c
> index 89a384b59c..9a32f5abfe 100644
> --- a/arch/arm/mach-socfpga/reset_manager_gen5.c
> +++ b/arch/arm/mach-socfpga/reset_manager_gen5.c
> @@ -61,14 +61,6 @@ void socfpga_per_reset_all(void)
> writel(0xffffffff, &reset_manager_base->per2_mod_reset);
> }
>
> -/*
> - * Release peripherals from reset based on handoff
> - */
> -void reset_deassert_peripherals_handoff(void)
> -{
> - writel(0, &reset_manager_base->per_mod_reset);
> -}
> -
> #define L3REGS_REMAP_LWHPS2FPGA_MASK 0x10
> #define L3REGS_REMAP_HPS2FPGA_MASK 0x08
> #define L3REGS_REMAP_OCRAM_MASK 0x01
> diff --git a/arch/arm/mach-socfpga/spl_gen5.c b/arch/arm/mach-socfpga/spl_gen5.c
> index bd2a9fe5ae..1a60cdc897 100644
> --- a/arch/arm/mach-socfpga/spl_gen5.c
> +++ b/arch/arm/mach-socfpga/spl_gen5.c
> @@ -175,8 +175,7 @@ void board_init_f(ulong dummy)
> sysmgr_pinmux_init();
> sysmgr_config_warmrstcfgio(0);
>
> - /* De-assert reset for peripherals and bridges based on handoff */
> - reset_deassert_peripherals_handoff();
> + /* Set bridges handoff value */
> socfpga_bridges_set_handoff_regs(true, true, true);
>
> debug("Unfreezing/Thaw all I/O banks\n");
>
More information about the U-Boot
mailing list