Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e216c2d5 authored by Dhaval Patel's avatar Dhaval Patel Committed by Gerrit - the friendly Code Review server
Browse files

msm: sde: rotator: release the second fence reference 



rotator driver marks the fence to null after passing
it to client. This is invalid starting from kernel
rev 4.9 because it uses the dma fence APIs. These
APIs takes two reference while creating the fence
and it is client's responsibility to release the
first fence.

Change-Id: I2c1d175144df5c5a75fb060a61ac1c23bdd34976
Signed-off-by: default avatarDhaval Patel <pdhaval@codeaurora.org>
parent 84a47e8c

drivers/media/platform/msm/sde/rotator/sde_rotator_dev.c

+0 −7
Original line number Diff line number Diff line
@@ -2613,13 +2613,6 @@ static long sde_rotator_private_ioctl(struct file *file, void *fh, 
				return ret;

			}



			/**

			 * Loose any reference to sync fence once we pass

			 * it to user. Driver does not clean up user

			 * unclosed fence descriptors.

			 */

			vbinfo->fence = NULL;



			/**

			 * Cache fence descriptor in case user calls this

			 * ioctl multiple times. Cached value would be stale