>@@ -47,7 +48,7 @@ static int __init olpc_pm_init(void) > uint32_t lo, hi; > int ret; > >- if (!machine_is_olpc) >+ if (machine_is_olpc()) > return -ENODEV; You lost the negation here