[PATCH u-boot-marvell 02/39] ddr: marvell: axp: fix array types have different bounds warning
Bin Meng
bmeng.cn at gmail.com
Mon Mar 8 07:50:32 CET 2021
On Sun, Mar 7, 2021 at 12:26 PM Marek Behún <marek.behun at nic.cz> wrote:
>
> The arrays `pbs_dq_mapping`, `div_ratio1to1` and `div_ratio2to1` have
> different bounds declared in header files where these variables are also
> defined from the ones declared in source files.
>
> This causes the compiler to complain (when building with LTO):
> ddr3_sdram.c:24:12: warning: type of ‘pbs_dq_mapping’ does not match
> original declaration
> [-Wlto-type-mismatch]
> ddr3_patterns_64bit.h:911:5: note: array types have different bounds
> ddr3_patterns_64bit.h:911:5: note: ‘pbs_dq_mapping’ was previously
> declared here
>
> ddr3_dfs.c:45:11: warning: type of ‘div_ratio1to1’ does not match
> original declaration [-Wlto-type-mismatch]
> ddr3_axp_vars.h:167:4: note: array types have different bounds
> ddr3_axp_vars.h:167:4: note: ‘div_ratio1to1’ was previously declared
> here
>
> ddr3_dfs.c:46:11: warning: type of ‘div_ratio2to1’ does not match
> original declaration [-Wlto-type-mismatch]
> ddr3_axp_vars.h:196:4: note: array types have different bounds
> ddr3_axp_vars.h:196:4: note: ‘div_ratio2to1’ was previously declared
> here
>
> CI managed to trigger this as an error when compiling with LTO for AXP.
>
> Fix this by using values from the header files, which seem to be the
> correct ones.
>
> Signed-off-by: Marek Behún <marek.behun at nic.cz>
> ---
> drivers/ddr/marvell/axp/ddr3_dfs.c | 4 ++--
> drivers/ddr/marvell/axp/ddr3_sdram.c | 2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
>
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
More information about the U-Boot
mailing list