[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <78856df3.17ba.1997acdc444.Coremail.dongxuyang@eswincomputing.com>
Date: Wed, 24 Sep 2025 16:18:49 +0800 (GMT+08:00)
From: "Xuyang Dong" <dongxuyang@...incomputing.com>
To: "Conor Dooley" <conor@...nel.org>
Cc: mturquette@...libre.com, sboyd@...nel.org, robh@...nel.org,
krzk+dt@...nel.org, conor+dt@...nel.org, linux-clk@...r.kernel.org,
devicetree@...r.kernel.org, linux-kernel@...r.kernel.org,
ningyu@...incomputing.com, linmin@...incomputing.com,
huangyifeng@...incomputing.com, pinkesh.vaghela@...fochips.com
Subject: Re: Re: [PATCH v5 1/2] dt-bindings: clock: eswin: Documentation for
eic7700 SoC
> >
> > Add device tree binding documentation for the ESWIN eic7700
> > clock controller module.
> >
> > Signed-off-by: Yifeng Huang <huangyifeng@...incomputing.com>
> > Signed-off-by: Xuyang Dong <dongxuyang@...incomputing.com>
> > ---
> > .../bindings/clock/eswin,eic7700-clock.yaml | 40 ++
> > .../dt-bindings/clock/eswin,eic7700-clock.h | 379 ++++++++++++++++++
> > 2 files changed, 419 insertions(+)
> > create mode 100644 Documentation/devicetree/bindings/clock/eswin,eic7700-clock.yaml
> > create mode 100644 include/dt-bindings/clock/eswin,eic7700-clock.h
> >
> > diff --git a/Documentation/devicetree/bindings/clock/eswin,eic7700-clock.yaml b/Documentation/devicetree/bindings/clock/eswin,eic7700-clock.yaml
> > new file mode 100644
> > index 000000000000..49053543ecfe
> > --- /dev/null
> > +++ b/Documentation/devicetree/bindings/clock/eswin,eic7700-clock.yaml
> > @@ -0,0 +1,40 @@
> > +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause)
> > +%YAML 1.2
> > +---
> > +$id: http://devicetree.org/schemas/clock/eswin,eic7700-clock.yaml#
> > +$schema: http://devicetree.org/meta-schemas/core.yaml#
> > +
> > +title: Eswin EIC7700 SoC clock controller
> > +
> > +maintainers:
> > + - Yifeng Huang <huangyifeng@...incomputing.com>
> > + - Xuyang Dong <dongxuyang@...incomputing.com>
> > +
> > +description:
> > + The clock controller generates and supplies clock to all the modules
> > + for eic7700 SoC.
> > +
> > +properties:
> > + compatible:
> > + const: eswin,eic7700-clock
> > +
> > + reg:
> > + maxItems: 1
> > +
> > + '#clock-cells':
> > + const: 1
> > +
> > +required:
> > + - compatible
> > + - reg
> > + - '#clock-cells'
> > +
> > +additionalProperties: false
> > +
> > +examples:
> > + - |
> > + clock-controller@...28000 {
> > + compatible = "eswin,eic7700-clock";
> > + reg = <0x51828000 0x200>;
> > + #clock-cells = <1>;
> > + };
>
> No clock input to this block? Surprised there's not some off-chip
> oscillator that provides a quality reference for the internal PLLs etc.
There is an oscillator as the clock input. The frequency is 24000000 Hz, and
the clock name is "xtal".
We will update the following description in the next patch.
properties:
compatible:
const: eswin,eic7700-clock
reg:
maxItems: 1
clocks:
items:
- description: External 24MHz oscillator clock
clock-names:
items:
- const: xtal
'#clock-cells':
const: 1
required:
- compatible
- reg
- clocks
- clock-names
- '#clock-cells'
additionalProperties: false
examples:
- |
clock-controller@...28000 {
compatible = "eswin,eic7700-clock";
reg = <0x51828000 0x200>;
clocks = <&xtal>;
clock-names = "xtal";
#clock-cells = <1>;
};
properties:
compatible:
const: eswin,eic7700-clock
reg:
maxItems: 1
clocks:
items:
- description: External 24MHz oscillator clock
clock-names:
items:
- const: xtal
'#clock-cells':
const: 1
required:
- compatible
- reg
- clocks
- clock-names
- '#clock-cells'
additionalProperties: false
Powered by blists - more mailing lists