Re: [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-17 Thread Naresh Kamboju
On 15 November 2017 at 18:28, Hans Verkuil wrote: > On 15/11/17 13:37, Arnd Bergmann wrote: >> An otherwise correct cleanup patch from Dan Carpenter turned a broken >> failure handling from a feature patch by Hans Verkuil into a kernel >> Oops, so bisection points to commit

Re: [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-16 Thread John Stultz
On Thu, Nov 16, 2017 at 2:20 PM, John Stultz wrote: > On Thu, Nov 16, 2017 at 1:50 PM, John Stultz wrote: >> On Wed, Nov 15, 2017 at 4:37 AM, Arnd Bergmann wrote: >>> An otherwise correct cleanup patch from Dan Carpenter turned a

Re: [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-16 Thread John Stultz
On Thu, Nov 16, 2017 at 1:50 PM, John Stultz wrote: > On Wed, Nov 15, 2017 at 4:37 AM, Arnd Bergmann wrote: >> An otherwise correct cleanup patch from Dan Carpenter turned a broken >> failure handling from a feature patch by Hans Verkuil into a kernel >>

Re: [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-16 Thread John Stultz
On Wed, Nov 15, 2017 at 4:37 AM, Arnd Bergmann wrote: > An otherwise correct cleanup patch from Dan Carpenter turned a broken > failure handling from a feature patch by Hans Verkuil into a kernel > Oops, so bisection points to commit 7af35b0addbc ("drm/kirin: Checking > for

Re: [PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-15 Thread Hans Verkuil
On 15/11/17 13:37, Arnd Bergmann wrote: > An otherwise correct cleanup patch from Dan Carpenter turned a broken > failure handling from a feature patch by Hans Verkuil into a kernel > Oops, so bisection points to commit 7af35b0addbc ("drm/kirin: Checking > for IS_ERR() instead of NULL") rather

[PATCH] [RFT] drm: adv7511/33: fix adv7511_cec_init() failure handling

2017-11-15 Thread Arnd Bergmann
An otherwise correct cleanup patch from Dan Carpenter turned a broken failure handling from a feature patch by Hans Verkuil into a kernel Oops, so bisection points to commit 7af35b0addbc ("drm/kirin: Checking for IS_ERR() instead of NULL") rather than 3b1b975003e4 ("drm: adv7511/33: add HDMI CEC