[PATCH master 1/7] ddr: imx8m: align function definition with prototype

Ahmad Fatoum a.fatoum at pengutronix.de
Sun Jun 4 23:29:33 PDT 2023


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 at pengutronix.de>
---
 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




More information about the barebox mailing list