From: Mauro Carvalho Chehab <mchehab@redhat.com>
To: Sakari Ailus <sakari.ailus@iki.fi>
Cc: "linux-media@vger.kernel.org" <linux-media@vger.kernel.org>,
Laurent Pinchart <laurent.pinchart@ideasonboard.com>,
David Cohen <dacohen@gmail.com>,
Sylwester Nawrocki <snjw23@gmail.com>,
Hans Verkuil <hverkuil@xs4all.nl>,
Andy Shevchenko <andriy.shevchenko@linux.intel.com>,
Tomasz Stanislawski <t.stanislaws@samsung.com>,
tuukkat76@gmail.com, Kamil Debski <k.debski@samsung.com>,
Kim HeungJun <riverful@gmail.com>,
teturtia@gmail.com, pradeep.sawlani@gmail.com
Subject: Re: [GIT PULL FOR v3.4] V4L2 subdev and sensor control changes and SMIA++ driver
Date: Mon, 19 Mar 2012 18:47:28 -0300 [thread overview]
Message-ID: <4F67A970.8090606@redhat.com> (raw)
In-Reply-To: <20120311165650.GA4220@valkosipuli.localdomain>
Em 11-03-2012 13:56, Sakari Ailus escreveu:
> Hi Mauro,
>
> This patchset adds
>
> - Integer menu controls,
> - Selection IOCTL for subdevs,
> - Sensor control improvements,
> - link_validate() media entity and V4L2 subdev pad ops,
> - OMAP 3 ISP driver improvements,
> - SMIA++ sensor driver and
> - Other V4L2 and media improvements (see individual patches)
>
> The previous patchset can be found here:
>
> <URL:http://www.spinics.net/lists/linux-media/msg45052.html>
>
> Compared to the patchset, I've dropped the rm-696 camera board code and will
> submit it through linux-omap later on. Other changes done to address review
> comments have been also done --- see the URL above for details.
>
> The following changes since commit 632fba4d012458fd5fedc678fb9b0f8bc59ceda2:
>
> [media] cx25821: Add a card definition for "No brand" cards that have: subvendor = 0x0000 subdevice = 0x0000 (2012-03-08 12:42:28 -0300)
>
> are available in the git repository at:
> ssh://linuxtv.org/git/sailus/media_tree.git media-for-3.4
>
> Jesper Juhl (1):
> adp1653: Remove unneeded include of version.h
>
> Laurent Pinchart (3):
> omap3isp: Prevent pipelines that contain a crashed entity from starting
> omap3isp: Fix crash caused by subdevs now having a pointer to devnodes
> omap3isp: Fix frame number propagation
>
> Sakari Ailus (37):
> v4l: Introduce integer menu controls
> v4l: Document integer menu controls
> vivi: Add an integer menu test control
> v4l: VIDIOC_SUBDEV_S_SELECTION and VIDIOC_SUBDEV_G_SELECTION IOCTLs
> v4l: vdev_to_v4l2_subdev() should have return type "struct v4l2_subdev *"
> v4l: Check pad number in get try pointer functions
> v4l: Support s_crop and g_crop through s/g_selection
> v4l: Add subdev selections documentation: svg and dia files
> v4l: Add subdev selections documentation
This patch broke docbook compilation:
HTML Documentation/DocBook/media_api.html
warning: failed to load external entity "/home/v4l/v4l/patchwork/Documentation/DocBook/vidioc-subdev-g-selection.xml"
/home/v4l/v4l/patchwork/Documentation/DocBook/dev-subdev.xml:310: parser error : Failure to process entity sub-subdev-g-selection
size configured using &sub-subdev-g-selection; and
^
/home/v4l/v4l/patchwork/Documentation/DocBook/dev-subdev.xml:310: parser error : Entity 'sub-subdev-g-selection' not defined
size configured using &sub-subdev-g-selection; and
^
/home/v4l/v4l/patchwork/Documentation/DocBook/dev-subdev.xml:468: parser error : chunk is not well balanced
^
/home/v4l/v4l/patchwork/Documentation/DocBook/v4l2.xml:476: parser error : Failure to process entity sub-dev-subdev
<section id="subdev"> &sub-dev-subdev; </section>
^
/home/v4l/v4l/patchwork/Documentation/DocBook/v4l2.xml:476: parser error : Entity 'sub-dev-subdev' not defined
<section id="subdev"> &sub-dev-subdev; </section>
^
/usr/bin/xmlto: line 568: 3232 Segmentation fault "/usr/bin/xsltproc" --nonet --xinclude --param passivetex.extensions '1' -o "/tmp/xmlto.J0M0go/media_api.proc" "/tmp/xmlto-xsl.GKa5kH" "/home/v4l/v4l/patchwork/Documentation/DocBook/media_api.xml"
/bin/cp: cannot stat `*.*htm*': No such file or directory
make[1]: *** [Documentation/DocBook/media_api.html] Error 1
make: *** [htmldocs] Error 2
Please fix.
Regards,
Mauro
> v4l: Mark VIDIOC_SUBDEV_G_CROP and VIDIOC_SUBDEV_S_CROP obsolete
> v4l: Image source control class
> v4l: Image processing control class
> v4l: Document raw bayer 4CC codes
> v4l: Add DPCM compressed raw bayer pixel formats
> media: Add link_validate() op to check links to the sink pad
> v4l: Improve sub-device documentation for pad ops
> v4l: Implement v4l2_subdev_link_validate()
> v4l: Allow changing control handler lock
> omap3isp: Support additional in-memory compressed bayer formats
> omap3isp: Move definitions required by board code under include/media.
> omap3: add definition for CONTROL_CAMERA_PHY_CTRL
> omap3isp: Move setting constaints above media_entity_pipeline_start
> omap3isp: Assume media_entity_pipeline_start may fail
> omap3isp: Add lane configuration to platform data
> omap3isp: Collect entities that are part of the pipeline
> omap3isp: Add information on external subdev to struct isp_pipeline
> omap3isp: Introduce isp_video_check_external_subdevs()
> omap3isp: Use external rate instead of vpcfg
> omap3isp: Default link validation for ccp2, csi2, preview and resizer
> omap3isp: Move CCDC link validation to ccdc_link_validate()
> omap3isp: Configure CSI-2 phy based on platform data
> omap3isp: Add resizer data rate configuration to resizer_link_validate
> omap3isp: Find source pad from external entity
> smiapp: Generic SMIA++/SMIA PLL calculator
> smiapp: Add driver
> omap3isp: Prevent crash at module unload
> omap3isp: Handle omap3isp_csi2_reset() errors
>
> Documentation/DocBook/media/Makefile | 4 +-
> Documentation/DocBook/media/v4l/compat.xml | 19 +-
> Documentation/DocBook/media/v4l/controls.xml | 168 ++
> Documentation/DocBook/media/v4l/dev-subdev.xml | 202 ++-
> Documentation/DocBook/media/v4l/pixfmt-srggb10.xml | 2 +-
> .../DocBook/media/v4l/pixfmt-srggb10dpcm8.xml | 29 +
> Documentation/DocBook/media/v4l/pixfmt.xml | 1 +
> .../media/v4l/subdev-image-processing-crop.dia | 614 +++++
> .../media/v4l/subdev-image-processing-crop.svg | 63 +
> .../media/v4l/subdev-image-processing-full.dia | 1588 +++++++++++
> .../media/v4l/subdev-image-processing-full.svg | 163 ++
> ...ubdev-image-processing-scaling-multi-source.dia | 1152 ++++++++
> ...ubdev-image-processing-scaling-multi-source.svg | 116 +
> Documentation/DocBook/media/v4l/v4l2.xml | 20 +-
> .../DocBook/media/v4l/vidioc-g-ext-ctrls.xml | 12 +
> .../DocBook/media/v4l/vidioc-queryctrl.xml | 39 +-
> .../DocBook/media/v4l/vidioc-subdev-g-crop.xml | 9 +-
> .../media/v4l/vidioc-subdev-g-selection.xml | 228 ++
> Documentation/media-framework.txt | 19 +
> Documentation/video4linux/4CCs.txt | 32 +
> Documentation/video4linux/v4l2-framework.txt | 21 +
> arch/arm/mach-omap2/control.h | 1 +
> drivers/media/media-entity.c | 57 +-
> drivers/media/video/Kconfig | 5 +
> drivers/media/video/Makefile | 3 +
> drivers/media/video/adp1653.c | 11 +-
> drivers/media/video/omap3isp/isp.c | 67 +-
> drivers/media/video/omap3isp/isp.h | 11 +-
> drivers/media/video/omap3isp/ispccdc.c | 76 +-
> drivers/media/video/omap3isp/ispccdc.h | 10 -
> drivers/media/video/omap3isp/ispccp2.c | 24 +-
> drivers/media/video/omap3isp/ispcsi2.c | 21 +-
> drivers/media/video/omap3isp/ispcsi2.h | 1 -
> drivers/media/video/omap3isp/ispcsiphy.c | 172 +-
> drivers/media/video/omap3isp/ispcsiphy.h | 25 +-
> drivers/media/video/omap3isp/isppreview.c | 1 +
> drivers/media/video/omap3isp/ispresizer.c | 16 +
> drivers/media/video/omap3isp/ispvideo.c | 341 ++--
> drivers/media/video/omap3isp/ispvideo.h | 5 +
> drivers/media/video/smiapp-pll.c | 419 +++
> drivers/media/video/smiapp-pll.h | 103 +
> drivers/media/video/smiapp/Kconfig | 13 +
> drivers/media/video/smiapp/Makefile | 3 +
> drivers/media/video/smiapp/smiapp-core.c | 2832 ++++++++++++++++++++
> drivers/media/video/smiapp/smiapp-debug.h | 32 +
> drivers/media/video/smiapp/smiapp-limits.c | 132 +
> drivers/media/video/smiapp/smiapp-limits.h | 128 +
> drivers/media/video/smiapp/smiapp-quirk.c | 264 ++
> drivers/media/video/smiapp/smiapp-quirk.h | 72 +
> drivers/media/video/smiapp/smiapp-reg-defs.h | 503 ++++
> drivers/media/video/smiapp/smiapp-reg.h | 122 +
> drivers/media/video/smiapp/smiapp-regs.c | 213 ++
> drivers/media/video/smiapp/smiapp-regs.h | 46 +
> drivers/media/video/smiapp/smiapp.h | 251 ++
> drivers/media/video/v4l2-ctrls.c | 133 +-
> drivers/media/video/v4l2-subdev.c | 143 +-
> drivers/media/video/vivi.c | 26 +-
> include/linux/v4l2-subdev.h | 41 +
> include/linux/videodev2.h | 26 +-
> include/media/media-entity.h | 5 +-
> include/media/omap3isp.h | 29 +
> include/media/smiapp.h | 83 +
> include/media/v4l2-ctrls.h | 15 +-
> include/media/v4l2-subdev.h | 49 +-
> 64 files changed, 10544 insertions(+), 487 deletions(-)
> create mode 100644 Documentation/DocBook/media/v4l/pixfmt-srggb10dpcm8.xml
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-crop.dia
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-crop.svg
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-full.dia
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-full.svg
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-scaling-multi-source.dia
> create mode 100644 Documentation/DocBook/media/v4l/subdev-image-processing-scaling-multi-source.svg
> create mode 100644 Documentation/DocBook/media/v4l/vidioc-subdev-g-selection.xml
> create mode 100644 Documentation/video4linux/4CCs.txt
> create mode 100644 drivers/media/video/smiapp-pll.c
> create mode 100644 drivers/media/video/smiapp-pll.h
> create mode 100644 drivers/media/video/smiapp/Kconfig
> create mode 100644 drivers/media/video/smiapp/Makefile
> create mode 100644 drivers/media/video/smiapp/smiapp-core.c
> create mode 100644 drivers/media/video/smiapp/smiapp-debug.h
> create mode 100644 drivers/media/video/smiapp/smiapp-limits.c
> create mode 100644 drivers/media/video/smiapp/smiapp-limits.h
> create mode 100644 drivers/media/video/smiapp/smiapp-quirk.c
> create mode 100644 drivers/media/video/smiapp/smiapp-quirk.h
> create mode 100644 drivers/media/video/smiapp/smiapp-reg-defs.h
> create mode 100644 drivers/media/video/smiapp/smiapp-reg.h
> create mode 100644 drivers/media/video/smiapp/smiapp-regs.c
> create mode 100644 drivers/media/video/smiapp/smiapp-regs.h
> create mode 100644 drivers/media/video/smiapp/smiapp.h
> create mode 100644 include/media/smiapp.h
>
>
> Kind regards,
>
next prev parent reply other threads:[~2012-03-19 21:47 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-03-11 16:56 [GIT PULL FOR v3.4] V4L2 subdev and sensor control changes and SMIA++ driver Sakari Ailus
2012-03-19 21:47 ` Mauro Carvalho Chehab [this message]
2012-03-19 22:54 ` Sakari Ailus
2012-03-21 13:45 ` Prabhakar Lad
2012-03-21 14:18 ` Sakari Ailus
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=4F67A970.8090606@redhat.com \
--to=mchehab@redhat.com \
--cc=andriy.shevchenko@linux.intel.com \
--cc=dacohen@gmail.com \
--cc=hverkuil@xs4all.nl \
--cc=k.debski@samsung.com \
--cc=laurent.pinchart@ideasonboard.com \
--cc=linux-media@vger.kernel.org \
--cc=pradeep.sawlani@gmail.com \
--cc=riverful@gmail.com \
--cc=sakari.ailus@iki.fi \
--cc=snjw23@gmail.com \
--cc=t.stanislaws@samsung.com \
--cc=teturtia@gmail.com \
--cc=tuukkat76@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).