From patchwork Tue Feb 20 10:43:33 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Biju Das X-Patchwork-Id: 13563857 Received: from relmlie6.idc.renesas.com (relmlor2.renesas.com [210.160.252.172]) by smtp.subspace.kernel.org (Postfix) with ESMTP id EFAE762178; Tue, 20 Feb 2024 10:43:49 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=210.160.252.172 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708425832; cv=none; b=prpszjHAdsBKwUNPDePVDKJJCTTn3SrzElNM9nQbnB9lBpvUs0nvrlQjgMeV3bteBhRHuqb86129YvVnbhLyIC+bWyn8iLXArIJiFcbsTpjKUGCfepsiaCd9Gf29E4K1UEHx+//vbQv/9xTOKeSgsgyjM8RQ/Yem63JQC0XX07o= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1708425832; c=relaxed/simple; bh=gJNDcnnI5rP7McNDysSXsx8zm9W44QaG4sKI/u39w2Y=; h=From:To:Cc:Subject:Date:Message-Id:MIME-Version; b=r5KfhBMM9qVg6oz0vIRU+IC9cLA0yo4F5RwPAc0oCBRQawONWeKEy0KFpqfmjQdh7EY4xZCax74O3CqycK7+D5b7kLZcnB/bVP8ImimFNc0KHw11LZtqxLtApu6Ngr70nxb8shjBSLxv9O/mBJgq5TJHRiLP8Z5vjPwbN/XInh4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com; spf=pass smtp.mailfrom=bp.renesas.com; arc=none smtp.client-ip=210.160.252.172 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=bp.renesas.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=bp.renesas.com X-IronPort-AV: E=Sophos;i="6.06,172,1705330800"; d="scan'208";a="198515560" Received: from unknown (HELO relmlir5.idc.renesas.com) ([10.200.68.151]) by relmlie6.idc.renesas.com with ESMTP; 20 Feb 2024 19:43:43 +0900 Received: from localhost.localdomain (unknown [10.226.92.246]) by relmlir5.idc.renesas.com (Postfix) with ESMTP id B40E24001942; Tue, 20 Feb 2024 19:43:38 +0900 (JST) From: Biju Das To: Michael Turquette , Stephen Boyd , Sakari Ailus , Laurent Pinchart , Mauro Carvalho Chehab , Hans Verkuil , Russell King Cc: Biju Das , linux-clk@vger.kernel.org, Geert Uytterhoeven , Prabhakar Mahadev Lad , Biju Das , linux-media@vger.kernel.org, linux-renesas-soc@vger.kernel.org Subject: [PATCH v2 0/3] Add clk_poll_disable_unprepare() Date: Tue, 20 Feb 2024 10:43:33 +0000 Message-Id: <20240220104336.260194-1-biju.das.jz@bp.renesas.com> X-Mailer: git-send-email 2.25.1 Precedence: bulk X-Mailing-List: linux-media@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Currently documentation related to disabling of clock do not state it is synchronous or asynchronous clock gating. So, make it clear it is driver dependent. The clk_disable_unprepare() doesn't guarantee that a clock is gated after the execution as it is driver dependent. The Renesas and most of the other platforms don't wait until clock is stopped because of performance reason. But these platforms wait while turning on the clock. The normal case for shutting down the clock is unbind/close/suspend or error paths in the driver. Not waiting for the shutting down the clock will improve the suspend time. But on RZ/G2L Camera Data Receiving Unit (CRU) IP, initially the vclk is on. Before enabling link reception, we need to wait for vclk to be off and after enabling reception, we need to turn the vlck on. Special cases like this requires a sync API for clock gating. Add clk_poll_disable_unprepare() to poll the clock gate operation that guarantees gating of clk after the execution. RFC->v2: * Updated cover letter description and header. * Created patch#1 for updating existing documentation. * Renamed clk_disable_unprepare_sync()-->clk_poll_disable_unprepare() * Redesigned to make use of __clk_is_enabled() to poll the clock gating. Biju Das (3): clk: Update API documentation related to clock disable clk: Add clk_poll_disable_unprepare() media: platform: rzg2l-cru: rzg2l-video: Use clk_poll_disable_unprepare() drivers/clk/clk.c | 26 +++++++++- .../platform/renesas/rzg2l-cru/rzg2l-csi2.c | 5 +- include/linux/clk-provider.h | 3 +- include/linux/clk.h | 49 ++++++++++++++++++- 4 files changed, 79 insertions(+), 4 deletions(-)