[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <36f2ng6bdovp63xmojrs4sxmbwnapl6oo2isehmiiuh6xl2les@7nfpou4ppkp7>
Date: Wed, 6 Nov 2024 12:57:29 +0100
From: Krzysztof Kozlowski <krzk@...nel.org>
To: Imran Shaik <quic_imrashai@...cinc.com>
Cc: Bjorn Andersson <andersson@...nel.org>,
Michael Turquette <mturquette@...libre.com>, Stephen Boyd <sboyd@...nel.org>, Rob Herring <robh@...nel.org>,
Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor Dooley <conor+dt@...nel.org>,
Ajit Pandey <quic_ajipan@...cinc.com>, Taniya Das <quic_tdas@...cinc.com>,
Jagadeesh Kona <quic_jkona@...cinc.com>, Satya Priya Kakitapalli <quic_skakitap@...cinc.com>,
linux-arm-msm@...r.kernel.org, linux-clk@...r.kernel.org, devicetree@...r.kernel.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 1/6] dt-bindings: clock: qcom: Add GPU clocks for
QCS8300
On Wed, Nov 06, 2024 at 03:21:56PM +0530, Imran Shaik wrote:
> The QCS8300 GPU clock controller is mostly identical to SA8775P, but
> QCS8300 has few additional clocks and minor differences. Hence, reuse
> SA8775P gpucc bindings and add additional clocks required for QCS8300.
>
> Signed-off-by: Imran Shaik <quic_imrashai@...cinc.com>
> ---
> Documentation/devicetree/bindings/clock/qcom,gpucc.yaml | 3 +++
> include/dt-bindings/clock/qcom,qcs8300-gpucc.h | 17 +++++++++++++++++
> 2 files changed, 20 insertions(+)
>
That's a bit unusual, I think none of other SoCs do it. Last time you
brought existing example which used the same header and prefixes.
Anyway, it's fine from DT point of view, if that's what SoC/clock
maintainers want so:
Acked-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>
Best regards,
Krzysztof
Powered by blists - more mailing lists