Dear Laurent Pinchart:
Thanks for your patches! Unfortunately the Media CI robot detected some
issues:
# Test virtme32:test-virtme
Error running test. Final Summary not found
# Test
checkpatch:./0006-media-v4l2-Add-Renesas-Camera-Receiver-Unit-pixel-fo.patch
checkpatch
WARNING: added, moved or
On Mon Jun 16 20:57:21 2025 +0200, Niklas Söderlund wrote:
> Depending on if the capture session deals with fields or whole frames
> interrupts can be generated at an end of field, or end of frame event.
> The interrupt mask is setup to generate an interrupt on one of the two
> events depending on
On Fri May 30 16:41:47 2025 +0100, Lad Prabhakar wrote:
> The FCPVD block on the RZ/V2N SoC is identical to the one found on the
> RZ/G2L SoC.
>
> No driver changes are required, as `renesas,fcpv` will be used as a
> fallback compatible string on the RZ/V2N SoC.
>
> Signed-off-by: Lad Prabhakar
On Mon Jun 16 09:18:50 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Mon Jun 16 09:18:51 2025 +0200, Bartosz Golaszewski wrote:
> GPIO core already handles missing direction_input/output() callbacks.
> The ones in this driver also effectively return magic numbers which is
> not optimal either (the number accidentally corresponds with -EPERM which
> is different f
On Thu Jun 12 19:59:01 2025 +0200, Niklas Söderlund wrote:
> The helper function to deal with calculating the link speed is designed
> in such a way that it returns the correct type bps (bits per second) for
> D-PHY and sps (symbols per second) for C-PHY. And for historical reasons
> the function
On Thu May 22 13:56:43 2025 -0400, Frank Li wrote:
> dma_set_mask_and_coherent() never return failure when mask bigger than
> 32bit.
>
> See commit f7ae20f2fc4e ("docs: dma: correct dma_set_mask() sample code")
>
> So remove return value check for dma_set_mask_and_coherent().
>
> Reviewed-by: La
On Sat Mar 29 15:26:42 2025 +0100, Christophe JAILLET wrote:
> 'struct i2c_algorithm' is not modified in this driver.
>
> Constifying this structure moves some data to a read-only section, so
> increase overall security, especially when the structure holds some
> function pointers.
>
> On a x86_6
On Wed May 21 13:34:03 2025 -0400, Frank Li wrote:
> Add compatible strings "nxp,imx95-jpgdec" and "nxp,imx95-jpgenc", which
> are backward compatible with "nxp,imx8qxp-jpgdec" and
> "nxp,imx8qxp-jpegenc". i.MX95 just need one power domain which combine
> wrap and all slots together. Reduce minItem
On Fri May 23 17:43:59 2025 +0200, Jernej Skrabec wrote:
> If VPU supports untiled output, it actually supports several different
> YUV 4:2:0 layouts, namely NV12, NV21, YUV420 and YVU420.
>
> Add support for all of them.
>
> Signed-off-by: Jernej Skrabec
> Reviewed-by: Paul Kocialkowski
> Sign
On Fri Jun 13 17:34:24 2025 +0200, Niklas Söderlund wrote:
> The link setup callback once acted on each VIN instance, and expected to
> be called once for each VIN instance. This have changed as the driver
> grew support for later hardware generations and the callback is now
> expected to setup li
On Thu May 22 13:56:39 2025 -0400, Frank Li wrote:
> Add binding documentation for i.MX8QXP and i.MX8QM ISI. The clock-names,
> power-domains, and ports differ significantly from the existing
> nxp,imx8-isi.yaml. Create a new file to avoid complex if-else branches.
>
> Add new file to MAINTAINERS.
On Wed May 28 23:22:14 2025 +0300, Dan Carpenter wrote:
> This sd_init() function reads the firmware. The firmware data holds a
> series of records and the function reads each record and sends the data
> to the device. The request_ihex_firmware() function
> calls ihex_validate_fw() which ensures
On Mon Jun 2 18:21:57 2025 +, Ricardo Ribalda wrote:
> HP Webcam HD 2300 does not seem to flip the FID bit according to spec.
>
> Device Descriptor:
> bLength18
> bDescriptorType 1
> bcdUSB 2.00
> bDeviceClass 239 Miscellaneous Device
>
On Mon Jun 16 09:18:56 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Fri May 23 17:14:30 2025 +0200, Stefan Klug wrote:
> Properly handle the return of rkisp1_params_init_vb2_queue(). It is very
> unlikely that this ever fails without code changes but should be handled
> anyways.
>
> While at it rename the error label for easier extension in the upcoming
> patch
On Thu Jun 12 19:59:04 2025 +0200, Niklas Söderlund wrote:
> Add D-PHY support for V4H in addition to the already supported C-PHY.
> The common start-up procedure for C-PHY and D-PHY is shared, only PHY
> setup differ. Extend the V4H setup with D-PHY support as documented in
> the datasheet (Rev.1
On Mon Jun 9 16:35:57 2025 +0200, Hans de Goede wrote:
> I'm moving all my kernel work over to using my kernel.org email address.
> Update .mailmap and MAINTAINER entries still using hdego...@redhat.com.
>
> Signed-off-by: Hans de Goede
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Ha
On Wed Jun 11 19:37:13 2025 +0100, Dave Stevenson wrote:
> Adds register setups for running the CSI lanes at 972Mbit/s,
> which allows 1080P50 UYVY down 2 lanes.
>
> Signed-off-by: Dave Stevenson
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Hans Verkuil
drivers/media/i2c/tc358743.c
On Fri Apr 11 17:36:00 2025 -0400, Frank Li wrote:
> Convert fsl-vdoa.txt to yaml format.
>
> Additional changes:
> - Add irq.h and imx6qdl-clock.h in example.
>
> Signed-off-by: Frank Li
> Reviewed-by: Rob Herring (Arm)
> Signed-off-by: Nicolas Dufresne
> Signed-off-by: Hans Verkuil
Patch c
On Thu May 15 16:11:09 2025 +0800, Kuan-Wei Chiu wrote:
> Refactor parity calculations to use the standard parity8() helper.
> This change eliminates redundant implementations.
>
> Co-developed-by: Yu-Chun Lin
> Signed-off-by: Yu-Chun Lin
> Signed-off-by: Kuan-Wei Chiu
> Reviewed-by: Hans Verku
On Thu May 22 10:09:54 2025 +0200, Wolfram Sang wrote:
> This driver passes the length of an i2c_msg directly to
> usb_control_msg(). If the message is now a read and of length 0, it
> violates the USB protocol and a warning will be printed. Enable the
> I2C_AQ_NO_ZERO_LEN_READ quirk for this adapt
On Fri Aug 23 17:02:17 2024 +0800, Jinjie Ruan wrote:
> Avoids the need for manual cleanup of_node_put() in early exits
> from the loop.
>
> Signed-off-by: Jinjie Ruan
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Hans Verkuil
drivers/media/platform/xilinx/xilinx-vipp.c | 7 ++-
On Sun Jun 15 21:33:53 2025 -0400, Alex Guo wrote:
> In w7090p_tuner_write_serpar, msg is controlled by user. When msg[0].buf is
> null and msg[0].len is zero, former checks on msg[0].buf would be passed. If
> accessing msg[0].buf[2] without sanity check, null pointer deref would
> happen. We ad
On Fri Aug 23 17:02:16 2024 +0800, Jinjie Ruan wrote:
> Avoids the need for manual cleanup of_node_put() in early exits
> from the loop.
>
> Signed-off-by: Jinjie Ruan
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Hans Verkuil
.../platform/st/sti/c8sectpfe/c8sectpfe-core.c | 34
On Sat Jun 14 21:37:23 2025 +0100, Dr. David Alan Gilbert wrote:
> The functions:
> vpu_color_get_default()
> vpu_color_check_full_range()
> vpu_color_check_primaries()
> vpu_color_check_transfers()
> vpu_color_check_matrix()
>
> have been unused since 2022's
> commit 1925665ef403 ("medi
On Mon Jun 16 01:23:59 2025 +0100, Dr. David Alan Gilbert wrote:
> The functions:
> v4l2_jpeg_parse_huffman_tables()
> v4l2_jpeg_parse_quantization_tables()
> v4l2_jpeg_parse_scan_header()
> and
> v4l2_jpeg_parse_frame_header()
>
> were added in 2020 by
> commit 50733b5b9102 ("media: add v
On Wed May 28 14:32:04 2025 +0300, Andy Shevchenko wrote:
> Besides list_entry_is_past_end() is the same as list_entry_is_head(),
> it's implemented in the list namespace. Fix both of the issue by replacing
> the custom version with list_entry_is_head() calls.
>
> Signed-off-by: Andy Shevchenko
>
On Tue Oct 29 14:47:05 2024 +0200, Tomi Valkeinen wrote:
> The driver checks if "vq->max_num_buffers + *nbuffers < 3", but
> vq->max_num_buffers is (by default) 32, so the check is never true. Nor
> does the check make sense.
>
> The original code in the BSP kernel was "vq->num_buffers + *nbuffers
On Mon Jun 16 20:57:22 2025 +0200, Niklas Söderlund wrote:
> Enable the VSYNC Rising Edge Detection interrupt and generate a
> FRAME_SYNC event form it. The interrupt is available on all supported
> models of the VIN (Gen2, Gen3 and Gen4).
>
> Signed-off-by: Niklas Söderlund
> Reviewed-by: Laur
On Fri May 9 09:58:09 2025 +0200, Krzysztof HaÅasa wrote:
> The error messages in the mxc_isi_crossbar_enable_streams() and
> mxc_isi_crossbar_disable_streams() functions are similar, with a single
> word difference between them. The word is moved out of the format string
> to a separate dev_err()
On Thu May 22 13:56:41 2025 -0400, Frank Li wrote:
> Preserve clarity by removing the unused 'offset' field in struct
> mxc_isi_reg, as it duplicates information already indicated by the mask
> and remains unused.
>
> Improve readability by replacing hex value masks with the BIT() macro.
>
> No f
On Tue Jun 10 17:55:27 2025 +0530, Jai Luthra wrote:
> RKISP supports a basic Wide Dynamic Range (WDR) module since the first
> iteration (v1.0) of the ISP. Add support for enabling and configuring it
> using extensible parameters.
>
> Also, to ease programming, switch to using macro variables for
On Mon Jun 16 00:58:01 2025 +0100, Dr. David Alan Gilbert wrote:
> vpdma_update_dma_addr() was added in 2016 as part of
> commit 2f88703a0bfd ("[media] media: ti-vpe: vpdma: Add multi-instance and
> multi-client support")
>
> but has remained unused.
>
> Remove it.
>
> I did see that there was a
On Mon Jun 2 11:27:51 2025 +0200, Martin Tůma wrote:
> Enumerate only the available (as given by the sysfs setup - our "EDID
> replacement") timings, not all theoretically possible. This is the video
> outputs part of the previous inputs patch that somehow got "lost in
> translation".
>
> Signed-
On Wed Jun 11 19:37:14 2025 +0100, Dave Stevenson wrote:
> The existing fixed value of 16 worked for UYVY 720P60 over
> 2 lanes at 594MHz, or UYVY 1080P60 over 4 lanes. (RGB888
> 1080P60 needs 6 lanes at 594MHz).
> It doesn't allow for lower resolutions to work as the FIFO
> underflows.
>
> 374 is
On Fri May 9 18:24:14 2025 +, Ricardo Ribalda wrote:
> Do uvc_pm_get before we call uvc_queue_streamon. Although the current
> code is correct, uvc_ioctl_streamon is allways called after uvc_pm_get,
> this change makes the code more resiliant to future changes.
>
> Suggested-by: Laurent Pincha
On Mon Jun 16 18:30:38 2025 +0200, Jacopo Mondi wrote:
> To detect leaks of display lists, store in the display list manager the
> number of allocated display lists when the manager is created and verify
> that when the display manager is reset the same number of lists is
> available in the free li
On Thu Jun 12 19:59:03 2025 +0200, Niklas Söderlund wrote:
> Prepare for adding D-PHY support to the driver by first updating the
> generic startup procedure that covers both C-PHY and D-PHY operations.
> The starting procedure where updated in later versions of the datasheet.
>
> Most of the con
On Wed May 21 09:54:07 2025 +0800, Ming Qian wrote:
> Applications may set data_offset when it refers to an output queue. So
> driver need to account for it when getting the start address of input
> image in the plane.
>
> Meanwhile the mxc-jpeg codec requires the address (plane address +
> data_o
On Fri Jun 13 09:15:57 2025 +0200, Krzysztof Kozlowski wrote:
> It is very useful to find driver implementing compatibles with `git grep
> compatible`, so driver should not use defines for that string, even if
> this means string will be effectively duplicated.
>
> Signed-off-by: Krzysztof Kozlows
On Sun May 25 10:11:44 2025 +0200, Christophe JAILLET wrote:
> The 'used' and 'new' bitmaps are local to this function, so there is no
> need to use atomic access because concurrency can not happen.
>
> Use the non-atomic __set_bit() to save a few cycles.
>
> Signed-off-by: Christophe JAILLET
>
On Sun Jun 15 21:32:31 2025 -0400, Alex Guo wrote:
> In dib7090p_rw_on_apb, msg is controlled by user. When msg[0].buf is null and
> msg[0].len is zero, former checks on msg[0].buf would be passed. If accessing
> msg[0].buf[2] without sanity check, null pointer deref would happen. We add
> check on
On Fri May 9 18:24:13 2025 +, Ricardo Ribalda wrote:
> Today uvc_ctrl_set_handle() covers two use-uses: setting the handle and
> clearing the handle. The only common code between the two cases is the
> lockdep_assert_held.
>
> The code looks cleaner if we split these two usecases in two functi
On Mon May 12 10:01:36 2025 +0800, Ming Qian wrote:
> For format H264 and HEVC, the firmware can report the parsed profile idc
> and level idc to driver, these information may be useful.
> Implement the H264 and HEVC profile and level control to report them.
>
> Signed-off-by: Ming Qian
> Signed-
On Thu May 22 13:56:42 2025 -0400, Frank Li wrote:
> Use devm_clk_bulk_get_all() helper to simplify clock handle code.
>
> No functional changes intended.
>
> Signed-off-by: Frank Li
> Reviewed-by: Laurent Pinchart
> Link: https://lore.kernel.org/r/20250522-8qxp_camera-v5-4-d4be869fd...@nxp.com
On Tue Jun 17 09:23:28 2025 +0200, Jacopo Mondi wrote:
> Add support for VSPX, a specialized version of the VSP2 that
> transfers data to the ISP. The VSPX is composed of two RPF units
> to read data from external memory and an IIF instance that performs
> transfer towards the ISP.
>
> The VSPX is
On Thu Feb 27 15:44:51 2025 +0800, Haoxiang Li wrote:
> Add video_device_release() in label 'err_m2m' to release the memory
> allocated by video_device_alloc() and prevent potential memory leaks.
> Remove the reduntant code in label 'err_m2m'.
>
> Fixes: a8ef0488cc59 ("media: imx: add csc/scaler m
On Mon Jun 16 09:18:55 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Fri May 9 02:17:50 2025 +0100, Dr. David Alan Gilbert wrote:
> saa7164_cmd_signal() last use was removed in 2009 by
> commit 39e469ab6dee ("V4L/DVB (12940): SAA7164: IRQ / message timeout
> related change")
>
> saa7164_buffer_display() was added in 2010 by
> commit add3f580a434 ("[media] saa716
On Thu May 8 17:00:15 2025 -0400, Nicolas Dufresne wrote:
> On errors, the rkvdec chip self resets. This can clear the addresses
> programmed in the iommu. This case is signaled by the
> RKVDEC_SOFTRESET_RDY status bit.
>
> Since the iommu framework does not have a restore functionality, and
> as
On Thu Mar 13 13:30:34 2025 +0200, Andy Shevchenko wrote:
> Replace the check for firmware registered I²C devices as the firmware node
> independently on type should be retrieved via dev_fwnode().
>
> Signed-off-by: Andy Shevchenko
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Hans V
On Thu May 22 13:56:48 2025 -0400, Frank Li wrote:
> Add support for i.MX8QXP, which has a dedicated control and status register
> (CSR) space. Enable obtaining the second register space and initializing
> PHY and link settings accordingly.
>
> Reviewed-by: Laurent Pinchart
> Signed-off-by: Frank
On Fri Jun 13 17:34:27 2025 +0200, Niklas Söderlund wrote:
> The R-Car VIN driver is needlessly complex and uses more then one
> v4l-async notifier to attach to all its subdevices. Prepare for unifying
> them by moving rvin_parallel_parse_of() to where it needs to be when
> they are unified.
>
>
On Fri May 23 14:28:41 2025 +, Ricardo Ribalda wrote:
> Now we are replacing most of the error codes with -ENODEV.
> Instead, Populate the error code from the functions called by
> uvc_probe().
>
> Take this opportunity to replace a generic error code from
> uvc_scan_device() into something mo
On Fri Jun 13 17:34:26 2025 +0200, Niklas Söderlund wrote:
> Prepare to move Gen2 and earlier models to media controller by
> generating a unique VIN group id for each VIN instance. On Gen3 and Gen4
> it is important to have a specific id in the group as media graph routes
> depend on this. On Gen
On Mon Jun 16 09:18:53 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Fri Jun 13 17:34:28 2025 +0200, Niklas Söderlund wrote:
> Use the __free(fwnode_handle) hooks to free the endpoints when the
> function exits to simplify the error paths and make the intent more
> clear.
>
> While at it correct the error message when failing to parse an endpoint
> to report th
On Fri Jun 13 17:34:33 2025 +0200, Niklas Söderlund wrote:
> Complete the conversion from soc_camera to a full fledge media
> controller enabled devices for all supported generations of the device.
> All work is already done as this is already supported on Gen3, and
> later.
>
> All that is missi
On Fri May 30 16:41:48 2025 +0100, Lad Prabhakar wrote:
> The VSPD block on the RZ/V2N SoC is identical to the one found on the
> RZ/G2L SoC.
>
> No driver changes are required, as `renesas,r9a07g044-vsp2` will be used
> as a fallback compatible string on the RZ/V2N SoC.
>
> Signed-off-by: Lad Pr
On Wed Jun 11 19:37:15 2025 +0100, Dave Stevenson wrote:
> The probe for the TC358743 reads the CHIPID register from
> the device and compares it to the expected value of 0.
> If the I2C request fails then that also returns 0, so
> the driver loads thinking that the device is there.
>
> Generally
On Thu May 22 13:56:47 2025 -0400, Guoniu.zhou wrote:
> Introduce `imx8mq_plat_data` along with enable/disable callback operations
> to facilitate support for new chips. No functional changes.
>
> Signed-off-by: Guoniu.zhou
> Reviewed-by: Laurent Pinchart
> Signed-off-by: Frank Li
> Link: https
On Fri Jun 13 17:34:32 2025 +0200, Niklas Söderlund wrote:
> Before moving Gen2 to media controller simplify the creation of controls
> by not exposing the sub-device controls on the video device. This could
> be done while enabling media controller but doing it separately reduces
> the changes ne
On Thu May 22 13:56:45 2025 -0400, Robert Chiras wrote:
> Add compatibles and platform data for i.MX8QM and i.MX8QXP platforms.
> i.MX8QM's IER register layout is difference with i.MX8QXP.
>
> Signed-off-by: Robert Chiras
> Reviewed-by: Laurent Pinchart
> Signed-off-by: Frank Li
> Link: https:/
On Thu May 22 13:56:44 2025 -0400, Frank Li wrote:
> Use dev_err_probe() simplify code. No functional changes intended.
>
> Reviewed-by: Laurent Pinchart
> Signed-off-by: Frank Li
> Link: https://lore.kernel.org/r/20250522-8qxp_camera-v5-6-d4be869fd...@nxp.com
> Signed-off-by: Laurent Pinchart
On Fri Jun 13 17:34:30 2025 +0200, Niklas Söderlund wrote:
> Prepare for Gen2 media graph support by always initializing a media pad
> for the VIN device.
>
> Signed-off-by: Niklas Söderlund
> Reviewed-by: Laurent Pinchart
> Link:
> https://lore.kernel.org/r/20250613153434.2001800-10-niklas.s
On Fri May 23 17:14:31 2025 +0200, Stefan Klug wrote:
> Add a RKISP1_CID_SUPPORTED_PARAMS_BLOCKS V4L2 control to be able to
> query the parameters blocks supported by the current kernel on the
> current hardware from user space.
>
> Signed-off-by: Stefan Klug
> Reviewed-by: Paul Elder
> Reviewed
On Mon Jun 16 09:18:52 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Mon Feb 17 16:46:54 2025 -0500, Nicolas Dufresne wrote:
> The desired clock frequency was correctly set to 400MHz in the device tree
> but was lowered by the driver to 300MHz breaking 4K 60Hz content playback.
> Fix the issue by removing the driver call to clk_set_rate(), which reduce
> the amou
On Thu Jun 12 19:59:02 2025 +0200, Niklas Söderlund wrote:
> Each AFE lane has a set of control and offset registers. The registers
> themself are undocumented but later datasheets program more of them with
> magic values. Before exploding the driver with more defines to name them
> all rework the
On Thu May 22 13:56:46 2025 -0400, Robert Chiras wrote:
> Add compatible strings for i.MX8QM/i.MX8QXP platform. Remove
> fsl,mipi-phy-gpr from required properties and add new reg space, since
> i.MX8QM and i.MX8QXP use dedicate control and status register(csr) space.
>
> Keep the same restriction
On Fri Aug 23 17:02:15 2024 +0800, Jinjie Ruan wrote:
> Avoids the need for manual cleanup of_node_put() in early exits
> from the loop.
>
> Signed-off-by: Jinjie Ruan
> Signed-off-by: Hans Verkuil
Patch committed.
Thanks,
Hans Verkuil
.../media/platform/samsung/exynos4-is/media-dev.c | 27
On Mon Apr 28 20:16:50 2025 +0200, Ludwig Disterhof wrote:
> When an program is streaming (ffplay) and another program (qv4l2)
> changes the TV standard from NTSC to PAL, the kernel crashes due to trying
> to copy to unmapped memory.
>
> Changing from NTSC to PAL increases the resolution in the us
On Tue Jun 10 21:41:07 2025 +0900, Youngjun Lee wrote:
> The buffer length check before calling uvc_parse_format() only ensured
> that the buffer has at least 3 bytes (buflen > 2), buf the function
> accesses buffer[3], requiring at least 4 bytes.
>
> This can lead to an out-of-bounds read if the
On Fri Jun 13 17:34:23 2025 +0200, Niklas Söderlund wrote:
> When the transition of Gen2 to use groups are complete the platform
> specific information can be retrieved from the group instead of being
> duplicated in each VIN's private data structure.
>
> Prepare for this by already adding the in
On Mon Jun 16 20:57:20 2025 +0200, Niklas Söderlund wrote:
> The call sites using the interrupt helper functions have all been
> reworked to only one for each. Fold each of them into the only call
> sites left.
>
> This fixes a possible interrupt loss in case an interrupt occurs between
> reading
On Fri May 9 18:24:15 2025 +, Ricardo Ribalda wrote:
> Declaring a variable for doing automatic cleanup is not a very common
> pattern. Replace the cleanup macro with manual cleanup to make the code
> simpler.
>
> Suggested-by: Laurent Pinchart
> Signed-off-by: Ricardo Ribalda
> Reviewed-by:
On Mon Jun 16 14:14:24 2025 +0200, Jacopo Mondi wrote:
> Add a function to perform soft reset of the FCP.
>
> It is intended to support the correct stop procedure of the VSPX-FCPVX
> and VSPD-FCPD pairs according to section "62.3.7.3 Reset Operation" of
> the R-Car Hardware Manual at revision 1.20
On Fri Jun 6 12:05:33 2025 +0300, Laurent Pinchart wrote:
> The imx-mipi-csis defines custom macros for the CSI-2 data types,
> duplicating the centralized macros defines in mipi-csi2.h. Replace them
> with the latter.
>
> Reviewed-by: Rui Miguel Silva
> Reviewed-by: Stefan Klug
> Reviewed-by: F
On Sat May 10 14:18:03 2025 +0800, chenchangcheng wrote:
> Some broken device return wrong dwMaxPayloadTransferSize fields as
> follows:
>
> [ 218.632537] uvcvideo: Device requested 2752512 B/frame bandwidth.
> [ 218.632598] uvcvideo: No fast enough alt setting for requested bandwidth.
>
> When
On Thu May 15 16:07:48 2025 +0800, Kuan-Wei Chiu wrote:
> Refactor parity calculations to use the standard parity8() helper.
> This change eliminates redundant implementations.
>
> Co-developed-by: Yu-Chun Lin
> Signed-off-by: Yu-Chun Lin
> Signed-off-by: Kuan-Wei Chiu
> Reviewed-by: Hans Verku
On Thu May 15 16:09:33 2025 +0800, Kuan-Wei Chiu wrote:
> Refactor parity calculations to use the standard parity8() helper.
> This change eliminates redundant implementations.
>
> Co-developed-by: Yu-Chun Lin
> Signed-off-by: Yu-Chun Lin
> Signed-off-by: Kuan-Wei Chiu
> Reviewed-by: Hans Verku
On Mon Jun 16 14:14:25 2025 +0200, Koji Matsuoka wrote:
> According to the R-Car Gen3 H/W manual v2.40, and R-Car Gen4 H/W manual
> v1.20, the FCP must be reset after resetting the VSPD, except for the
> VSPDL. Do so.
>
> Signed-off-by: Koji Matsuoka
> Signed-off-by: LUU HOAI
> Signed-off-by: Ja
On Fri Jun 13 17:34:31 2025 +0200, Niklas Söderlund wrote:
> On Gen2 where sub-devices where not exposed to user-space the field
> TB/BT ordering was controlled by a hack in the VIN driver. Before
> converting it to media device model where the subdevice is exposed
> remove that hack.
>
> Signed-
On Fri Jun 13 17:34:22 2025 +0200, Niklas Söderlund wrote:
> When extending the driver with Gen4 support the iteration of over
> possible remote subdevices changed from being R-Car CSI-2 Rx only to
> also cover R-Car CSISP instances. In two loops updating the bounds
> variable was missed.
>
> Thi
On Fri Jun 13 17:34:34 2025 +0200, Niklas Söderlund wrote:
> With Gen2 converted to use the common media device there is only one
> caller left for the helper to notify a video device of an event, fold it
> in.
>
> Signed-off-by: Niklas Söderlund
> Reviewed-by: Laurent Pinchart
> Link:
> http
On Mon Jun 16 18:30:37 2025 +0200, Jacopo Mondi wrote:
> In order to detect invalid usage pattern such as double list_put()
> calls, add an 'allocated' flag to each display list. Set the flag
> whenever a list is get() and clear it when the list is put(). Warn if a
> list not marked as allocated is
On Fri Jun 6 03:04:59 2025 +, Gui-Dong Han wrote:
> In the interrupt handler rain_interrupt(), the buffer full check on
> rain->buf_len is performed before acquiring rain->buf_lock. This
> creates a Time-of-Check to Time-of-Use (TOCTOU) race condition, as
> rain->buf_len is concurrently accesse
On Thu May 22 13:56:40 2025 -0400, Frank Li wrote:
> Allow num_sources (drvdata: num_channels) to be greater than num_sink
> (drvdata: num_ports + 1).
>
> ISI support stream multiplexing, such as differentiates multiple cameras
> from a single 2-lane MIPI input, or duplicates input stream into mul
On Fri Jun 13 17:34:25 2025 +0200, Niklas Söderlund wrote:
> Prepare for more failed probe conditions that needs cleanup by
> converting the error path to use labels.
>
> Signed-off-by: Niklas Söderlund
> Reviewed-by: Laurent Pinchart
> Link:
> https://lore.kernel.org/r/20250613153434.2001800
On Mon Jun 16 09:18:54 2025 +0200, Bartosz Golaszewski wrote:
> struct gpio_chip now has callbacks for setting line values that return
> an integer, allowing to indicate failures. Convert the driver to using
> them.
>
> Signed-off-by: Bartosz Golaszewski
> Signed-off-by: Hans Verkuil
Patch comm
On Wed Jun 11 19:37:16 2025 +0100, Dave Stevenson wrote:
> When calling tc358743_set_fmt, the code was calling tc358743_get_fmt
> to choose a valid format. However that sets the colorspace
> based on information read back from the chip, not the colour
> format requested.
>
> The result was that if
On Fri Jun 13 17:34:29 2025 +0200, Niklas Söderlund wrote:
> The VIN usage of v4l-async is complex and stems from organic growth of
> the driver of supporting both private local subdevices (Gen2, Gen3) and
> subdevices shared between all VIN instances (Gen3 and Gen4).
>
> The driver used a separa
On Thu May 22 10:09:55 2025 +0200, Wolfram Sang wrote:
> The HW can only do write-then-read transactions. This is a common
> limitation, so we can add an adapter quirk flag to let the I2C core
> enforce the checks instead of open coding them.
>
> Signed-off-by: Wolfram Sang
> Signed-off-by: Hans
On Fri May 2 07:48:28 2025 +, Ricardo Ribalda wrote:
> To implement VIDIOC_QUERYCTRL, we need to know the minimum, maximum,
> step and flags of the control. For some of the controls, this involves
> querying the actual hardware.
>
> Some non-compliant cameras produce errors when we query them.
On Wed Jun 4 14:38:48 2025 +, James Cowgill wrote:
> The `separate_colour_plane_flag` element is only present in the SPS if
> `chroma_format_idc == 3`, so the corresponding flag should be disabled
> whenever that is not the case and not just on profiles where
> `chroma_format_idc` is not presen
On Fri Jun 13 14:21:51 2025 +0300, Cosmin Tanislav wrote:
> Replace the static transmit buffer with a dynamically allocated one,
> removing the limit imposed on the number of pulses to transmit.
>
> Calculate the number of pulses for each duration in the received buffer
> ahead of time, while also
On Fri Jun 13 14:21:52 2025 +0300, Cosmin Tanislav wrote:
> Carrier frequency is currently unconstrained, allowing the SPI transfer
> to be allocated and filled only for it to be later rejected by the SPI
> controller since the frequency is too large.
>
> Add a check to constrain the carrier frequ
On Fri Jun 13 14:21:53 2025 +0300, Cosmin Tanislav wrote:
> Switch to u64 arithmetic and use DIV_ROUND_CLOSEST_ULL() to avoid
> the overflow.
>
> buffer[i] is unsigned int and is limited by the lirc core to
> IR_MAX_DURATION, which is 50.
>
> idata->freq is u32, which has a max value of 0xFFF
Dear Bryan O'Donoghue:
Thanks for your patches! Unfortunately the Media CI robot has not been
able to test them.
Make sure that the whole series 20250630121704.260831-1-...@kernel.org is
available at lore. And that it can be cherry-picked on top the "next"
branch of "https://gitlab.freedesktop.or
100 matches
Mail list logo
