On 07/29/2016 07:40 PM, Niklas Söderlund wrote: > Hi, > > This series add V4L2_FIELD_ALTERNATE support to the rcar-vin driver and > changes the field mode reported by adv7180 from V4L2_FIELD_INTERLACED to > V4L2_FIELD_ALTERNATE. > > The change field mode reported by adv7180 was first done by Steve > Longerbeam (https://lkml.org/lkml/2016/7/23/107), I have keept and > reworked Steves patch to report V4L2_FIELD_ALTERNATE instead of > V4L2_FIELD_SEQ_{TB,BT}, after discussions on #v4l this seems more > correct. > > The rcar-vin changes contains some bug fixes needed to enable > V4L2_FIELD_ALTERNATE. > > All work is based on top of media-next and is tested on Koelsch. > > This series touch two drivers which is not a good thing. But I could not > figure out a good way to post them separately since if the adv7180 parts > where too be merged before the rcar-vin changes the driver would stop to > work on the Koelsch. If some one wants this series split in two let me > know. When you post v2, please also run 'v4l2-compliance -f': this will test all format/field combinations. I also recommend testing with qv4l2 if you can to verify that it looks correct. The qv4l2 utility is guaranteed to handle all field settings correctly. Regards, Hans > > Niklas Söderlund (5): > media: rcar-vin: allow field to be changed > media: rcar-vin: fix bug in scaling > media: rcar-vin: fix height for TOP and BOTTOM fields > media: rcar-vin: add support for V4L2_FIELD_ALTERNATE > media: adv7180: fill in mbus format in set_fmt > > Steve Longerbeam (1): > media: adv7180: fix field type > > drivers/media/i2c/adv7180.c | 21 ++-- > drivers/media/platform/rcar-vin/rcar-dma.c | 26 +++-- > drivers/media/platform/rcar-vin/rcar-v4l2.c | 151 ++++++++++++++++------------ > 3 files changed, 123 insertions(+), 75 deletions(-) >