On 7/25/12 7:44 PM, Konke Radlow wrote:
> +static void print_rds_af(struct v4l2_rds_af_set *af_set)
> +{
> + int counter = 0;
> +
> + printf("\nAnnounced AFs: %u", af_set->announced_af);
> + for (int i = 0; i < af_set->size && i < af_set->announced_af; i++, counter++) {
> + if (af_set->af[i] >= 87500000 ) {
> + printf("\nAF%02d: %.1fMHz", counter, af_set->af[i] / 1000000.0);
> + continue;
> + }
> + printf("\nAF%02d: %.1fkHz", counter, af_set->af[i] / 1000.0);
> + }
> +}
> +
> +static void print_rds_pi(const struct v4l2_rds *handle)
> +{
> + printf("\nArea Coverage: %s", v4l2_rds_get_coverage_str(handle));
> +}
> +
> +static void print_rds_data(struct v4l2_rds *handle, uint32_t updated_fields)
> +{
> + if (params.options[OptPrintBlock])
> + updated_fields = 0xFFFFFFFF;
You could use UINT32_MAX here
> +
> + if (updated_fields & V4L2_RDS_PI &&
> + handle->valid_fields & V4L2_RDS_PI) {
> + printf("\nPI: %04x", handle->pi);
> + print_rds_pi(handle);
> + }
> +static int parse_cl(int argc, char **argv)
> +{
> + int i = 0;
> + int idx = 0;
> + int opt = 0;
> + char short_options[26 * 2 * 2 + 1];
Where comes the 26 and 2 from?
Could this be (ARRAY_SIZE(long_options) + 1 ) * 2?
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
