firmware: microchip: don't unconditionally print validation success

[ Upstream commit 6e3b7e862e ]

If validation fails, both prints are made. Skip the success one in the
failure case.

Fixes: ec5b0f1193 ("firmware: microchip: add PolarFire SoC Auto Update support")
Signed-off-by: Conor Dooley <conor.dooley@microchip.com>
Signed-off-by: Sasha Levin <sashal@kernel.org>
This commit is contained in:
Conor Dooley 2024-04-10 12:58:05 +01:00 committed by Greg Kroah-Hartman
parent 254c58d614
commit 4006e50609
1 changed files with 2 additions and 0 deletions

View File

@ -206,10 +206,12 @@ static int mpfs_auto_update_verify_image(struct fw_upload *fw_uploader)
if (ret | response->resp_status) {
dev_warn(priv->dev, "Verification of Upgrade Image failed!\n");
ret = ret ? ret : -EBADMSG;
goto free_message;
}
dev_info(priv->dev, "Verification of Upgrade Image passed!\n");
free_message:
devm_kfree(priv->dev, message);
free_response:
devm_kfree(priv->dev, response);