Received: by 2002:a05:6a10:1287:0:0:0:0 with SMTP id d7csp6369688pxv; Thu, 29 Jul 2021 12:52:12 -0700 (PDT) X-Google-Smtp-Source: ABdhPJzyjlkYcH2sOUod0zv0PRjyVs18adJRfrc1g4wuwQDe3tYohxUlese3LxlMiCKbCg2Z6bkS X-Received: by 2002:a5d:9e03:: with SMTP id h3mr5637056ioh.7.1627588331865; Thu, 29 Jul 2021 12:52:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1627588331; cv=none; d=google.com; s=arc-20160816; b=XdJgttQQs4iankL0UqqjAaY2lgU/PDKJAzVRwT5VEaLJoAegkSbdHxZ5C4LWgLgXCL QNbSA6wwSUXzUWiXW5530gv8UBkFSx36sVH+Ub/lHCILGBRYXYgd5fu+2hZQgkQix+kB q6Y12Eb+8zQJZY2+b2wGeEmBHcVd2BRocE+zWJEnV8dTvNDZ628zF2orsmPiwspAHmH7 SeLEatCa9i9d8Pc7NT99rhPdiAKPkBdj//34mmgZoyjgiiUJ1hQmIhNU49xZxQ1cIJtZ RtVNfExRqtRaKzqdkLCXmOeTD0VCDZGQOvz2aDGPi8WUBzutzrT9nc8cGYpRl9GzrvDy d7AQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=a1f/4Mzvy91uylKaEBSa+SbUv2oypgU64QN38rZ/W1A=; b=g5vkAMrJ4R7geSo0I8xeUIWz9607MW7WS6uakKOrJiaikbksOgL454lPXqAIfzXzZ0 VzGmdbYXzPxLIYIgVmnY+f/zloyJlgAtVH+sLaXazmE8j94Oh+Hb66l40LAdUsBgDKkR 35A+zKRxT74kFpvExE5X7+IU8FPxxW0GSqeTB6mpNt9joWgbto91mNc+eNzo/3Begp9p obIDDz6jXR+pfHTSuaPcT24td1m760U0uqINoJchPDS18x8UgYNwidfRBxgqFmw94jFJ bohGTvlQWTLsarRN40OcJp/2zPQTcuqNdNBjz3n1nyDrAfyEBqvpwyoGcGhmZgtx06x/ rCUw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gerhold.net header.s=strato-dkim-0002 header.b=BhFcK3pV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id v13si4716036ilj.65.2021.07.29.12.51.59; Thu, 29 Jul 2021 12:52:11 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@gerhold.net header.s=strato-dkim-0002 header.b=BhFcK3pV; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S230056AbhG2TvM (ORCPT + 99 others); Thu, 29 Jul 2021 15:51:12 -0400 Received: from mo4-p01-ob.smtp.rzone.de ([85.215.255.53]:29746 "EHLO mo4-p01-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229606AbhG2TvL (ORCPT ); Thu, 29 Jul 2021 15:51:11 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1627588261; s=strato-dkim-0002; d=gerhold.net; h=In-Reply-To:References:Message-ID:Subject:Cc:To:From:Date:Cc:Date: From:Subject:Sender; bh=a1f/4Mzvy91uylKaEBSa+SbUv2oypgU64QN38rZ/W1A=; b=BhFcK3pVObxgbgbV6J1xl7tDYIcvcsOU6ZLYorGXAWqwHPUe2gEwIbvGh/j5F63rvm XZudNTAUZFotrbXXFDcjDo7N9schc9DbXE0dH+dYZldo0aCHzeob3yid8SmfRXCIWJzf KJTRsqB+Z8kZZJbucHz3eYSSMYEh79Y0Y6me2QB2PJzXC6bDG1Mleo5vTcwAvt/Q3nvU ax6+Fz5bqesHLub+E37q4wkquBD1Nt2BMZmoca33kxypXWrPsFyLeq60GrpY9DeBLMTJ T4zdtqrjxfy6u/zGiY42LM1HPSOAR43XHwViPsYFy9AM6FJYHvOV2Bi78eGArf/VHitk Tgnw== Authentication-Results: strato.com; dkim=none X-RZG-AUTH: ":P3gBZUipdd93FF5ZZvYFPugejmSTVR2nRPhVOQ/OcYgojyw4j34+u261EJF5OxJD4peA8paM1A==" X-RZG-CLASS-ID: mo00 Received: from gerhold.net by smtp.strato.de (RZmta 47.28.1 DYNA|AUTH) with ESMTPSA id g02a44x6TJp06W6 (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Thu, 29 Jul 2021 21:51:00 +0200 (CEST) Date: Thu, 29 Jul 2021 21:50:54 +0200 From: Stephan Gerhold To: Rob Herring Cc: "David S. Miller" , Jakub Kicinski , Loic Poulain , Sergey Ryazanov , Johannes Berg , Bjorn Andersson , Andy Gross , Vinod Koul , Aleksander Morgado , netdev@vger.kernel.org, linux-arm-msm@vger.kernel.org, dmaengine@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, phone-devel@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, Jeffrey Hugo Subject: Re: [RFC PATCH net-next 1/4] dt-bindings: dmaengine: bam_dma: Add remote power collapse mode Message-ID: References: <20210719145317.79692-1-stephan@gerhold.net> <20210719145317.79692-2-stephan@gerhold.net> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thu, Jul 29, 2021 at 01:36:31PM -0600, Rob Herring wrote: > On Mon, Jul 19, 2021 at 04:53:14PM +0200, Stephan Gerhold wrote: > > In some configurations, the BAM DMA controller is set up by a remote > > processor and the local processor can simply start making use of it > > without setting up the BAM. This is already supported using the > > "qcom,controlled-remotely" property. > > > > However, for some reason another possible configuration is that the > > remote processor is responsible for powering up the BAM, but we are > > still responsible for initializing it (e.g. resetting it etc). Add > > a "qcom,remote-power-collapse" property to describe that configuration. > > > > Signed-off-by: Stephan Gerhold > > --- > > NOTE: This is *not* a compile-time requirement for the BAM-DMUX driver > > so this could also go through the dmaengine tree. > > > > Also note that there is an ongoing effort to convert these bindings > > to DT schema but sadly there were not any updates for a while. :/ > > https://lore.kernel.org/linux-arm-msm/20210519143700.27392-2-bhupesh.sharma@linaro.org/ > > --- > > Documentation/devicetree/bindings/dma/qcom_bam_dma.txt | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/dma/qcom_bam_dma.txt b/Documentation/devicetree/bindings/dma/qcom_bam_dma.txt > > index cf5b9e44432c..362a4f0905a8 100644 > > --- a/Documentation/devicetree/bindings/dma/qcom_bam_dma.txt > > +++ b/Documentation/devicetree/bindings/dma/qcom_bam_dma.txt > > @@ -15,6 +15,8 @@ Required properties: > > the secure world. > > - qcom,controlled-remotely : optional, indicates that the bam is controlled by > > remote proccessor i.e. execution environment. > > +- qcom,remote-power-collapse : optional, indicates that the bam is powered up by > > + a remote processor but must be initialized by the local processor. > > Wouldn't 'qcom,remote-power' or 'qcom,remote-powered' be sufficient? I > don't understand what 'collapse' means here. Doesn't sound good though. > Yeah I can't think of any significant meaning of the "collapse" part for the bindings, I probably just picked it up somewhere while trying to find some information about how the BAM DMUX setup works. :) Just one question, would you prefer "qcom,remote-powered" or rather "qcom,powered-remotely" for consistency with the existing "qcom,controlled-remotely"? Both sounds fine to me. Thanks! Stephan