Re: ANY field

[Gabor Kerenyi <wom@xxxxxxxxxxx>]

On Friday 28 February 2003 13:31, Gerd Knorr wrote:
> Gabor Kerenyi <wom@xxxxxxxxxxx> writes:
> > I think the V4L2_FIELD_ANY should be assigned to this
> > feature, but now it is mapped to the bottom field only.
>
> No.  ANY is "driver is free to pick anything".  You are looking for
> V4L2_FIELD_ALTERNATE.

Thanks. I tried but it does not work.
1. When I try to set the field to this value I get EINVAL
Ok, at try_fmt there was no case for this value, so I added.
Then it was working but only the bottom picture was captured
(25 fps instead of 50).

BTW: there is a bug at try_fmt function
in the last case branch. At V4L2_FIELD_SEQ_TB a break is
missing:

switch (field) {
                case V4L2_FIELD_TOP:
                case V4L2_FIELD_BOTTOM:
                        maxh = maxh/2;
                        break;
                case V4L2_FIELD_INTERLACED:
                        break;
                case V4L2_FIELD_SEQ_TB:
                        if (fmt->flags & FORMAT_FLAGS_PLANAR)
                                return -EINVAL;
                        break;
                default:
                        return -EINVAL;
}

gabor