Update core.c

1:The control flow was simplified by using else if statements instead of goto structure.

2:Error conditions are handled more clearly.

3:The device_unlock call at the end of the function is guaranteed in all cases.
This commit is contained in:
Okan Tümüklü 2024-09-30 01:23:29 +03:00 committed by GitHub
parent 9852d85ec9
commit 5b036330b2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194

View File

@ -40,27 +40,19 @@ int nfc_fw_download(struct nfc_dev *dev, const char *firmware_name)
if (dev->shutting_down) {
rc = -ENODEV;
goto error;
}
if (dev->dev_up) {
}else if (dev->dev_up) {
rc = -EBUSY;
goto error;
}
if (!dev->ops->fw_download) {
}else if (!dev->ops->fw_download) {
rc = -EOPNOTSUPP;
goto error;
}
}else{
dev->fw_download_in_progress = true;
rc = dev->ops->fw_download(dev, firmware_name);
if (rc)
dev->fw_download_in_progress = false;
}
dev->fw_download_in_progress = true;
rc = dev->ops->fw_download(dev, firmware_name);
if (rc)
dev->fw_download_in_progress = false;
error:
device_unlock(&dev->dev);
return rc;
device_unlock(&dev->dev);
return rc;
}
/**