On 2016-06-27 22:20, Guenter Roeck wrote:
On 06/27/2016 04:26 PM, Thomas Pedersen wrote:
From: Matthew McClintock <mmcclint@xxxxxxxxxxxxxx>
Update the compatible string to align with driver
CC: linux-watchdog@xxxxxxxxxxxxxxx
Signed-off-by: Matthew McClintock <mmcclint@xxxxxxxxxxxxxx>
---
Documentation/devicetree/bindings/watchdog/qcom-wdt.txt | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt
b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt
index 4726924..60bb2f98 100644
--- a/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt
+++ b/Documentation/devicetree/bindings/watchdog/qcom-wdt.txt
@@ -4,9 +4,8 @@ Qualcomm Krait Processor Sub-system (KPSS) Watchdog
Required properties :
- compatible : shall contain only one of the following:
- "qcom,kpss-wdt-msm8960"
- "qcom,kpss-wdt-apq8064"
- "qcom,kpss-wdt-ipq8064"
+ "qcom,kpss-timer"
+ "qcom,scss-timer"
- reg : shall contain base register location and length
- clocks : shall contain the input clock
Rob's earlier feedback to the same patch you had submitted in March:
"Keep the SoC specific ones even if they are not used. The DTS should
have both strings."
Why would we add the SoC specific compatible strings to the DTS if
they're not
even used?
--
thomas
--
To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
