linux-kernel - Re: [PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused cable

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <20150929110724.GA15035@ulmo.nvidia.com>
Date:	Tue, 29 Sep 2015 13:07:25 +0200
From:	Thierry Reding <treding@...dia.com>
To:	Philipp Zabel <p.zabel@...gutronix.de>
CC:	Russell King - ARM Linux <linux@....linux.org.uk>,
	Douglas Anderson <dianders@...omium.org>, <airlied@...ux.ie>,
	<fabio.estevam@...escale.com>, <ykk@...k-chips.com>,
	<andy.yan@...k-chips.com>, <dri-devel@...ts.freedesktop.org>,
	<linux-kernel@...r.kernel.org>, <kernel@...gutronix.de>
Subject: Re: [PATCH] CHROMIUM: drm: bridge/dw_hdmi: Eliminate unused
 cable_plugin

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 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 implement whatever feature / fix was needed then we can drop this
> > > > patch.  The 'cable_plugin' member has been unused since the code was
> > > > first added in (9aaf880 imx-drm: Add mx6 hdmi transmitter support).
> > > > 
> > > > Signed-off-by: Douglas Anderson <dianders@...omium.org>
> > > > ---
> > > >  drivers/gpu/drm/bridge/dw_hdmi.c | 9 ---------
> > > >  1 file changed, 9 deletions(-)
> > > 
> > > Except for the CHROMIUM: prefix this looks good to me:
> > > 
> > > Reviewed-by: Thierry Reding <treding@...dia.com>
> 
> This seems to be similar to Sascha's "drm: bridge/dw_hdmi: remove unused
> code" patch, except that the hdmi_disable_overflow_interrupts function
> could be removed too.
> 
> > > Russell, do you have patches to this driver queued for v4.4 and plan to
> > > pick this up into your tree or should I take it?
> > 
> > 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: bridge/dw_hdmi: remove CEC engine register definitions
> > drm: bridge/dw_hdmi-cec: add Designware HDMI CEC driver
> > cec: add HDMI CEC input driver
> > cec: add HDMI CEC core driver
> > drm: bridge/dw_hdmi: replace CTS calculation for the ACR
> > drm: bridge/dw_hdmi: remove ratio support from ACR code
> > drm: bridge/dw_hdmi: adjust pixel clock values in N calculation
> > drm: bridge/dw_hdmi: avoid being recursive in N calculation
> > drm: bridge/dw_hdmi-ahb-audio: allow larger buffer sizes
> > drm: bridge/dw_hdmi-ahb-audio: basic support for multi-channel PCM audio
> > drm: bridge/dw_hdmi-ahb-audio: parse ELD from HDMI driver
> > drm: bridge/dw_hdmi-ahb-audio: add audio driver
> > drm: bridge/dw_hdmi: improve HDMI enable/disable handling
> > drm: bridge/dw_hdmi: add connector mode forcing
> > drm: bridge/dw_hdmi: add support for interlaced video modes
> > gpu: imx: fix support for interlaced modes
> > gpu: imx: simplify sync polarity setting
> > 
> > I haven't yet decided what, if anything, from that stack I'm going to
> > try to get into the next merge window.  Given the lack of interest last
> > time I posted these patches, I'm loosing interest myself in trying to
> > get them merged, especially ones which are getting on for being 2 years
> > old.
> 
> I'm still very interested to see at least the "gpu: imx: fix support for
> interlaced modes" and "gpu: imx: simplify sync polarity setting" merged.
> May I take them into the imx-drm tree separately?

The "gpu: imx:" patches sound like they are standalone, so taking them
through the imx-drm tree would be the easiest.

Thierry

Download attachment "signature.asc" of type "application/pgp-signature" (820 bytes)