Commit c1f1d76c authored by Ding Xiang's avatar Ding Xiang Committed by Mauro Carvalho Chehab
Browse files

media: atomisp: remove redundant NULL check of "params" 

The check result of (!A || (A && B)) is equivalent to (!A || B),
so remove redundant NULL check of "params"

Link: https://lore.kernel.org/linux-media/20201117081058.673291-1-dingxiang@cmss.chinamobile.com


Signed-off-by: default avatarDing Xiang <dingxiang@cmss.chinamobile.com>
Signed-off-by: default avatarMauro Carvalho Chehab <mchehab+huawei@kernel.org>
parent a53ff69f

drivers/staging/media/atomisp/pci/sh_css_params.c

+1 −3
Original line number Diff line number Diff line
@@ -4649,10 +4649,8 @@ ia_css_dvs2_6axis_config_allocate(const struct ia_css_stream *stream) 
	params = stream->isp_params_configs;



	/* Backward compatibility by default consider pipe as Video*/

	if (!params || (params &&

			!params->pipe_dvs_6axis_config[IA_CSS_PIPE_ID_VIDEO])) {

	if (!params || !params->pipe_dvs_6axis_config[IA_CSS_PIPE_ID_VIDEO])

		goto err;

	}



	dvs_config = kvcalloc(1, sizeof(struct ia_css_dvs_6axis_config),

			      GFP_KERNEL);