Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-06 Thread Archit Taneja
On 08/05/2017 02:53 PM, Hans Verkuil wrote: Archit, I posted a v3 that added a missing cec_notifier_put to the remove() function. Apparently you missed that. The chunk in question from the v3 patch is this: I did. Sorry, about that. There was an issue with my mailer. @@ -2469,6 +2489,9 @@

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-05 Thread Hans Verkuil
Archit, I posted a v3 that added a missing cec_notifier_put to the remove() function. Apparently you missed that. The chunk in question from the v3 patch is this: @@ -2469,6 +2489,9 @@ static void __dw_hdmi_remove(struct dw_hdmi *hdmi) /* Disable all interrupts */ hdmi_writeb(hdmi

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-04 Thread Archit Taneja
On 07/31/2017 08:55 PM, Hans Verkuil wrote: On 07/31/2017 04:29 PM, Russell King wrote: Add CEC notifier support to the HDMI bridge driver, so that the CEC part of the IP can receive its physical address. Signed-off-by: Russell King Acked-by: Hans Verkuil Queued to drm-misc-next. Than

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-02 Thread Russell King - ARM Linux
On Wed, Aug 02, 2017 at 04:17:15PM +0200, Hans Verkuil wrote: > Yes, you should. Well spotted, I missed that one. Great, another respin. I give up. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up accordin

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-02 Thread Hans Verkuil
On 08/02/17 16:11, Laurent Pinchart wrote: > Hi Russell, > > Thank you for the patch. > > On Monday 31 Jul 2017 15:29:41 Russell King wrote: >> Add CEC notifier support to the HDMI bridge driver, so that the CEC >> part of the IP can receive its physical address. >> >> Signed-off-by: Russell King

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-08-02 Thread Laurent Pinchart
Hi Russell, Thank you for the patch. On Monday 31 Jul 2017 15:29:41 Russell King wrote: > Add CEC notifier support to the HDMI bridge driver, so that the CEC > part of the IP can receive its physical address. > > Signed-off-by: Russell King > --- > drivers/gpu/drm/bridge/synopsys/Kconfig |

[PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-07-31 Thread Russell King
Add CEC notifier support to the HDMI bridge driver, so that the CEC part of the IP can receive its physical address. Signed-off-by: Russell King --- drivers/gpu/drm/bridge/synopsys/Kconfig | 1 + drivers/gpu/drm/bridge/synopsys/dw-hdmi.c | 22 +- 2 files changed, 22 insert

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-07-31 Thread Hans Verkuil
On 07/31/2017 04:29 PM, Russell King wrote: > Add CEC notifier support to the HDMI bridge driver, so that the CEC > part of the IP can receive its physical address. > > Signed-off-by: Russell King Acked-by: Hans Verkuil Regards, Hans > --- > drivers/gpu/drm/bridge/synopsys/Kconfig

Re: [PATCH v2 1/4] drm/bridge: dw-hdmi: add cec notifier support

2017-07-31 Thread Neil Armstrong
On 07/31/2017 04:29 PM, Russell King wrote: > Add CEC notifier support to the HDMI bridge driver, so that the CEC > part of the IP can receive its physical address. > > Signed-off-by: Russell King > --- > drivers/gpu/drm/bridge/synopsys/Kconfig | 1 + > drivers/gpu/drm/bridge/synopsys/dw-hdmi