diff options
| author | Zhao Liu <zhao1.liu@intel.com> | 2024-04-18 18:04:33 +0800 |
|---|---|---|
| committer | Philippe Mathieu-Daudé <philmd@linaro.org> | 2024-04-25 12:48:12 +0200 |
| commit | e0ddabc6d4cfef4a5c7f154f0b0ad00dbf9a18d0 (patch) | |
| tree | f846f108e6961b91f994551cf4f2643001f9e9b6 /include/hw | |
| parent | a133d207a8fefe934eb808c2b1ee8f2c695cb528 (diff) | |
| download | qemu-e0ddabc6d4cfef4a5c7f154f0b0ad00dbf9a18d0.zip qemu-e0ddabc6d4cfef4a5c7f154f0b0ad00dbf9a18d0.tar.gz qemu-e0ddabc6d4cfef4a5c7f154f0b0ad00dbf9a18d0.tar.bz2 | |
hw/cxl/cxl-cdat: Make cxl_doe_cdat_init() return boolean
As error.h suggested, the best practice for callee is to return
something to indicate success / failure.
With returned boolean, there's no need to dereference @errp to check
failure case.
Suggested-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Zhao Liu <zhao1.liu@intel.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Acked-by: Jonathan Cameron <Jonathan.Cameron@huawei.com>
Message-ID: <20240418100433.1085447-4-zhao1.liu@linux.intel.com>
Signed-off-by: Philippe Mathieu-Daudé <philmd@linaro.org>
Diffstat (limited to 'include/hw')
| -rw-r--r-- | include/hw/cxl/cxl_component.h | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/hw/cxl/cxl_component.h b/include/hw/cxl/cxl_component.h index 5012fab..945ee6f 100644 --- a/include/hw/cxl/cxl_component.h +++ b/include/hw/cxl/cxl_component.h @@ -273,7 +273,7 @@ hwaddr cxl_decode_ig(int ig); CXLComponentState *cxl_get_hb_cstate(PCIHostState *hb); bool cxl_get_hb_passthrough(PCIHostState *hb); -void cxl_doe_cdat_init(CXLComponentState *cxl_cstate, Error **errp); +bool cxl_doe_cdat_init(CXLComponentState *cxl_cstate, Error **errp); void cxl_doe_cdat_release(CXLComponentState *cxl_cstate); void cxl_doe_cdat_update(CXLComponentState *cxl_cstate, Error **errp); |