From: Chris Wilson <chris@chris-wilson.co.uk>
To: Rodrigo Vivi <rodrigo.vivi@gmail.com>, Daniel Vetter <daniel@ffwll.ch>
Cc: intel-gfx@lists.freedesktop.org
Subject: Re: [PATCH] drm/i915: Insert i915_preliminary_hw_support variable.
Date: Thu, 18 Oct 2012 08:35:01 +0100 [thread overview]
Message-ID: <b94cdc$70ga66@fmsmga001.fm.intel.com> (raw)
In-Reply-To: <CABVU7+u46o2PvzL-h4Cgns6ZXBTr-8KGAm3RdJp8ojGmzc=p3g@mail.gmail.com>
On Wed, 17 Oct 2012 21:55:10 -0300, Rodrigo Vivi <rodrigo.vivi@gmail.com> wrote:
> On Wed, Oct 17, 2012 at 4:22 PM, Daniel Vetter <daniel@ffwll.ch> wrote:
>
> > On Mon, Oct 15, 2012 at 05:16:23PM -0300, Rodrigo Vivi wrote:
> > > On the worst scenario, users with new hardwares and old kernel from
> > enabling times can get black screens.
> > > So, now on, this i915_perliminary_hw_support variable shall be used by
> > all upcoming platforms that are still under enabling.
> > >
> > > Although it is uncomfortable for developers use this extra variable it
> > brings more stability for end users.
> > >
> > > Signed-off-by: Rodrigo Vivi <rodrigo.vivi@gmail.com>
> >
> > Merged, with the linewrap in the commit message fixed and the module param
> > shrunk by the redudant i915_ prefix.
> >
>
> Thanks!
> To be honest I don't like any of those i915_... all redundants i195.i915_
There weren't meant to be there, they were just cut'n'pasting the
variable name into the wrong field and then enshrined as a public
interface.
>
>
>
> >
> > Thanks, Daniel
> > --
> > Daniel Vetter
> > Software Engineer, Intel Corporation
> > +41 (0) 79 365 57 48 - http://blog.ffwll.ch
> >
>
>
>
> --
> Rodrigo Vivi
> Blog: http://blog.vivi.eng.br
Non-text part: text/html
> _______________________________________________
> Intel-gfx mailing list
> Intel-gfx@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/intel-gfx
--
Chris Wilson, Intel Open Source Technology Centre
prev parent reply other threads:[~2012-10-18 7:35 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-10-15 20:16 [PATCH] drm/i915: Insert i915_preliminary_hw_support variable Rodrigo Vivi
2012-10-15 23:39 ` Dave Airlie
2012-10-17 19:22 ` Daniel Vetter
2012-10-18 0:55 ` Rodrigo Vivi
2012-10-18 7:05 ` Daniel Vetter
2012-10-18 7:35 ` Chris Wilson [this message]
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='b94cdc$70ga66@fmsmga001.fm.intel.com' \
--to=chris@chris-wilson.co.uk \
--cc=daniel@ffwll.ch \
--cc=intel-gfx@lists.freedesktop.org \
--cc=rodrigo.vivi@gmail.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.