From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ej2-f17.google.com (mail-ej2-f17.google.com [74.125.228.145]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id B414453B351 for ; Thu, 17 Sep 2026 17:05:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=74.125.228.145 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789664759; cv=none; b=I6M9xEOVGWompLWCutvcG03tfVxLf3wLn/wTXeKnJncXSRB8Rbbs/IRT9KhPbyZ4VE84A4KjuFjMkAZ9fF8TDW0HDrRyaEqOaxQAMDSP7Ak+Uy0FAZcyEa14KIX6gBHAoMPehm/G3HdTz+cYS+ODiHgF3ODftNe+ki9pZJf/5Jg= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789664759; c=relaxed/simple; bh=RoY3EB5k0t9YlxK6r86vJXHGTsBlXmMy09ib9fpZf8s=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=buVFtI305vGtg+LgT8KQYg38b1sU/YtAbv6STFL9hcb26b6urGToPV34EJmgUaIy9bFtkIZuux6uN6uAv+FLvxIVqt/OpJRAOK5uq7mV2VoHJLLUP2S1En5tEMIPFAnZ0ZKeZuw2ROKtJfCzWWpWQMxXzEj6cFyMsOH8/7Seu3A= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b=eIpeir7G; arc=none smtp.client-ip=74.125.228.145 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com header.b="eIpeir7G" Received: by mail-ej2-f17.google.com with SMTP id a640c23a62f3a-c29d50b7cf9so183896666b.2 for ; Thu, 17 Sep 2026 10:05:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20251104; t=1789664756; x=1790269556; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=nT/QhDBYyAoNvkb2DNjew9sHsXIHdx5twpGKwbTIO+4=; b=eIpeir7GMm5JARi/Ic/exd4EalL80vvx8iByFTJ0bHqNI0+zlQZQWiPOd8wB3p3SsH 8pqM6F6kj/BmpRKbR6v1+0/eFI2yZxYwC/sj4OZJXfcNT8o0dbh6TPgHsE0awXBAKOcn LU4Rb0bcbD5S7oOCJzLc5JEdNwOU8rkK5yKifgCGDtg0wjI/YGP32t86lzQH9Zv6akM+ Y3tZGNp45vPGb/D5xx1JRIvGpDwkOt6us3H67MhJ161RFRLLijvdJhgdQhbpcjviVz85 CrWJsGm0t+xGgLEQH+/K0qpjCAPoGB8fDTyytQph5iBB5zOqsu6oZEtautGct3UMimY3 85DQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1789664756; x=1790269556; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=nT/QhDBYyAoNvkb2DNjew9sHsXIHdx5twpGKwbTIO+4=; b=cW+bXhsqT4+qMGivIptentWcyfW2yKQGbZmP8/GCVIcf1us8yorfjgp0a0xXzZg3Kf tJt1r+Ahj56FulycOByJmGrqr+YYchtroj3BaVHnNrJUM/pTrv4qqtgYhV4OGc5WBtmV ELgHoTmb/U1jCwhNFd9qcitjL1nOfimLSOo073gn3bZ/WCbdMQK4S9ZX8zSpTQ22Om4P xHROoskM/vD4q1zLO4AqSFH9CSkUYmVrk6QFcpJ9h0QbPeO24nmXDRs1nQe49gptYK7K ydASqAoOwgjhKiohsNEhoyvx2w12gysKh2czifB9cJfcIjpAZZUpNNFuavY3pIPTtzs2 3e4w== X-Gm-Message-State: AFuF++kcVMDEGY+naGaJ474Rf0ODwGnueXWrEyIxxs7418tJFRoaNK0H OwBZ3tDZPsEepEBi2/LNu7taSz7VbE+alkXB8n4C8VM6b3Kz/hGwBDUC X-Gm-Gg: AYBFou2In3+cmJNtjPzghqv3CQJCwxBWBw+S1v/zuLjHBQYfVfIWM4a0+vtpSUgo6mW Dpw/PyqsPl8dJWI0Z6b/DuUdX7/kjhI2EbzwfMcoXyg/jyppDLWtXzzDWD0rXdSbtAjgLrTTHg+ Wdpu8wbZ35KdcSqFpN/v92SH0B+y1OhNs3m9t+W6VsyaOhd+0TVn62+hJu6zWayNrq1mtL0XZ6e hE3Oan6juAtsghsgPdkJVmWPrdbVx4+QqoWrtOjNB+VYO6AYlPvW7lnrJBQIfz31kz9cg6+n4F4 wBIfWNUg962zQaDfnDAtW5KHfAyM+9lfJO4d8m7kwZi9eTgiSQ4/Bb8XmfIBZWsgxhVJC13XhDo nFlaxheaJq+Wb+9BbifPQoF+Cko5zKuAes+mPrsZhXdpf5vTNeXAZzKX/hzrmzEnFan6RdpmKUt WDXsAgUkK/wXYIpvpH5Tf9gDCjz49+4jdWXtPdVL+AFznXk4qqSAMnHpy6jCgCpIe4aof52prcQ wdMTGjRtd6L9zNF X-Received: by 2002:a17:906:7944:b0:c29:4d53:787f with SMTP id a640c23a62f3a-c29e532ccedmr581845366b.41.1789664755628; Thu, 17 Sep 2026 10:05:55 -0700 (PDT) Received: from [10.43.58.190] ([185.94.190.186]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-c29de4893aasm312844766b.30.2026.09.17.10.05.51 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 17 Sep 2026 10:05:53 -0700 (PDT) Message-ID: <39fdddbf-0fdb-40e7-9cd5-c8f857ee8b9d@gmail.com> Date: Thu, 17 Sep 2026 19:05:50 +0200 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v5 2/2] clk: Add gpio-locked fixed clock driver To: Jerome Brunet , Vyacheslav Yurkov via B4 Relay , Michael Turquette , Stephen Boyd , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Brian Masney , Brian Masney , Jerome Brunet , Jyri Sarha Cc: linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, Vyacheslav Yurkov References: <20260915-feature-clock-guard-v5-0-42ab5dc3a6aa@bruker.com> <20260915-feature-clock-guard-v5-2-42ab5dc3a6aa@bruker.com> <1j33vaesh7.fsf@starbuckisacylon.baylibre.com> <1jtsnpcn8l.fsf@starbuckisacylon.baylibre.com> Content-Language: en-US From: Vyacheslav Yurkov In-Reply-To: <1jtsnpcn8l.fsf@starbuckisacylon.baylibre.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 16.09.2026 15:57, Jerome Brunet wrote: >>>> +/* We can't enable the clock, but the Common Clock Framework calls only >>>> + * enable() not is_enabled() >>>> + */ >>>> +static int gpio_locked_clk_enable(struct clk_hw *hw) >>>> +{ >>>> + return gpio_locked_clk_is_enabled(hw); >>>> +} > > You need to explain your problem a bit more because this is not OK. Your > clock should really just provide .is_enabled() AFAICT When a peripheral driver uses devm_clk_get_enabled(), this is resolved to clk_prepare() and clk_enable(). Neither of them check for .is_enabled(). Is that a flaw in the CCF or expected behavior? >>>> + >>>> +/* We have to implement it, but we are not going to control >>>> + * parent clock selection >>>> + */ >>>> +static u8 gpio_locked_clk_get_parent(struct clk_hw *hw) >>>> +{ >>>> + return 0; >>>> +} > > Same, I dont get why you need that. Not needed if there a single parent This is a remnant from v4, thanks for spotting it. Slava