From mboxrd@z Thu Jan 1 00:00:00 1970 Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:46090 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750828Ab3IPHAs (ORCPT ); Mon, 16 Sep 2013 03:00:48 -0400 Message-ID: <5236AC66.8090201@ti.com> Date: Mon, 16 Sep 2013 12:29:50 +0530 From: Archit Taneja MIME-Version: 1.0 To: , CC: , , Subject: Re: [PATCH v4 0/4] v4l: VPE mem to mem driver References: <1376996457-17275-1-git-send-email-archit@ti.com> <1378462346-10880-1-git-send-email-archit@ti.com> In-Reply-To: <1378462346-10880-1-git-send-email-archit@ti.com> Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-media-owner@vger.kernel.org List-ID: Hi Hans, Laurent, On Friday 06 September 2013 03:42 PM, Archit Taneja wrote: > VPE(Video Processing Engine) is an IP found on DRA7xx, this series adds VPE as a > mem to mem v4l2 driver, and VPDMA as a helper library. > > The first version of the patch series described VPE in detail, you can have a > look at it here: > > http://www.spinics.net/lists/linux-media/msg66518.html > > Changes in v4: > - Control ID for the driver reserved in v4l2-controls.h > - Some fixes/clean ups suggested by Hans. > - A small hack done in VPE's probe to use a fixed 64K resource size, this > is needed as the DT bindings will split the addresses accross VPE > submodules, the driver currently works with register offsets from the top > level VPE base. The driver can be modified later to support multiple > ioremaps of the sub modules. > - Addition of sync on channel descriptors for input DMA channels, this > ensures the VPDMA list is stalled in the rare case of an input channel's > DMA getting completed after all the output channel DMAs. > - Removed the DT and hwmod patches from this series. DRA7xx support is not > yet got in the 3.12 merge window. Will deal with those separately. I incorporated your comments and suggestions from the previous series. Wanted to know if you think it looks good enough to get merged now? Thanks! Archit From mboxrd@z Thu Jan 1 00:00:00 1970 From: Archit Taneja Subject: Re: [PATCH v4 0/4] v4l: VPE mem to mem driver Date: Mon, 16 Sep 2013 12:29:50 +0530 Message-ID: <5236AC66.8090201@ti.com> References: <1376996457-17275-1-git-send-email-archit@ti.com> <1378462346-10880-1-git-send-email-archit@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset="ISO-8859-1"; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from bear.ext.ti.com ([192.94.94.41]:46090 "EHLO bear.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750828Ab3IPHAs (ORCPT ); Mon, 16 Sep 2013 03:00:48 -0400 In-Reply-To: <1378462346-10880-1-git-send-email-archit@ti.com> Sender: linux-omap-owner@vger.kernel.org List-Id: linux-omap@vger.kernel.org To: hverkuil@xs4all.nl, laurent.pinchart@ideasonboard.com Cc: linux-media@vger.kernel.org, linux-omap@vger.kernel.org, tomi.valkeinen@ti.com Hi Hans, Laurent, On Friday 06 September 2013 03:42 PM, Archit Taneja wrote: > VPE(Video Processing Engine) is an IP found on DRA7xx, this series adds VPE as a > mem to mem v4l2 driver, and VPDMA as a helper library. > > The first version of the patch series described VPE in detail, you can have a > look at it here: > > http://www.spinics.net/lists/linux-media/msg66518.html > > Changes in v4: > - Control ID for the driver reserved in v4l2-controls.h > - Some fixes/clean ups suggested by Hans. > - A small hack done in VPE's probe to use a fixed 64K resource size, this > is needed as the DT bindings will split the addresses accross VPE > submodules, the driver currently works with register offsets from the top > level VPE base. The driver can be modified later to support multiple > ioremaps of the sub modules. > - Addition of sync on channel descriptors for input DMA channels, this > ensures the VPDMA list is stalled in the rare case of an input channel's > DMA getting completed after all the output channel DMAs. > - Removed the DT and hwmod patches from this series. DRA7xx support is not > yet got in the 3.12 merge window. Will deal with those separately. I incorporated your comments and suggestions from the previous series. Wanted to know if you think it looks good enough to get merged now? Thanks! Archit