Hi Arnd, > -----Original Message----- > From: Arnd Bergmann [mailto:arnd@xxxxxxxx] > Sent: Thursday, May 05, 2016 4:32 PM > To: linuxppc-dev@xxxxxxxxxxxxxxxx > Cc: Yangbo Lu; linux-mmc@xxxxxxxxxxxxxxx; devicetree@xxxxxxxxxxxxxxx; > linux-arm-kernel@xxxxxxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; > linux-clk@xxxxxxxxxxxxxxx; linux-i2c@xxxxxxxxxxxxxxx; iommu@lists.linux- > foundation.org; netdev@xxxxxxxxxxxxxxx; Mark Rutland; > ulf.hansson@xxxxxxxxxx; Russell King; Bhupesh Sharma; Joerg Roedel; > Santosh Shilimkar; Yang-Leo Li; Scott Wood; Rob Herring; Claudiu Manoil; > Kumar Gala; Xiaobo Xie; Qiang Zhao > Subject: Re: [v10, 7/7] mmc: sdhci-of-esdhc: fix host version for T4240- > R1.0-R2.0 > > On Thursday 05 May 2016 11:12:30 Yangbo Lu wrote: > > > > + fsl_guts_init(); > > + svr = fsl_guts_get_svr(); > > + if (svr) { > > + esdhc->soc_ver = SVR_SOC_VER(svr); > > + esdhc->soc_rev = SVR_REV(svr); > > + } else { > > + dev_err(&pdev->dev, "Failed to get SVR value!\n"); > > + } > > + > > > > > Sorry for jumping in again after not participating in the discussion for > the past few versions. > > What happened to my suggestion of making this a platform-independent > interface to avoid the link time dependency? > > Specifically, why not add an exported function to drivers/base/soc.c that > uses glob_match() for comparing a string in the device driver to the ID > of the SoC that is set by whatever SoC identifying driver the platform > has? > > Arnd [Lu Yangbo-B47093] I think this has been discussed in v6. You can find Scott's comments about this in below link. https://patchwork.kernel.org/patch/8544501/ Thanks. -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html