From: Krzysztof Kozlowski <krzk@kernel.org>
To: Geert Uytterhoeven <geert@linux-m68k.org>
Cc: Rob Herring <robh@kernel.org>,
Sudeep Holla <sudeep.holla@kernel.org>,
Cristian Marussi <cristian.marussi@arm.com>,
Krzysztof Kozlowski <krzk+dt@kernel.org>,
Conor Dooley <conor+dt@kernel.org>,
Saravana Kannan <saravanak@kernel.org>,
Ulf Hansson <ulfh@kernel.org>,
"Rafael J . Wysocki" <rafael@kernel.org>,
Philipp Zabel <p.zabel@pengutronix.de>,
Michael Turquette <mturquette@baylibre.com>,
Stephen Boyd <sboyd@kernel.org>,
Brian Masney <bmasney@redhat.com>,
Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>,
Vinod Koul <vkoul@kernel.org>,
Konrad Dybcio <konrad.dybcio@oss.qualcomm.com>,
Kevin Hilman <khilman@baylibre.com>,
Florian Fainelli <florian.fainelli@broadcom.com>,
Wolfram Sang <wsa+renesas@sang-engineering.com>,
Marek Vasut <marek.vasut+renesas@mailbox.org>,
Kuninori Morimoto <kuninori.morimoto.gx@renesas.com>,
arm-scmi@vger.kernel.org, linux-arm-kernel@lists.infradead.org,
devicetree@vger.kernel.org, linux-pm@vger.kernel.org,
linux-clk@vger.kernel.org, linux-renesas-soc@vger.kernel.org,
linux-kernel@vger.kernel.org
Subject: Re: [PATCH v2 05/10] dt-bindings: clock: renesas,r8a78000-cpg: Add firmware property
Date: Wed, 12 Aug 2026 12:38:12 +0200 [thread overview]
Message-ID: <7b066c1f-20b7-407f-9268-e9dbeafab004@kernel.org> (raw)
In-Reply-To: <CAMuHMdXJFdJQP1GVXrOWRuYO8Si_WF5X+vpzmAgTZY+tQstR1A@mail.gmail.com>
On 12/08/2026 12:02, Geert Uytterhoeven wrote:
> Hi Krzysztof,
>
> On Wed, 12 Aug 2026 at 11:51, Krzysztof Kozlowski <krzk@kernel.org> wrote:
>> On 12/08/2026 11:47, Geert Uytterhoeven wrote:
>>>>>>>>> + firmware:
>>>>>>>>
>>>>>>>> RPi is doing similar, but I have doubts. It looks way too close to
>>>>>>>> generic "syscon" which is disallowed.
>>>>>>>>
>>>>>>>> I think this should be vendor+purpose specific, e.g.:
>>>>>>>> "renesas,scmi-transport" or "renesas,scmi-firmware"
>>>>>>
>>>>>> If you can tell me the resource that a 'firmware' provider provides then
>>>>>> it is fine, but you can't.
>>>>>
>>>>> It provides a firmware API, which is not available until the firmware
>>>>> provider is initialized.
>>>>
>>>> I don't know what "a firmware API" is. What does it provide? I know what
>>>
>>> It can provide lots of things (everything we are no longer allowed to
>>> touch directly from Linux ;-)
>>>
>>>> a clock, GPIO, reset, interrupt, etc. provider is.
>>>
>>> A clock is an object you can access through the clock API.
>>> (s/clock/.../ for the other cases).
>>> Firmware is a service (virtual object) you can access through a
>>> firmware API.
>>>
>>> In all cases, the consumer of the object or service needs to know
>>> which object or service to use (there can be multiple).
>>>
>>> In all cases you cannot access the object or service until it is
>>> available (initialized and registered), for which we have -EPROBE_DEFER,
>>> and/or "[PATCH v2 02/10] of: property: fw_devlink: Add support for
>>> firmware".
>>>
>>> Thanks for your understanding ;-)
>>
>> I think that's the difference here. Clock is not an object, but a signal
>> on a wire or within IC coming from one device (provider) to another
>> (consumer). Firmware is not a signal, not a wire, not anything which can
>> be provided from one to another.
>
> Oh yes, it is a service provided by the secure world or a different
> CPU core to the application core running Linux.
Service is still not a resource. Not mentioning it is way too generic,
because anything can be a service. We have entire services-based economies.
Best regards,
Krzysztof
next prev parent reply other threads:[~2026-08-12 10:38 UTC|newest]
Thread overview: 29+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-08 10:16 [PATCH v2 00/10] R-Car X5H Ironhide CPG/MDLC SCMI remapping support Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 01/10] firmware: arm_scmi: Add scmi_get_base_info() Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 02/10] of: property: fw_devlink: Add support for firmware Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 03/10] pmdomain: Make genpd_get_from_provider() public Geert Uytterhoeven
2026-07-14 12:31 ` Ulf Hansson
2026-07-08 10:16 ` [PATCH v2 04/10] reset: Add reset_controller_get_provider() Geert Uytterhoeven
2026-07-10 14:49 ` Philipp Zabel
2026-07-08 10:16 ` [PATCH v2 05/10] dt-bindings: clock: renesas,r8a78000-cpg: Add firmware property Geert Uytterhoeven
2026-07-10 10:21 ` Krzysztof Kozlowski
2026-07-10 11:02 ` Geert Uytterhoeven
2026-07-16 21:49 ` Rob Herring
2026-08-03 12:01 ` Geert Uytterhoeven
2026-08-11 16:33 ` Rob Herring
2026-08-12 9:47 ` Geert Uytterhoeven
2026-08-12 9:50 ` Krzysztof Kozlowski
2026-08-12 10:02 ` Geert Uytterhoeven
2026-08-12 10:38 ` Krzysztof Kozlowski [this message]
2026-08-12 11:14 ` Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 06/10] dt-bindings: power: renesas,r8a78000-mdlc: " Geert Uytterhoeven
2026-07-17 8:07 ` Sudeep Holla
2026-07-08 10:16 ` [PATCH v2 07/10] clk: renesas: r8a78000: Add SCMI remapping support Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 08/10] pmdomain: " Geert Uytterhoeven
2026-07-08 10:16 ` [PATCH v2 09/10] arm64: dts: renesas: ironhide: Enable SCMI devpd, sys, clock, and reset Geert Uytterhoeven
2026-07-08 15:42 ` [PATCH v2 00/10] R-Car X5H Ironhide CPG/MDLC SCMI remapping support Marek Vasut
2026-07-08 16:04 ` Geert Uytterhoeven
2026-07-08 16:48 ` Marek Vasut
2026-07-08 16:00 ` [PATCH v2 10/10] arm64: dts: renesas: ironhide: Add CPG/MDLC firmware properties Geert Uytterhoeven
2026-07-17 8:05 ` [PATCH v2 00/10] R-Car X5H Ironhide CPG/MDLC SCMI remapping support Sudeep Holla
2026-08-07 16:57 ` Geert Uytterhoeven
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=7b066c1f-20b7-407f-9268-e9dbeafab004@kernel.org \
--to=krzk@kernel.org \
--cc=arm-scmi@vger.kernel.org \
--cc=bartosz.golaszewski@oss.qualcomm.com \
--cc=bmasney@redhat.com \
--cc=conor+dt@kernel.org \
--cc=cristian.marussi@arm.com \
--cc=devicetree@vger.kernel.org \
--cc=florian.fainelli@broadcom.com \
--cc=geert@linux-m68k.org \
--cc=khilman@baylibre.com \
--cc=konrad.dybcio@oss.qualcomm.com \
--cc=krzk+dt@kernel.org \
--cc=kuninori.morimoto.gx@renesas.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-clk@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pm@vger.kernel.org \
--cc=linux-renesas-soc@vger.kernel.org \
--cc=marek.vasut+renesas@mailbox.org \
--cc=mturquette@baylibre.com \
--cc=p.zabel@pengutronix.de \
--cc=rafael@kernel.org \
--cc=robh@kernel.org \
--cc=saravanak@kernel.org \
--cc=sboyd@kernel.org \
--cc=sudeep.holla@kernel.org \
--cc=ulfh@kernel.org \
--cc=vkoul@kernel.org \
--cc=wsa+renesas@sang-engineering.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.