drm/nouveau: Fix a sparse warning.
authorFrancisco Jerez <currojerez@riseup.net>
Tue, 13 Jul 2010 14:16:26 +0000 (16:16 +0200)
committerBen Skeggs <bskeggs@redhat.com>
Mon, 26 Jul 2010 01:42:00 +0000 (11:42 +1000)
It doesn't like variable length arrays.

Signed-off-by: Francisco Jerez <currojerez@riseup.net>
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nouveau_bios.c

index 0eb1b5ad0db117e9b2cd43dbe25fcd31541d51d4..31183a41b8d69dc607d232dd9487436cecc928cd 100644 (file)
@@ -209,20 +209,20 @@ static struct methods shadow_methods[] = {
        { "PCIROM", load_vbios_pci, true },
        { "ACPI", load_vbios_acpi, true },
 };
+#define NUM_SHADOW_METHODS ARRAY_SIZE(shadow_methods)
 
 static bool NVShadowVBIOS(struct drm_device *dev, uint8_t *data)
 {
-       const int nr_methods = ARRAY_SIZE(shadow_methods);
        struct methods *methods = shadow_methods;
        int testscore = 3;
-       int scores[nr_methods], i;
+       int scores[NUM_SHADOW_METHODS], i;
 
        if (nouveau_vbios) {
-               for (i = 0; i < nr_methods; i++)
+               for (i = 0; i < NUM_SHADOW_METHODS; i++)
                        if (!strcasecmp(nouveau_vbios, methods[i].desc))
                                break;
 
-               if (i < nr_methods) {
+               if (i < NUM_SHADOW_METHODS) {
                        NV_INFO(dev, "Attempting to use BIOS image from %s\n",
                                methods[i].desc);
 
@@ -234,7 +234,7 @@ static bool NVShadowVBIOS(struct drm_device *dev, uint8_t *data)
                NV_ERROR(dev, "VBIOS source \'%s\' invalid\n", nouveau_vbios);
        }
 
-       for (i = 0; i < nr_methods; i++) {
+       for (i = 0; i < NUM_SHADOW_METHODS; i++) {
                NV_TRACE(dev, "Attempting to load BIOS image from %s\n",
                         methods[i].desc);
                data[0] = data[1] = 0;  /* avoid reuse of previous image */
@@ -245,7 +245,7 @@ static bool NVShadowVBIOS(struct drm_device *dev, uint8_t *data)
        }
 
        while (--testscore > 0) {
-               for (i = 0; i < nr_methods; i++) {
+               for (i = 0; i < NUM_SHADOW_METHODS; i++) {
                        if (scores[i] == testscore) {
                                NV_TRACE(dev, "Using BIOS image from %s\n",
                                         methods[i].desc);