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 vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id C8F4EC433EF for ; Mon, 17 Jan 2022 10:31:23 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S238632AbiAQKbW (ORCPT ); Mon, 17 Jan 2022 05:31:22 -0500 Received: from foss.arm.com ([217.140.110.172]:56348 "EHLO foss.arm.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S238606AbiAQKbV (ORCPT ); Mon, 17 Jan 2022 05:31:21 -0500 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7D1A96D; Mon, 17 Jan 2022 02:31:20 -0800 (PST) Received: from bogus (unknown [10.57.38.209]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B7C0D3F73D; Mon, 17 Jan 2022 02:31:17 -0800 (PST) Date: Mon, 17 Jan 2022 10:31:00 +0000 From: Sudeep Holla To: Stephen Boyd Cc: Cristian Marussi , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, Michael Turquette , linux-clk@vger.kernel.org Subject: Re: [PATCH v8 11/11] clk: scmi: Support atomic clock enable/disable API Message-ID: <20220117103100.c6ltxnz4iqmbc4b5@bogus> References: <20211220195646.44498-1-cristian.marussi@arm.com> <20211220195646.44498-12-cristian.marussi@arm.com> <20220114230839.ABD63C36AE9@smtp.kernel.org> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20220114230839.ABD63C36AE9@smtp.kernel.org> Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Jan 14, 2022 at 03:08:37PM -0800, Stephen Boyd wrote: > Quoting Cristian Marussi (2021-12-20 11:56:46) > > Support also atomic enable/disable clk_ops beside the bare non-atomic one > > (prepare/unprepare) when the underlying SCMI transport is configured to > > support atomic transactions for synchronous commands. > > > > Cc: Michael Turquette > > Cc: Stephen Boyd > > Cc: linux-clk@vger.kernel.org > > Signed-off-by: Cristian Marussi > > --- > > NOTE THAT STILL THERE'S NO FINE GRAIN CONTROL OVER SELECTION > > OF WHICH CLOCK DEVICES CAN SUPPORT ATOMIC AND WHICH SHOULD NOT > > BASED ON CLOCK DEVICES ENABLE LATENCY. > > THIS HAS STILL TO BE ADDED IN SCMI PROTOCOL SPEC. > > Why are you yelling on the internet? :-) I guess I need to ack this. > It is for the partners who request such changes. We are trying to prototype and share the code and ask for feedback before we finalise the specification. In fact it is other way around for you 😁. Not to ack as it is not yet final 😉. At least I need to wait until the spec contents are finalised before I can merge with your ack 😁. But I agree RFC would have indicated that along with the above background instead of *yelling*. Cristian assumed everyone is aware of the content as quite a few are involved in offline discussions. > Acked-by: Stephen Boyd Thanks anyways, will use it if nothing changes. -- Regards, Sudeep 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 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 smtp.lore.kernel.org (Postfix) with ESMTPS id E8A60C433F5 for ; Mon, 17 Jan 2022 10:32:52 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc: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=fadwi1YUcxkq3RQ0ZAZEiNBhayefrl8iWcOQtNfw7tU=; b=xA4/5DqZ4AKqdC Zj+rouFFxeDY0UI8Eq5NSV9DATJWryDSSiSjl/tuwkeQP3Z7A5xIP+QVzcI4ITgddzoZkfQ0fCrC1 soCuNXABMw4J7U0e6x9fqhel4tpuP7vQWHsYZbirP/EcfkR2UJITldRofyAEBXDOJQwqrQ5MHW8ld LRZt87uufAxvu0Eeg1NDRJgQA20p4Qy314eaqKYppwHZXvpkZga5OnHouZH84h+IjobZ6mAWNtI3M LyWBh7KV7HGpzvItj4ol2mSf0FtV1yAW/HQvrmSsqAIhfM1PCB8yS6b6JHtfgWZ3LNTavqUtLUtdt y5zQJv5bNQTfGIUCfgJA==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n9PIN-00EVeA-2u; Mon, 17 Jan 2022 10:31:27 +0000 Received: from foss.arm.com ([217.140.110.172]) by bombadil.infradead.org with esmtp (Exim 4.94.2 #2 (Red Hat Linux)) id 1n9PIJ-00EVct-LY for linux-arm-kernel@lists.infradead.org; Mon, 17 Jan 2022 10:31:25 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 7D1A96D; Mon, 17 Jan 2022 02:31:20 -0800 (PST) Received: from bogus (unknown [10.57.38.209]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id B7C0D3F73D; Mon, 17 Jan 2022 02:31:17 -0800 (PST) Date: Mon, 17 Jan 2022 10:31:00 +0000 From: Sudeep Holla To: Stephen Boyd Cc: Cristian Marussi , Sudeep Holla , linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org, james.quinlan@broadcom.com, Jonathan.Cameron@Huawei.com, f.fainelli@gmail.com, etienne.carriere@linaro.org, vincent.guittot@linaro.org, souvik.chakravarty@arm.com, Michael Turquette , linux-clk@vger.kernel.org Subject: Re: [PATCH v8 11/11] clk: scmi: Support atomic clock enable/disable API Message-ID: <20220117103100.c6ltxnz4iqmbc4b5@bogus> References: <20211220195646.44498-1-cristian.marussi@arm.com> <20211220195646.44498-12-cristian.marussi@arm.com> <20220114230839.ABD63C36AE9@smtp.kernel.org> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20220114230839.ABD63C36AE9@smtp.kernel.org> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20220117_023123_797568_002BF0DE X-CRM114-Status: GOOD ( 12.17 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org T24gRnJpLCBKYW4gMTQsIDIwMjIgYXQgMDM6MDg6MzdQTSAtMDgwMCwgU3RlcGhlbiBCb3lkIHdy b3RlOgo+IFF1b3RpbmcgQ3Jpc3RpYW4gTWFydXNzaSAoMjAyMS0xMi0yMCAxMTo1Njo0NikKPiA+ IFN1cHBvcnQgYWxzbyBhdG9taWMgZW5hYmxlL2Rpc2FibGUgY2xrX29wcyBiZXNpZGUgdGhlIGJh cmUgbm9uLWF0b21pYyBvbmUKPiA+IChwcmVwYXJlL3VucHJlcGFyZSkgd2hlbiB0aGUgdW5kZXJs eWluZyBTQ01JIHRyYW5zcG9ydCBpcyBjb25maWd1cmVkIHRvCj4gPiBzdXBwb3J0IGF0b21pYyB0 cmFuc2FjdGlvbnMgZm9yIHN5bmNocm9ub3VzIGNvbW1hbmRzLgo+ID4gCj4gPiBDYzogTWljaGFl bCBUdXJxdWV0dGUgPG10dXJxdWV0dGVAYmF5bGlicmUuY29tPgo+ID4gQ2M6IFN0ZXBoZW4gQm95 ZCA8c2JveWRAa2VybmVsLm9yZz4KPiA+IENjOiBsaW51eC1jbGtAdmdlci5rZXJuZWwub3JnCj4g PiBTaWduZWQtb2ZmLWJ5OiBDcmlzdGlhbiBNYXJ1c3NpIDxjcmlzdGlhbi5tYXJ1c3NpQGFybS5j b20+Cj4gPiAtLS0KPiA+IE5PVEUgVEhBVCBTVElMTCBUSEVSRSdTIE5PIEZJTkUgR1JBSU4gQ09O VFJPTCBPVkVSIFNFTEVDVElPTgo+ID4gT0YgV0hJQ0ggQ0xPQ0sgREVWSUNFUyBDQU4gU1VQUE9S VCBBVE9NSUMgQU5EIFdISUNIIFNIT1VMRCBOT1QKPiA+IEJBU0VEIE9OIENMT0NLIERFVklDRVMg RU5BQkxFIExBVEVOQ1kuCj4gPiBUSElTIEhBUyBTVElMTCBUTyBCRSBBRERFRCBJTiBTQ01JIFBS T1RPQ09MIFNQRUMuCj4gCj4gV2h5IGFyZSB5b3UgeWVsbGluZyBvbiB0aGUgaW50ZXJuZXQ/IDot KSBJIGd1ZXNzIEkgbmVlZCB0byBhY2sgdGhpcy4KPgoKSXQgaXMgZm9yIHRoZSBwYXJ0bmVycyB3 aG8gcmVxdWVzdCBzdWNoIGNoYW5nZXMuIFdlIGFyZSB0cnlpbmcgdG8gcHJvdG90eXBlCmFuZCBz aGFyZSB0aGUgY29kZSBhbmQgYXNrIGZvciBmZWVkYmFjayBiZWZvcmUgd2UgZmluYWxpc2UgdGhl IHNwZWNpZmljYXRpb24uCgpJbiBmYWN0IGl0IGlzIG90aGVyIHdheSBhcm91bmQgZm9yIHlvdSDw n5iBLiBOb3QgdG8gYWNrIGFzIGl0IGlzIG5vdCB5ZXQgZmluYWwK8J+YiS4gQXQgbGVhc3QgSSBu ZWVkIHRvIHdhaXQgdW50aWwgdGhlIHNwZWMgY29udGVudHMgYXJlIGZpbmFsaXNlZCBiZWZvcmUg SQpjYW4gbWVyZ2Ugd2l0aCB5b3VyIGFjayDwn5iBLiBCdXQgSSBhZ3JlZSBSRkMgd291bGQgaGF2 ZSBpbmRpY2F0ZWQgdGhhdCBhbG9uZwp3aXRoIHRoZSBhYm92ZSBiYWNrZ3JvdW5kIGluc3RlYWQg b2YgKnllbGxpbmcqLiBDcmlzdGlhbiBhc3N1bWVkIGV2ZXJ5b25lCmlzIGF3YXJlIG9mIHRoZSBj b250ZW50IGFzIHF1aXRlIGEgZmV3IGFyZSBpbnZvbHZlZCBpbiBvZmZsaW5lIGRpc2N1c3Npb25z LgoKPiBBY2tlZC1ieTogU3RlcGhlbiBCb3lkIDxzYm95ZEBrZXJuZWwub3JnPgoKVGhhbmtzIGFu eXdheXMsIHdpbGwgdXNlIGl0IGlmIG5vdGhpbmcgY2hhbmdlcy4KCi0tIApSZWdhcmRzLApTdWRl ZXAKCl9fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fCmxpbnV4 LWFybS1rZXJuZWwgbWFpbGluZyBsaXN0CmxpbnV4LWFybS1rZXJuZWxAbGlzdHMuaW5mcmFkZWFk Lm9yZwpodHRwOi8vbGlzdHMuaW5mcmFkZWFkLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2xpbnV4LWFy bS1rZXJuZWwK