The values of enum dram_type and enum ddrc_type are defined, so they don't overlap, which allows ddr_cfg_phy() to accept them OR-d. It's thus wrong to use only one enum of them in the prototype, so adjust it with the definition. Signed-off-by: Ahmad Fatoum <a.fatoum@xxxxxxxxxxxxxx> --- include/soc/imx8m/ddr.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/include/soc/imx8m/ddr.h b/include/soc/imx8m/ddr.h index 0b3c4d47e39d..e268892957db 100644 --- a/include/soc/imx8m/ddr.h +++ b/include/soc/imx8m/ddr.h @@ -432,7 +432,7 @@ static inline int imx8mp_ddr_init(struct dram_timing_info *dram_timing, return imx8m_ddr_init(dram_timing, DDRC_TYPE_MP | dram_type); } -int ddr_cfg_phy(struct dram_timing_info *timing_info, enum ddrc_type ddrc_type); +int ddr_cfg_phy(struct dram_timing_info *timing_info, unsigned type); void load_lpddr4_phy_pie(void); void ddrphy_trained_csr_save(struct dram_cfg_param *param, unsigned int num); void dram_config_save(struct dram_timing_info *info, unsigned long base); -- 2.39.2
