From: Tomi Valkeinen <tomi.valkeinen@ti.com> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org, linux-media@vger.kernel.org, Bryan Wu <bryan.wu@canonical.com>, Richard Purdie <rpurdie@rpsys.net>, Marcus Lorentzon <marcus.lorentzon@linaro.org>, Sumit Semwal <sumit.semwal@ti.com>, Archit Taneja <archit@ti.com>, Sebastien Guiriec <s-guiriec@ti.com>, Inki Dae <inki.dae@samsung.com>, Kyungmin Park <kyungmin.park@samsung.com> Subject: Re: [RFC 0/5] Generic panel framework Date: Mon, 20 Aug 2012 14:39:30 +0300 [thread overview] Message-ID: <1345462770.2684.23.camel@deskari> (raw) In-Reply-To: <4948190.AFNtaaFKXQ@avalon> [-- Attachment #1: Type: text/plain, Size: 2284 bytes --] On Sat, 2012-08-18 at 03:16 +0200, Laurent Pinchart wrote: > Hi Tomi, > mipi-dbi-bus might not belong to include/video/panel/ though, as it can be > used for non-panel devices (at least in theory). The future mipi-dsi-bus > certainly will. They are both display busses. So while they could be used for anything, I find it quite unlikely as there are much better alternatives for generic bus needs. > Would you be able to send incremental patches on top of v2 to implement the > solution you have in mind ? It would be neat if you could also implement mipi- > dsi-bus for the OMAP DSS and test the code with a real device :-) Yes, I'd like to try this out on OMAP, both DBI and DSI. However, I fear it'll be quite complex due to the dependencies all around we have in the current driver. We're working on simplifying things so that it'll be easier to try thing like the panel framework, though, so we're going in the right direction. > > Generally about locks, if we define that panel ops may only be called > > exclusively, does it simplify things? I think we can make such > > requirements, as there should be only one display framework that handles > > the panel. Then we don't need locking for things like enable/disable. > > Pushing locking to callers would indeed simplify panel drivers, but we need to > make sure we won't need to expose a panel to several callers in the future. I have a feeling that would be a bad idea. Display related stuff are quite sensitive to any delays, so any extra transactions over, say, DSI bus could cause a noticeable glitch on the screen. I'm not sure what are all the possible ops that a panel can offer, but I think all that affect the display or could cause delays should be handled by one controlling entity (drm or such). The controlling entity needs to handle locking anyway, so in that sense I don't think it's an extra burden for it. The things that come to my mind that could possibly cause calls to the panel outside drm: debugfs, sysfs, audio, backlight. Of those, I think backlight should go through drm. Audio, no idea. debugfs and sysfs locking needs to be handled by the panel driver, and they are a bit problematic as I guess having them requires full locking. Tomi [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 836 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: Tomi Valkeinen <tomi.valkeinen@ti.com> To: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org, linux-leds@vger.kernel.org, linux-media@vger.kernel.org, Bryan Wu <bryan.wu@canonical.com>, Richard Purdie <rpurdie@rpsys.net>, Marcus Lorentzon <marcus.lorentzon@linaro.org>, Sumit Semwal <sumit.semwal@ti.com>, Archit Taneja <archit@ti.com>, Sebastien Guiriec <s-guiriec@ti.com>, Inki Dae <inki.dae@samsung.com>, Kyungmin Park <kyungmin.park@samsung.com> Subject: Re: [RFC 0/5] Generic panel framework Date: Mon, 20 Aug 2012 11:39:30 +0000 [thread overview] Message-ID: <1345462770.2684.23.camel@deskari> (raw) In-Reply-To: <4948190.AFNtaaFKXQ@avalon> [-- Attachment #1: Type: text/plain, Size: 2284 bytes --] On Sat, 2012-08-18 at 03:16 +0200, Laurent Pinchart wrote: > Hi Tomi, > mipi-dbi-bus might not belong to include/video/panel/ though, as it can be > used for non-panel devices (at least in theory). The future mipi-dsi-bus > certainly will. They are both display busses. So while they could be used for anything, I find it quite unlikely as there are much better alternatives for generic bus needs. > Would you be able to send incremental patches on top of v2 to implement the > solution you have in mind ? It would be neat if you could also implement mipi- > dsi-bus for the OMAP DSS and test the code with a real device :-) Yes, I'd like to try this out on OMAP, both DBI and DSI. However, I fear it'll be quite complex due to the dependencies all around we have in the current driver. We're working on simplifying things so that it'll be easier to try thing like the panel framework, though, so we're going in the right direction. > > Generally about locks, if we define that panel ops may only be called > > exclusively, does it simplify things? I think we can make such > > requirements, as there should be only one display framework that handles > > the panel. Then we don't need locking for things like enable/disable. > > Pushing locking to callers would indeed simplify panel drivers, but we need to > make sure we won't need to expose a panel to several callers in the future. I have a feeling that would be a bad idea. Display related stuff are quite sensitive to any delays, so any extra transactions over, say, DSI bus could cause a noticeable glitch on the screen. I'm not sure what are all the possible ops that a panel can offer, but I think all that affect the display or could cause delays should be handled by one controlling entity (drm or such). The controlling entity needs to handle locking anyway, so in that sense I don't think it's an extra burden for it. The things that come to my mind that could possibly cause calls to the panel outside drm: debugfs, sysfs, audio, backlight. Of those, I think backlight should go through drm. Audio, no idea. debugfs and sysfs locking needs to be handled by the panel driver, and they are a bit problematic as I guess having them requires full locking. Tomi [-- Attachment #2: This is a digitally signed message part --] [-- Type: application/pgp-signature, Size: 836 bytes --]
next prev parent reply other threads:[~2012-08-20 11:39 UTC|newest] Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top 2012-08-17 0:49 [RFC 0/5] Generic panel framework Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-08-17 0:49 ` [RFC 1/5] video: Add generic display panel core Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-09-04 9:24 ` Sascha Hauer 2012-09-04 9:24 ` Sascha Hauer 2012-09-13 1:40 ` Laurent Pinchart 2012-09-13 1:40 ` Laurent Pinchart 2012-09-13 11:29 ` Sascha Hauer 2012-09-13 11:29 ` Sascha Hauer 2012-09-13 19:32 ` Robert Schwebel 2012-09-13 19:32 ` Robert Schwebel 2012-08-17 0:49 ` [RFC 2/5] video: panel: Add dummy panel support Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-08-17 0:49 ` [RFC 3/5] video: panel: Add MIPI DBI bus support Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-08-17 9:03 ` Tomi Valkeinen 2012-08-17 9:03 ` Tomi Valkeinen 2012-08-17 10:02 ` Laurent Pinchart 2012-08-17 10:02 ` Laurent Pinchart 2012-08-17 10:51 ` Tomi Valkeinen 2012-08-17 10:51 ` Tomi Valkeinen 2012-08-17 12:33 ` Laurent Pinchart 2012-08-17 12:33 ` Laurent Pinchart 2012-08-17 13:06 ` Tomi Valkeinen 2012-08-17 13:06 ` Tomi Valkeinen 2012-08-17 14:06 ` Laurent Pinchart 2012-08-17 14:06 ` Laurent Pinchart 2012-08-17 0:49 ` [RFC 4/5] video: panel: Add R61505 panel support Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-08-17 0:49 ` [RFC 5/5] video: panel: Add R61517 " Laurent Pinchart 2012-08-17 0:49 ` Laurent Pinchart 2012-08-17 1:33 ` [RFC 0/5] Generic panel framework Jingoo Han 2012-08-17 1:33 ` Jingoo Han 2012-08-17 8:38 ` Tomi Valkeinen 2012-08-17 8:38 ` Tomi Valkeinen 2012-08-17 11:10 ` Laurent Pinchart 2012-08-17 11:10 ` Laurent Pinchart 2012-08-17 11:42 ` Tomi Valkeinen 2012-08-17 11:42 ` Tomi Valkeinen 2012-08-18 1:16 ` Laurent Pinchart 2012-08-18 1:16 ` Laurent Pinchart 2012-08-18 1:16 ` Laurent Pinchart 2012-08-20 11:39 ` Tomi Valkeinen [this message] 2012-08-20 11:39 ` Tomi Valkeinen 2012-08-20 23:29 ` Laurent Pinchart 2012-08-20 23:29 ` Laurent Pinchart 2012-08-20 23:29 ` Laurent Pinchart 2012-08-21 5:49 ` Tomi Valkeinen 2012-08-21 5:49 ` Tomi Valkeinen 2012-08-21 9:23 ` Laurent Pinchart 2012-08-21 9:23 ` Laurent Pinchart 2012-08-23 6:23 ` Jun Nie 2012-08-23 6:23 ` Jun Nie 2012-09-04 8:20 ` Zhou Zhu 2012-09-04 8:20 ` Zhou Zhu 2012-10-30 16:35 ` Laurent Pinchart 2012-10-30 16:35 ` Laurent Pinchart 2012-10-30 16:23 ` Laurent Pinchart 2012-10-30 16:23 ` Laurent Pinchart 2012-10-20 14:18 ` Inki Dae 2012-10-20 14:18 ` Inki Dae 2012-09-19 11:45 ` Tomi Valkeinen 2012-09-19 11:45 ` Tomi Valkeinen 2012-10-31 13:13 ` Laurent Pinchart 2012-10-31 13:13 ` Laurent Pinchart 2012-10-31 14:20 ` Tomi Valkeinen 2012-10-31 14:20 ` Tomi Valkeinen 2012-10-31 14:20 ` Tomi Valkeinen 2012-10-20 13:10 ` Inki Dae 2012-10-20 13:10 ` Inki Dae 2012-10-20 14:22 ` Inki Dae 2012-10-20 14:22 ` Inki Dae 2012-10-31 13:28 ` Laurent Pinchart 2012-10-31 13:28 ` Laurent Pinchart
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1345462770.2684.23.camel@deskari \ --to=tomi.valkeinen@ti.com \ --cc=archit@ti.com \ --cc=bryan.wu@canonical.com \ --cc=dri-devel@lists.freedesktop.org \ --cc=inki.dae@samsung.com \ --cc=kyungmin.park@samsung.com \ --cc=laurent.pinchart@ideasonboard.com \ --cc=linux-fbdev@vger.kernel.org \ --cc=linux-leds@vger.kernel.org \ --cc=linux-media@vger.kernel.org \ --cc=marcus.lorentzon@linaro.org \ --cc=rpurdie@rpsys.net \ --cc=s-guiriec@ti.com \ --cc=sumit.semwal@ti.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.