On Thursday, September 20, 2012, Thomas Abraham <thomas.abraham@xxxxxxxxxx> wrote: > On 19 September 2012 19:12, Seungwon Jeon <tgih.jun@xxxxxxxxxxx> wrote: > > On Tuesday, September 18, 2012, Thomas Abraham <thomas.abraham@xxxxxxxxxx> wrote: > >> +int dw_mci_exynos_probe(struct platform_device *pdev) > >> +{ > >> + struct dw_mci_drv_data *drv_data; > >> + const struct of_device_id *match; > >> + > >> + match = of_match_node(dw_mci_exynos_match, pdev->dev.of_node); > >> + drv_data = match->data; > > In non-dt case, match has NULL. > > Needed to check "pdev->dev.of_node" > > Since all current users in mainline of this driver on Exynos platforms > are device tree based, and since Exynos4 and Exynos5 have device tree > support, I have mainly been adding device tree support here. Only for > non-exynos portions of the code, I have ensured that both dt and > legacy modes are supported. Yes. I know, then dw_mmc-exynos.c should have a dependency of CONFIG_OF? It needs to prevent kernel panic at least. Thanks, Seungwon Jeon Thanks, Seungwon Jeon > > Thanks, > Thomas. > > [...] > -- > 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 -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html