dri-devel Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
To: Thierry Reding <thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Cc: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>,
	Alexandre Courbot
	<acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	Terje Bergstrom
	<tbergstrom-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>,
	"linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>,
	"dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org"
	<dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org>,
	"linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org"
	<linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org>
Subject: Re: [PATCH 4/4] ARM: tegra: roth: add display DT node
Date: Mon, 21 Jul 2014 23:51:53 +0100	[thread overview]
Message-ID: <20140721225153.GJ17528@sirena.org.uk> (raw)
In-Reply-To: <20140721211630.GA12076@mithrandir>

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

On Mon, Jul 21, 2014 at 11:16:32PM +0200, Thierry Reding wrote:
> On Mon, Jul 21, 2014 at 09:35:27AM -0600, Stephen Warren wrote:

> > > vdd-2v8-display needs to remain always-on however. Here we may hit a
> > > limitation of the simple-panel driver, where only one power supply can
> > > be provided.

> > Can't we fix the simple-panel driver to allow a list of regulators in
> > the property?

> I have no objection to allowing that. But I don't think there's a way to
> do that with the current regulator API. You can use the bulk API, but
> that requires separate properties, not multiple regulators in one
> property.

> Perhaps one idea to solve this would be to make the regulator API return
> a regulator handle that in fact controls an array of regulators? Adding
> Mark.

I'm really not comfortable with that idea, it seems like most of the
users would be abusing it - one of the biggest issues is always getting
people to understand that their driver may be used in other systems with
the supplies mapped differently.  If you were going to do something
along those lines you'd need to do something that enumerated all the
supply properties on the device.

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

  reply	other threads:[~2014-07-21 22:51 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-07-02 12:19 [PATCH 0/4] drm/dsi/tegra: continuous clock support Alexandre Courbot
2014-07-02 12:19 ` [PATCH 1/4] drm/dsi: Add flag for continuous clock behavior Alexandre Courbot
2014-07-03  8:23   ` Andrzej Hajda
     [not found]     ` <53B5131B.1060702-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-07-04  5:57       ` Alexandre Courbot
     [not found]         ` <CAAVeFu+sTWhwCTd9ntqdJQA1aaFV93zHnyRSj6WA-8VicFuvgQ-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-07-04  9:53           ` Andrzej Hajda
     [not found]             ` <53B679A0.1060306-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-07-06  9:30               ` Alexandre Courbot
2014-07-02 12:19 ` [PATCH 2/4] drm/tegra: dsi - Handle continuous clock flag Alexandre Courbot
2014-07-02 12:19 ` [PATCH 3/4] drm/panel: LG LH500WX1-SD03 uses continuous clock Alexandre Courbot
2014-07-02 12:19 ` [PATCH 4/4] ARM: tegra: roth: add display DT node Alexandre Courbot
2014-07-02 15:55   ` Stephen Warren
     [not found]     ` <53B42B60.1000600-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-07-03  3:10       ` Alexandre Courbot
2014-07-21 15:35         ` Stephen Warren
     [not found]           ` <53CD333F.2070805-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2014-07-21 21:16             ` Thierry Reding
2014-07-21 22:51               ` Mark Brown [this message]
     [not found]                 ` <20140721225153.GJ17528-GFdadSzt00ze9xe1eoZjHA@public.gmane.org>
2014-07-22  4:39                   ` Alexandre Courbot
     [not found]                     ` <CAAVeFuKtVYNP5LFXGCEpkUtCCrK6AP+9WEVPDPs45RwAPvKKRg-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2014-07-22  9:45                       ` Mark Brown
     [not found]   ` <1404303560-32209-5-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-07-03  3:15     ` [PATCH v2] " Alexandre Courbot

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=20140721225153.GJ17528@sirena.org.uk \
    --to=broonie-dgejt+ai2ygdnm+yrofe0a@public.gmane.org \
    --cc=acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
    --cc=linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
    --cc=swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org \
    --cc=tbergstrom-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=thierry.reding-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.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