[Intel-gfx] [PATCH 2/2] drm/i915: Add lvds_channel module option

Takashi Iwai tiwai at suse.de
Sat Mar 17 09:00:25 CET 2012


At Fri, 16 Mar 2012 22:51:22 -0300,
Eugeni Dodonov wrote:
> 
> [1  <text/plain; ISO-8859-1 (7bit)>]
> On Fri, Mar 16, 2012 at 18:41, Takashi Iwai <tiwai at suse.de> wrote:
> 
> > Add a new module optoin lvds_channel to specify the LVDS channel mode
> > explicitly instead of probing the LVDS register value set by BIOS.
> > This will be helpful when VBT is broken or incompatible with the
> > current code.
> >
> > Signed-off-by: Takashi Iwai <tiwai at suse.de>
> > ---
> >  drivers/gpu/drm/i915/i915_drv.c      |    6 ++++++
> >  drivers/gpu/drm/i915/i915_drv.h      |    1 +
> >  drivers/gpu/drm/i915/intel_display.c |    5 +++++
> >  3 files changed, 12 insertions(+)
> >
> > diff --git a/drivers/gpu/drm/i915/i915_drv.c
> > b/drivers/gpu/drm/i915/i915_drv.c
> > index 308f819..eee6804 100644
> > --- a/drivers/gpu/drm/i915/i915_drv.c
> > +++ b/drivers/gpu/drm/i915/i915_drv.c
> > @@ -80,6 +80,12 @@ MODULE_PARM_DESC(lvds_downclock,
> >                "Use panel (LVDS/eDP) downclocking for power savings "
> >                "(default: false)");
> >
> > +int i915_lvds_channel __read_mostly;
> > +module_param_named(lvds_channel, i915_lvds_channel, int, 0400);
> >
> 
> As a minor bikeshedding, perhaps this variable would make more sense if
> called 'i915_lvds_channel_mode' instead?
> 
> I am fine with either spelling though, it just looks more semantically
> correct to me. But in any case,

OK, I'll fix in the next spin.


thanks,

Takashi

> 
> Reviewed-by: Eugeni Dodonov <eugeni.dodonov at intel.com>
> 
> -- 
> Eugeni Dodonov
> <http://eugeni.dodonov.net/>
> [2  <text/html; ISO-8859-1 (quoted-printable)>]
> 



More information about the Intel-gfx mailing list