On 2/28/20 4:57 AM, Sergey Senozhatsky wrote: > On (20/02/27 13:36), Hans Verkuil wrote: > [..] >>> other changes, then set ``count`` to 0, ``memory`` to >>> ``V4L2_MEMORY_MMAP`` and ``format.type`` to the buffer type. >>> * - __u32 >>> - - ``reserved``\ [7] >>> + - ``flags`` >>> + - Specifies additional buffer management attributes. >>> + See :ref:`memory-flags`. Old drivers and applications must set it to >>> + zero. >> >> Drop the last sentence, it's not relevant. >> >>> + >>> + * - __u32 >>> + - ``reserved``\ [6] >>> - A place holder for future extensions. Drivers and applications >>> must set the array to zero. >> >> Old drivers and applications still think reserved is [7] and will zero this. > > Just to make sure, does this mean that you also want me to drop the > "Drivers and applications must set the array to zero" sentence? Not for the reserved field, only for the flags field. Regards, Hans > > -ss >
