fixup! gpio-fsm: Avoid truncation of delay jiffies

The kernel's time unit of jiffies should be stored as an unsigned long
value. Storing it as an unsigned int, as gpio-fsm did, leads to
truncation and malfunction when the kernel is built for a 64-bit
platform.

Signed-off-by: Phil Elwell <[email protected]>

media: imx219: Advertise embedded data node on media pad 1

This commit updates the imx219 driver to adverise support for embedded
data streams.  This can then be used by the bcm2835-unicam driver, which
has recently been updated to expose the embedded data stream to
userland.

The imx219 sensor subdevice overloads the media pad to differentiate
between image stream (pad 0) and embedded data stream (pad 1) when
performing the v4l2_subdev_pad_ops functions.

Signed-off-by: Naushir Patuck <[email protected]>
Signed-off-by: Dave Stevenson <[email protected]>

imx708: Do not reset vblank to a default value

imx708_set_framing_limits resets the vblank control to the mode default
value unconditionally. This causes it to overwrite the user specified
vblank and exposure control values when starting the sensor, since
it is called when handling V4L2_CID_WIDE_DYNAMIC_RANGE.

Remove this call to s_ctrl as it is unnecessary.

Signed-off-by: Naushir Patuck <[email protected]>

imx708: Do not reset vblank to a default value

imx708_set_framing_limits resets the vblank control to the mode default
value unconditionally. This causes it to overwrite the user specified
vblank and exposure control values when starting the sensor, since
it is called when handling V4L2_CID_WIDE_DYNAMIC_RANGE.

Remove this call to s_ctrl as it is unnecessary.

Signed-off-by: Naushir Patuck <[email protected]>

drm/connector: Set DDC pointer in drmm_connector_init

Commit 35a3b82 ("drm/connector: Introduce drmm_connector_init")
introduced the function drmm_connector_init() with a parameter for an
optional ddc pointer to the i2c controller used to access the DDC bus.

However, the underlying call to __drm_connector_init() was always
setting it to NULL instead of passing the ddc argument around.

This resulted in unexpected null pointer dereference on platforms
expecting to get a DDC controller.

Fixes: 35a3b82 ("drm/connector: Introduce drmm_connector_init")
Signed-off-by: Maxime Ripard <[email protected]>

overlays: Add README entry for i2c-rtc,rv3032

Signed-off-by: Phil Elwell <[email protected]>

media: i2c: arducam-pivariety: Add custom controls

Add support for strobe_shift, strobe_width and mode custom controls.

Signed-off-by: Lee Jackson <[email protected]>

overlays: Mark more overlays as Pi4-specific

Signed-off-by: Phil Elwell <[email protected]>

brcmfmac: Restore ISO3166 and 0 rev as a fallback

Commit b0b524f ("brcmfmac: use ISO3166 country code and 0 rev
as fallback") was a welcome addition for Raspberry Pi as it replaced
an equivalent bit of downstream code. Commit 151a7c1 reverted
it because it broke BCM4359/9 devices.

This is a renamed reversion of the reversion.

See: raspberrypi/Raspberry-Pi-OS-64bit#217

Signed-off-by: Phil Elwell <[email protected]>

bcm2835-codec: Return empty buffers to the VPU instead of queueing to…

… vbuf2

The encoder can skip frames totally should rate control overshoot
the target bitrate too far. In this situation it generates an
output buffer of length 0.
V4L2 treats a buffer of length 0 as an end of stream flag, which is
not appropriate in this case, therefore we can not return that buffer
to the client.

The driver was returning the buffer to videobuf2 in the QUEUED state,
however that buffer was then not dequeued again, so the number of
buffers was reduced each time this happened. In the pathological
case of using GStreamer's videotestsrc in mode 1 for noise, this happens
sufficiently frequently to totally stall the pipeline.

If the port is still enabled then return the buffer straight back to
the VPU rather than to videobuf2.

Signed-off-by: Dave Stevenson <[email protected]>