Thread (1 message) flat view 1 message, 1 author, 2017-12-15

[RFC PATCH 1/2] ASoC: add support of mclk clock providers in wm8894 driver

From: ckeepax@opensource.cirrus.com (Charles Keepax)
Date: 2017-12-15 11:53:49
Also in: alsa-devel, lkml

On Thu, Dec 14, 2017 at 05:30:25PM +0000, Mark Brown wrote:
On Thu, Dec 14, 2017 at 05:53:57PM +0100, Olivier Moysan wrote:
quoted
+	pdata->mclk1 = devm_clk_get(wm8994->dev, "MCLK1");
+	if (IS_ERR(pdata->mclk1))
+		pdata->mclk1 = NULL;
These should special case -EPROBE_DEFER so we defer properly if we need
to (and ideally log an error in case there was a MCLK and we legit ran
into an error).
We probably want the special case on there being no clock at all
which should silently proceed as this code does and then actual
errors and PROBE_DEFERs can log and fail.

Thanks,
Charles
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help