On 08/27/2013 02:36 AM, Ezequiel Garcia wrote: > On Mon, Aug 26, 2013 at 10:08:43AM -0400, Jason Cooper wrote: >> On Fri, Aug 23, 2013 at 07:12:20PM -0300, Ezequiel Garcia wrote: >>> Change the 'reg' property meaning, by defining two required cells. >>> It's important to note this commit breaks DT-compatibility for this >>> device. >>> >>> Cc: devicetree@xxxxxxxxxxxxxxx >>> Signed-off-by: Ezequiel Garcia <ezequiel.garcia@xxxxxxxxxxxxxxxxxx> >>> --- >>> Documentation/devicetree/bindings/watchdog/orion-wdt.txt | 7 +++++-- >>> 1 file changed, 5 insertions(+), 2 deletions(-) >>> >>> diff --git a/Documentation/devicetree/bindings/watchdog/orion-wdt.txt b/Documentation/devicetree/bindings/watchdog/orion-wdt.txt >>> index 5dc8d30..a74c9c8 100644 >>> --- a/Documentation/devicetree/bindings/watchdog/orion-wdt.txt >>> +++ b/Documentation/devicetree/bindings/watchdog/orion-wdt.txt >>> @@ -3,7 +3,9 @@ >>> Required Properties: >>> >>> - Compatibility : "marvell,orion-wdt" >>> -- reg : Address of the timer registers >>> +- reg : Two cells are required: >>> + First cell contains the global timer control register. >>> + Second cell contains the watchdog counter register. BTW, you don't have 2 cells here, but 4 cells (assuming typical #address-cells and #size-cells values). Instead, you have 2 register specifiers. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
