Hello,
this is an updated version of the changeset extending struct v4l2_mbus_framefmt
with new framesamples field.
Changes since v1:
- Docbook documentation improvements
- drivers that are exposing a sub-device node are modified to initialize
the new struct v4l2_mbus_framefmt member to 0 if they support only
uncompressed formats
Changes since v3:
- dropped patches for m5mols and s5p-fimc drivers which are unchanged;
- in teh subdev drivers use local copy of the format data structure to clear
all structure members which are unused by a driver, in way that it don't break
after new fields are added to struct v4l2_mbus_framefmt;
The omap3isp changes are only compile tested. I'd like to ask someone who
has access to the hardware to test the patch.
The proposed semantics for the framesamples parameter is as follows:
- the value is propagated at video pipeline entities where 'code' indicates
compressed format;
- the subdevs adjust the value if needed;
- although currently there is only one compressed data format at the media
bus - V4L2_MBUS_FMT_JPEG_1X8 which corresponds to V4L2_PIX_FMT_JPEG and
one sample at the media bus equals to one byte in memory, it is assumed
that the host knows exactly what is framesamples/sizeimage ratio and it will
validate framesamples/sizeimage values before starting streaming;
- the host will query internally a relevant subdev to properly handle 'sizeimage'
at the VIDIOC_TRY/S_FMT ioctl
The initial RFC can be found here:
http://www.mail-archive.com/linux-media@xxxxxxxxxxxxxxx/msg39321.html
Sylwester Nawrocki (2):
v4l: Add new framesamples field to struct v4l2_mbus_framefmt
v4l: Update subdev drivers to handle framesamples parameter
Documentation/DocBook/media/v4l/dev-subdev.xml | 10 +++++-
Documentation/DocBook/media/v4l/subdev-formats.xml | 9 +++++-
drivers/media/video/noon010pc30.c | 3 ++
drivers/media/video/omap3isp/ispccdc.c | 12 +++++--
drivers/media/video/omap3isp/ispccp2.c | 31 +++++++++++--------
drivers/media/video/omap3isp/ispcsi2.c | 12 +++++--
drivers/media/video/omap3isp/isppreview.c | 23 +++++++++-----
drivers/media/video/omap3isp/ispresizer.c | 19 +++++++++---
drivers/media/video/s5k6aa.c | 2 +
include/linux/v4l2-mediabus.h | 4 ++-
10 files changed, 87 insertions(+), 38 deletions(-)
--
1.7.8
--
Best regards,
Sylwester Nawrocki
--
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
