Hi Dan,

   > I don't think that's upstream suitable. It will cripple any
   > non-XO system that boots a kernel with that option
   > included. Remember that distributors like Fedora ship their
   > generic kernels to the whole world with CONFIG_OLPC enabled.

Thanks, you're right.

   > At the very least there should be a runtime "am i an XO" check
   > inside the ifdef..although ideally we could find (with Harald) a
   > less ugly way of doing this.

"if (machine_is_olpc())" is the right way to do this check, but it's
returning false on the 1.5 board at the moment.  I'll commit a patch
that uses that once it's hooked up, and we can use that patch to
start a discussion upstream about the preferred way to do this.

- Chris.
-- 
Chris Ball   <[email protected]>
_______________________________________________
Devel mailing list
[email protected]
http://lists.laptop.org/listinfo/devel

Reply via email to