On 04/12/2016 06:42 PM, Javier Martinez Canillas wrote: > Hello, > > This is a second version of an RFC patch series that adds MC input connector > support to the tvp5150 driver. The first RFC version was [0]. > > The patches are RFC because a previous version was merged and later reverted > since the approach was found to be inadequate. So I preferred to post this > approach as RFC to discuss it first. > > The main difference with v1 is that a single sink pad is used for the tvp5150 > (instead of using a pad per each input pin) as suggested by Mauro and Hans. > > The mc_nextgen_test dot output after applying the series can be found at [1] > and the graph png generated using the dot tool is at [2]. > > I tested these patches on an IGEPv2 by capturing using both Composite inputs. > > [0]: https://www.mail-archive.com/linux-media@vger.kernel.org/msg95389.html > [1]: http://hastebin.com/yiduhonome.tex > [2]: http://i.imgur.com/EyFtVtJ.png?1 > > Best regards, > Javier > > Changes in v2: > - Remove from the changelog a mention of devices that multiplex the > physical RCA connectors to be used for the S-Video Y and C signals > since it's a special case and it doesn't really work on the IGEPv2. > - Use a single sink pad for the demod and map the connectors as entities > so the mux is made via links. Suggested by Mauro and Hans. > > Javier Martinez Canillas (2): > [media] tvp5150: Add input connectors DT bindings > [media] tvp5150: Replace connector support according to DT binding > > .../devicetree/bindings/media/i2c/tvp5150.txt | 59 ++++++++ > drivers/media/i2c/tvp5150.c | 155 > +++++++++++++++------ > 2 files changed, 170 insertions(+), 44 deletions(-) >
Any comments about this series? Best regards, -- Javier Martinez Canillas Open Source Group Samsung Research America -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
