Commit cf56f4f2 authored by Rikard Falkeborn's avatar Rikard Falkeborn Committed by Mauro Carvalho Chehab
Browse files

media: s5p-jpeg: Constify struct v4l2_m2m_ops 



The only usage of the v4l2_m2m_ops structs is to pass their address to
v4l2_m2m_init() which takes a pointer to const struct v4l2_m2m_ops as
argument. Make them const to allow the compiler to put them in read-only
memory.

Signed-off-by: default avatarRikard Falkeborn <rikard.falkeborn@gmail.com>
Signed-off-by: default avatarHans Verkuil <hverkuil-cisco@xs4all.nl>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent 0407c49e

drivers/media/platform/s5p-jpeg/jpeg-core.c

+3 −3
Original line number Diff line number Diff line
@@ -2425,17 +2425,17 @@ static int s5p_jpeg_job_ready(void *priv) 
	return 1;

}



static struct v4l2_m2m_ops s5p_jpeg_m2m_ops = {

static const struct v4l2_m2m_ops s5p_jpeg_m2m_ops = {

	.device_run	= s5p_jpeg_device_run,

	.job_ready	= s5p_jpeg_job_ready,

};



static struct v4l2_m2m_ops exynos3250_jpeg_m2m_ops = {

static const struct v4l2_m2m_ops exynos3250_jpeg_m2m_ops = {

	.device_run	= exynos3250_jpeg_device_run,

	.job_ready	= s5p_jpeg_job_ready,

};



static struct v4l2_m2m_ops exynos4_jpeg_m2m_ops = {

static const struct v4l2_m2m_ops exynos4_jpeg_m2m_ops = {

	.device_run	= exynos4_jpeg_device_run,

	.job_ready	= s5p_jpeg_job_ready,

};

drivers/media/platform/s5p-jpeg/jpeg-core.h

+1 −1
Original line number Diff line number Diff line
@@ -142,7 +142,7 @@ struct s5p_jpeg_variant { 
	unsigned int		hw3250_compat:1;

	unsigned int		htbl_reinit:1;

	unsigned int		hw_ex4_compat:1;

	struct v4l2_m2m_ops	*m2m_ops;

	const struct v4l2_m2m_ops *m2m_ops;

	irqreturn_t		(*jpeg_irq)(int irq, void *priv);

	const char		*clk_names[JPEG_MAX_CLOCKS];

	int			num_clocks;