On Tue, May 30, 2017 at 05:50:24PM -0500, Rob Herring wrote: > On Sat, May 20, 2017 at 04:32:12PM +0300, Cosar Dindar wrote: > > Add device tree binding for STM32F4. > > > > Signed-off-by: Cosar Dindar <cosardindar@xxxxxxxxx> > > --- > > Documentation/devicetree/bindings/crypto/st,stm32-crc.txt | 4 ++-- > > 1 file changed, 2 insertions(+), 2 deletions(-) > > > > diff --git a/Documentation/devicetree/bindings/crypto/st,stm32-crc.txt b/Documentation/devicetree/bindings/crypto/st,stm32-crc.txt > > index 3ba92a5..7b30f1e 100644 > > --- a/Documentation/devicetree/bindings/crypto/st,stm32-crc.txt > > +++ b/Documentation/devicetree/bindings/crypto/st,stm32-crc.txt > > @@ -1,7 +1,7 @@ > > * STMicroelectronics STM32 CRC > > > > Required properties: > > -- compatible: Should be "st,stm32f7-crc". > > +- compatible: Can be either "st,stm32f7-crc" or "st,srm32f4-crc". > > Here you say either, but the example has both. > Thanks for reviewing. Here it could be changed as : Should be one of the following string "st,stm32f7-crc" "st,stm32f4-crc" Also, example may not contain both binding strings, I think. It could be leaved unchanged as in the previous version. I will make these changes with this way if it is OK. > > - reg: The address and length of the peripheral registers space > > - clocks: The input clock of the CRC instance > > > > @@ -10,7 +10,7 @@ Optional properties: none > > Example: > > > > crc: crc@40023000 { > > - compatible = "st,stm32f7-crc"; > > + compatible = "st,stm32f7-crc", "st,stm32f4-crc"; > > reg = <0x40023000 0x400>; > > clocks = <&rcc 0 12>; > > }; > > -- > > 2.7.4 > >