On Tue 18 September 2012 12:53:02 Hans Verkuil wrote: > Hi Prabhakar, > > This patch series does some driver cleanup and reorganizes the config > structs that are used to set up subdevices. > > The current driver associates an input or output with a subdev, but multiple > inputs may use the same subdev and some inputs may not use a subdev at all > (this is the case for our hardware). > > Several other things were also configured in the wrong structure. For > example the vpif_interface struct is really part of the channel config > and has nothing to do with the subdev. > > What is missing here is that the output doesn't have the same flexibility > as the input when it comes to configuration. It would be good if someone > can pick this up as a follow-up since it's unlikely I'll be working on > that. > > What would also be nice is that by leaving the inputs or outputs for the > second channel empty (NULL) in the configuration you can disable the second > video node, e.g. trying to use it will always result in ENODEV or something. > > This patch series will at least make things more flexible. I forgot to mention that this patch series is on top of this git tree: http://git.linuxtv.org/mhadli/v4l-dvb-davinci_devices.git/shortlog/refs/heads/pull_vpif Regards, Hans -- 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