Re: [PATCHv2] drm: adv7511/33: Fix adv7511_cec_init() failure handling

2017-11-21 Thread Hans Verkuil
On 11/21/2017 07:48 AM, Laurent Pinchart wrote: > Hi Hans, > > Thank you for the patch. > > On Monday, 20 November 2017 22:57:34 EET Hans Verkuil wrote: >> If the device tree for a board did not specify a cec clock, then >> adv7511_cec_init would return an error, which would cause

Re: [PATCHv2] drm: adv7511/33: Fix adv7511_cec_init() failure handling

2017-11-20 Thread Laurent Pinchart
Hi Hans, Thank you for the patch. On Monday, 20 November 2017 22:57:34 EET Hans Verkuil wrote: > If the device tree for a board did not specify a cec clock, then > adv7511_cec_init would return an error, which would cause adv7511_probe() > to fail and thus there is no HDMI output. > > There is

Re: [PATCHv2] drm: adv7511/33: Fix adv7511_cec_init() failure handling

2017-11-20 Thread John Stultz
On Mon, Nov 20, 2017 at 12:57 PM, Hans Verkuil wrote: > If the device tree for a board did not specify a cec clock, then > adv7511_cec_init would return an error, which would cause adv7511_probe() > to fail and thus there is no HDMI output. > > There is no need to have

[PATCHv2] drm: adv7511/33: Fix adv7511_cec_init() failure handling

2017-11-20 Thread Hans Verkuil
If the device tree for a board did not specify a cec clock, then adv7511_cec_init would return an error, which would cause adv7511_probe() to fail and thus there is no HDMI output. There is no need to have adv7511_probe() fail if the CEC initialization fails, so just change adv7511_cec_init() to