Re: [PATCH] drm/bridge: adv7511: Fix cec clock EPROBE_DEFER handling

2020-05-01 Thread Vincent Whitchurch
On Mon, Apr 06, 2020 at 02:58:17AM +0200, Laurent Pinchart wrote: > On Tue, Mar 31, 2020 at 04:16:29PM +0200, Vincent Whitchurch wrote: > > int adv7511_cec_init(struct device *dev, struct adv7511 *adv7511) > > { > > unsigned int offset = adv7511->type == ADV7533 ? > >

Re: [PATCH] drm/bridge: adv7511: Fix cec clock EPROBE_DEFER handling

2020-04-05 Thread Laurent Pinchart
Hi Vincent, Thank you for the patch. On Tue, Mar 31, 2020 at 04:16:29PM +0200, Vincent Whitchurch wrote: > If adv7511's devm_clk_get() for the cec clock returns -EPROBE_DEFER, we > end up in an infinite probe loop. This happens: > > (1) adv7511's probe is called. > > (2) adv7511's probe

[PATCH] drm/bridge: adv7511: Fix cec clock EPROBE_DEFER handling

2020-04-01 Thread Vincent Whitchurch
If adv7511's devm_clk_get() for the cec clock returns -EPROBE_DEFER, we end up in an infinite probe loop. This happens: (1) adv7511's probe is called. (2) adv7511's probe adds some secondary i2c devices which bind to the dummy driver and thus call driver_deferred_probe_trigger() and