From mboxrd@z Thu Jan 1 00:00:00 1970 From: Vinod Koul Subject: Re: [PATCH V1] dmaengine: tegra: add dma driver Date: Wed, 25 Apr 2012 15:03:45 +0530 Message-ID: <1335346425.31825.118.camel@vkoul-udesk3> References: <1334912896-4614-1-git-send-email-ldewangan@nvidia.com> <1334920487.31825.98.camel@vkoul-udesk3> <4F9153AF.7020901@nvidia.com> <1335170491.31825.104.camel@vkoul-udesk3> <4F95483F.5020005@nvidia.com> <20120423130659.GA22186@n2100.arm.linux.org.uk> <4F95567E.3090605@nvidia.com> <4F97BD67.1050300@nvidia.com> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4F97BD67.1050300-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> Sender: linux-tegra-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Laxman Dewangan Cc: Russell King - ARM Linux , "dan.j.williams-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org" , "grant.likely-s3s/WqlpOiPyB63q8FvJNQ@public.gmane.org" , "rob.herring-bsGFqQB8/DxBDgjK7y7TUQ@public.gmane.org" , "linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org" , Stephen Warren , "linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-tegra@vger.kernel.org On Wed, 2012-04-25 at 14:31 +0530, Laxman Dewangan wrote: > > I had a communication with Russell on another change thread and > understand that some more work need to be done in virt_chan to support > cyclic one. > > I want to have the cyclic dma support in my driver so I want to go on > following steps: > 1. I will post the patch for tegra dma which will not use the virt_dma > so that my driver will be independent of Russel's change. which driver? A new driver for cyclic support only or on top of Russell's changes? > 2. Once the tegra dma is part of tree, I can move all my dma client to > use the dmaengine based driver and remove old style dma. > 3. Till that time Russell's will have already virt_dma support for > cyclic one and so I will change tegra_dma.c to use virt_dma. > > > Let me know your opinion so that I can plan my patch/change > accordingly. -- ~Vinod From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757288Ab2DYJhq (ORCPT ); Wed, 25 Apr 2012 05:37:46 -0400 Received: from mga09.intel.com ([134.134.136.24]:30481 "EHLO mga09.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756991Ab2DYJho (ORCPT ); Wed, 25 Apr 2012 05:37:44 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.67,352,1309762800"; d="scan'208";a="133578625" Subject: Re: [PATCH V1] dmaengine: tegra: add dma driver From: Vinod Koul To: Laxman Dewangan Cc: Russell King - ARM Linux , "dan.j.williams@intel.com" , "grant.likely@secretlab.ca" , "rob.herring@calxeda.com" , "linux-kernel@vger.kernel.org" , "devicetree-discuss@lists.ozlabs.org" , Stephen Warren , "linux-tegra@vger.kernel.org" In-Reply-To: <4F97BD67.1050300@nvidia.com> References: <1334912896-4614-1-git-send-email-ldewangan@nvidia.com> <1334920487.31825.98.camel@vkoul-udesk3> <4F9153AF.7020901@nvidia.com> <1335170491.31825.104.camel@vkoul-udesk3> <4F95483F.5020005@nvidia.com> <20120423130659.GA22186@n2100.arm.linux.org.uk> <4F95567E.3090605@nvidia.com> <4F97BD67.1050300@nvidia.com> Content-Type: text/plain; charset="UTF-8" Date: Wed, 25 Apr 2012 15:03:45 +0530 Message-ID: <1335346425.31825.118.camel@vkoul-udesk3> Mime-Version: 1.0 X-Mailer: Evolution 2.28.3 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, 2012-04-25 at 14:31 +0530, Laxman Dewangan wrote: > > I had a communication with Russell on another change thread and > understand that some more work need to be done in virt_chan to support > cyclic one. > > I want to have the cyclic dma support in my driver so I want to go on > following steps: > 1. I will post the patch for tegra dma which will not use the virt_dma > so that my driver will be independent of Russel's change. which driver? A new driver for cyclic support only or on top of Russell's changes? > 2. Once the tegra dma is part of tree, I can move all my dma client to > use the dmaengine based driver and remove old style dma. > 3. Till that time Russell's will have already virt_dma support for > cyclic one and so I will change tegra_dma.c to use virt_dma. > > > Let me know your opinion so that I can plan my patch/change > accordingly. -- ~Vinod