On Thu, May 01, 2014 at 07:56:43PM +0800, Daniel Kurtz wrote: > DRM_IOCTL_MODE_SETPLANE crtc_x, crtc_y are s32. > This is to allow a destination location that is partially off screen. > > Make this more obvious to users of libdrm by using signed crtc_x/_y > parameters for drmModeSetPlane() as well. > > Signed-off-by: Daniel Kurtz <djkurtz@xxxxxxxxxxxx> Reviewed-by: Ville Syrjälä <ville.syrjala@xxxxxxxxxxxxxxx> and pushed. > --- > xf86drmMode.c | 2 +- > xf86drmMode.h | 2 +- > 2 files changed, 2 insertions(+), 2 deletions(-) > > diff --git a/xf86drmMode.c b/xf86drmMode.c > index a6bb2ee..7ca89b3 100644 > --- a/xf86drmMode.c > +++ b/xf86drmMode.c > @@ -906,7 +906,7 @@ int drmModePageFlip(int fd, uint32_t crtc_id, uint32_t fb_id, > > int drmModeSetPlane(int fd, uint32_t plane_id, uint32_t crtc_id, > uint32_t fb_id, uint32_t flags, > - uint32_t crtc_x, uint32_t crtc_y, > + int32_t crtc_x, int32_t crtc_y, > uint32_t crtc_w, uint32_t crtc_h, > uint32_t src_x, uint32_t src_y, > uint32_t src_w, uint32_t src_h) > diff --git a/xf86drmMode.h b/xf86drmMode.h > index 9bcb1d1..6eab565 100644 > --- a/xf86drmMode.h > +++ b/xf86drmMode.h > @@ -445,7 +445,7 @@ extern drmModePlaneResPtr drmModeGetPlaneResources(int fd); > extern drmModePlanePtr drmModeGetPlane(int fd, uint32_t plane_id); > extern int drmModeSetPlane(int fd, uint32_t plane_id, uint32_t crtc_id, > uint32_t fb_id, uint32_t flags, > - uint32_t crtc_x, uint32_t crtc_y, > + int32_t crtc_x, int32_t crtc_y, > uint32_t crtc_w, uint32_t crtc_h, > uint32_t src_x, uint32_t src_y, > uint32_t src_w, uint32_t src_h); > -- > 1.9.1.423.g4596e3a > > _______________________________________________ > dri-devel mailing list > dri-devel@xxxxxxxxxxxxxxxxxxxxx > http://lists.freedesktop.org/mailman/listinfo/dri-devel -- Ville Syrjälä Intel OTC _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
