From mboxrd@z Thu Jan 1 00:00:00 1970 From: Chee, Tien Fong Date: Thu, 3 Jan 2019 07:20:38 +0000 Subject: [U-Boot] [PATCH v2] misc: fs_loader: Add support for initializing MMC In-Reply-To: <1546133783-4369-1-git-send-email-tien.fong.chee@intel.com> References: <1546133783-4369-1-git-send-email-tien.fong.chee@intel.com> Message-ID: <1546500038.9838.0.camel@intel.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 8bit To: u-boot@lists.denx.de On Sun, 2018-12-30 at 09:36 +0800, tien.fong.chee at intel.com wrote: > From: Tien Fong Chee > > Firmware loader would encounter problem if the MMC is accessed before > initializing it. This patch would adding the support of probing block > device and initializing MMC before the MMC is accessed by firmware > loader. > > Signed-off-by: Tien Fong Chee > --- > > Changes in v2: > - Initializing MMC through probing the blk device Any comment for this patch? > --- >  drivers/misc/fs_loader.c | 33 +++++++++++++++++++++++++++++++++ >  1 file changed, 33 insertions(+) > > diff --git a/drivers/misc/fs_loader.c b/drivers/misc/fs_loader.c > index 57a14a3..5858073 100644 > --- a/drivers/misc/fs_loader.c > +++ b/drivers/misc/fs_loader.c > @@ -12,6 +12,7 @@ >  #include >  #include >  #include > +#include >  #include >   >  DECLARE_GLOBAL_DATA_PTR; > @@ -252,6 +253,38 @@ static int fs_loader_ofdata_to_platdata(struct > udevice *dev) >   >  static int fs_loader_probe(struct udevice *dev) >  { > +#if defined(CONFIG_DM_MMC) && defined(CONFIG_BLK) > + int ret; > + struct device_platdata *plat = dev->platdata; > + > + ret = mmc_initialize(NULL); > + if (ret) { > + debug("MMC: could not initialize mmc. error: %d\n", > ret); > + > + return ret; > + } > + > + if (plat->phandlepart.phandle) { > + ofnode node = ofnode_get_by_phandle(plat- > >phandlepart.phandle); > + > + struct udevice *mmc_dev = NULL; > + > + ret = device_get_global_by_ofnode(node, &mmc_dev); > + if (!ret) { > + struct mmc *mmc = mmc_get_mmc_dev(mmc_dev); > + struct udevice *dev; > + > + ret = blk_get_from_parent(mmc->dev, &dev); > + if (ret) { > + debug("MMC: No block device: %d\n", > + ret); > + > + return ret; > + } > + } > + } > +#endif > + >   return 0; >  }; >