[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAL_JsqLj8WbP=oXAovyVFOc-58eFr5xS5EJK=kpAK-eT7_TyNw@mail.gmail.com>
Date: Fri, 31 Jan 2020 10:43:25 -0600
From: Rob Herring <robh@...nel.org>
To: Douglas Anderson <dianders@...omium.org>
Cc: Andy Gross <agross@...nel.org>,
Bjorn Andersson <bjorn.andersson@...aro.org>,
Stephen Boyd <sboyd@...nel.org>,
Jeffrey Hugo <jhugo@...eaurora.org>,
Taniya Das <tdas@...eaurora.org>,
Jeffrey Hugo <jeffrey.l.hugo@...il.com>,
linux-arm-msm <linux-arm-msm@...r.kernel.org>,
Harigovindan P <harigovi@...eaurora.org>,
devicetree@...r.kernel.org, Matthias Kaehlcke <mka@...omium.org>,
Kalyan Thota <kalyan_t@...eaurora.org>,
Mark Rutland <mark.rutland@....com>,
linux-clk <linux-clk@...r.kernel.org>,
"Kristian H. Kristensen" <hoegsberg@...omium.org>,
Michael Turquette <mturquette@...libre.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 07/15] dt-bindings: clock: Fix qcom,gpucc bindings for sdm845/sc7180/msm8998
On Thu, Jan 30, 2020 at 3:12 PM Douglas Anderson <dianders@...omium.org> wrote:
>
> The qcom,gpucc bindings had a few problems with them:
>
> 1. When things were converted to yaml the name of the "gpll0 main"
> clock got changed from "gpll0" to "gpll0_main". Change it back for
> msm8998.
>
> 2. Apparently there is a push not to use purist aliases for clocks but
> instead to just use the internal Qualcomm names. For sdm845 and
> sc7180 (where the drivers haven't already been changed) move in
> this direction.
>
> Things were also getting complicated harder to deal with by jamming
> several SoCs into one file. Splitting simplifies things.
>
> Fixes: 5c6f3a36b913 ("dt-bindings: clock: Add YAML schemas for the QCOM GPUCC clock bindings")
> Signed-off-by: Douglas Anderson <dianders@...omium.org>
> ---
>
> Changes in v3:
> - Added pointer to inlude file in description.
> - Everyone but msm8998 now uses internal QC names.
> - Fixed typo grpahics => graphics
> - Split bindings into 3 files.
>
> Changes in v2:
> - Patch ("dt-bindings: clock: Fix qcom,gpucc...") new for v2.
>
> .../devicetree/bindings/clock/qcom,gpucc.yaml | 72 -------------------
> .../bindings/clock/qcom,msm8998-gpucc.yaml | 66 +++++++++++++++++
> .../bindings/clock/qcom,sc7180-gpucc.yaml | 72 +++++++++++++++++++
> .../bindings/clock/qcom,sdm845-gpucc.yaml | 72 +++++++++++++++++++
> 4 files changed, 210 insertions(+), 72 deletions(-)
> delete mode 100644 Documentation/devicetree/bindings/clock/qcom,gpucc.yaml
> create mode 100644 Documentation/devicetree/bindings/clock/qcom,msm8998-gpucc.yaml
> create mode 100644 Documentation/devicetree/bindings/clock/qcom,sc7180-gpucc.yaml
> create mode 100644 Documentation/devicetree/bindings/clock/qcom,sdm845-gpucc.yaml
I'm not seeing any differences in sdm845 and sc7180. Do those really
need to be separate? It doesn't have to be all combined or all
separate.
Rob
Powered by blists - more mailing lists