Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]
Groups > linux.kernel > #1619134 > unrolled thread
| Started by | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| First post | 2017-04-08 00:00 +0200 |
| Last post | 2017-04-08 00:10 +0200 |
| Articles | 5 — 3 participants |
Back to article view | Back to linux.kernel
[PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer Linus Walleij <linus.walleij@linaro.org> - 2017-04-08 00:00 +0200
Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-04-08 00:10 +0200
Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer Rob Herring <robh+dt@kernel.org> - 2017-04-08 00:10 +0200
Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-04-08 00:20 +0200
Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer Daniel Lezcano <daniel.lezcano@linaro.org> - 2017-04-08 00:10 +0200
| From | Linus Walleij <linus.walleij@linaro.org> |
|---|---|
| Date | 2017-04-08 00:00 +0200 |
| Subject | [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer |
| Message-ID | <ttK5b-4R7-9@gated-at.bofh.it> |
It turns out that the Cortina Gemini timer block is just a
standard IP block from Faraday Technology named FTTMR010.
In order to make things clear and understandable, we rename the
bindings with a Faraday compatible as primary and the Cortina
gemini as a more specific case.
For the plain Faraday timer we require two clock references,
while the Gemini can keep it's syscon lookup pattern.
Cc: devicetree@vger.kernel.org
Signed-off-by: Linus Walleij <linus.walleij@linaro.org>
---
.../bindings/timer/cortina,gemini-timer.txt | 22 ---------------
.../devicetree/bindings/timer/faraday,fttmr010.txt | 33 ++++++++++++++++++++++
2 files changed, 33 insertions(+), 22 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt
create mode 100644 Documentation/devicetree/bindings/timer/faraday,fttmr010.txt
diff --git a/Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt b/Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt
deleted file mode 100644
index 16ea1d3b2e9e..000000000000
--- a/Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt
+++ /dev/null
@@ -1,22 +0,0 @@
-Cortina Systems Gemini timer
-
-This timer is embedded in the Cortina Systems Gemini SoCs.
-
-Required properties:
-
-- compatible : Must be "cortina,gemini-timer"
-- reg : Should contain registers location and length
-- interrupts : Should contain the three timer interrupts with
- flags for rising edge
-- syscon : a phandle to the global Gemini system controller
-
-Example:
-
-timer@43000000 {
- compatible = "cortina,gemini-timer";
- reg = <0x43000000 0x1000>;
- interrupts = <14 IRQ_TYPE_EDGE_RISING>, /* Timer 1 */
- <15 IRQ_TYPE_EDGE_RISING>, /* Timer 2 */
- <16 IRQ_TYPE_EDGE_RISING>; /* Timer 3 */
- syscon = <&syscon>;
-};
diff --git a/Documentation/devicetree/bindings/timer/faraday,fttmr010.txt b/Documentation/devicetree/bindings/timer/faraday,fttmr010.txt
new file mode 100644
index 000000000000..b73ca6cd07f8
--- /dev/null
+++ b/Documentation/devicetree/bindings/timer/faraday,fttmr010.txt
@@ -0,0 +1,33 @@
+Faraday Technology timer
+
+This timer is a generic IP block from Faraday Technology, embedded in the
+Cortina Systems Gemini SoCs and other designs.
+
+Required properties:
+
+- compatible : Must be one of
+ "faraday,fttmr010"
+ "cortina,gemini-timer"
+- reg : Should contain registers location and length
+- interrupts : Should contain the three timer interrupts usually with
+ flags for falling edge
+
+Optionally required properties:
+
+- clocks : a clock to provide the tick rate for "faraday,fttmr010"
+- clock-names : should be "EXTCLK" and "PCLK" for the external tick timer
+ and peripheral clock respectively, for "faraday,fttmr010"
+- syscon : a phandle to the global Gemini system controller if the compatible
+ type is "cortina,gemini-timer"
+
+Example:
+
+timer@43000000 {
+ compatible = "faraday,fttmr010";
+ reg = <0x43000000 0x1000>;
+ interrupts = <14 IRQ_TYPE_EDGE_FALLING>, /* Timer 1 */
+ <15 IRQ_TYPE_EDGE_FALLING>, /* Timer 2 */
+ <16 IRQ_TYPE_EDGE_FALLING>; /* Timer 3 */
+ clocks = <&extclk>, <&pclk>;
+ clock-names = "EXTCLK", "PCLK";
+};
--
2.9.3
[toc] | [next] | [standalone]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2017-04-08 00:10 +0200 |
| Subject | Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer |
| Message-ID | <ttKeS-59B-3@gated-at.bofh.it> |
| In reply to | #1619134 |
On 08/04/2017 00:01, Rob Herring wrote: > On Fri, Apr 7, 2017 at 4:55 PM, Linus Walleij <linus.walleij@linaro.org> wrote: >> It turns out that the Cortina Gemini timer block is just a >> standard IP block from Faraday Technology named FTTMR010. >> >> In order to make things clear and understandable, we rename the >> bindings with a Faraday compatible as primary and the Cortina >> gemini as a more specific case. >> >> For the plain Faraday timer we require two clock references, >> while the Gemini can keep it's syscon lookup pattern. >> >> Cc: devicetree@vger.kernel.org >> Signed-off-by: Linus Walleij <linus.walleij@linaro.org> >> --- >> .../bindings/timer/cortina,gemini-timer.txt | 22 --------------- >> .../devicetree/bindings/timer/faraday,fttmr010.txt | 33 ++++++++++++++++++++++ >> 2 files changed, 33 insertions(+), 22 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt >> create mode 100644 Documentation/devicetree/bindings/timer/faraday,fttmr010.txt > > Does the -M option not work here? > > In any case, > > Acked-by: Rob Herring <robh@kernel.org> Thanks. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
[toc] | [prev] | [next] | [standalone]
| From | Rob Herring <robh+dt@kernel.org> |
|---|---|
| Date | 2017-04-08 00:10 +0200 |
| Message-ID | <ttKeS-59B-5@gated-at.bofh.it> |
| In reply to | #1619134 |
On Fri, Apr 7, 2017 at 4:55 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > It turns out that the Cortina Gemini timer block is just a > standard IP block from Faraday Technology named FTTMR010. > > In order to make things clear and understandable, we rename the > bindings with a Faraday compatible as primary and the Cortina > gemini as a more specific case. > > For the plain Faraday timer we require two clock references, > while the Gemini can keep it's syscon lookup pattern. > > Cc: devicetree@vger.kernel.org > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > --- > .../bindings/timer/cortina,gemini-timer.txt | 22 --------------- > .../devicetree/bindings/timer/faraday,fttmr010.txt | 33 ++++++++++++++++++++++ > 2 files changed, 33 insertions(+), 22 deletions(-) > delete mode 100644 Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt > create mode 100644 Documentation/devicetree/bindings/timer/faraday,fttmr010.txt Does the -M option not work here? In any case, Acked-by: Rob Herring <robh@kernel.org>
[toc] | [prev] | [next] | [standalone]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2017-04-08 00:20 +0200 |
| Subject | Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer |
| Message-ID | <ttKox-5cY-7@gated-at.bofh.it> |
| In reply to | #1619138 |
On Fri, Apr 07, 2017 at 05:01:53PM -0500, Rob Herring wrote: > On Fri, Apr 7, 2017 at 4:55 PM, Linus Walleij <linus.walleij@linaro.org> wrote: > > It turns out that the Cortina Gemini timer block is just a > > standard IP block from Faraday Technology named FTTMR010. > > > > In order to make things clear and understandable, we rename the > > bindings with a Faraday compatible as primary and the Cortina > > gemini as a more specific case. > > > > For the plain Faraday timer we require two clock references, > > while the Gemini can keep it's syscon lookup pattern. > > > > Cc: devicetree@vger.kernel.org > > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > > --- > > .../bindings/timer/cortina,gemini-timer.txt | 22 --------------- > > .../devicetree/bindings/timer/faraday,fttmr010.txt | 33 ++++++++++++++++++++++ > > 2 files changed, 33 insertions(+), 22 deletions(-) > > delete mode 100644 Documentation/devicetree/bindings/timer/cortina,gemini-timer.txt > > create mode 100644 Documentation/devicetree/bindings/timer/faraday,fttmr010.txt > > Does the -M option not work here? > > In any case, > > Acked-by: Rob Herring <robh@kernel.org> Thanks. -- <http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook | <http://twitter.com/#!/linaroorg> Twitter | <http://www.linaro.org/linaro-blog/> Blog
[toc] | [prev] | [next] | [standalone]
| From | Daniel Lezcano <daniel.lezcano@linaro.org> |
|---|---|
| Date | 2017-04-08 00:10 +0200 |
| Subject | Re: [PATCH 1/3 RESEND] clocksource: Augment bindings for Faraday timer |
| Message-ID | <ttKeS-59B-11@gated-at.bofh.it> |
| In reply to | #1619134 |
On Fri, Apr 07, 2017 at 11:55:13PM +0200, Linus Walleij wrote: > It turns out that the Cortina Gemini timer block is just a > standard IP block from Faraday Technology named FTTMR010. > > In order to make things clear and understandable, we rename the > bindings with a Faraday compatible as primary and the Cortina > gemini as a more specific case. > > For the plain Faraday timer we require two clock references, > while the Gemini can keep it's syscon lookup pattern. > > Cc: devicetree@vger.kernel.org > Signed-off-by: Linus Walleij <linus.walleij@linaro.org> > --- Series applied. Thanks. -- Daniel
[toc] | [prev] | [standalone]
Back to top | Article view | linux.kernel
csiph-web