- 30 Sep, 2022 18 commits
-
-
kyrie wu authored
Add jpeg decoding worker to ensure that three HWs run in parallel in MT8195. Signed-off-by:
kyrie wu <kyrie.wu@mediatek.com> Signed-off-by:
irui wang <irui.wang@mediatek.com> Signed-off-by:
Hans Verkuil <hverkuil-cisco@xs4all.nl>
-
kyrie wu authored
Generalizes jpegdec timeout func interfaces to handle HW timeout. Signed-off-by:
-
kyrie wu authored
support jpegdec multi-hardware includes HW0/HW1/HW2. Signed-off-by:
-
kyrie wu authored
mtk jpeg decoder is built as a module, export some functions to make them visible by other modules. Signed-off-by:
-
kyrie wu authored
Add mediatek,mt8195-jpgdec compatible to binding document. Signed-off-by:
Rob Herring <robh@kernel.org> Acked-by:
Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org> Signed-off-by:
-
kyrie wu authored
Add stop cmd interface for jpgenc to stop stream Signed-off-by:
-
kyrie wu authored
There are two HWs in mt8195. Since the two HWs run in parallel, it is necessary to reorder the output images to ensure that the order is consistent with the input images. Signed-off-by:
-
kyrie wu authored
Add jpeg encoding worker to ensure that two HWs run in parallel in MT8195. Signed-off-by:
-
kyrie wu authored
Generalizes jpegenc timeout func interfaces to handle HW timeout. Signed-off-by:
-
kyrie wu authored
support jpeg encode multi-hardware includes HW0 and HW1. Signed-off-by:
kernel test robot <lkp@intel.com> Signed-off-by:
-
kyrie wu authored
mtk jpeg encoder is built as a module, export some functions to make them visible by other modules. Signed-off-by:
-
kyrie wu authored
Add mediatek,mt8195-jpgenc compatible to binding document. Signed-off-by:
-
Ming Qian authored
There is a hardware bug that the interrupt STMBUF_HALF may be triggered after or when disable interrupt. It may led to unexpected kernel panic. And interrupt STMBUF_HALF and STMBUF_RTND have no other effect. So disable them and the unused interrupts. meanwhile clear the interrupt status when disable interrupt. Signed-off-by:
Ming Qian <ming.qian@nxp.com> Reviewed-by:
Mirela Rabulea <mirela.rabulea@nxp.com> Signed-off-by:
-
Ming Qian authored
In dynamic resolution change, streamoff and streamon on the capture queue may be called, the V4L2_DEC_CMD_STOP cmd may be called before driver parsed the jpeg header. don't clear the stop state in streamoff of handling the dynamic resolution change, otherwise the drain may not complete. Fixes: 4911c5ac ("media: imx-jpeg: Implement drain using v4l2-mem2mem helpers") Signed-off-by:
-
Jianhua Lin authored
Add compatible for MT8188 jpeg encoder. Signed-off-by:
Jianhua Lin <jianhua.lin@mediatek.com> Acked-by:
-
Jianhua Lin authored
Add compatible for MT8188 jpeg decoder. Signed-off-by:
-
Ming Qian authored
Add a timeout mechanism for each frame. If the frame can't be decoded or encoded, driver can cancel it to avoid hang. Fixes: 2db16c6e ("media: imx-jpeg: Add V4L2 driver for i.MX8 JPEG Encoder/Decoder") Signed-off-by:
-
Jason Wang authored
The `dma_alloc_coherent()' already zeroes out memory for us, so we don't need the redundant memset(). Signed-off-by:
Jason Wang <wangborong@cdjrlc.com> Signed-off-by:
-
- 27 Sep, 2022 6 commits
-
-
Mauro Carvalho Chehab authored
* fixes: media: rkvdec: Disable H.264 error detection media: mediatek: vcodec: Drop platform_get_resource(IORESOURCE_IRQ) media: dvb_vb2: fix possible out of bound access media: v4l2-ioctl.c: fix incorrect error path media: v4l2-compat-ioctl32.c: zero buffer passed to v4l2_compat_get_array_args() media: uvcvideo: Fix InterfaceProtocol for Quanta camera
-
Nicolas Dufresne authored
Quite often, the HW get stuck in error condition if a stream error was detected. As documented, the HW should stop immediately and self reset. There is likely a problem or a miss-understanding of the self reset mechanism, as unless we make a long pause, the next command will then report an error even if there is no error in it. Disabling error detection fixes the issue, and let the decoder continue after an error. This patch is safe for backport into older kernels. Fixes: cd33c830 ("media: rkvdec: Add the rkvdec driver") Signed-off-by:
Nicolas Dufresne <nicolas.dufresne@collabora.com> Reviewed-by:
Brian Norris <briannorris@chromium.org> Tested-by:
Ezequiel Garcia <ezequiel@vanguardiasur.com.ar> Signed-off-by:
Mauro Carvalho Chehab <mchehab@kernel.org>
-
Nícolas F. R. A. Prado authored
Commit a1a2b712 ("of/platform: Drop static setup of IRQ resource from DT core") removed support for calling platform_get_resource(..., IORESOURCE_IRQ, ...) on DT-based drivers, but the probe() function of mtk-vcodec's encoder was still making use of it. This caused the encoder driver to fail probe. Since the platform_get_resource() call was only being used to check for the presence of the interrupt (its returned resource wasn't even used) and platform_get_irq() was already being used to get the IRQ, simply drop the use of platform_get_resource(IORESOURCE_IRQ) and handle the failure of platform_get_irq(), to get the driver probing again. [hverkuil: drop unused struct resource *res] Fixes: a1a2b712 ("of/platform: Drop static setup of IRQ resource from DT core") Signed-off-by:
Nícolas F. R. A. Prado <nfraprado@collabora.com> Reviewed-by:
AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com> Signed-off-by:
-
Hangyu Hua authored
vb2_core_qbuf and vb2_core_querybuf don't check the range of b->index controlled by the user. Fix this by adding range checking code before using them. Fixes: 57868acc ("media: videobuf2: Add new uAPI for DVB streaming I/O") Signed-off-by:
Hangyu Hua <hbh25y@gmail.com> Reviewed-by:
Sergey Senozhatsky <senozhatsky@chromium.org> Signed-off-by:
-
Hans Verkuil authored
If allocating array_buf fails, or copying data from userspace into that buffer fails, then just free memory and return the error. Don't attempt to call video_put_user() since there is no point, and it would copy back data on error even if INFO_FL_ALWAYS_COPY wasn't set. So if writing the array back to userspace fails, then don't go to out_array_args, instead just continue with the regular code that just returns the error unless 'always_copy' is set. Update the VIDIOC_G/S/TRY_EXT_CTRLS ioctls to set the ALWAYS_COPY flag since they now need it. Before this worked due to this buggy code, but now that that is fixed these ioctls need to set this flag explicitly. Signed-off-by:
-
Hans Verkuil authored
The v4l2_compat_get_array_args() function can leave uninitialized memory in the buffer it is passed. So zero it before copying array elements from userspace into the buffer. Signed-off-by:
Laurent Pinchart <laurent.pinchart@ideasonboard.com> Signed-off-by:
-
- 24 Sep, 2022 16 commits
-
-
Paul Kocialkowski authored
The buffer structure is a top-level definition, put it in the main header to keep things tidy. No functional change intended. Signed-off-by:
Paul Kocialkowski <paul.kocialkowski@bootlin.com> Reviewed-by:
Maxime Ripard <maxime@cerno.tech> Signed-off-by:
-
Paul Kocialkowski authored
Introduce some helpers for buffer and general video configuration. Signed-off-by:
Jernej Skrabec <jernej.skrabec@gmail.com> Signed-off-by:
-
Paul Kocialkowski authored
In order to keep the power use count fields balanced when link changes happen between v4l2_pipeline_pm_get/set calls (in open/close), the link_notify media operation callback needs to be registered. Signed-off-by:
-
Paul Kocialkowski authored
The driver does not expose controls directly and thus does not need a controls handler for its own use. Controls attached to subdevs used to be exposed that way, however this can easily lead to issue when multiple subdevs attached to the same v4l2 device expose the same controls. Subdev controls should be set through each individual subdev node instead. Signed-off-by:
-
Paul Kocialkowski authored
There is no particular need to register the media device in the subdev notify complete callback. Register it in the v4l2 code instead where it's more in-context. Signed-off-by:
-
Paul Kocialkowski authored
The video structure is part of the main csi device structure, so pass pointers to that top-level structure directly. This makes it easier to navigate and access other elements. No functional change intended. Signed-off-by:
-
Paul Kocialkowski authored
Some code cleanups, renames, variable lowerings and moving things around for better organization. No functional change intended. Signed-off-by:
-
Paul Kocialkowski authored
Various cosmetic improvements to the v4l2 registration code, with renames, lowerings, etc. The cleanup function is moved down after setup. No functional change intended. Signed-off-by:
-
Paul Kocialkowski authored
Update the option title and help, group related options together, add dependency on VIDEO_DEV since the driver uses it and update the description. Signed-off-by:
-
Paul Kocialkowski authored
Wrap the clock and reset preparation into runtime pm functions for better organization of the code. Also fix the clock and reset enable order to first deassert reset, as recommended in Allwinner literature. Make the driver depend on PM while at it since runtime pm is mandatory for the driver to work. Signed-off-by:
-
Paul Kocialkowski authored
Introduce a proper variant structure with the module clock rate instead of hardcoding it with a manual check on the compatible. Signed-off-by:
-
Paul Kocialkowski authored
In some situations the default rate of the module clock is not the required one for operation (for example when reconfiguring the clock tree to use a different parent). As a result, always set the correct rate for the clock (and take care of cleanup). Signed-off-by:
-
Paul Kocialkowski authored
Various renames, variables lowering and other cosmetic changes in the platform-support code. No functional change intended. Signed-off-by:
-
Paul Kocialkowski authored
Merge contents of structs sun6i_csi and sun6i_csi_dev into a main sun6i_csi_device structure holding a sun6i_csi_v4l2 struct for things related to v4l2, as well as the already-existing sun6i_csi_video and sun6i_csi_config which are left unchanged. This mostly simplifies accessing stuff by having a single main structure accessible to every part of the code instead of a private definition. Also solve some kerneldoc warnings by describing return codes while at it. No functional change is intended in this commit, variables are just moved around (cosmetics). Signed-off-by:
-
Paul Kocialkowski authored
Add proper defines for driver name and description instead of MODULE_NAME and hardcoding (cosmetics). Also rework the description while at it to mention the hardware generation that the driver supports and remove the video capture mentions since it applies to the whole media device. Signed-off-by:
-
Paul Kocialkowski authored
The driver relies on the reset controller API to work, so add RESET_CONTROLLER as one of its Kconfig dependencies. Fixes: 50e76151 ("media: platform: Add Cedrus VPU decoder driver") Signed-off-by:
-
