> There is a function buried in the depths of the code that can be called
> from both modules and at init time. If its called at init time it
> displays the penguin logo everyone wants. If its called later it doesnt.
> So either there is a "penguin has been freed" flag or you pass "is being
> called by a module" through every single layer of code to that deeply
> buried function.
isnt it cleaner to add something like this to chdev_init:
#if defined (CONFIG_FB)
fbmem_init();
+ show_fb_logos();
#endif
this is never executed during module init. The solution is IMO not to push
a flag deep within some function, but to split the function into two
orthogonal parts. This needs no per-fbdriver #ifdefs.
-- mingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.rutgers.edu
Please read the FAQ at http://www.tux.org/lkml/faq.html