2015-05-23 6:44 GMT+08:00 Suman Anna <s-a...@ti.com>: > Hi Barry, > > On 05/19/2015 01:41 AM, Barry Song wrote: >> From: Wei Chen <wei.c...@csr.com> >> >> The Hardware Spinlock device on atlas7 provides hardware assistance >> for synchronization between the multiple processors in the system >> (dual Cortex-A7, CAN bus Cortex-M3 and audio DSP). >> This patch adds the DT bindings information for this hwspinlock >> module. >> >> Cc: Suman Anna <s-a...@ti.com> >> Cc: Bjorn Andersson <bj...@kryo.se> >> Signed-off-by: Wei Chen <wei.c...@csr.com> >> Signed-off-by: Barry Song <baohua.s...@csr.com> >> --- >> .../devicetree/bindings/hwlock/sirf,hwspinlock.txt | 25 >> ++++++++++++++++++++++ >> 1 file changed, 25 insertions(+) >> create mode 100644 >> Documentation/devicetree/bindings/hwlock/sirf,hwspinlock.txt >> >> diff --git a/Documentation/devicetree/bindings/hwlock/sirf,hwspinlock.txt >> b/Documentation/devicetree/bindings/hwlock/sirf,hwspinlock.txt >> new file mode 100644 >> index 0000000..cc6d351 >> --- /dev/null >> +++ b/Documentation/devicetree/bindings/hwlock/sirf,hwspinlock.txt >> @@ -0,0 +1,25 @@ >> +SIRF Hardware spinlock device Binding >> +----------------------------------------------- >> + >> +Required properties : >> +- compatible : shall contain only one of the following: >> + "sirf,hwspinlock" >> + >> +- reg : the register address of hwspinlock > > Also suggest to document the value to be used for #hwlock-cells, the > generic hwlock binding does not mention that.
do you think whether we can put one line like? #hwlock-cells : <1>, as required by generic hwspinlock binding. > >> + >> +Please look at the generic hwlock binding for usage information for >> consumers, >> +"Documentation/devicetree/bindings/hwlock/hwlock.txt" >> + > > regards > Suman > -barry -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
