From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=no autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id D2EBEC43331 for ; Mon, 11 Nov 2019 09:01:11 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id A2B2F206BA for ; Mon, 11 Nov 2019 09:01:11 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="qVa9uT5O"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="e69J39U/" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org A2B2F206BA Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=D3VbpDOHy4x7i/6466bMJrWapHUhSHQqOP/wqat2LdE=; b=qVa9uT5Oh8Agm5 fw9x1T6YtvfeDFKKqyXjwq3YYvAVOZDoJl5B0/d+bPOswuTRiNuojuM0ZUYAgtu035OzlnfnqPqL0 88Lvz4y6Tfb7yJnceoQrhI1D/E+ZffG8OWLCk2LFR0yYypCTbctfcB33UOvQNCwgsQdw73CoOaDtT eDKaqiWYec6A949Jlilk/FfA+Cf+DwnARiUv8V9cKG2LgV8Juejcqq6lU1JCOimnVNxv7WwohkbB5 aA8poP0+rL7jZJwMRXNwUFTQLK4JolqY6tEztMGrZMyI6fpDjwptNRsi1f57YbgJ/2uXD/eYVdFJl W5Fl5hxmFTkryUvxMNbQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1iU5ZM-0000MZ-3R; Mon, 11 Nov 2019 09:01:08 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iU5ZI-0000Lj-RK for linux-arm-kernel@lists.infradead.org; Mon, 11 Nov 2019 09:01:06 +0000 Received: from localhost (unknown [106.201.42.77]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 60B34206BA; Mon, 11 Nov 2019 09:01:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1573462864; bh=z/j9crcHsk7S4aU9B2I6s0jBvYdfKN9sp40Q4ClK9J4=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=e69J39U/DZqIZdafDyLMGrY+J4YYs6R8wCZts1udWAxa3Xhh2I/nPeAbcxehxjL/F IlHNETuf6ciTeHdl4WU+En9BixkF7hVwiHh8IFPPhLOrfJAlgJNT8lmZaw7kmMydY9 CTHYUhLs399fKoWZGmHus2OLDQB3QXS1NUL58kPc= Date: Mon, 11 Nov 2019 14:30:57 +0530 From: Vinod Koul To: Peter Ujfalusi Subject: Re: [PATCH v4 09/15] dmaengine: ti: New driver for K3 UDMA - split#1: defines, structs, io func Message-ID: <20191111090057.GT952516@vkoul-mobl> References: <20191101084135.14811-1-peter.ujfalusi@ti.com> <20191101084135.14811-10-peter.ujfalusi@ti.com> <20191111052828.GN952516@vkoul-mobl> <00777586-a3ac-2404-5226-e8c887936a32@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <00777586-a3ac-2404-5226-e8c887936a32@ti.com> User-Agent: Mutt/1.12.1 (2019-06-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191111_010104_907755_4B692A07 X-CRM114-Status: GOOD ( 12.15 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: nm@ti.com, devicetree@vger.kernel.org, grygorii.strashko@ti.com, lokeshvutla@ti.com, j-keerthy@ti.com, linux-kernel@vger.kernel.org, t-kristo@ti.com, tony@atomide.com, robh+dt@kernel.org, ssantosh@kernel.org, dmaengine@vger.kernel.org, dan.j.williams@intel.com, linux-arm-kernel@lists.infradead.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 11-11-19, 10:33, Peter Ujfalusi wrote: > On 11/11/2019 7.28, Vinod Koul wrote: > > On 01-11-19, 10:41, Peter Ujfalusi wrote: > >> + struct udma_static_tr static_tr; > >> + char *name; > >> + > >> + struct udma_tchan *tchan; > >> + struct udma_rchan *rchan; > >> + struct udma_rflow *rflow; > >> + > >> + bool psil_paired; > >> + > >> + int irq_num_ring; > >> + int irq_num_udma; > >> + > >> + bool cyclic; > >> + bool paused; > >> + > >> + enum udma_chan_state state; > >> + struct completion teardown_completed; > >> + > >> + u32 bcnt; /* number of bytes completed since the start of the channel */ > >> + u32 in_ring_cnt; /* number of descriptors in flight */ > >> + > >> + bool pkt_mode; /* TR or packet */ > >> + bool needs_epib; /* EPIB is needed for the communication or not */ > >> + u32 psd_size; /* size of Protocol Specific Data */ > >> + u32 metadata_size; /* (needs_epib ? 16:0) + psd_size */ > >> + u32 hdesc_size; /* Size of a packet descriptor in packet mode */ > >> + bool notdpkt; /* Suppress sending TDC packet */ > >> + int remote_thread_id; > >> + u32 src_thread; > >> + u32 dst_thread; > >> + enum psil_endpoint_type ep_type; > >> + bool enable_acc32; > >> + bool enable_burst; > >> + enum udma_tp_level channel_tpl; /* Channel Throughput Level */ > >> + > >> + /* dmapool for packet mode descriptors */ > >> + bool use_dma_pool; > >> + struct dma_pool *hdesc_pool; > >> + > >> + u32 id; > >> + enum dma_transfer_direction dir; > > > > why does channel have this, it already exists in descriptor > > The channel can not change role, it is set when it was requested. In the how do you do this on set? The channel is requested, we do not know the direction. When prep_ is invoked we know it.. -- ~Vinod _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel