> +++ b/hw/versatile_i2c.c > @@ -0,0 +1,102 @@ > +/* > + * Versatile I2C controller
I'd go for "ARM Versatile I2C controller". I wish companies wouldn't pick generic terms as product names :-( That's not your fault though! > + * > + * Copyright (C) 2012 Oskar Andero <oskar.and...@gmail.com> AFAICS You've copy/pasted this from versatilepb.c, so you should inherit the copyright from there. Adding a copyright line for yourself is OK (though I'm not entirely sure which code you'd actually be able to claim). Changing from my lazy one-line statement to the more verbose boilerplate is also OK. Paul