…
> +++ b/drivers/accel/amdxdna/aie2_pci.c
> @@ -0,0 +1,182 @@
…
> +static int aie2_init(struct amdxdna_dev *xdna)
> +{
…
> +     release_firmware(fw);
> +     return 0;

I find the following statements more appropriate here
(so that a bit of redundant code can be avoided).

        ret = 0;
        goto release_fw;

…
> +release_fw:
> +     release_firmware(fw);
> +
> +     return ret;
> +}
…


Regards,
Markus

Reply via email to