Commit cf1d4fdb authored by Krzysztof Kozlowski's avatar Krzysztof Kozlowski Committed by Mathieu Poirier
Browse files

remoteproc: omap: Simplify returning syscon PTR_ERR 



No need to store PTR_ERR into temporary, local 'ret' variable.

Signed-off-by: default avatarKrzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
Link: https://lore.kernel.org/r/20250111-b4-syscon-phandle-args-remoteproc-v1-2-73ed6fafa1e3@linaro.org


Signed-off-by: default avatarMathieu Poirier <mathieu.poirier@linaro.org>
parent bd047185

drivers/remoteproc/omap_remoteproc.c

+2 −5
Original line number Diff line number Diff line
@@ -1137,7 +1137,6 @@ static int omap_rproc_get_boot_data(struct platform_device *pdev, 
	struct device_node *np = pdev->dev.of_node;

	struct omap_rproc *oproc = rproc->priv;

	const struct omap_rproc_dev_data *data;

	int ret;



	data = of_device_get_match_data(&pdev->dev);

	if (!data)
@@ -1153,10 +1152,8 @@ static int omap_rproc_get_boot_data(struct platform_device *pdev, 


	oproc->boot_data->syscon =

			syscon_regmap_lookup_by_phandle(np, "ti,bootreg");

	if (IS_ERR(oproc->boot_data->syscon)) {

		ret = PTR_ERR(oproc->boot_data->syscon);

		return ret;

	}

	if (IS_ERR(oproc->boot_data->syscon))

		return PTR_ERR(oproc->boot_data->syscon);



	if (of_property_read_u32_index(np, "ti,bootreg", 1,

				       &oproc->boot_data->boot_reg)) {