Skip to contentSkip to conversation

v4l2_ctrl_new_std_volatile()?

DISCUSSION Sakari Ailus 2 participants 2 messages Jun 10, 2011 Media (V4L/DVB)

lore
Share & tools
lore
#0 lore raw

Hi Hans,

The v4l2_ctrl_new_std() doesn't allow setting the is_volatile bit in the bit field. The adp1653 driver needs that for the faults control.

Would you prefer just setting the bit in the driver, or, as Laurent suggested, a new v4l2_ctrl_new_std_volatile() function which would mark the control as volatile by itself?

Regards,

Signature · 2 lines
Sakari Ailus
[email protected]
#1 lore raw
Quoted 8 lines · On Friday, June 10, 2011 10:47:09 Sakari Ailus wrote:

Hi Hans,

The v4l2_ctrl_new_std() doesn't allow setting the is_volatile bit in the bit field. The adp1653 driver needs that for the faults control.

Would you prefer just setting the bit in the driver, or, as Laurent suggested, a new v4l2_ctrl_new_std_volatile() function which would mark the control as volatile by itself?

Just set the bit in the driver.

If this will start happening a lot, then we can consider adding a special function for this, but volatile controls are still pretty rare and I do not think it warrants a special function just for that.

Regards,

Hans