Re: [PATCH 09/10] drm/vkms: plane_state->fb iff plane_state->crtc

2020-01-15 Thread Rodrigo Siqueira
Hi Sam, On 01/15, Sam Ravnborg wrote: > Hi Rodrigo. > > On Tue, Jan 14, 2020 at 06:50:13PM -0500, Rodrigo Siqueira wrote: > > On 12/13, Daniel Vetter wrote: > > > Checking both is one too much, so wrap a WARN_ON around it to stope > > > the copypasta. > > > > > > Signed-off-by: Daniel Vetter >

Re: [PATCH 09/10] drm/vkms: plane_state->fb iff plane_state->crtc

2020-01-14 Thread Sam Ravnborg
Hi Rodrigo. On Tue, Jan 14, 2020 at 06:50:13PM -0500, Rodrigo Siqueira wrote: > On 12/13, Daniel Vetter wrote: > > Checking both is one too much, so wrap a WARN_ON around it to stope > > the copypasta. > > > > Signed-off-by: Daniel Vetter > > Cc: Rodrigo Siqueira > > Cc: Haneen Mohammed > > Cc

Re: [PATCH 09/10] drm/vkms: plane_state->fb iff plane_state->crtc

2020-01-14 Thread Rodrigo Siqueira
On 12/13, Daniel Vetter wrote: > Checking both is one too much, so wrap a WARN_ON around it to stope > the copypasta. > > Signed-off-by: Daniel Vetter > Cc: Rodrigo Siqueira > Cc: Haneen Mohammed > Cc: Daniel Vetter > --- > drivers/gpu/drm/vkms/vkms_plane.c | 2 +- > 1 file changed, 1 inserti