Em Mon, 6 May 2019 07:47:13 +0200 Marco Felsch <m.felsch@xxxxxxxxxxxxxx> escreveu: > Hi Mauro, > > I know you are busy but can you have a look on it? You should really trust on the sub-maintainers for such kind of reviews :-) I'll take a look today. > > Regards, > Marco > > On 19-04-15 14:44, Marco Felsch wrote: > > Hi, > > > > many thanks to Hans and Jacopo for the feedack :) this v6 address the > > comments both made on my v5 [1]. > > > > In short this is round fixes just some minor issues rather than major > > ones so the diff to the v5 is really small. The changed patches contain > > the changelog so I omit it here. > > > > I've tested it on a custom hardware but I can't test the em28xx usb > > use-case since I haven't such a device. So other testers are welcome :) > > > > Looking forward for your feedack, > > > > Marco > > > > [1] https://patchwork.kernel.org/cover/10886903/ > > > > Javier Martinez Canillas (1): > > partial revert of "[media] tvp5150: add HW input connectors support" > > > > Marco Felsch (11): > > dt-bindings: connector: analog: add tv norms property > > media: v4l2-fwnode: add v4l2_fwnode_connector > > media: v4l2-fwnode: add initial connector parsing support > > media: tvp5150: add input source selection of_graph support > > media: dt-bindings: tvp5150: Add input port connectors DT bindings > > media: tvp5150: add FORMAT_TRY support for get/set selection handlers > > media: tvp5150: add s_power callback > > media: dt-bindings: tvp5150: cleanup bindings stlye > > media: dt-bindings: tvp5150: add optional tvnorms documentation > > media: tvp5150: add support to limit tv norms on connector > > media: tvp5150: make debug output more readable > > > > Michael Tretter (1): > > media: tvp5150: initialize subdev before parsing device tree > > > > .../display/connector/analog-tv-connector.txt | 4 + > > .../devicetree/bindings/media/i2c/tvp5150.txt | 125 +++- > > drivers/media/i2c/tvp5150.c | 672 +++++++++++++----- > > drivers/media/v4l2-core/v4l2-fwnode.c | 111 +++ > > include/dt-bindings/media/tvnorms.h | 56 ++ > > include/dt-bindings/media/tvp5150.h | 2 - > > include/media/v4l2-connector.h | 30 + > > include/media/v4l2-fwnode.h | 49 ++ > > 8 files changed, 859 insertions(+), 190 deletions(-) > > create mode 100644 include/dt-bindings/media/tvnorms.h > > create mode 100644 include/media/v4l2-connector.h > > > > -- > > 2.20.1 > > > > > > > Thanks, Mauro