> +++ 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

Reply via email to