Re: [PATCH 1/2] [media] ov7670: Return the real error code

2017-08-08 Thread Sakari Ailus
On Tue, Aug 08, 2017 at 08:56:14AM -0300, Fabio Estevam wrote: > Hi Sakari, > > On Tue, Aug 8, 2017 at 8:24 AM, Sakari Ailus wrote: > > > I don't think -EPROBE_DEFER is returned by clk_get() if the clock can't be > > found. The clock providers often are e.g. ISP drivers that may well be > > Yes

Re: [PATCH 1/2] [media] ov7670: Return the real error code

2017-08-08 Thread Fabio Estevam
Hi Sakari, On Tue, Aug 8, 2017 at 8:24 AM, Sakari Ailus wrote: > I don't think -EPROBE_DEFER is returned by clk_get() if the clock can't be > found. The clock providers often are e.g. ISP drivers that may well be Yes, it is. Please check: commit a34cd4666f3da84228a82f70c94b8d9b692034ea Author

Re: [PATCH 1/2] [media] ov7670: Return the real error code

2017-08-08 Thread Sakari Ailus
On Wed, Jul 19, 2017 at 12:34:18AM -0300, Fabio Estevam wrote: > From: Fabio Estevam > > When devm_clk_get() fails the real error code should be propagated, > instead of always returning -EPROBE_DEFER. > > Signed-off-by: Fabio Estevam Hi Fabio, I don't think -EPROBE_DEFER is returned by clk_g

[PATCH 1/2] [media] ov7670: Return the real error code

2017-07-18 Thread Fabio Estevam
From: Fabio Estevam When devm_clk_get() fails the real error code should be propagated, instead of always returning -EPROBE_DEFER. Signed-off-by: Fabio Estevam --- drivers/media/i2c/ov7670.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/media/i2c/ov7670.c b/driver