[alsa-devel] [PATCH] ASoC: sgtl5000: defer the probe if clock is not found

Shawn Guo shawn.guo at linaro.org
Mon Jul 15 18:19:43 CEST 2013


On Mon, Jul 15, 2013 at 05:12:50PM +0100, Mark Brown wrote:
> On Mon, Jul 15, 2013 at 10:55:10PM +0800, Shawn Guo wrote:
> > It's not always the case that clock is already available when sgtl5000
> > get probed at the first time, e.g. the clock is provided by CPU DAI
> > which may be probed after sgtl5000.  So let's defer the probe when
> > devm_clk_get() call fails and give it chance to try later.
> 
> My previous comments about this being better handled in the clock core
> appear to have been ignored and...

I'm not really fond of fixing a regression for -rc with involving clock
core change.  I think we can do that when that becomes a really common
pattern.

> 
> > +		/* Defer the probe to see if the clk will be provided later */
> > +		return ret == -ENOENT ? -EPROBE_DEFER : ret;
> 
> ...please try to avoid the ternery operator, it's not helpful for
> legibility.

Okay, will change that to respect your taste.

Shawn



More information about the Alsa-devel mailing list