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=-4.1 required=3.0 tests=BAYES_00,DKIMWL_WL_HIGH, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS 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 B01FEC433E6 for ; Mon, 31 Aug 2020 11:07:52 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 8D20C2054F for ; Mon, 31 Aug 2020 11:07:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598872072; bh=C+1fKhJBNw/Y3PJKxGvNWyUc7dhncI98yBJO4d14IJo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:List-ID:From; b=i3BShFG4LCnMcv5/xoWeQeOcG9EZj2uoolhFduY//N7nXeWa9kPQl0ApUSs0P4oJ+ a3q9XDW972hZX6rmxf5yYZ6BEpWC+YS41SduMy5T0wVQ01FN9KGwB2yCiFnpkfrYkd Vhs73NcyrxqRw/KmZChvQTEaQiu75NqNylCsnX60= Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726249AbgHaLHA (ORCPT ); Mon, 31 Aug 2020 07:07:00 -0400 Received: from mail.kernel.org ([198.145.29.99]:54562 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726224AbgHaLBT (ORCPT ); Mon, 31 Aug 2020 07:01:19 -0400 Received: from localhost (unknown [122.171.38.130]) (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 A6F482072D; Mon, 31 Aug 2020 11:00:35 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1598871636; bh=C+1fKhJBNw/Y3PJKxGvNWyUc7dhncI98yBJO4d14IJo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=WfQm7zL27aM+2aGqyAcsedQ8Dko+y1Kwvx25LUAl+6/suNTRdkCJhJbgYopap3ZIR uScl0ijE0IlvZu3amNL60oaIxi7/6b1o5xnnK/lj5pIGpgjIGR+Bo76uf7Uw2xT3Hu 3jo3iO1ofbIm0W3j2MhexTl2pwXlZqHridtCUqQM= Date: Mon, 31 Aug 2020 16:30:32 +0530 From: Vinod Koul To: "Reddy, MallikarjunaX" Cc: Rob Herring , dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, andriy.shevchenko@intel.com, cheol.yong.kim@intel.com, qi-ming.wu@intel.com, chuanhua.lei@linux.intel.com, malliamireddy009@gmail.com Subject: Re: [PATCH v5 1/2] dt-bindings: dma: Add bindings for intel LGM SOC Message-ID: <20200831110032.GN2639@vkoul-mobl> References: <68c77fd2ffb477aa4a52a58f8a26bfb191d3c5d1.1597381889.git.mallikarjunax.reddy@linux.intel.com> <20200814203222.GA2674896@bogus> <7cdc0587-8b4f-4360-a303-1541c9ad57b2@linux.intel.com> <20200825112107.GN2639@vkoul-mobl> <20200828104530.GT2639@vkoul-mobl> <09547b0e-1c2e-d916-d4c0-f66b0110e173@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <09547b0e-1c2e-d916-d4c0-f66b0110e173@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 31-08-20, 16:06, Reddy, MallikarjunaX wrote: > Hi Vinod, > > Thanks for the review. Please see my comment inline. > > On 8/28/2020 6:45 PM, Vinod Koul wrote: > > On 27-08-20, 17:54, Reddy, MallikarjunaX wrote: > > > Hi Vinod, > > > Thanks for the review comments. > > > > > > On 8/25/2020 7:21 PM, Vinod Koul wrote: > > > > On 18-08-20, 15:00, Reddy, MallikarjunaX wrote: > > > > > > > > > > > + > > > > > > > + intel,chans: > > > > > > > + $ref: /schemas/types.yaml#/definitions/uint32-array > > > > > > > + description: > > > > > > > + The channels included on this port. Format is channel start > > > > > > > + number and how many channels on this port. > > > > > > Why does this need to be in DT? This all seems like it can be in the dma > > > > > > cells for each client. > > > > > (*ABC) > > > > > Yes. We need this. > > > > > for dma0(lgm-cdma) old SOC supports 16 channels and the new SOC supports 22 > > > > > channels. and the logical channel mapping for the peripherals also differ > > > > > b/w old and new SOCs. > > > > > > > > > > Because of this hardware limitation we are trying to configure the total > > > > > channels and port-channel mapping dynamically from device tree. > > > > > > > > > > based on port name we are trying to configure the default values for > > > > > different peripherals(ports). > > > > > Example: burst length is not same for all ports, so using port name to do > > > > > default configurations. > > > > Sorry that does not make sense to me, why not specify the values to be > > > > used here instead of defining your own name scheme! > > > OK. Agreed. I will remove port name from DT and only use intel,chans > > what is intel,chans, why not use dma-channels? >  The intel,chans says about the channels included on the correspondng port. What do you mean by a port here? > Format is channel start number and how many channels on this port. It is perfectly reasonable to have 16 channels but linux not use use all, lets say from 5th channel channel onwards So you need to use standard dma-channels also with dma-channel-mask to specify which channels linux can use >  The reasong behind using this attribute instead of standrad dma-channels > is... > > > DMA_VER22 HW supports 22 channels. But there is a hole in HW, total it can > use only 16. > > Old soc supports 4ports and 16 channels. > New soc supports 6ports and 22 channels. > (old and new soc carry the same version VER22) > > port channel mapping for the old and new soc also not the same. > old soc: logical channels:(Rx, Tx) > 0, 1 - SPI0 > 2, 3 - SPI1 > 4, 5 - HSNAND > 12, 14, 13, 15 - Memcopy > > New soc: Logical channels(Rx, Tx) > 0, 1 - SPI0 > 2, 3 - SPI1 > 4, 5 - SPI2 > 6, 7 - SPI3 > 8, 9 - HSNAND > 10 to 21 - Mcopy Mapping is different, client can set that channel required in dmas property and use a specific required channel. > Because of these reasons we are trying to use "intel,chans" attribute, and > reading then number of channels from the dt. > Advantaage: > 1. we can map the channels correspondign to port > 2. Dynamically configure the channels (due to hw limitation) > > If this is not ok, please suggest us the better way to handle this. > > -- ~Vinod