Em Thu, 2 Aug 2018 12:16:41 +0200 Marco Felsch <m.felsch@xxxxxxxxxxxxxx> escreveu: > > > Did you drop the DT of_graph support patch? It was there on your first > > > tvp5150 branch. > > > > Yes. As discussed, I'm waiting for a replacement patch from you. So, > > after testing, I removed it, in order to make simpler to add your > > replacement patch. > > > > IMO, the proper mapping is one input linked to (up to) 3 connectors. > > I tought it would be okay to have more than 1 input pad since the > .sig_type pad property. So the tvp5150 media entity can be represented > like the physical tvp5150 chip. As I said, tvp5150 has internally two physical inputs only: AIP1A and AIP1B. IMO, it should be creating 3 PADS (two inputs and the output one), e. g. something like (names here are just a suggestion): TVP5150_PAD_IN_AIP1A, TVP5150_PAD_IN_AIP1B, TVP5150_PAD_OUT The S-video connector (if present) should be linked to both inputs. I discussed this with other core maintainers: we all have the same opinion about that. I'll post a separate e-mail from the discussions for you and others to comment. It would need some logic that will enforce that just one connector link will be active on any given time (e. g. when a link is enabled, it should disable the other links). > > I will fix it, if it isn't the right way. Ok. Thanks, Mauro -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
