From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41492) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZHa1m-0002zo-QY for qemu-devel@nongnu.org; Tue, 21 Jul 2015 12:04:23 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZHa1i-00030J-Gk for qemu-devel@nongnu.org; Tue, 21 Jul 2015 12:04:22 -0400 Received: from greensocs.com ([193.104.36.180]:54740) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZHa1i-00030D-6d for qemu-devel@nongnu.org; Tue, 21 Jul 2015 12:04:18 -0400 Message-ID: <55AE6D7D.8080707@greensocs.com> Date: Tue, 21 Jul 2015 18:04:13 +0200 From: Frederic Konrad MIME-Version: 1.0 References: <1436199721-9902-1-git-send-email-fred.konrad@greensocs.com> <1436199721-9902-7-git-send-email-fred.konrad@greensocs.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH V3 6/8] Introduce xilinx dpdma. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Hyun Kwon , "qemu-devel@nongnu.org" Cc: "peter.maydell@linaro.org" , "mark.burton@greensocs.com" , "guillaume.delbergue@greensocs.com" , Peter Crosthwaite On 11/07/2015 02:29, Hyun Kwon wrote: > Hi Fred, > > Thanks for the patch. > >> -----Original Message----- >> From: fred.konrad@greensocs.com [mailto:fred.konrad@greensocs.com] >> Sent: Monday, July 06, 2015 9:22 AM >> To: qemu-devel@nongnu.org >> Cc: peter.maydell@linaro.org; Peter Crosthwaite; Hyun Kwon; >> guillaume.delbergue@greensocs.com; mark.burton@greensocs.com; >> fred.konrad@greensocs.com >> Subject: [PATCH V3 6/8] Introduce xilinx dpdma. >> >> From: KONRAD Frederic >> >> This is the implementation of the DPDMA. >> >> Signed-off-by: KONRAD Frederic >> --- >> hw/dma/Makefile.objs | 1 + >> hw/dma/xlnx_dpdma.c | 789 >> +++++++++++++++++++++++++++++++++++++++++++++++++++ >> hw/dma/xlnx_dpdma.h | 77 +++++ >> 3 files changed, 867 insertions(+) >> create mode 100644 hw/dma/xlnx_dpdma.c >> create mode 100644 hw/dma/xlnx_dpdma.h >> >> diff --git a/hw/dma/Makefile.objs b/hw/dma/Makefile.objs >> index 0e65ed0..5451836 100644 >> --- a/hw/dma/Makefile.objs >> +++ b/hw/dma/Makefile.objs >> @@ -8,6 +8,7 @@ common-obj-$(CONFIG_XILINX_AXI) += xilinx_axidma.o >> common-obj-$(CONFIG_ETRAXFS) += etraxfs_dma.o >> common-obj-$(CONFIG_STP2000) += sparc32_dma.o >> common-obj-$(CONFIG_SUN4M) += sun4m_iommu.o >> +obj-$(CONFIG_XLNX_ZYNQMP) += xlnx_dpdma.o >> >> obj-$(CONFIG_OMAP) += omap_dma.o soc_dma.o >> obj-$(CONFIG_PXA2XX) += pxa2xx_dma.o >> diff --git a/hw/dma/xlnx_dpdma.c b/hw/dma/xlnx_dpdma.c >> new file mode 100644 >> index 0000000..a327085 >> --- /dev/null >> +++ b/hw/dma/xlnx_dpdma.c >> @@ -0,0 +1,789 @@ >> +/* >> + * xlnx_dpdma.c > [snip] > >> + >> +size_t xlnx_dpdma_start_operation(XlnxDPDMAState *s, uint8_t channel, >> + bool one_desc) >> +{ >> + uint64_t desc_addr; >> + uint64_t source_addr[6]; >> + DPDMADescriptor desc; >> + bool done = false; >> + size_t ptr = 0; >> + >> + assert(channel <= 5); >> + >> + /* Trigger a VSYNC IRQ when the graphic callback asks for data. */ >> + if (channel == 3) { >> + s->registers[DPDMA_ISR] |= (1 << 27); >> + xlnx_dpdma_update_irq(s); >> + } > I'd rather make this VSYNC interrupt generation as a separate function, and have it called by whenever xlnx_dp_update_display() is called. Please see my comment in patch 7/8 as well. > > Thanks, > -hyun Ok I do that. Thanks, Fred > > > This email and any attachments are intended for the sole use of the named recipient(s) and contain(s) confidential information that may be proprietary, privileged or copyrighted under applicable law. If you are not the intended recipient, do not read, copy, or forward this email message or any attachments. Delete this email message and any attachments immediately. >