[PATCH v3 1/3] lmb: Fix lmb property's defination under struct lmb
Michal Simek
michal.simek at amd.com
Fri Jul 15 09:44:12 CEST 2022
On 7/7/22 10:45, Michal Simek wrote:
> From: Ashok Reddy Soma <ashok.reddy.soma at xilinx.com>
>
> Under struct lmb {} the lmb property's should be defined only if
> CONFIG_LMB_MEMORY_REGIONS is defined.
>
> Signed-off-by: Ashok Reddy Soma <ashok.reddy.soma at xilinx.com>
> Signed-off-by: Michal Simek <michal.simek at xilinx.com>
>
> Signed-off-by: Michal Simek <michal.simek at amd.com>
> ---
>
> (no changes since v2)
>
> Changes in v2:
> - Also fix lib/lmb.c
>
> include/lmb.h | 2 +-
> lib/lmb.c | 2 +-
> 2 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/include/lmb.h b/include/lmb.h
> index ab277ca80004..1476d78c2823 100644
> --- a/include/lmb.h
> +++ b/include/lmb.h
> @@ -68,7 +68,7 @@ struct lmb_region {
> struct lmb {
> struct lmb_region memory;
> struct lmb_region reserved;
> -#if !IS_ENABLED(CONFIG_LMB_USE_MAX_REGIONS)
> +#if IS_ENABLED(CONFIG_LMB_MEMORY_REGIONS)
> struct lmb_property memory_regions[CONFIG_LMB_MEMORY_REGIONS];
> struct lmb_property reserved_regions[CONFIG_LMB_RESERVED_REGIONS];
> #endif
> diff --git a/lib/lmb.c b/lib/lmb.c
> index f72996a4248e..f21fe672ae96 100644
> --- a/lib/lmb.c
> +++ b/lib/lmb.c
> @@ -108,7 +108,7 @@ void lmb_init(struct lmb *lmb)
> #if IS_ENABLED(CONFIG_LMB_USE_MAX_REGIONS)
> lmb->memory.max = CONFIG_LMB_MAX_REGIONS;
> lmb->reserved.max = CONFIG_LMB_MAX_REGIONS;
> -#else
> +#elif IS_ENABLED(CONFIG_LMB_MEMORY_REGIONS)
> lmb->memory.max = CONFIG_LMB_MEMORY_REGIONS;
> lmb->reserved.max = CONFIG_LMB_RESERVED_REGIONS;
> lmb->memory.region = lmb->memory_regions;
applied all.
M
More information about the U-Boot
mailing list