Linux on ARM based TI OMAP SoCs
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Tony Lindgren <tony@atomide.com>
Cc: Igor Grinberg <grinberg@compulab.co.il>,
	linux-omap@vger.kernel.org, Archit Taneja <archit@ti.com>,
	Mike Rapoport <mike@compulab.co.il>
Subject: Re: [PATCH 08/28] ARM: OMAP: CM-T35: Kconfig option for the display options
Date: Fri, 29 Mar 2013 06:27:45 +0200	[thread overview]
Message-ID: <51551841.9070704@ti.com> (raw)
In-Reply-To: <20130328212837.GY10155@atomide.com>

[-- Attachment #1: Type: text/plain, Size: 2838 bytes --]

On 2013-03-28 23:28, Tony Lindgren wrote:
> * Tomi Valkeinen <tomi.valkeinen@ti.com> [130328 10:16]:
>> On 2013-03-28 18:53, Tony Lindgren wrote:
>>>
>>> Yes that's not nice from usability point of view. Can we still switch
>>> between LCD and DVI during the runtime? I thought the issue was registering
>>> multiple LCD panels where only one can exist at a time?
>>
>> I guess I could've been more verbose in my descriptions. And I agree
>> this is not a nice change.
>>
>> No, we can't switch between the LCD and DVI. But that's not strictly DSS
>> issue. The selection between LCD and DVI (or any other displays on the
>> same bus) are board specific things, sometimes requiring board specific
>> gpios or even things like i2c commands to muxes.
>>
>> It works now because we have custom callbacks in the board files to do
>> those things, but it'll break with DT.
> 
> Well people are already used to changing between LCD and DVI.. So we
> need some way of doing it with DT also.
> 
> How do you plan to change between LCD and DVI with DT? By using some
> custom driver modules?

I don't know, but I guess we need board specific drivers.

Do you know if there are other similar cases for other busses? I mean, I
think this is a similar case than, say, having two i2c devices with the
same i2c-id on the same bus. Only one of the devices can be enabled, the
other must be disconnected. And how the connect/disconnect is made is
board specific.

> There's the capebus coming that can be used for that too, but in most
> cases all the hardware is permanently connected with LCD and DVI. So
> sounds like capebus should only be needed for the add on boards.

True.

Well, depends on how you look at it. From one point of view there's no
real difference whether the disabled device is physically on the board
or not, as it has to be disconnected. The details depend on the bus, but
for example for DSI I think the disabled device has to be totally
removed from the DSI bus with some mux or such.

But, from the other point of view, the devices are there, on the board,
and in some cases the HW has been designed to support switching the
displays during runtime.

So what I wish is that I could make the linux device for the display to
be removed when it's disabled. I believe that can be done, but I guess
it requires a board specific driver, with some custom user interface to
do that.

And it would be different user interface than currently, which is again
not so nice. Currently the devices are always there, with their sysfs
files, and those sysfs files are used to enable/disable the devices. If
the device is removed, the sysfs files get removed also...

But I would really like to get rid of the current model, which I think
was a very bad design decision.

 Tomi



[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 899 bytes --]

  reply	other threads:[~2013-03-29  4:27 UTC|newest]

Thread overview: 50+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-03-28 12:48 [PATCH 00/28] OMAP: DSS related board file changes Tomi Valkeinen
2013-03-28 12:48 ` [PATCH 01/28] ARM: OMAP: Remove unnecessary platform callbacks for VENC devices Tomi Valkeinen
2013-03-28 15:18   ` Igor Grinberg
2013-03-28 12:48 ` [PATCH 02/28] ARM: OMAP: don't use reset_gpio from omap4_panda_dvi_device Tomi Valkeinen
2013-03-28 12:48 ` [PATCH 03/28] ARM: OMAP: Overo: Kconfig option for the display options Tomi Valkeinen
2013-03-28 12:48 ` [PATCH 04/28] ARM: OMAP: 3430SDP: " Tomi Valkeinen
2013-03-28 12:48 ` [PATCH 05/28] ARM: OMAP: 4430SDP: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 06/28] ARM: OMAP: DEVKIT8000: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 07/28] ARM: OMAP: OMAP3EVM: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 08/28] ARM: OMAP: CM-T35: " Tomi Valkeinen
2013-03-28 16:09   ` Igor Grinberg
2013-03-28 16:53     ` Tony Lindgren
2013-03-28 17:12       ` Tomi Valkeinen
2013-03-28 21:28         ` Tony Lindgren
2013-03-29  4:27           ` Tomi Valkeinen [this message]
2013-03-29 15:47             ` Tony Lindgren
2013-03-28 17:02     ` Tomi Valkeinen
2013-03-31  9:17       ` Igor Grinberg
2013-04-02  8:07         ` Tomi Valkeinen
2013-04-02 11:20           ` Igor Grinberg
2013-03-28 12:49 ` [PATCH 09/28] ARM: OMAP: AM3517EVM: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 10/28] ARM: OMAP: Panda: use new TFP410 platform driver Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 11/28] ARM: OMAP: Panda & SDP: use new HDMI driver Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 12/28] ARM: OMAP: 4430SDP: use new Taal driver Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 13/28] ARM: OMAP: Beagle: use new TFP410 platform driver Tomi Valkeinen
2013-03-28 12:52   ` Peter Meerwald
2013-03-28 12:49 ` [PATCH 14/28] ARM: OMAP: Stalker: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 15/28] ARM: OMAP: OMAP3EVM: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 16/28] ARM: OMAP: IGEP0020: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 17/28] ARM: OMAP: Devkit8000: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 18/28] ARM: OMAP: CM-T35: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 19/28] ARM: OMAP: AM3517EVM: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 20/28] ARM: OMAP: 3430SDP: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 21/28] ARM: OMAP: Overo: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 22/28] ARM: OMAP: Overo: use new generic-dpi-panel " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 23/28] ARM: OMAP: LDP: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 24/28] ARM: OMAP: H4: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 25/28] ARM: OMAP: Devkit8000: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 26/28] ARM: OMAP: CM-T35: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 27/28] ARM: OMAP: AM3517EVM: " Tomi Valkeinen
2013-03-28 12:49 ` [PATCH 28/28] ARM: OMAP: 2430SDP: " Tomi Valkeinen
2013-03-28 15:31 ` [PATCH 00/28] OMAP: DSS related board file changes Igor Grinberg
2013-03-28 16:48   ` Tomi Valkeinen
2013-03-28 16:57     ` Tony Lindgren
2013-03-31  8:41       ` Igor Grinberg
2013-03-31  8:39     ` Igor Grinberg
2013-03-31 15:03       ` Greg Kroah-Hartman
2013-04-02  8:01       ` Tomi Valkeinen
2013-04-02  8:40         ` Igor Grinberg
2013-04-03  9:36 ` Tomi Valkeinen

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=51551841.9070704@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=archit@ti.com \
    --cc=grinberg@compulab.co.il \
    --cc=linux-omap@vger.kernel.org \
    --cc=mike@compulab.co.il \
    --cc=tony@atomide.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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox