linux-fbdev.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Tomi Valkeinen <tomi.valkeinen@ti.com>
To: linux-arm-kernel@lists.infradead.org
Subject: Re: [PATCH 11/11] ARM: versatile: move CLCD configuration to device tree
Date: Thu, 18 Feb 2016 13:37:01 +0000	[thread overview]
Message-ID: <56C5C8FD.7090004@ti.com> (raw)
In-Reply-To: <20160218131233.GP19428@n2100.arm.linux.org.uk>

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

On 18/02/16 15:12, Russell King - ARM Linux wrote:
> On Thu, Feb 18, 2016 at 01:52:32PM +0200, Tomi Valkeinen wrote:
>> In my opinion the best option would be to use DT overlays, but so that
>> the bootloader would supply them, or construct the dtb. But afaik that's
>> not possible at the moment. And perhaps I think that's the best option
>> only because I don't work with the bootloaders =).
>>
>> So, I don't like this, but I don't have a good suggestion how to do it
>> better with the infrastructure in place at the moment.
> 
> The danger of that position is that we end up with nothing happening,
> and the problem remaining unresolved, which then pushes people into
> maintaining patches out of mainline just to have a working setup -
> which then pushes people to vendor trees.

I agree.

I didn't express my position clearly: I'm ok with the approach, if we
don't come up with anything better.

But if we go with this approach, it must be understood that it may cause
problems later. It's not the most maintainable approach.

I'd also like to have an ack from the DT maintainers, as I think this is
somewhat of an abuse of the DT.

 Tomi


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

  reply	other threads:[~2016-02-18 13:37 UTC|newest]

Thread overview: 71+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-02-04 14:04 [PATCH 00/11] CLCD Nomadik+Versatile support Linus Walleij
2016-02-04 14:04 ` [PATCH 01/11] video: ARM CLCD: backlight support for OF Linus Walleij
2016-02-04 14:04 ` [PATCH 02/11] video: ARM CLCD: support DT signal inversion flags Linus Walleij
2016-02-17 18:10   ` [02/11] " Ray Jui
2016-02-19  8:46     ` Linus Walleij
2016-02-20  1:23       ` Ray Jui
2016-02-20 11:46         ` Linus Walleij
2016-02-04 14:04 ` [PATCH 03/11] video: ARM CLCD: support pads connected in reverse order Linus Walleij
2016-02-04 14:04 ` [PATCH 04/11] video: ARM CLCD: support Nomadik variant Linus Walleij
2016-02-04 14:04 ` [PATCH 05/11] video: ARM CLCD: add special board and panel hooks for Nomadik Linus Walleij
2016-02-04 14:04 ` [PATCH 06/11] Documentation/DT: add blurb for IB2 syscon to Versatile Linus Walleij
2016-02-04 14:04 ` [PATCH 07/11] Documentation/DT: add Versatile display bindings Linus Walleij
2016-02-17  9:11   ` Tomi Valkeinen
2016-02-18 20:48     ` Linus Walleij
2016-02-04 14:04 ` [PATCH 08/11] video: ARM CLCD: add special panel hook for Versatiles Linus Walleij
2016-02-04 14:04 ` [PATCH 09/11] ARM: PB11MPCore: define a standard VGA panel Linus Walleij
2016-02-04 14:04 ` [PATCH 10/11] ARM: PB1176: " Linus Walleij
2016-02-04 14:04 ` [PATCH 11/11] ARM: versatile: move CLCD configuration to device tree Linus Walleij
2016-02-17  9:09   ` Tomi Valkeinen
2016-02-17  9:41     ` Russell King - ARM Linux
2016-02-17 16:17     ` Linus Walleij
2016-02-17 21:32       ` Russell King - ARM Linux
2016-02-18 11:52         ` Tomi Valkeinen
2016-02-18 13:12           ` Russell King - ARM Linux
2016-02-18 13:37             ` Tomi Valkeinen [this message]
2016-02-18 20:31               ` Linus Walleij
2016-02-21 22:39           ` Linus Walleij
2016-02-22 15:41             ` Tomi Valkeinen
2016-02-22 15:54               ` Linus Walleij
2016-02-23  9:08   ` Linus Walleij
2016-02-23  9:34     ` Arnd Bergmann
2016-02-23 10:10       ` Linus Walleij
2016-02-23 11:22         ` Arnd Bergmann
2016-02-23 13:00           ` Tomi Valkeinen
2016-02-23 13:16             ` Linus Walleij
2016-02-23 13:38               ` Tomi Valkeinen
2016-02-24 10:53                 ` Russell King - ARM Linux
2016-02-24 11:35                   ` Tomi Valkeinen
2016-02-25 14:04                 ` Linus Walleij
2016-02-25 16:08                   ` Arnd Bergmann
2016-02-25 16:22                   ` Russell King - ARM Linux
2016-02-25 16:45                   ` Tomi Valkeinen
2016-02-25 16:57                     ` Russell King - ARM Linux
2016-02-25 19:30                     ` Linus Walleij
2016-02-26 10:47                       ` Tomi Valkeinen
2016-03-05 16:57                         ` Linus Walleij
2016-03-07  7:36                           ` Tomi Valkeinen
2016-02-25 19:32                     ` Linus Walleij
2016-02-23  9:58     ` Tomi Valkeinen
2016-02-23 10:32       ` Adam Ford
2016-02-23 10:59         ` Pantelis Antoniou
2016-02-23 11:56       ` Peter Maydell
2016-02-23 12:01         ` Russell King - ARM Linux
2016-02-23 13:45           ` Tom Rini
2016-02-23 12:45         ` Tomi Valkeinen
2016-02-23 13:49           ` Peter Maydell
2016-02-24 12:06             ` Tomi Valkeinen
2016-02-24 10:46           ` Russell King - ARM Linux
2016-02-24 11:21             ` Tomi Valkeinen
2016-02-24 11:35               ` Russell King - ARM Linux
2016-02-24 11:47                 ` Tomi Valkeinen
2016-02-24 12:13               ` Pantelis Antoniou
2016-02-25 13:43                 ` Linus Walleij
2016-02-25 13:56                   ` Tomi Valkeinen
2016-02-25 14:35                   ` Pantelis Antoniou
2016-02-25 15:36                     ` Linus Walleij
2016-02-25 15:40                       ` Pantelis Antoniou
2016-02-23 13:08         ` Linus Walleij
2016-02-15 23:34 ` [PATCH 00/11] CLCD Nomadik+Versatile support Linus Walleij
2016-02-16 13:29   ` Tomi Valkeinen
2016-02-16 22:30     ` Linus Walleij

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=56C5C8FD.7090004@ti.com \
    --to=tomi.valkeinen@ti.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    /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;
as well as URLs for NNTP newsgroup(s).