[U-Boot] [PATCH 09/10] ddr: altera: Staticize global variables

George Broz brozgeo at gmail.com
Sat Apr 9 00:15:50 CEST 2016


On 6 April 2016 at 19:04, Marek Vasut <marex at denx.de> wrote:
> Just staticize global variables in sequencer, since there is no
> point in having these symbols available outside of the DDR code.
>
> Signed-off-by: Marek Vasut <marex at denx.de>
> Cc: Dinh Nguyen <dinguyen at opensource.altera.com>
> Cc: Chin Liang See <clsee at altera.com>
> ---
>  drivers/ddr/altera/sequencer.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
>
> diff --git a/drivers/ddr/altera/sequencer.c b/drivers/ddr/altera/sequencer.c
> index c41555f..5ea53ad 100644
> --- a/drivers/ddr/altera/sequencer.c
> +++ b/drivers/ddr/altera/sequencer.c
> @@ -57,7 +57,7 @@ const struct socfpga_sdram_misc_config *misccfg;
>         STATIC_SKIP_DELAY_LOOPS)
>
>  /* calibration steps requested by the rtl */
> -u16 dyn_calib_steps;
> +static u16 dyn_calib_steps;
>
>  /*
>   * To make CALIB_SKIP_DELAY_LOOPS a dynamic conditional option
> @@ -68,13 +68,13 @@ u16 dyn_calib_steps;
>   * zero when skipping
>   */
>
> -u16 skip_delay_mask;   /* mask off bits when skipping/not-skipping */
> +static u16 skip_delay_mask;    /* mask off bits when skipping/not-skipping */
>
>  #define SKIP_DELAY_LOOP_VALUE_OR_ZERO(non_skip_value) \
>         ((non_skip_value) & skip_delay_mask)
>
> -struct gbl_type *gbl;
> -struct param_type *param;
> +static struct gbl_type *gbl;
> +static struct param_type *param;
>
>  static void set_failing_group_stage(u32 group, u32 stage,
>         u32 substage)
> --
> 2.7.0
>

Tested on: SoCKit, DE0_Nano_SoC
Tested-by: George Broz <brozgeo at gmail.com>


More information about the U-Boot mailing list