On Wednesday 17 June 2009 17:02:01 Karicheri, Muralidharan wrote: > >> <snip> > > > >Can you post your latest proposal for the s_bus op? > > > >I propose a few changes: the name of the struct should be something like > >v4l2_bus_settings, the master/slave bit should be renamed to something > >like 'host_is_master', and we should have two widths: subdev_width and > >host_width. > > > >That way the same structure can be used for both host and subdev, unless > >some of the polarities are inverted. In that case you need to make two > >structs, one for host and one for the subdev. > > > >It is possible to add info on inverters to the struct, but unless > > inverters are used a lot more frequently than I expect I am inclined > > not to do that at this time. > > [MK]Today I am planning to send my v3 version of the vpfe capture patch > and also tvp514x patch since Vaibhav is pre-occupied with some other > activities. I have discussed the changes with Vaibhav for this driver. > > For s_bus, I will try if I can send a patch today. BTW, do you expect me > to add one bool for active high, one for active low etc as done in SoC > camera ? Since I remain opposed to autonegotiation, there is IMO no need for this. Regards, Hans -- Hans Verkuil - video4linux developer - sponsored by TANDBERG Telecom -- 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
