On Thu, 22 Jul 2021 14:30:05 +0800 Icenowy Zheng <icen...@sipeed.com> wrote:
Hi Icenowy, > Allwinner R329 has no MMC2. > > Only include the code of MMC2 if the base address of it is defined. > > Signed-off-by: Icenowy Zheng <icen...@sipeed.com> One day (TM) we will hopefully get somehow rid of those #ifdefs somehow, but until then it looks OK. Reviewed-by: Andre Przywara <andre.przyw...@arm.com> Thanks! Andre > --- > drivers/mmc/sunxi_mmc.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/mmc/sunxi_mmc.c b/drivers/mmc/sunxi_mmc.c > index 178b8cf106..6b809c001f 100644 > --- a/drivers/mmc/sunxi_mmc.c > +++ b/drivers/mmc/sunxi_mmc.c > @@ -73,10 +73,12 @@ static int mmc_resource_init(int sdc_no) > priv->reg = (struct sunxi_mmc *)SUNXI_MMC1_BASE; > priv->mclkreg = &ccm->sd1_clk_cfg; > break; > +#ifdef SUNXI_MMC2_BASE > case 2: > priv->reg = (struct sunxi_mmc *)SUNXI_MMC2_BASE; > priv->mclkreg = &ccm->sd2_clk_cfg; > break; > +#endif > #ifdef SUNXI_MMC3_BASE > case 3: > priv->reg = (struct sunxi_mmc *)SUNXI_MMC3_BASE;