All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH v2] rtc: s3c: Document in binding that only s3c6410 needs a src clk
@ 2016-02-15 14:11 ` Javier Martinez Canillas
  0 siblings, 0 replies; 13+ messages in thread
From: Javier Martinez Canillas @ 2016-02-15 14:11 UTC (permalink / raw)
  To: linux-kernel, Rob Herring
  Cc: Alexandre Belloni, linux-samsung-soc, Krzysztof Kozlowski,
	rtc-linux, devicetree, Javier Martinez Canillas

The S3C binding doc says that the RTC and RTC source clocks are required
but the S3C driver supports different HW IP and only the s3c6410 needs a
source clock.

Fix the binding explaining that the source clock is only needed for the
s3c6410-rtc compatible controller.

Reported-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Signed-off-by: Javier Martinez Canillas <javier@osg.samsung.com>
Acked-by: Alexandre Belloni <alexandre.belloni@free-electrons.com>

---
Hello,

This patch depends on a previous one that was picked by Rob Herring so
this one has to go through his tree again and not through the RTC tree
to avoid conflicts.

Best regards,
Javier

Changes in v2:
- Remove the mention to a commit sha-1 that's not in mainline (Alexandre).
- Fix typo in commit message (Alexandre).
- Add Alexandre's Acked-by tag.

 Documentation/devicetree/bindings/rtc/s3c-rtc.txt | 7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/Documentation/devicetree/bindings/rtc/s3c-rtc.txt b/Documentation/devicetree/bindings/rtc/s3c-rtc.txt
index 1068ffce9f91..fdde63a5419c 100644
--- a/Documentation/devicetree/bindings/rtc/s3c-rtc.txt
+++ b/Documentation/devicetree/bindings/rtc/s3c-rtc.txt
@@ -15,9 +15,10 @@ Required properties:
   is the rtc tick interrupt. The number of cells representing a interrupt
   depends on the parent interrupt controller.
 - clocks: Must contain a list of phandle and clock specifier for the rtc
-          and source clocks.
-- clock-names: Must contain "rtc" and "rtc_src" entries sorted in the
-               same order as the clocks property.
+          clock and in the case of a s3c6410 compatible controller, also
+          a source clock.
+- clock-names: Must contain "rtc" and for a s3c6410 compatible controller,
+               a "rtc_src" sorted in the same order as the clocks property.
 
 Example:
 
-- 
2.5.0

^ permalink raw reply related	[flat|nested] 13+ messages in thread

end of thread, other threads:[~2016-04-21 13:09 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-02-15 14:11 [PATCH v2] rtc: s3c: Document in binding that only s3c6410 needs a src clk Javier Martinez Canillas
2016-02-15 14:11 ` [rtc-linux] " Javier Martinez Canillas
2016-02-15 23:22 ` Krzysztof Kozlowski
2016-02-15 23:22   ` [rtc-linux] " Krzysztof Kozlowski
2016-02-18 14:38 ` Rob Herring
2016-02-18 14:38   ` [rtc-linux] " Rob Herring
2016-03-17 22:31   ` Alexandre Belloni
2016-03-17 22:31     ` [rtc-linux] " Alexandre Belloni
2016-03-28 22:19 ` [rtc-linux] " Alexandre Belloni
2016-04-21 12:46   ` Javier Martinez Canillas
2016-04-21 12:46     ` Javier Martinez Canillas
2016-04-21 13:09     ` Alexandre Belloni
2016-04-21 13:09       ` Alexandre Belloni

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.