[Nouveau] module parameter description fix

soulpatch soulpa7ch at gmail.com
Thu Feb 25 21:31:16 PST 2010


On Fri, 2010-02-26 at 00:23 -0500, Tavian Barnes wrote:
> On 25 February 2010 23:36, soulpatch <l337_sp33k at yahoo.ca> wrote:
> > I found a bug in the nouveau kernel module. It looks like a cut and
> > paste error on the definitions of some module parameters in
> > nouveau_drv.c. The bug exhibits when displaying module parameter
> > information with the modinfo utility. Here's a patch:
> >
> > diff --git a/drivers/gpu/drm/nouveau/nouveau_drv.c b/drivers/gpu/drm/nouveau/nouveau_drv.c
> > index da3b93b..874adf5 100644
> > --- a/drivers/gpu/drm/nouveau/nouveau_drv.c
> > +++ b/drivers/gpu/drm/nouveau/nouveau_drv.c
> > @@ -75,11 +75,11 @@ MODULE_PARM_DESC(ignorelid, "Ignore ACPI lid status");
> >  int nouveau_ignorelid = 0;
> >  module_param_named(ignorelid, nouveau_ignorelid, int, 0400);
> >
> > -MODULE_PARM_DESC(noagp, "Disable all acceleration");
> > +MODULE_PARM_DESC(noaccel, "Disable all acceleration");
> >  int nouveau_noaccel = 0;
> >  module_param_named(noaccel, nouveau_noaccel, int, 0400);
> >
> > -MODULE_PARM_DESC(noagp, "Disable fbcon acceleration");
> > +MODULE_PARM_DESC(nofbaccel, "Disable fbcon acceleration");
> >  int nouveau_nofbaccel = 0;
> >  module_param_named(nofbaccel, nouveau_nofbaccel, int, 0400);
> >
> >
> > Cheers,
> >        soulpatch
> >
> >
> > _______________________________________________
> > Nouveau mailing list
> > Nouveau at lists.freedesktop.org
> > http://lists.freedesktop.org/mailman/listinfo/nouveau
> >
> 
> Looks like 933236b4a6ae9cb917f80ef1effe964323d7f25a just fixed this yesterday.
> 

Yeah, I fixed it yesterday too. I guess I should have done another pull
before generating the diff today. Glad it got fixed though.

soulpatch



More information about the Nouveau mailing list