EDAC/qcom: Remove superfluous return variable assignment in qcom_llcc_core_setup()

Mani-Sadhasivam · andersson · commit 3d49f7406b5d · 2023-05-26T20:56:54.000-07:00
"ret" variable will be assigned on both success and failure cases. So there is no need to initialize it during start of qcom_llcc_core_setup(). Signed-off-by: Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> Signed-off-by: Bjorn Andersson <andersson@kernel.org> Link: https://lore.kernel.org/r/20230517114635.76358-2-manivannan.sadhasivam@linaro.org
diff --git a/drivers/edac/qcom_edac.c b/drivers/edac/qcom_edac.c
@@ -170,7 +170,7 @@ static int qcom_llcc_core_setup(struct regmap *llcc_bcast_regmap)
 static int
 qcom_llcc_clear_error_status(int err_type, struct llcc_drv_data *drv)
 {
-	int ret = 0;
+	int ret;
 
 	switch (err_type) {
 	case LLCC_DRAM_CE:

Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ static int qcom_llcc_core_setup(struct regmap *llcc_bcast_regmap)`
`170`	`170`	`static int`
`171`	`171`	`qcom_llcc_clear_error_status(int err_type, struct llcc_drv_data *drv)`
`172`	`172`	`{`
`173`		`- int ret = 0;`
	`173`	`+ int ret;`
`174`	`174`
`175`	`175`	`switch (err_type) {`
`176`	`176`	`case LLCC_DRAM_CE:`