make vesafb build without CONFIG_MTRR

vesafb did not build without CONFIG_MTRR.

From: Jan Beulich <[email protected]>

(actual patch attached)

vesafb did not build without CONFIG_MTRR.

From: Jan Beulich <[email protected]>

@@ -419,6 +419,7 @@ static int __init vesafb_probe(struct de
 	 * region already (FIXME) */
 	request_region(0x3c0, 32, "vesafb");
 
+#ifdef CONFIG_MTRR
 	if (mtrr) {
 		unsigned int temp_size = size_total;
 		unsigned int type = 0;

@@ -456,6 +457,7 @@ static int __init vesafb_probe(struct de
 			} while (temp_size >= PAGE_SIZE && rc == -EINVAL);
 		}
 	}
+#endif
 	
 	info->fbops = &vesafb_ops;
 	info->var = vesafb_defined;

I wasn't able to reproduce your problem.

Please send the error message and the a complete .config for reproducing it.

cu
Adrian

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

reproducing

Hmm, yes. The change came from the Xen kernel, which for some reason has CONFIG_MTRR but doesn't compile the respective source file on x86-64. When I ran into that, I didn't realize that include/asm/mtrr.h has wrappers for the !CONFIG_MTRR case.

But anyway, the code now framed by the conditionals is dead code without CONFIG_MTRR anyway. So if the number of #ifdef-s is important I'm OK with withrawing that patch, but then I'd like to see the same happen in all the other frame buffer drivers (after all it was the fact that the conditionals were there everywhere else I checked that made me not look at include/asm/mtrr.h)...

Jan