On Wed, Nov 21, 2012 at 06:41:01PM +0530, Philip, Avinash wrote:
[...]
> +static const struct of_device_id ecap_of_match[] = {
> +	{
> +		.compatible	= "ti,am33xx-ecap",
> +	},

Same here, can be shorter by putting it on one line instead of three.

Thierry
[unhandled content-type:application/pgp-signature]