mtd: spi-nor: spansion: return method directly

ambarus · ambarus · commit abfac0f3a457 · 2023-07-22T11:17:07.000+03:00
Remove unnecessary handling of method's return code and return the method directly. Link: https://lore.kernel.org/r/20230721170911.13502-1-tudor.ambarus@linaro.org Signed-off-by: Tudor Ambarus <tudor.ambarus@linaro.org>
diff --git a/drivers/mtd/spi-nor/spansion.c b/drivers/mtd/spi-nor/spansion.c
@@ -656,13 +656,7 @@ static int s28hx_t_post_bfpt_fixup(struct spi_nor *nor,
 				   const struct sfdp_parameter_header *bfpt_header,
 				   const struct sfdp_bfpt *bfpt)
 {
-	int ret;
-
-	ret = cypress_nor_set_addr_mode_nbytes(nor);
-	if (ret)
-		return ret;
-
-	return 0;
+	return cypress_nor_set_addr_mode_nbytes(nor);
 }
 
 static void s28hx_t_late_init(struct spi_nor *nor)

Original file line number	Diff line number	Diff line change
`@@ -656,13 +656,7 @@ static int s28hx_t_post_bfpt_fixup(struct spi_nor *nor,`
`656`	`656`	`const struct sfdp_parameter_header *bfpt_header,`
`657`	`657`	`const struct sfdp_bfpt *bfpt)`
`658`	`658`	`{`
`659`		`- int ret;`
`660`		`-`
`661`		`- ret = cypress_nor_set_addr_mode_nbytes(nor);`
`662`		`- if (ret)`
`663`		`- return ret;`
`664`		`-`
`665`		`- return 0;`
	`659`	`+ return cypress_nor_set_addr_mode_nbytes(nor);`
`666`	`660`	`}`
`667`	`661`
`668`	`662`	`static void s28hx_t_late_init(struct spi_nor *nor)`