On 27 Nov 2016 15:16, Stafford Horne wrote:
> +/* Search FEATURE for a register REGNO and return its name. */

GNU style puts two spaces after periods

> +char *
> +tdesc_find_register_name (const struct tdesc_feature *feature,
> +			   int regno)
> +char *
> +tdesc_find_register_group_name (const struct tdesc_feature *feature,
> +			   int regno)

seems like return values for both of these should be const
-mike