Use resource_size() instead of open coding res->end - res->start + 1.
Include linux/ioport.h explicitly, since the file relies on struct resource and now on the helper as well. Signed-off-by: Aryan Sinha <[email protected]> --- v2: elaborate more in the commit message, and include linux/ioport.h. Both as per Rodrigo suggestions. Link to v1: https://lore.kernel.org/all/[email protected]/ drivers/gpu/drm/xe/xe_svm.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/gpu/drm/xe/xe_svm.c b/drivers/gpu/drm/xe/xe_svm.c index b228a737c..a9b087c43 100644 --- a/drivers/gpu/drm/xe/xe_svm.c +++ b/drivers/gpu/drm/xe/xe_svm.c @@ -3,6 +3,7 @@ * Copyright © 2024 Intel Corporation */ +#include <linux/ioport.h> #include <linux/pci-p2pdma.h> #include <drm/drm_drv.h> @@ -1848,7 +1849,7 @@ static struct xe_pagemap *xe_pagemap_create(struct xe_device *xe, struct xe_vram out_no_pages: drm_pagemap_release_owner(&xpagemap->peer); out_no_owner: - devm_release_mem_region(dev, res->start, res->end - res->start + 1); + devm_release_mem_region(dev, res->start, resource_size(res)); out_err: drm_pagemap_put(dpagemap); return ERR_PTR(err); -- 2.46.0
