On 12 September 2013 14:26, Viresh Kumar <[email protected]> wrote: > On 12 September 2013 13:52, Sachin Kamat <[email protected]> wrote: >> sdhci_probe only checks for IS_ERR. We would need to change it to >> IS_ERR_OR_NULL (which I do not prefer personally as there is some >> discussion for its removal). In that case we would need to return as I >> mentioned in my earlier email. Let me know your opinion. > > The problem with pdata normally is that it can't be NULL. Its not like clk/ > regulator framework where NULL's are valid... > > And that's why we are checking NULL later in the probe().. Probably you can > just move that up a bit.. and nothing else..
Yes you are right. I had not seen that check which is done later in probe. I will leave the probe as it is and modify the sdhci_probe_config_dt as per your suggestion. -- With warm regards, Sachin -- To unsubscribe from this list: send the line "unsubscribe linux-mmc" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
