On Thu, Oct 24, 2019 at 08:50:05AM +0200, Maxime Ripard wrote: > Hi, > > On Wed, Oct 23, 2019 at 10:10:14PM +0200, Corentin Labbe wrote: > > This patch adds documentation for Device-Tree bindings of the > > Security System cryptographic offloader driver. > > > > Signed-off-by: Corentin Labbe <clabbe.montj...@gmail.com> > > --- > > .../bindings/crypto/allwinner,sun8i-ss.yaml | 64 +++++++++++++++++++ > > 1 file changed, 64 insertions(+) > > create mode 100644 > > Documentation/devicetree/bindings/crypto/allwinner,sun8i-ss.yaml > > > > diff --git > > a/Documentation/devicetree/bindings/crypto/allwinner,sun8i-ss.yaml > > b/Documentation/devicetree/bindings/crypto/allwinner,sun8i-ss.yaml > > new file mode 100644 > > index 000000000000..99b7736975bc > > --- /dev/null > > +++ b/Documentation/devicetree/bindings/crypto/allwinner,sun8i-ss.yaml > > @@ -0,0 +1,64 @@ > > +# SPDX-License-Identifier: GPL-2.0 > > +%YAML 1.2 > > +--- > > +$id: http://devicetree.org/schemas/crypto/allwinner,sun8i-ss.yaml# > > +$schema: http://devicetree.org/meta-schemas/core.yaml# > > + > > +title: Allwinner Security System v2 driver > > + > > +maintainers: > > + - Corentin Labbe <corentin.la...@gmail.com> > > + > > +properties: > > + compatible: > > + enum: > > + - allwinner,sun8i-a83t-crypto > > + - allwinner,sun9i-a80-crypto > > + > > + reg: > > + maxItems: 1 > > + > > + interrupts: > > + maxItems: 1 > > + > > + clocks: > > + items: > > + - description: Bus clock > > + - description: Module clock > > + > > + clock-names: > > + items: > > + - const: bus > > + - const: mod > > + > > + resets: > > + maxItems: 1 > > The A83t at least has a reset line, so please make a condition to have > it required. >
Hello The A80 have one also, so I need to set minItems: 1 But setting both minItems: 1 and maxItems:1 lead to a check failure: properties:resets: {'minItems': 1, 'maxItems': 1} is not valid under any of the given schemas How to do that ? Furthermore, I try to do that for interrupts and reg, since they are also mandatory and same failure. > > + reset-names: > > + const: bus > > You don't need reset-names at all in that binding. Fixed Thanks -- You received this message because you are subscribed to the Google Groups "linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an email to linux-sunxi+unsubscr...@googlegroups.com. To view this discussion on the web, visit https://groups.google.com/d/msgid/linux-sunxi/20191024093118.GA15113%40Red.