> Subject: RE: [PATCH] mmc: Parse new binding for eMMC fixed driver type > > > > > From: Peng Fan <[email protected]> > > Sent: Tuesday, July 14, 2020 4:52 PM > > To: Tokumoto, Shunsuke/�员� 俊介 <[email protected]> > > Cc: [email protected]; Iida, Yasushi/��田 康志 > > <[email protected]> > > Subject: RE: [PATCH] mmc: Parse new binding for eMMC fixed driver type > > > > > Subject: [PATCH] mmc: Parse new binding for eMMC fixed driver type > > > > > > Parse the new binding and store it in the mmc config struct after > > > doing some sanity checks. The code is designed to support fixed mmc > > > driver type if we ever need that. > > > > > > Signed-off-by: Shunsuke Tokumoto <[email protected]> > > > Signed-off-by: Yasushi Iida <[email protected]> > > > --- > > > drivers/mmc/mmc-uclass.c | 30 > > ++++++++++++++++++++++++++++++ > > > drivers/mmc/mmc.c | 8 ++++++++ > > > include/mmc.h | 3 +++ > > > 3 files changed, 41 insertions(+) > > > > > > diff --git a/drivers/mmc/mmc-uclass.c b/drivers/mmc/mmc-uclass.c > > index > > > c5b7872900..e67aae451f 100644 > > > --- a/drivers/mmc/mmc-uclass.c > > > +++ b/drivers/mmc/mmc-uclass.c > > > @@ -162,6 +162,36 @@ int dm_mmc_deferred_probe(struct udevice > *dev) > > > if (ops->deferred_probe) > > > return ops->deferred_probe(dev); > > > > > > + /* Check eMMC driver type selection */ > > > + val = dev_read_u32_default(dev, "fixed-emmc-driver-type", 0); > > > > Has the bindings been accepted by Linux Kernel? > > > > The bindings "fixed-emmc-driver-type" has already included in the Linux > Kernel. > The others, "fixed-emmc-driver-type-hs200" and > "fixed-emmc-driver-type-hs400" are not included in the Linux Kernel.
Then please mark u-boot,xxx for U-Boot specific properties and add some doc in U-Boot. If the bindings are under review by Linux Kernel, please share me the link and see how it goes. Thanks, Peng. > > > Regards, > > Peng. > > Best regards > Shunsuke

