On Sun, Mar 7, 2021 at 12:26 PM Marek Behún <marek.be...@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.be...@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...@gmail.com>
