On 3/8/2016 10:33 AM, Sinan Kaya wrote:
> +     int rc = 0;
> +

I should have initialized this to -1. 

> +     vdev->reset = vfio_platform_lookup_reset(vdev->compat, vdev->acpihid,
> +                                              &vdev->reset_module);
> +     if (vdev->reset)
> +             return 0;
> +
> +     if (vdev->acpihid)
> +             rc = request_module("vfio-reset:%s", vdev->acpihid);
> +
> +     if (rc && vdev->compat)
> +             rc = request_module("vfio-reset:%s", vdev->compat);
> +


-- 
Sinan Kaya
Qualcomm Technologies, Inc. on behalf of Qualcomm Innovation Center, Inc.
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux 
Foundation Collaborative Project

Reply via email to