[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