Em Thu, 03 Mar 2016 00:58:31 +0200 Laurent Pinchart <laurent.pinchart@xxxxxxxxxxxxxxxx> escreveu: > (Disclaimer: There are lots of thoughts in this e-mail, sometimes in a bit of > a random order. I would thus recommend reading through it completely before > starting to write a reply.) I did read the entire e-mail. There are interesting things there, but we're diverging from what it is needed. I intend to discuss about that later, but let's focus on the problem. See below. > > For S-Video, we may not need to represent two pads. > > Unless I'm mistaken, that's one of the fundamental questions we've been trying > to answer through our discussions on this topic. And I really think we should > answer it, it's the core of the problem we're trying to solve. No, the core problem we're trying to solve are a way simpler than that. 1) how we'll call the entities that represent the connection with external hardware; 2) how we document it? 3) how we map the cases where the S-Video adapter is used for composite. For the first question, it seems that the current namespace is OK, e. g. keep naming them as: #define MEDIA_ENT_F_CONN_RF (MEDIA_ENT_F_BASE + 0x30001) #define MEDIA_ENT_F_CONN_SVIDEO (MEDIA_ENT_F_BASE + 0x30002) #define MEDIA_ENT_F_CONN_COMPOSITE (MEDIA_ENT_F_BASE + 0x30003) For the second question, it was addressed on this patch: https://patchwork.linuxtv.org/patch/33287/ For the third question, I can see only two possibilities: a) create just one entity for S-Video, with 2 pads. if S-Video is connected to it, both pads will be active; if Composite is connected to it, just one pad will be active. b) create a separate entity for "Composite over S-Video". Questions (1) and (2) should be answered for Kernel 4.5. Question (3) was rised by saa7134 driver. We don't need to provide a solution for 4.5 (although it would be really great if we could do it), as, right now, the "composite over S-Video" inputs are not mapped via MC API: the driver just ignores them when creating the connector entities. Thanks, Mauro -- To unsubscribe from this list: send the line "unsubscribe linux-media" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
