Hi Andrew, On 2017-07-30 19:07:38 +0200, Andrew Lunn wrote: > Hi Niklas > > > @@ -2041,6 +2073,11 @@ static int ravb_probe(struct platform_device *pdev) > > > > priv->chip_id = chip_id; > > > > + /* Get clock, if not found that's OK but Wake-On-Lan is unavailable */ > > + priv->clk = devm_clk_get(&pdev->dev, NULL); > > + if (IS_ERR(priv->clk)) > > + priv->clk = NULL; > > Can you get EPROBE_DEFER returned? I don't think so, but I'm not sure :-) The clock I'm trying to get is the module clock of the ravb itself, so if that clock is not available (and enabled) no register writes to the ravb would be possible in the first place, so i guess it's safe to assume -EPROBE_DEFER can not happen here? I'm just trying to play it safe here since the clock is only needed to support WoL, I though it best to not change behavior here. Try to get the clock, if we can great we can do WoL if not then user-space will be prevented from enabling WoL and nothing in the current behavior changes. > > Andrew -- Regards, Niklas Söderlund
