[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <dd90b445-bafb-46d4-8cec-e0877cf425b3@amlogic.com>
Date: Mon, 8 Dec 2025 16:40:57 +0800
From: Jian Hu <jian.hu@...ogic.com>
To: Krzysztof Kozlowski <krzk@...nel.org>
Cc: Jerome Brunet <jbrunet@...libre.com>,
Xianwei Zhao <xianwei.zhao@...ogic.com>, Chuan Liu <chuan.liu@...ogic.com>,
Neil Armstrong <neil.armstrong@...aro.org>,
Kevin Hilman <khilman@...libre.com>, Stephen Boyd <sboyd@...nel.org>,
Michael Turquette <mturquette@...libre.com>, robh+dt <robh+dt@...nel.org>,
Rob Herring <robh@...nel.org>, devicetree <devicetree@...r.kernel.org>,
linux-clk <linux-clk@...r.kernel.org>,
linux-amlogic <linux-amlogic@...ts.infradead.org>,
linux-kernel <linux-kernel@...r.kernel.org>,
linux-arm-kernel <linux-arm-kernel@...ts.infradead.org>
Subject: Re: [PATCH v6 2/5] dt-bindings: clock: add Amlogic T7 SCMI clock
controller
Hi, Krzysztof
Thans for your review.
On 12/8/2025 2:17 PM, Krzysztof Kozlowski wrote:
> [ EXTERNAL EMAIL ]
>
> On Thu, Dec 04, 2025 at 01:36:31PM +0800, Jian Hu wrote:
>> Add DT bindings for the SCMI clock controller of the Amlogic T7 SoC family.
>>
>> Signed-off-by: Jian Hu <jian.hu@...ogic.com>
>> Acked-by: Rob Herring (Arm) <robh@...nel.org>
>> ---
>> include/dt-bindings/clock/amlogic,t7-scmi.h | 47 +++++++++++++++++++++
>> 1 file changed, 47 insertions(+)
>> create mode 100644 include/dt-bindings/clock/amlogic,t7-scmi.h
>>
> Where is any binding doc for this? Why is this a separate patch?
The ARM SCMI device tree binding specification is located at
./Documentation/devicetree/bindings/firmware/arm,scmi.yaml.
Certain secure clocks on the T7 rely on the ARM SCMI driver stack, which
is officially supported by ARM.
The kernel-side SCMI client implementation resides in
./drivers/firmware/arm_scmi/.
To enable ARM SCMI on T7, three components are needed:
- Kernel-side definition of ARM SCMI clock indices (this patch addresses
this component);
- SCMI server implementation in the ARM Trusted Firmware (ATF) running
at Exception Level 3 (EL3), which has been integrated into the bootloader;
- Device Tree Source (DTS) configuration for ARM SCMI clock nodes (the
DTS changes will be submitted after the T7 clock driver patches are
merged upstream).
>
> Best regards,
> Krzysztof
>
Powered by blists - more mailing lists