From: "Ville Syrjälä" <ville.syrjala@linux.intel.com>
To: Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx <intel-gfx@lists.freedesktop.org>
Subject: Re: [RFC 09/12] drm/i915/config: Add VBT settings configuration.
Date: Fri, 15 May 2015 12:39:20 +0300 [thread overview]
Message-ID: <20150515093920.GW18908@intel.com> (raw)
In-Reply-To: <20150224205216.GG24485@phenom.ffwll.local>
On Tue, Feb 24, 2015 at 09:52:16PM +0100, Daniel Vetter wrote:
> On Tue, Feb 24, 2015 at 10:37:10AM -0800, Bob Paauwe wrote:
> > On Tue, 24 Feb 2015 14:57:48 +0100
> > Daniel Vetter <daniel@ffwll.ch> wrote:
> > > As Jani points out we already have vbt headaches, it would be good if we
> > > only have those once ;-)
> > > -Daniel
> >
> > I've been trying to think of a better way but not really coming up with
> > something that scales well. EMGD took the approach of create
> > configuration for only those values that we had customer requests for.
> > And that was primarily just the panel power up/down timing sequence and
> > some backlight control. We never tried to replicate everything that
> > could be set via vbt.
>
> If it's only a very few select things we could just add them as atomic
> properties to connectors. No need for anything special, and we could even
> use that to debug panel issues at runtime.
I think properties should only be used for something that the user
might actually want to configure. Not sure what exactly we're talking
about here, but eg. panel timings are definitely not in that category
IMO.
>
> We need a lot of the dp things already anyway for the dp validation stuff,
> but since that was debug Dave shot down my idea to just go with props. But
> here we have another user, and this one will have full abi compat
> requirements. So no longer any reasons to not go with props imo.
>
> Or else we just smash an entire vbt into acpi, for my that's ok (we need
> to deal with vbt anyway).
> -Daniel
> --
> Daniel Vetter
> Software Engineer, Intel Corporation
> +41 (0) 79 365 57 48 - http://blog.ffwll.ch
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Ville Syrjälä
Intel OTC
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
next prev parent reply other threads:[~2015-05-15 9:39 UTC|newest]
Thread overview: 32+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-02-12 23:41 [RFC 00/12] i915 init-time configuration Bob Paauwe
2015-02-12 23:41 ` [RFC 01/12] drm/i915/config: Initial framework Bob Paauwe
2015-02-13 8:19 ` Jani Nikula
2015-02-24 16:17 ` Daniel Vetter
2015-02-24 17:41 ` Bob Paauwe
2015-02-24 20:47 ` Daniel Vetter
2015-02-12 23:41 ` [RFC 02/12] drm/i915/config: Introduce intel_output_name Bob Paauwe
2015-02-13 8:12 ` Jani Nikula
2015-02-12 23:41 ` [RFC 03/12] drm/i915/config: Add init-time configuration of bits per color Bob Paauwe
2015-02-12 23:41 ` [RFC 04/12] drm/i915/config: Set dp panel fitter property based on init-time config Bob Paauwe
2015-02-12 23:41 ` [RFC 05/12] drm/i915/config: Set general connector properties using config Bob Paauwe
2015-02-12 23:41 ` [RFC 06/12] drm/i915/config: Split out allocation of list nodes Bob Paauwe
2015-02-13 8:15 ` Jani Nikula
2015-02-12 23:41 ` [RFC 07/12] drm/i915/config: Get workaround information from configuration Bob Paauwe
2015-02-24 13:51 ` Daniel Vetter
2015-02-24 17:54 ` Bob Paauwe
2015-02-12 23:41 ` [RFC 08/12] drm/i915/config: Use workarounds list " Bob Paauwe
2015-02-12 23:41 ` [RFC 09/12] drm/i915/config: Add VBT settings configuration Bob Paauwe
2015-02-24 13:57 ` Daniel Vetter
2015-02-24 18:37 ` Bob Paauwe
2015-02-24 20:52 ` Daniel Vetter
2015-02-24 22:23 ` Bob Paauwe
2015-05-15 9:39 ` Ville Syrjälä [this message]
2015-05-20 17:07 ` Bob Paauwe
2015-05-21 8:37 ` Daniel Vetter
2015-05-26 21:20 ` Bob Paauwe
2015-05-27 12:23 ` Daniel Vetter
2015-02-12 23:41 ` [RFC 10/12] drm/i915/config: Introduce a test table and code to make use of it Bob Paauwe
2015-02-12 23:41 ` [RFC 11/12] drm/i915/config: Add workaround properties to ACPI table Bob Paauwe
2015-02-12 23:41 ` [RFC 12/12] drm/i915/config: Add ACPI device examples for VBT configuration Bob Paauwe
2015-02-13 8:08 ` [RFC 00/12] i915 init-time configuration Jani Nikula
2015-02-13 17:17 ` Bob Paauwe
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=20150515093920.GW18908@intel.com \
--to=ville.syrjala@linux.intel.com \
--cc=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.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