From mboxrd@z Thu Jan 1 00:00:00 1970 From: Mauro Carvalho Chehab Subject: Re: [PATCH v2 02/14] v4l: vsp1: Don't recycle active list at display start Date: Thu, 20 Jul 2017 10:51:38 -0300 Message-ID: <20170720105138.395541e4@vento.lan> References: <20170626181226.29575-1-laurent.pinchart+renesas@ideasonboard.com> <20170626181226.29575-3-laurent.pinchart+renesas@ideasonboard.com> Mime-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: Sender: linux-renesas-soc-owner@vger.kernel.org To: Kieran Bingham Cc: Laurent Pinchart , dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org List-Id: dri-devel@lists.freedesktop.org Em Thu, 13 Jul 2017 18:02:20 +0100 Kieran Bingham escreveu: > Hi Laurent, > > On 26/06/17 19:12, Laurent Pinchart wrote: > > When the display start interrupt occurs, we know that the hardware has > > finished loading the active display list. The driver then proceeds to > > recycle the list, assuming it won't be needed anymore. > > > > This assumption holds true for headerless display lists, as the VSP > > doesn't reload the list for the next frame if it hasn't changed. > > However, this isn't true anymore for header display lists, as they are > > loaded at every frame start regardless of whether they have been > > updated. > > > > To prepare for header display lists usage in display pipelines, we need > > to postpone recycling the list until it gets replaced by a new one > > through a page flip. The driver already does so in the frame end > > interrupt handler, so all we need is to skip list recycling in the > > display start interrupt handler. > > > > While the active list can be recycled at display start for headerless > > display lists, there's no real harm in postponing that to the frame end > > interrupt handler in all cases. This simplifies interrupt handling as we > > don't need to process the display start interrupt anymore. > > > > Signed-off-by: Laurent Pinchart > > Ok, I had skipped this one as I was concerned about its effects in relation to > 11/14 but I see how that's working now. > > Reviewed-by: Kieran Bingham Acked-by: Mauro Carvalho Chehab Thanks, Mauro