Commit fa9f443f authored by Mauro Carvalho Chehab's avatar Mauro Carvalho Chehab
Browse files

media: rcar_fdp1: simplify error check logic at fdp_open() 



Avoid some code duplication by moving the common error path
logic at fdp_open().

Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent d07bb970

drivers/media/platform/rcar_fdp1.c

+8 −7
Original line number Diff line number Diff line
@@ -2117,9 +2117,7 @@ static int fdp1_open(struct file *file) 


	if (ctx->hdl.error) {

		ret = ctx->hdl.error;

		v4l2_ctrl_handler_free(&ctx->hdl);

		kfree(ctx);

		goto done;

		goto error_ctx;

	}



	ctx->fh.ctrl_handler = &ctx->hdl;
@@ -2133,10 +2131,7 @@ static int fdp1_open(struct file *file) 


	if (IS_ERR(ctx->fh.m2m_ctx)) {

		ret = PTR_ERR(ctx->fh.m2m_ctx);



		v4l2_ctrl_handler_free(&ctx->hdl);

		kfree(ctx);

		goto done;

		goto error_ctx;

	}



	/* Perform any power management required */
@@ -2147,6 +2142,12 @@ static int fdp1_open(struct file *file) 
	dprintk(fdp1, "Created instance: %p, m2m_ctx: %p\n",

		ctx, ctx->fh.m2m_ctx);



	mutex_unlock(&fdp1->dev_mutex);

	return 0;



error_ctx:

	v4l2_ctrl_handler_free(&ctx->hdl);

	kfree(ctx);

done:

	mutex_unlock(&fdp1->dev_mutex);

	return ret;