Re: [STLinux Kernel] [PATCH 08/12] rtc: bindings: Supply knowledge of a third supported device - clocksource

From: Lee Jones
Date: Thu May 07 2015 - 15:21:52 EST


> > Signed-off-by: Lee Jones <lee.jones@xxxxxxxxxx>
> > ---
> > 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.

--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org â Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/