The driver's memory regions are currently just ioremap()ed, but not
reserved through a request. That's not a bug, but having the request is
a little more robust.

Implement the region-request through the corresponding managed
devres-function.

Signed-off-by: Philipp Stanner <pstan...@redhat.com>
---
 drivers/gpu/drm/imx/dcss/dcss-dev.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/imx/dcss/dcss-dev.c 
b/drivers/gpu/drm/imx/dcss/dcss-dev.c
index 4f3af0dfb344..efd3a998652d 100644
--- a/drivers/gpu/drm/imx/dcss/dcss-dev.c
+++ b/drivers/gpu/drm/imx/dcss/dcss-dev.c
@@ -177,10 +177,10 @@ struct dcss_dev *dcss_dev_create(struct device *dev, bool 
hdmi_output)
                return ERR_PTR(-ENODEV);
        }
 
-       res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
-       if (!res) {
-               dev_err(dev, "cannot get memory resource\n");
-               return ERR_PTR(-EINVAL);
+       res = devm_platform_get_resource(pdev, IORESOURCE_MEM, 0, "dcss");
+       if (IS_ERR(res)) {
+               dev_err(dev, "cannot get / request memory resource\n");
+               return res;
        }
 
        dcss = kzalloc(sizeof(*dcss), GFP_KERNEL);
-- 
2.43.0

Reply via email to