[PATCH v2 00/10] [media] tvp5150: add MC and DT support

Javier Martinez Canillas
2016-01-07T12:47:05Z

Hello,

One of my testing platforms for the MC next gen [0] work has been an OMAP3
board (IGEPv2) with a tvp5151 video decoder attached to the OMAP3ISP block.

I've been using some patches from Laurent Pinchart that adds MC support to
the tvp5150 driver. The patches were never posted to the list and it seems
he doesn't have time to continue working on this so I have taken them from
his personal tree [1] and submitting now for review.

The series also contains patches that adds DT support to the driver so it
can be used in DT based platforms.

This is the second version of the series that fixes issues addressed by
Rob Herring and Laurent Pinchart. The first patch-set version was [2].

To test, the following media pipeline was used:

$ media-ctl -r -l '"tvp5150 1-005c":0->"OMAP3 ISP CCDC":0[1], "OMAP3 ISP CCDC":1->"OMAP3 ISP CCDC output":0[1]'
$ media-ctl -v --set-format '"OMAP3 ISP CCDC":0 [UYVY2X8 720x240 field:alternate]'
$ media-ctl -v --set-format '"OMAP3 ISP CCDC":1 [UYVY2X8 720x240 field:interlaced-tb]'

And frames captured with the yavta tool:

$ yavta -f UYVY -s 720x480 -n 1 --field interlaced-tb --capture=1 -F /dev/video2
$ raw2rgbpnm -f UYVY -s 720x480 frame-000000.bin frame-000000.pnm

The patches are on top of [0] not because is a depedency but just to avoid
merge conflicts and I don't expect them to be picked before that anyways.

Best regards,
Javier

[0]: http://lists.infradead.org/pipermail/linux-arm-kernel/2015-August/367109.html
[1]: http://git.linuxtv.org/pinchartl/media.git/log/?h=omap3isp/tvp5151
[2]: https://lkml.org/lkml/2016/1/4/216

Changes in v2:
- Fix indentation of the DTS example. Suggested by Rob Herring.
- Rename powerdown-gpios to pdn-gpios to match the pin name in
the datasheet. Suggested by Laurent Pinchart.
- Add optional properties for the video endpoint and list the supported
values. Suggested by Laurent Pinchart.
- Add Reviewed-by tag from Laurent Pinchart to patch 8/10.
- Include missing linux/gpio/consumer.h header. Reported by kbuild test robot.
- Keep the headers sorted alphabetically. Suggested by Laurent Pinchart.
- Rename powerdown to pdn to match datasheet pin. Suggested by Laurent Pinchart.
- Embed mbus_type into struct tvp5150. Suggested by Laurent Pinchart.
- Remove platform data support. Suggested by Laurent Pinchart.
- Check if the hsync, vsync and field even active properties are correct.
Suggested by Laurent Pinchart.

Eduard Gavin (1):
[media] tvp5150: Add OF match table

Javier Martinez Canillas (3):
[media] tvp5150: Add device tree binding document
[media] tvp5150: Initialize the chip on probe
[media] tvp5150: Configure data interface via DT

Laurent Pinchart (6):
[media] tvp5150: Restructure version detection
[media] tvp5150: Add tvp5151 support
[media] tvp5150: Add pad-level subdev operations
[media] tvp5150: Add pixel rate control support
[media] tvp5150: Add s_stream subdev operation support
[media] tvp5150: Add g_mbus_config subdev operation support

.../devicetree/bindings/media/i2c/tvp5150.txt | 45 ++++
drivers/media/i2c/tvp5150.c | 269 +++++++++++++++++----
2 files changed, 268 insertions(+), 46 deletions(-)
create mode 100644 Documentation/devicetree/bindings/media/i2c/tvp5150.txt

--
2.4.3