On 12/09/16 07:53 PM, Hans de Goede wrote: > Before this commit crtc_bounds() did not take reflection into account, > when using reflection with 0 / 180 degree rotation this was not an > issue because of the default in the switch-case doing the right thing. > > But when using 90 / 270 degree rotation we would also end up in the > default which is wrong in this case. This would lead to the cursor > being constrained to a height x height area of the monitor. > > This commit masks out the reflection bits for the switch-case, > making crtc_bounds return the correct bounds and fixing the > problematic cursor constraining. > > Signed-off-by: Hans de Goede <[email protected]> > --- > randr/rrcrtc.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/randr/rrcrtc.c b/randr/rrcrtc.c > index 7636591..5d404e8 100644 > --- a/randr/rrcrtc.c > +++ b/randr/rrcrtc.c > @@ -281,7 +281,7 @@ crtc_bounds(RRCrtcPtr crtc, int *left, int *right, int > *top, int *bottom) > *left = crtc->x; > *top = crtc->y; > > - switch (crtc->rotation) { > + switch (crtc->rotation & 0xf) { > case RR_Rotate_0: > case RR_Rotate_180: > default: >
Reviewed-by: Michel Dänzer <[email protected]> -- Earthling Michel Dänzer | http://www.amd.com Libre software enthusiast | Mesa and X developer _______________________________________________ [email protected]: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: https://lists.x.org/mailman/listinfo/xorg-devel
