drm/nouveau/devinit: move simple pll setting routines to devinit

Dan Carpenter dan.carpenter at oracle.com
Wed Jan 28 11:28:21 PST 2015


[ Hm...  That's weird.  I don't know why this old bug is showing up in
  my new bugs pile.  Oh well, it looks valid. ]

Hello Ben Skeggs,

The patch 88524bc06926: "drm/nouveau/devinit: move simple pll setting
routines to devinit" from Mar 5, 2013, leads to the following static
checker warning:

	drivers/gpu/drm/nouveau/nvkm/subdev/devinit/nv04.c:404 nv04_devinit_fini()
	warn: impossible condition '(priv->owner < 0) => (0-255 < 0)'

drivers/gpu/drm/nouveau/nvkm/subdev/devinit/nv04.c
   402  
   403          /* unslave crtcs */
   404          if (priv->owner < 0)
                    ^^^^^^^^^^^^^^^
This condition is never true.  We could change the -1 in
nv04_devinit_ctor() to 0xff or make owner an int.

   405                  priv->owner = nv_rdvgaowner(priv);
   406          nv_wrvgaowner(priv, 0);
   407          return 0;
   408  }

regards,
dan carpenter


More information about the dri-devel mailing list