Mauro Carvalho Chehab wrote:
>>>     Video device drivers should use the v4l2_fh pointer as their
>>>     file->private_data.

...

>> The struct v4l2_fh is meant to be a part of the driver's own file handle
>> structure. It does not attempt to offer drivers anything directly but
>> generic functionality that depends on file handle specific data such as
>> V4L2 events. E.g.
>>
>> struct my_video_fh {
>>      blah;
>>      struct v4l2_fh fh;
>>      blah;
>> };

An additional comment about this: 

On your comment, you said that "Video device drivers should use the v4l2_fh 
pointer as their file->private_data.".
This can be understood as:
        file->private_data = v4l2_fh;

But a driver following your example, should do, instead:
        file->private_data = my_video_fh;

Please, be sure that you document it clearly when you submit it again.

-- 

Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to