linux.git/drivers/staging/media, branch v6.9

media: imx: csc/scaler: fix v4l2_ctrl_handler memory leak

2024-02-16T10:46:32+00:00

Free the memory allocated in v4l2_ctrl_handler_init on release.

Fixes: a8ef0488cc59 ("media: imx: add csc/scaler mem2mem device")
Signed-off-by: Lucas Stach 
Reviewed-by: Philipp Zabel 
Signed-off-by: Hans Verkuil

staging: media: starfive: Set 16 bpp for capture_raw device

2024-02-16T10:46:32+00:00

For StarFive JH7110 Camera Subsystem, capture_raw video device output
raw10 pixelformat requires 16bit of alignment.

Fixes: e080f339c80a ("media: staging: media: starfive: camss: Add capture driver")
Signed-off-by: Changhuang Liang 
Signed-off-by: Hans Verkuil

media: staging: imx: controls are from another device, mark this

2024-02-16T10:46:32+00:00

The last argument of v4l2_ctrl_add_handler() indicates whether the controls
you add are from a control handler owned by another driver (true) or from
the same driver (false). In this case the last argument was incorrectly set
to false. The control handlers are from different devices.

Signed-off-by: Hans Verkuil

media: cedrus: h265: Fix configuring bitstream size

2024-02-16T10:46:31+00:00

bit_size field holds size of slice, not slice + header. Because of HW
quirks, driver can't program in just slice, but also preceding header.
But that means that currently used bit_size is wrong (too small).
Instead, just use size of whole buffer. There is no harm in doing this.

Fixes: 86caab29da78 ("media: cedrus: Add HEVC/H.265 decoding support")
Suggested-by: Paul Kocialkowski 
Signed-off-by: Jernej Skrabec 
Signed-off-by: Hans Verkuil

media: atomisp: don't use sizeof(NULL)

2024-02-05T11:57:46+00:00

Check the size of another pointer instead of NULL. This fixes this
smatch warning:

drivers/staging/media/atomisp/pci/sh_css.c:3609 ia_css_pipe_enqueue_buffer() warn: sizeof(NUMBER)?

Signed-off-by: Hans Verkuil

media: atomisp: make dbgopt static

2024-02-05T11:57:46+00:00

This fixes a sparse warning:

drivers/staging/media/atomisp/pci/atomisp_drvfs.c:40:14: warning: symbol 'dbgopt' was not declared. Should it be static?

Signed-off-by: Hans Verkuil

media: staging: meson: Fix kerneldoc

2024-02-05T11:57:45+00:00

Remove documentation from missing field.

Signed-off-by: Ricardo Ribalda 
Reviewed-by: Randy Dunlap 
Signed-off-by: Hans Verkuil

media: atomisp: Update TODO

2024-02-01T06:05:13+00:00

Remove the TODO items for using the main (drivers/media/i2c) ov2680 and
ov5693 drivers and removing the atomisp specific ones, this has been done.

Remove the TODO item for gracefully handling missing firmware the
"media: atomisp: Bind and do power-management without firmware" changes
have fixed this.

Signed-off-by: Hans de Goede 
Signed-off-by: Mauro Carvalho Chehab

media: atomisp: Remove unnecessary msleep(10) from atomisp_mrfld_power() error path

2024-02-01T06:05:08+00:00

Remove unnecessary msleep(10) from atomisp_mrfld_power() error-exit path,
the success exit from atomisp_mrfld_power() happens if a test succeeds
inside the do { } while loop above the msleep().

The error-exit path with the removed msleep is only hit it the power-on is
not reflected in the iUNIT ISPSSPM0 status bits after a timeout of 50 ms.
Sleeping an extra 10 ms in the timeout path makes little sense.

Signed-off-by: Hans de Goede 
Signed-off-by: Mauro Carvalho Chehab

media: atomisp: Bind and do power-management without firmware

2024-02-01T06:04:58+00:00

The ISP needs to be turned off in a special manner for the SoC
to be able to reach S0i3 during suspend.

When the firmware is missing still bind to the PCI device and
run in pm-only mode which takes care of turning the ISP off
(including turning it off again if the firmware has turned it
on during resume).

In this new pm-only mode the atomisp driver works exactly the same
as the non-staging, pm-only drivers/platform/x86/intel/atomisp2/pm.c
driver.

Signed-off-by: Hans de Goede 
Signed-off-by: Mauro Carvalho Chehab