All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Tony Lindgren <tony@atomide.com>, Sebastian Reichel <sre@debian.org>
Cc: linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org,
	devicetree@vger.kernel.org, Archit Taneja <archit@ti.com>,
	Darren Etheridge <detheridge@ti.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Stefan Roese <stefan.roese@gmail.com>,
	Robert Nelson <robertcnelson@gmail.com>,
	"Dr . H . Nikolaus Schaller" <hns@goldelico.com>,
	Marek Belisko <marek@goldelico.com>
Subject: Re: [PATCH 00/26] OMAPDSS: DT support (Christmas edition)
Date: Thu, 05 Dec 2013 09:41:42 +0000	[thread overview]
Message-ID: <52A04A56.60403@ti.com> (raw)
In-Reply-To: <20131204160119.GA9433@earth.universe>

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

On 2013-12-04 18:01, Sebastian Reichel wrote:
> Hi,
> 
> On Wed, Dec 04, 2013 at 02:28:27PM +0200, Tomi Valkeinen wrote:
>> Here's a new version for DT support to OMAP Display Subsystem. See
>> http://article.gmane.org/gmane.linux.ports.arm.omap/102689 for the intro of the
>> previous version, which contains thoughts about the related problems.
>>
>> The major change in this version is the use of V4L2 and CDF style port/endpoint
>> style in the DT data. However, note that even if the DT data contains proper
>> port & endpoint data, the drivers only use the first endpoint. This is to
>> simplify the patches, as adding full support for the ports and endpoints to the
>> drivers will be a big task. This approach still works with more or less all the
>> boards, as the only cases where the simpler model is an issue are the boards
>> with multiple display devices connected to a single output.
>>
>> Laurent, I'd appreciate if you could have a look at the .dts changes, to see if
>> there's anything that's clearly not CDF compatible.
>>
>> The patches can also be found from:
>> git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git work/dss-dt
> 
> Will this be ready for 3.14? Depending on that I would handle

That is my hope. We have been missing DSS DT bindings for too long, and
maintaining and adding new mixed DT/non-DT panels, as done for omap4
now, is just a big headache.

The bindings in my series are probably not perfect, but I believe they
should be fine enough. I didn't get any NACKs for the previous versions
of this series, and the only reason I haven't been pushing these to
mainline is that I haven't been satisfied with the bindings.

Laurent promised to have a look at the bindings to see if he notices
anything that would conflict with Common Display Framework bindings. If
he's ok with the bindings, I think we can decide next week if these will
be merged for 3.14.

> omapdss support for the N900 differently. If this code gets merged
> into 3.14 I will update the N900 patches [0] accordingly. Otherwise
> I would suggest to take the N900 patches as is for 3.14 and I will
> take care of adding the omapdss DT support to the panel (and the
> N900.dts) separately.

I really would not like to add any more hacks for DSS.

 Tomi



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

WARNING: multiple messages have this Message-ID (diff)
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: Tony Lindgren <tony@atomide.com>, Sebastian Reichel <sre@debian.org>
Cc: linux-omap@vger.kernel.org, linux-fbdev@vger.kernel.org,
	devicetree@vger.kernel.org, Archit Taneja <archit@ti.com>,
	Darren Etheridge <detheridge@ti.com>,
	Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
	Stefan Roese <stefan.roese@gmail.com>,
	Robert Nelson <robertcnelson@gmail.com>,
	"Dr . H . Nikolaus Schaller" <hns@goldelico.com>,
	Marek Belisko <marek@goldelico.com>
Subject: Re: [PATCH 00/26] OMAPDSS: DT support (Christmas edition)
Date: Thu, 5 Dec 2013 11:41:42 +0200	[thread overview]
Message-ID: <52A04A56.60403@ti.com> (raw)
In-Reply-To: <20131204160119.GA9433@earth.universe>

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

On 2013-12-04 18:01, Sebastian Reichel wrote:
> Hi,
> 
> On Wed, Dec 04, 2013 at 02:28:27PM +0200, Tomi Valkeinen wrote:
>> Here's a new version for DT support to OMAP Display Subsystem. See
>> http://article.gmane.org/gmane.linux.ports.arm.omap/102689 for the intro of the
>> previous version, which contains thoughts about the related problems.
>>
>> The major change in this version is the use of V4L2 and CDF style port/endpoint
>> style in the DT data. However, note that even if the DT data contains proper
>> port & endpoint data, the drivers only use the first endpoint. This is to
>> simplify the patches, as adding full support for the ports and endpoints to the
>> drivers will be a big task. This approach still works with more or less all the
>> boards, as the only cases where the simpler model is an issue are the boards
>> with multiple display devices connected to a single output.
>>
>> Laurent, I'd appreciate if you could have a look at the .dts changes, to see if
>> there's anything that's clearly not CDF compatible.
>>
>> The patches can also be found from:
>> git://git.kernel.org/pub/scm/linux/kernel/git/tomba/linux.git work/dss-dt
> 
> Will this be ready for 3.14? Depending on that I would handle

That is my hope. We have been missing DSS DT bindings for too long, and
maintaining and adding new mixed DT/non-DT panels, as done for omap4
now, is just a big headache.

The bindings in my series are probably not perfect, but I believe they
should be fine enough. I didn't get any NACKs for the previous versions
of this series, and the only reason I haven't been pushing these to
mainline is that I haven't been satisfied with the bindings.

Laurent promised to have a look at the bindings to see if he notices
anything that would conflict with Common Display Framework bindings. If
he's ok with the bindings, I think we can decide next week if these will
be merged for 3.14.

> omapdss support for the N900 differently. If this code gets merged
> into 3.14 I will update the N900 patches [0] accordingly. Otherwise
> I would suggest to take the N900 patches as is for 3.14 and I will
> take care of adding the omapdss DT support to the panel (and the
> N900.dts) separately.

I really would not like to add any more hacks for DSS.

 Tomi



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

  reply	other threads:[~2013-12-05  9:41 UTC|newest]

Thread overview: 183+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-12-04 12:28 [PATCH 00/26] OMAPDSS: DT support (Christmas edition) Tomi Valkeinen
2013-12-04 12:28 ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 01/26] OMAPDSS: rename display-sysfs 'name' entry Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 02/26] OMAPDSS: DSI: fix fifosize Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 03/26] ARM: OMAP: remove DSS DT hack Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 04/26] OMAPDSS: remove DT hacks for regulators Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 05/26] ARM: OMAP2+: add omapdss_init_of() Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-11 23:10   ` Laurent Pinchart
2013-12-11 23:10     ` Laurent Pinchart
2013-12-12  7:30     ` Tomi Valkeinen
2013-12-12  7:30       ` Tomi Valkeinen
2013-12-13  8:32       ` Archit Taneja
2013-12-13  8:44         ` Archit Taneja
2013-12-13  8:40         ` Tomi Valkeinen
2013-12-13  8:40           ` Tomi Valkeinen
2013-12-13 17:07         ` Tony Lindgren
2013-12-13 17:07           ` Tony Lindgren
2013-12-04 12:28 ` [PATCH 06/26] OMAPDSS: if dssdev->name==NULL, use alias Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-11 23:13   ` Laurent Pinchart
2013-12-11 23:13     ` Laurent Pinchart
2013-12-11 23:56     ` Laurent Pinchart
2013-12-11 23:56       ` Laurent Pinchart
2013-12-12  7:41       ` Tomi Valkeinen
2013-12-12  7:41         ` Tomi Valkeinen
     [not found]         ` <52A968BD.20304-l0cyMroinI0@public.gmane.org>
2013-12-12 10:05           ` Sebastian Reichel
2013-12-12 10:05             ` Sebastian Reichel
2013-12-12 13:22             ` Laurent Pinchart
2013-12-12 13:22               ` Laurent Pinchart
2013-12-12 14:13             ` Tomi Valkeinen
2013-12-12 14:13               ` Tomi Valkeinen
2013-12-12 14:15               ` Laurent Pinchart
2013-12-12 14:15                 ` Laurent Pinchart
2013-12-12 14:19                 ` Tomi Valkeinen
2013-12-12 14:19                   ` Tomi Valkeinen
2013-12-12 17:31                   ` Sebastian Reichel
2013-12-12 17:31                     ` Sebastian Reichel
2013-12-13 12:01               ` Tomi Valkeinen
2013-12-13 12:01                 ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 07/26] OMAPDSS: get dssdev->alias from DT alias Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 08/26] OMAPFB: clean up default display search Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 09/26] OMAPFB: search for default display with DT alias Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 10/26] OMAPDSS: add of helpers Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-11 23:19   ` Laurent Pinchart
2013-12-11 23:19     ` Laurent Pinchart
2013-12-12  7:48     ` Tomi Valkeinen
2013-12-12  7:48       ` Tomi Valkeinen
2013-12-13  2:37       ` Laurent Pinchart
2013-12-13  2:37         ` Laurent Pinchart
2013-12-04 12:28 ` [PATCH 11/26] OMAPDSS: Add DT support to DSS, DISPC, DPI, HDMI, VENC Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 12/26] OMAPDSS: Add DT support to DSI Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 13/26] ARM: omap3.dtsi: add omapdss information Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-05 17:05   ` Tony Lindgren
2013-12-05 17:05     ` Tony Lindgren
2013-12-09 12:45     ` Tomi Valkeinen
2013-12-09 12:45       ` Tomi Valkeinen
2013-12-09 18:04       ` Tony Lindgren
2013-12-09 18:04         ` Tony Lindgren
2013-12-11 23:44       ` Laurent Pinchart
2013-12-11 23:44         ` Laurent Pinchart
2013-12-12  8:38         ` Tomi Valkeinen
2013-12-12  8:38           ` Tomi Valkeinen
2013-12-12 21:59           ` Tony Lindgren
2013-12-13  3:27             ` Laurent Pinchart
2013-12-13  3:27               ` Laurent Pinchart
2013-12-13 10:18               ` Tomi Valkeinen
2013-12-13 10:18                 ` Tomi Valkeinen
     [not found]                 ` <52AADEF3.9040808-l0cyMroinI0@public.gmane.org>
2013-12-13 17:10                   ` Tony Lindgren
2013-12-13 17:10                     ` Tony Lindgren
2013-12-13  3:24           ` Laurent Pinchart
2013-12-13  3:24             ` Laurent Pinchart
2013-12-13  9:29             ` Tomi Valkeinen
2013-12-13  9:29               ` Tomi Valkeinen
2013-12-16 10:49             ` Tomi Valkeinen
2013-12-16 10:49               ` Tomi Valkeinen
     [not found]               ` <52AEDA9F.2020609-l0cyMroinI0@public.gmane.org>
2013-12-16 13:55                 ` Laurent Pinchart
2013-12-16 13:55                   ` Laurent Pinchart
2013-12-04 12:28 ` [PATCH 14/26] ARM: omap4.dtsi: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 15/26] ARM: omap4-panda.dts: add display information Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-06  8:57   ` Javier Martinez Canillas
2013-12-06  8:57     ` Javier Martinez Canillas
2013-12-09 12:56     ` Tomi Valkeinen
2013-12-09 12:56       ` Tomi Valkeinen
2013-12-09 15:09       ` Javier Martinez Canillas
2013-12-09 15:09         ` Javier Martinez Canillas
2013-12-09 15:30         ` Tomi Valkeinen
2013-12-09 15:30           ` Tomi Valkeinen
2013-12-09 16:53           ` Javier Martinez Canillas
2013-12-09 16:53             ` Javier Martinez Canillas
2013-12-10 10:56             ` Enric Balletbo Serra
2013-12-10 10:56               ` Enric Balletbo Serra
2013-12-10 12:10               ` Tomi Valkeinen
2013-12-10 12:10                 ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 16/26] ARM: omap4-sdp.dts: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-13  9:27   ` Archit Taneja
2013-12-13  9:39     ` Archit Taneja
2013-12-13  9:39     ` Tomi Valkeinen
2013-12-13  9:39       ` Tomi Valkeinen
2013-12-13  9:58       ` Archit Taneja
2013-12-13  9:58         ` Archit Taneja
2013-12-13 10:15         ` Tomi Valkeinen
2013-12-13 10:15           ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 17/26] ARM: omap3-tobi.dts: add lcd (TEST) Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-06 10:18   ` Florian Vaussard
2013-12-06 10:18     ` Florian Vaussard
2013-12-10 12:18     ` Tomi Valkeinen
2013-12-10 12:18       ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 18/26] ARM: omap3-beagle.dts: add display information Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-06  8:41   ` Javier Martinez Canillas
2013-12-06  8:41     ` Javier Martinez Canillas
2013-12-09 12:06     ` Tomi Valkeinen
2013-12-09 12:06       ` Tomi Valkeinen
2013-12-09 12:16       ` Javier Martinez Canillas
2013-12-09 12:16         ` Javier Martinez Canillas
2013-12-04 12:28 ` [PATCH 19/26] ARM: omap3-beagle-xm.dts: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 20/26] OMAPDSS: panel-dsi-cm: Add DT support Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 21/26] OMAPDSS: encoder-tfp410: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 22/26] OMAPDSS: connector-dvi: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 23/26] OMAPDSS: encoder-tpd12s015: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 24/26] OMAPDSS: hdmi-connector: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 25/26] OMAPDSS: panel-dpi: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
2013-12-04 12:28 ` [PATCH 26/26] OMAPDSS: connector-analog-tv: " Tomi Valkeinen
2013-12-04 12:28   ` Tomi Valkeinen
     [not found] ` <1386160133-24026-1-git-send-email-tomi.valkeinen-l0cyMroinI0@public.gmane.org>
2013-12-04 16:01   ` [PATCH 00/26] OMAPDSS: DT support (Christmas edition) Sebastian Reichel
2013-12-04 16:01     ` Sebastian Reichel
2013-12-05  9:41     ` Tomi Valkeinen [this message]
2013-12-05  9:41       ` Tomi Valkeinen
2013-12-07  3:48 ` Javier Martinez Canillas
2013-12-07  3:48   ` Javier Martinez Canillas
2013-12-07  4:28   ` Javier Martinez Canillas
2013-12-07  4:28     ` Javier Martinez Canillas
2013-12-09 12:01     ` Tomi Valkeinen
2013-12-09 12:01       ` Tomi Valkeinen
2013-12-09 12:23       ` Javier Martinez Canillas
2013-12-09 12:23         ` Javier Martinez Canillas
2013-12-12  0:39 ` Laurent Pinchart
2013-12-12  0:39   ` Laurent Pinchart
2013-12-12  8:54   ` Tomi Valkeinen
2013-12-12  8:54     ` Tomi Valkeinen
2013-12-13  3:45     ` Laurent Pinchart
2013-12-13  3:45       ` Laurent Pinchart
2013-12-13  8:16       ` Geert Uytterhoeven
2013-12-13  8:16         ` Geert Uytterhoeven
2013-12-13 10:05       ` Tomi Valkeinen
2013-12-13 10:05         ` Tomi Valkeinen
2013-12-13 14:37         ` Laurent Pinchart
2013-12-13 14:37           ` Laurent Pinchart
2013-12-13 15:47           ` Tomi Valkeinen
2013-12-13 15:47             ` Tomi Valkeinen
2013-12-13 17:22             ` Tony Lindgren
2013-12-13 17:22               ` Tony Lindgren
2013-12-14  7:34               ` Tomi Valkeinen
2013-12-14  7:34                 ` Tomi Valkeinen
2013-12-14 14:09                 ` Tony Lindgren
2013-12-14 14:09                   ` Tony Lindgren
2013-12-16  7:24                   ` Tomi Valkeinen
2013-12-16  7:24                     ` Tomi Valkeinen
2013-12-18  0:30                     ` Tony Lindgren
2013-12-18  0:30                       ` Tony Lindgren

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=52A04A56.60403@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=archit@ti.com \
    --cc=detheridge@ti.com \
    --cc=devicetree@vger.kernel.org \
    --cc=hns@goldelico.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=linux-fbdev@vger.kernel.org \
    --cc=linux-omap@vger.kernel.org \
    --cc=marek@goldelico.com \
    --cc=robertcnelson@gmail.com \
    --cc=sre@debian.org \
    --cc=stefan.roese@gmail.com \
    --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 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.