[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-25 Thread Ying Liu
On Mon, Oct 24, 2016 at 7:50 PM, Philipp Zabel wrote: > Am Freitag, den 21.10.2016, 16:49 +0800 schrieb Ying Liu: >> On Fri, Oct 21, 2016 at 4:18 PM, Philipp Zabel >> wrote: >> > Am Freitag, den 21.10.2016, 13:45 +0800 schrieb Ying Liu: >> >> On Thu, Oct 20, 2016 at 9:29 PM, Philipp Zabel > >>

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-24 Thread Philipp Zabel
Am Freitag, den 21.10.2016, 16:49 +0800 schrieb Ying Liu: > On Fri, Oct 21, 2016 at 4:18 PM, Philipp Zabel > wrote: > > Am Freitag, den 21.10.2016, 13:45 +0800 schrieb Ying Liu: > >> On Thu, Oct 20, 2016 at 9:29 PM, Philipp Zabel > >> wrote: > >> > Am Donnerstag, den 20.10.2016, 16:51 +0800 sch

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-21 Thread Ying Liu
On Fri, Oct 21, 2016 at 4:18 PM, Philipp Zabel wrote: > Am Freitag, den 21.10.2016, 13:45 +0800 schrieb Ying Liu: >> On Thu, Oct 20, 2016 at 9:29 PM, Philipp Zabel >> wrote: >> > Am Donnerstag, den 20.10.2016, 16:51 +0800 schrieb Ying Liu: >> >> >> Does the clip thing potentially change the use

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-21 Thread Ying Liu
On Thu, Oct 20, 2016 at 9:29 PM, Philipp Zabel wrote: > Am Donnerstag, den 20.10.2016, 16:51 +0800 schrieb Ying Liu: >> >> Does the clip thing potentially change the user's request by force? >> >> For example, the user request an unreasonable big resolution. >> > >> > The user is allowed to ask f

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-21 Thread Philipp Zabel
Am Freitag, den 21.10.2016, 13:45 +0800 schrieb Ying Liu: > On Thu, Oct 20, 2016 at 9:29 PM, Philipp Zabel > wrote: > > Am Donnerstag, den 20.10.2016, 16:51 +0800 schrieb Ying Liu: > >> >> Does the clip thing potentially change the user's request by force? > >> >> For example, the user request an

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-20 Thread Ying Liu
On Thu, Oct 20, 2016 at 4:41 PM, Ville Syrjälä wrote: > On Thu, Oct 20, 2016 at 03:53:46PM +0800, Ying Liu wrote: >> On Wed, Oct 19, 2016 at 6:21 PM, Philipp Zabel >> wrote: >> > Use drm_plane_helper_check_state to clip raw user coordinates to crtc >> > bounds. This checks for full plane cover

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-20 Thread Ville Syrjälä
On Thu, Oct 20, 2016 at 04:51:30PM +0800, Ying Liu wrote: > On Thu, Oct 20, 2016 at 4:41 PM, Ville Syrjälä > wrote: > > On Thu, Oct 20, 2016 at 03:53:46PM +0800, Ying Liu wrote: > >> On Wed, Oct 19, 2016 at 6:21 PM, Philipp Zabel > >> wrote: > >> > Use drm_plane_helper_check_state to clip raw

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-20 Thread Ying Liu
On Wed, Oct 19, 2016 at 6:21 PM, Philipp Zabel wrote: > Use drm_plane_helper_check_state to clip raw user coordinates to crtc > bounds. This checks for full plane coverage and scaling already, so > we can drop some custom checks. Use the clipped coordinates everywhere. > > Suggested-by: Ville Syr

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-20 Thread Philipp Zabel
Am Donnerstag, den 20.10.2016, 16:51 +0800 schrieb Ying Liu: > >> Does the clip thing potentially change the user's request by force? > >> For example, the user request an unreasonable big resolution. > > > > The user is allowed to ask for destination coordinates extending outside > > the crtc dime

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-20 Thread Ville Syrjälä
On Thu, Oct 20, 2016 at 03:53:46PM +0800, Ying Liu wrote: > On Wed, Oct 19, 2016 at 6:21 PM, Philipp Zabel > wrote: > > Use drm_plane_helper_check_state to clip raw user coordinates to crtc > > bounds. This checks for full plane coverage and scaling already, so > > we can drop some custom checks.

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-19 Thread Ville Syrjälä
On Wed, Oct 19, 2016 at 12:21:17PM +0200, Philipp Zabel wrote: > Use drm_plane_helper_check_state to clip raw user coordinates to crtc > bounds. This checks for full plane coverage and scaling already, so > we can drop some custom checks. Use the clipped coordinates everywhere. > > Suggested-by: V

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-19 Thread Philipp Zabel
Am Mittwoch, den 19.10.2016, 13:31 +0300 schrieb Ville Syrjälä: > On Wed, Oct 19, 2016 at 12:21:17PM +0200, Philipp Zabel wrote: > > Use drm_plane_helper_check_state to clip raw user coordinates to crtc > > bounds. This checks for full plane coverage and scaling already, so > > we can drop some c

[PATCH 10/10] drm/imx: ipuv3-plane: use drm_plane_helper_check_state, clipped coordinates

2016-10-19 Thread Philipp Zabel
Use drm_plane_helper_check_state to clip raw user coordinates to crtc bounds. This checks for full plane coverage and scaling already, so we can drop some custom checks. Use the clipped coordinates everywhere. Suggested-by: Ville Syrjälä Signed-off-by: Philipp Zabel --- drivers/gpu/drm/imx/ip