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
_______________________________________________
Davinci-linux-open-source mailing list
[email protected]
http://linux.davincidsp.com/mailman/listinfo/davinci-linux-open-source

Reply via email to