[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-10-05 Thread Russell King - ARM Linux
On Sat, Sep 26, 2015 at 05:32:12PM -0300, Fabio Estevam wrote: > On Mon, Sep 21, 2015 at 11:15 AM, Russell King - ARM Linux > wrote: > > > My current patch stack for imx-drm related stuff looks like this at > > present: > > > > drm: bridge/dw_hdmi: place PHY into low power mode when disabled > >

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-10-05 Thread Fabio Estevam
On Mon, Oct 5, 2015 at 4:17 PM, Russell King - ARM Linux wrote: > On Sat, Sep 26, 2015 at 05:32:12PM -0300, Fabio Estevam wrote: >> On Mon, Sep 21, 2015 at 11:15 AM, Russell King - ARM Linux >> wrote: >> >> > My current patch stack for imx-drm related stuff looks like this at >> > present: >> >

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-29 Thread Russell King - ARM Linux
On Tue, Sep 29, 2015 at 01:07:25PM +0200, Thierry Reding wrote: > On Fri, Sep 25, 2015 at 10:29:51AM +0200, Philipp Zabel wrote: > > Am Montag, den 21.09.2015, 15:15 +0100 schrieb Russell King - ARM Linux: > > > On Mon, Sep 21, 2015 at 11:51:06AM +0200, Thierry Reding wrote: > > > > On Wed, Sep

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-29 Thread Thierry Reding
On Fri, Sep 25, 2015 at 10:29:51AM +0200, Philipp Zabel wrote: > Am Montag, den 21.09.2015, 15:15 +0100 schrieb Russell King - ARM Linux: > > On Mon, Sep 21, 2015 at 11:51:06AM +0200, Thierry Reding wrote: > > > On Wed, Sep 16, 2015 at 01:41:38PM -0700, Douglas Anderson wrote: > > > > There's a

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-26 Thread Fabio Estevam
On Mon, Sep 21, 2015 at 11:15 AM, Russell King - ARM Linux wrote: > My current patch stack for imx-drm related stuff looks like this at > present: > > drm: bridge/dw_hdmi: place PHY into low power mode when disabled > drm: bridge/dw_hdmi: start of support for pixel doubled modes > drm:

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-25 Thread Philipp Zabel
Am Montag, den 21.09.2015, 15:15 +0100 schrieb Russell King - ARM Linux: > On Mon, Sep 21, 2015 at 11:51:06AM +0200, Thierry Reding wrote: > > On Wed, Sep 16, 2015 at 01:41:38PM -0700, Douglas Anderson wrote: > > > There's a member in 'struct dw_hdmi' called cable_plugin. It's never > > > set to

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-21 Thread Russell King - ARM Linux
On Mon, Sep 21, 2015 at 11:51:06AM +0200, Thierry Reding wrote: > On Wed, Sep 16, 2015 at 01:41:38PM -0700, Douglas Anderson wrote: > > There's a member in 'struct dw_hdmi' called cable_plugin. It's never > > set to anything anywhere so thus is always false. There's a bit of code > > checking

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-21 Thread Thierry Reding
On Wed, Sep 16, 2015 at 01:41:38PM -0700, Douglas Anderson wrote: > There's a member in 'struct dw_hdmi' called cable_plugin. It's never > set to anything anywhere so thus is always false. There's a bit of code > checking it, but since it's always false this must be dead code. > Eliminate it. >

[PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable_plugin

2015-09-16 Thread Douglas Anderson
There's a member in 'struct dw_hdmi' called cable_plugin. It's never set to anything anywhere so thus is always false. There's a bit of code checking it, but since it's always false this must be dead code. Eliminate it. Note: if someone wants to figure out the intention of the original code and