# Re: \[PATCH 7/7\] OMAPDSS: HDMI: hot plug detect fix

- From: Tomi Valkeinen \<hidden\>
- Date: 2012-03-08 07:35:13 
- List: linux-fbdev
- Page: https://ratatoskr.run/linux-fbdev/2012/03/13462752
- Thread: https://ratatoskr.run/linux-fbdev/2012/03/13462814/t.md

```text
On Wed, 2012-03-07 at 12:01 -0800, Greg KH wrote:
> On Thu, Mar 01, 2012 at 02:26:35PM +0200, Tomi Valkeinen wrote:
> > From: Rob Clark <redacted>
> > 
> > The "OMAPDSS: HDMI: PHY burnout fix" commit switched the HDMI driver
> > over to using a GPIO for plug detect.  Unfortunately the ->detect()
> > method was not also updated, causing HDMI to no longer work for the
> > omapdrm driver (because it would actually check if a connection was
> > detected before attempting to enable display).
> > 
> > Signed-off-by: Rob Clark <redacted>
> > Signed-off-by: Tomi Valkeinen <redacted>
> 
> You forgot to tell me what the git commit id is for this patch (it's
> ca888a7958b3d808e4efd08ceff88913f4212c69, right?)

Yes, that's the one. It wasn't in Linus's tree yet, only in fbdev tree,
so I wasn't sure what the commit id is.

> And why isn't this needed for the 3.0 kernel as well?

The detect() function is not present in 3.0, so there was nothing to
break.

 Tomi

```

## Attachments

- [signature.asc](https://ratatoskr.run/linux-fbdev/2012/03/13462752/attachment/0) (application/pgp-signature, 836 bytes)