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 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 862D6C433FE for ; Mon, 18 Oct 2021 12:56:21 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 6E14D61351 for ; Mon, 18 Oct 2021 12:56:21 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S231466AbhJRM6a (ORCPT ); Mon, 18 Oct 2021 08:58:30 -0400 Received: from mo4-p01-ob.smtp.rzone.de ([85.215.255.54]:18822 "EHLO mo4-p01-ob.smtp.rzone.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S230526AbhJRM6a (ORCPT ); Mon, 18 Oct 2021 08:58:30 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; t=1634561774; 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=OqccIkY4i1t4VJRk9mPtA2mqAlP8NFJGUylFRJUf6so=; b=emhYqEC1M0Czer1t0/DniSH/7MJn6qmwkRVR+AIC9QU0Kgdgr/ysxF0cFISY2nntww ZGo9fiSp87F4bYLp28466WnF7UQtpQ3xsSJtyNQVT+fBG/be9UYjFMqDa8D3Ydl7HwU0 KK5CKMC9820YJZH5jxjq2IIbS432RyqVHsO9pq0XnUbwa7leL2WWwuNvDsLznupg3VjZ KoPH3Ktl7HRMmOFvxVehv9c1ZAAA8BPRRtYXE1N1P1MH16TSK7yJ8jW+SyAYso2JjxIm 04GKNPzezm6VMPbvZM8X+PydhmpMLcpF9pg99PBc7jEfl2srVf81yAzpvbxNCHf0u8/v BW5Q== Authentication-Results: strato.com; dkim=none X-RZG-AUTH: ":P3gBZUipdd93FF5ZZvYFPugejmSTVR2nRPhVOQ/OcYgojyw4j34+u267FZF9PwpcNKLVrKw5+aY=" X-RZG-CLASS-ID: mo00 Received: from gerhold.net by smtp.strato.de (RZmta 47.33.8 AUTH) with ESMTPSA id 301038x9ICuAWJs (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256 bits)) (Client did not present a certificate); Mon, 18 Oct 2021 14:56:10 +0200 (CEST) Date: Mon, 18 Oct 2021 14:56:05 +0200 From: Stephan Gerhold To: Bhupesh Sharma Cc: "David S. Miller" , Jakub Kicinski , Loic Poulain , Sergey Ryazanov , Johannes Berg , Bjorn Andersson , Andy Gross , Vinod Koul , Rob Herring , Aleksander Morgado , netdev@vger.kernel.org, MSM , dmaengine@vger.kernel.org, devicetree , Linux Kernel Mailing List , phone-devel@vger.kernel.org, ~postmarketos/upstreaming@lists.sr.ht, Jeffrey Hugo Subject: Re: [PATCH net-next v2 1/4] dt-bindings: dmaengine: bam_dma: Add "powered remotely" mode Message-ID: References: <20211011141733.3999-1-stephan@gerhold.net> <20211011141733.3999-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-arm-msm@vger.kernel.org On Mon, Oct 18, 2021 at 05:04:31PM +0530, Bhupesh Sharma wrote: > On Mon, 11 Oct 2021 at 20:12, 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,powered-remotely" property to describe that configuration. > > > > Signed-off-by: Stephan Gerhold > > --- > > Changes since RFC: > > - Rename qcom,remote-power-collapse -> qcom,powered-remotely > > for consistency with "qcom,controlled-remotely" > > > > 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/ > > Seems you missed the latest series posted last week - [1]. Sorry I got > a bit delayed posting it due to being caught up in other patches. > > Maybe you can rebase your patch on the same and use the YAML bindings > for the qcom,bam_dma controller. > > [1]. https://lore.kernel.org/linux-arm-msm/20211013105541.68045-1-bhupesh.sharma@linaro.org/T/#t > Ah, you're right sorry! Seems like you sent it two days after I sent the v2 of this patch. Thanks a lot for continuing work on this! :) Since I already sent v3 of this patch earlier, I think it is best if I wait a bit first and see if Vinod has any comments or still wants to take it for 5.16. Should be simple to rebase either of our patches on the other one. Thanks! Stephan