Hi, On Thu, Sep 15, 2011 at 11:27 AM, Tomi Valkeinen <tomi.valkeinen@xxxxxx> wrote: > On Thu, 2011-09-15 at 11:11 +0530, K, Mythri P wrote: >> Hi, >> >> On Wed, Sep 14, 2011 at 7:41 PM, Tomi Valkeinen <tomi.valkeinen@xxxxxx> wrote: > >> > Yes, you are right, detect() does not "know" if the monitor has changed >> > between polls, so both notification and polling are needed. I >> > implemented only polling as there's no HPD event mechanism yet in >> > omapdss, and also because this was simple and gives DRM basic ability to >> > detect a monitor. >> > >> If it is needed for DRM then it is fine, but with detect renamed to >> poll. By next week i should have a patch ready for HPD event >> mechanism. > > What is wrong with "detect"? It detects if there's a display connected. > It can be used in polling manner, trying it every n seconds, but it > should also be used even if you use HPD event. I think the normal > sequence would be something like: > > 1) register HPD event > 2) use detect() to see if a monitor is already connected > I guess polling ever few seconds to detect would be waste of CPU cycles when there is already a mechanism in the H/w to detect the connection. Current sequence : Enable display ( Irrespective of whether the cable is connected on not) Sequence with HPD: 1.Register for HPD connect. 2.Enable display 3.Notify DRM/Audio/Kernel component that wants to listen to this event. Thanks and regards, Mythri. -- To unsubscribe from this list: send the line "unsubscribe linux-omap" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
