Hello, I've done the modifications I suggested. Defines are now part of genesys_glXXX files. And a couple of functions close to hardware moved away from genesys.c . Needed register bits are used through helper functions. I'm sorry but I think you'll have to merge your changes to genesys_gl841.c by hand first before check-in your changes. Regards, Stef