From: Thierry Reding <tred...@nvidia.com>

There's no reason whatsoever why this should ever be negative. The same
goes for the number of pipes added to the DRM device.

Cc: Philipp Zabel <p.zabel at pengutronix.de>
Acked-by: Philipp Zabel <p.zabel at pengutronix.de>
Signed-off-by: Thierry Reding <treding at nvidia.com>
---
 drivers/gpu/drm/imx/imx-drm-core.c | 6 +++---
 drivers/gpu/drm/imx/imx-drm.h      | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/drivers/gpu/drm/imx/imx-drm-core.c 
b/drivers/gpu/drm/imx/imx-drm-core.c
index 74f505b0dd02..9ff1b780f453 100644
--- a/drivers/gpu/drm/imx/imx-drm-core.c
+++ b/drivers/gpu/drm/imx/imx-drm-core.c
@@ -39,20 +39,20 @@ struct imx_drm_component {
 struct imx_drm_device {
        struct drm_device                       *drm;
        struct imx_drm_crtc                     *crtc[MAX_CRTC];
-       int                                     pipes;
+       unsigned int                            pipes;
        struct drm_fbdev_cma                    *fbhelper;
 };

 struct imx_drm_crtc {
        struct drm_crtc                         *crtc;
-       int                                     pipe;
+       unsigned int                            pipe;
        struct imx_drm_crtc_helper_funcs        imx_drm_helper_funcs;
 };

 static int legacyfb_depth = 16;
 module_param(legacyfb_depth, int, 0444);

-int imx_drm_crtc_id(struct imx_drm_crtc *crtc)
+unsigned int imx_drm_crtc_id(struct imx_drm_crtc *crtc)
 {
        return crtc->pipe;
 }
diff --git a/drivers/gpu/drm/imx/imx-drm.h b/drivers/gpu/drm/imx/imx-drm.h
index 28e776d8d9d2..eebf0e2fefd0 100644
--- a/drivers/gpu/drm/imx/imx-drm.h
+++ b/drivers/gpu/drm/imx/imx-drm.h
@@ -12,7 +12,7 @@ struct drm_framebuffer;
 struct imx_drm_crtc;
 struct platform_device;

-int imx_drm_crtc_id(struct imx_drm_crtc *crtc);
+unsigned int imx_drm_crtc_id(struct imx_drm_crtc *crtc);

 struct imx_drm_crtc_helper_funcs {
        int (*enable_vblank)(struct drm_crtc *crtc);
-- 
2.5.0

Reply via email to