| Message ID | 1431005924-21777-9-git-send-email-lee.jones@linaro.org |
|---|---|
| State | Superseded |
| Headers | show |
Hi Lee, On Thu, 07 May 2015, Lee Jones wrote: > Signed-off-by: Lee Jones <lee.jones@linaro.org> > --- > Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt | 15 +++++++++------ > 1 file changed, 9 insertions(+), 6 deletions(-) > > diff --git a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > index 73407f50..daf8826 100644 > --- a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > +++ b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > @@ -1,20 +1,23 @@ > STMicroelectronics Low Power Controller (LPC) - RTC > =================================================== > > -LPC currently supports Watchdog OR Real Time Clock functionality. > +LPC currently supports Watchdog OR Real Time Clock OR Clocksource > +functionality. > > [See: ../watchdog/st_lpc_wdt.txt for Watchdog options] > +[See: ../timer/st,stih407-lpc for Clocksource options] > > Required properties > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > - "st,stih415-lpc" "st,stid127-lpc" > +- compatible : Must be: "st,stih407-lpc" You are removing the compatibles documentation for the other supported SoC's like stih416-lpc stih415-lpc here. regards, Peter. -- To unsubscribe from this list: send the line "unsubscribe linux-watchdog" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
> > Signed-off-by: Lee Jones <lee.jones@linaro.org> > > --- > > Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt | 15 +++++++++------ > > 1 file changed, 9 insertions(+), 6 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > > index 73407f50..daf8826 100644 > > --- a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > > +++ b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt > > @@ -1,20 +1,23 @@ > > STMicroelectronics Low Power Controller (LPC) - RTC > > =================================================== > > > > -LPC currently supports Watchdog OR Real Time Clock functionality. > > +LPC currently supports Watchdog OR Real Time Clock OR Clocksource > > +functionality. > > > > [See: ../watchdog/st_lpc_wdt.txt for Watchdog options] > > +[See: ../timer/st,stih407-lpc for Clocksource options] > > > > Required properties > > > > -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" > > - "st,stih415-lpc" "st,stid127-lpc" > > +- compatible : Must be: "st,stih407-lpc" > > You are removing the compatibles documentation for the other supported SoC's like > stih416-lpc stih415-lpc here. It would be pretty difficult to do that by mistake. :) See the comment I made to the Watchdog bindings patch for more info.
diff --git a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt index 73407f50..daf8826 100644 --- a/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt +++ b/Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt @@ -1,20 +1,23 @@ STMicroelectronics Low Power Controller (LPC) - RTC =================================================== -LPC currently supports Watchdog OR Real Time Clock functionality. +LPC currently supports Watchdog OR Real Time Clock OR Clocksource +functionality. [See: ../watchdog/st_lpc_wdt.txt for Watchdog options] +[See: ../timer/st,stih407-lpc for Clocksource options] Required properties -- compatible : Must be one of: "st,stih407-lpc" "st,stih416-lpc" - "st,stih415-lpc" "st,stid127-lpc" +- compatible : Must be: "st,stih407-lpc" - reg : LPC registers base address + size - interrupts : LPC interrupt line number and associated flags - clocks : Clock used by LPC device (See: ../clock/clock-bindings.txt) -- st,lpc-mode : The LPC can run either one of two modes ST_LPC_MODE_RTC [0] or - ST_LPC_MODE_WDT [1]. One (and only one) mode must be - selected. +- st,lpc-mode : The LPC can run either one of three modes: + ST_LPC_MODE_RTC [0] + ST_LPC_MODE_WDT [1] + ST_LPC_MODE_CLKSRC [2] + One (and only one) mode must be selected. Example: lpc@fde05000 {
Signed-off-by: Lee Jones <lee.jones@linaro.org> --- Documentation/devicetree/bindings/rtc/rtc-st-lpc.txt | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-)