diff options
| author | Yu Jiaoliang <yujiaoliang@vivo.com> | 2024-09-06 15:01:09 +0800 |
|---|---|---|
| committer | Matthew Brost <matthew.brost@intel.com> | 2024-09-12 12:18:21 -0700 |
| commit | bbb1ed0b4437ef728569457a136540ce2e6b11c4 (patch) | |
| tree | a7ae28b3959af5103da2e899d85c3acf03a0c8d8 /drivers/gpu | |
| parent | f96dbf7c321d70834d46f3aedb75a671e839b51e (diff) | |
| download | lwn-bbb1ed0b4437ef728569457a136540ce2e6b11c4.tar.gz lwn-bbb1ed0b4437ef728569457a136540ce2e6b11c4.zip | |
drm/xe: Use ERR_CAST to return an error-valued pointer
Instead of directly casting and returning an error-valued pointer,
use ERR_CAST to make the error handling more explicit and improve
code clarity.
Signed-off-by: Yu Jiaoliang <yujiaoliang@vivo.com>
Reviewed-by: Matthew Brost <matthew.brost@intel.com>
Signed-off-by: Matthew Brost <matthew.brost@intel.com>
Link: https://patchwork.freedesktop.org/patch/msgid/20240906070109.1852860-1-yujiaoliang@vivo.com
Diffstat (limited to 'drivers/gpu')
| -rw-r--r-- | drivers/gpu/drm/xe/xe_sa.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/xe/xe_sa.c b/drivers/gpu/drm/xe/xe_sa.c index fe2cb2a96f78..e055bed7ae55 100644 --- a/drivers/gpu/drm/xe/xe_sa.c +++ b/drivers/gpu/drm/xe/xe_sa.c @@ -53,7 +53,7 @@ struct xe_sa_manager *xe_sa_bo_manager_init(struct xe_tile *tile, u32 size, u32 if (IS_ERR(bo)) { drm_err(&xe->drm, "failed to allocate bo for sa manager: %ld\n", PTR_ERR(bo)); - return (struct xe_sa_manager *)bo; + return ERR_CAST(bo); } sa_manager->bo = bo; sa_manager->is_iomem = bo->vmap.is_iomem; |