On Fri, Dec 13, 2013 at 12:26 PM, Felipe Balbi <balbi@xxxxxx> wrote: > Hi, > > On Fri, Dec 13, 2013 at 09:23:36AM +0800, Peter Chen wrote: >> @@ -226,6 +235,16 @@ static int mxs_phy_probe(struct platform_device *pdev) >> >> platform_set_drvdata(pdev, mxs_phy); >> >> + if (mxs_phy->data->flags & MXS_PHY_HAS_ANATOP) { > > instead of the flag, you can check if that binding exist and use the > binding itself as a flag. > If flag MXS_PHY_HAS_ANATOP is existed, that binding must need to be existed, or it is an error. syscon_regmap_lookup_by_phandle will return error is the binding is not existed. -- BR, Peter Chen -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html