From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-5.3 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SIGNED_OFF_BY, SPF_HELO_NONE,SPF_PASS,USER_AGENT_SANE_1 autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A2D3CC33C9E for ; Tue, 14 Jan 2020 16:02:15 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 7056524656 for ; Tue, 14 Jan 2020 16:02:15 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="tJnk8NpO"; dkim=fail reason="signature verification failed" (2048-bit key) header.d=st.com header.i=@st.com header.b="DWk0khVZ" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 7056524656 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=st.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date: Message-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description :Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=6yAaNwlwtWAnx/6Ez/a6mj5M7NTF3M7rItLAlmM/JxA=; b=tJnk8NpO+DplDd MLm3lrHqTf0O5b/p7Kve1phnHLIefp5KehkDAvpRbNUZHViEBuYRPW+Sa9aDb+Lo0x/8q7LF7U27g syIY8YAFYPzrC+H/qlNGoBLScrX0BahIuQ2+AgGBmLvyNwsjtHMNUHUVvYIVXJbLyZeDuaFcxdv5A vvd4hX95MLKR9KMHD2Pv3kN4cND9Nh+mn8Je93OTOLOy4i6eriiKpxN/HER1lpfpKMkcjMzk0+cWl UbOhcPb94vN5m0AAe1pvcOSfDi8LwJRaukQx6qdRlbyQ3K9Sk/DRPSpLdIipj0vOG3RJEcDnOLEf/ WRICxYinW81ZaczErEXQ==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1irOdy-00030o-DI; Tue, 14 Jan 2020 16:02:14 +0000 Received: from mx08-00178001.pphosted.com ([91.207.212.93] helo=mx07-00178001.pphosted.com) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1irOds-0002zT-9P for linux-arm-kernel@lists.infradead.org; Tue, 14 Jan 2020 16:02:13 +0000 Received: from pps.filterd (m0046661.ppops.net [127.0.0.1]) by mx07-00178001.pphosted.com (8.16.0.42/8.16.0.42) with SMTP id 00EFqmCb023389; Tue, 14 Jan 2020 17:01:55 +0100 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=st.com; h=subject : to : cc : references : from : message-id : date : mime-version : in-reply-to : content-type : content-transfer-encoding; s=STMicroelectronics; bh=iqcbAO7gn5oc2a+bEcf0WOjlJoRulApDnHTG5smDQ34=; b=DWk0khVZQxpCW7NxPaFvTtniHCfvkDLVAsv5oVI0FSt7q8G4/RtcRqOT+KOIsoNbTRSc l1VmZtujDMkOFgjxAaqefoqNxQb9G8WKVqcOBL1Ir+u6SW4NfagLfjZ0FQLwK9IC6oym YaCXXUL+ZJBODdlCll1CNR935UKHZmdOZKzFyhWSF8mTA1cTQl/r25KU1ZLcd44hQ2Tl TApcE4LAReDFVWKGkv+z0mr02RMokcLgCf4KhvfBKehWZxlJVMaJQSuUHdHtKRGo2lu6 uRQvl5itEy/A7DfyZYMFJHfZmyx1HICO4oezIsP1KmppLA/qokU4li0N6II4gC9fy+Tw 7w== Received: from beta.dmz-eu.st.com (beta.dmz-eu.st.com [164.129.1.35]) by mx07-00178001.pphosted.com with ESMTP id 2xf7jpebd8-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 14 Jan 2020 17:01:55 +0100 Received: from euls16034.sgp.st.com (euls16034.sgp.st.com [10.75.44.20]) by beta.dmz-eu.st.com (STMicroelectronics) with ESMTP id A2FB1100044; Tue, 14 Jan 2020 17:01:54 +0100 (CET) Received: from Webmail-eu.st.com (sfhdag5node3.st.com [10.75.127.15]) by euls16034.sgp.st.com (STMicroelectronics) with ESMTP id 6AC7A2C38A2; Tue, 14 Jan 2020 17:01:54 +0100 (CET) Received: from [10.48.0.71] (10.75.127.46) by SFHDAG5NODE3.st.com (10.75.127.15) with Microsoft SMTP Server (TLS) id 15.0.1473.3; Tue, 14 Jan 2020 17:01:53 +0100 Subject: Re: [PATCH v2] dt-bindings: iio: adc: stm32-adc: convert bindings to json-schema To: Rob Herring References: <1575649028-10909-1-git-send-email-fabrice.gasnier@st.com> <20191217234345.GA7738@bogus> From: Fabrice Gasnier Message-ID: Date: Tue, 14 Jan 2020 17:01:53 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:60.0) Gecko/20100101 Thunderbird/60.9.0 MIME-Version: 1.0 In-Reply-To: <20191217234345.GA7738@bogus> Content-Language: en-US X-Originating-IP: [10.75.127.46] X-ClientProxiedBy: SFHDAG3NODE1.st.com (10.75.127.7) To SFHDAG5NODE3.st.com (10.75.127.15) X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10434:6.0.138, 18.0.572 definitions=2020-01-14_04:2020-01-14, 2020-01-14 signatures=0 X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200114_080208_616004_25BD4102 X-CRM114-Status: GOOD ( 21.99 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: mark.rutland@arm.com, devicetree@vger.kernel.org, lars@metafoo.de, olivier.moysan@st.com, alexandre.torgue@st.com, linux-iio@vger.kernel.org, pmeerw@pmeerw.net, linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, mcoquelin.stm32@gmail.com, knaack.h@gmx.de, linux-stm32@st-md-mailman.stormreply.com, jic23@kernel.org Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On 12/18/19 12:43 AM, Rob Herring wrote: > On Fri, Dec 06, 2019 at 05:17:08PM +0100, Fabrice Gasnier wrote: >> Convert the STM32 ADC binding to DT schema format using json-schema >> >> Signed-off-by: Fabrice Gasnier >> --- >> Note: this applies on top of IIO tree currently (iio-for-5.5c). >> >> Changes in V2: >> - Take almost all of Rob suggestions (removed reg generic description, >> added minItems, maxItems, st,max-clk-rate-hz range, drop some pipes, >> simplify clock-names, remove unneeded allOfs) >> - For now, keep all in one file despite there are lots of if/thens in the >> bindings >> --- >> .../devicetree/bindings/iio/adc/st,stm32-adc.txt | 149 ------- >> .../devicetree/bindings/iio/adc/st,stm32-adc.yaml | 454 +++++++++++++++++++++ >> 2 files changed, 454 insertions(+), 149 deletions(-) >> delete mode 100644 Documentation/devicetree/bindings/iio/adc/st,stm32-adc.txt >> create mode 100644 Documentation/devicetree/bindings/iio/adc/st,stm32-adc.yaml > > [snip] >> + >> + st,adc-channels: >> + description: | >> + List of single-ended channels muxed for this ADC. It can have up to: >> + - 16 channels, numbered from 0 to 15 (for in0..in15) on stm32f4 >> + - 20 channels, numbered from 0 to 19 (for in0..in19) on stm32h7 and >> + stm32mp1. >> + allOf: >> + - $ref: /schemas/types.yaml#/definitions/uint32-array >> + [snip] >> + >> + allOf: >> + - if: >> + properties: >> + compatible: >> + contains: >> + const: st,stm32f4-adc >> + >> + then: >> + properties: >> + reg: >> + enum: >> + - 0x0 >> + - 0x100 >> + - 0x200 >> + >> + interrupts: >> + minimum: 0 >> + maximum: 2 >> + >> + assigned-resolution-bits: >> + enum: [6, 8, 10, 12] >> + default: 12 >> + >> + st,adc-channels: >> + minItems: 1 >> + maxItems: 16 >> + minimum: 0 >> + maximum: 15 > > You are mixing array and scalar constraints here. You need: > > minItems: 1 > maxItems:16 > items: > minimum: 0 > maximum: 15 > > Update dtschema. It will now catch this. There's a few others too. Hi Rob, Sorry for the late reply. I updated dtschema. Now it catches it. I've tried your suggestion, but when I test it, I don't get any error on maxItems. In the example: "st,adc-channels = <0>, <1>, ... more than 16 items;" Is it possible I face some other issue with dtschema ? I tried another way below... Not sure that's correct. But it catches errors on maxItems: st,adc-channels: allOf: - minItems: 1 maxItems: 16 items: minimum: 0 maximum: 15 Error message is ... "is too long" with bad example above. Please advise, Regards, Fabrice > > >> + >> + st,adc-diff-channels: false >> + >> + st,min-sample-time-nsecs: >> + minItems: 1 >> + maxItems: 16 >> + minimum: 80 >> + >> + required: >> + - clocks >> + >> + - if: >> + properties: >> + compatible: >> + contains: >> + enum: >> + - st,stm32h7-adc >> + - st,stm32mp1-adc >> + >> + then: >> + properties: >> + reg: >> + enum: >> + - 0x0 >> + - 0x100 >> + >> + interrupts: >> + minimum: 0 >> + maximum: 1 >> + >> + assigned-resolution-bits: >> + enum: [8, 10, 12, 14, 16] >> + default: 16 >> + >> + st,adc-channels: >> + minItems: 1 >> + maxItems: 20 >> + minimum: 0 >> + maximum: 19 >> + >> + st,min-sample-time-nsecs: >> + minItems: 1 >> + maxItems: 20 >> + minimum: 40 >> + >> + additionalProperties: false >> + >> + anyOf: >> + - required: >> + - st,adc-channels >> + - required: >> + - st,adc-diff-channels >> + >> + required: >> + - compatible >> + - reg >> + - interrupts >> + - '#io-channel-cells' >> + >> +examples: >> + - | >> + // Example 1: with stm32f429 >> + adc123: adc@40012000 { >> + compatible = "st,stm32f4-adc-core"; >> + reg = <0x40012000 0x400>; >> + interrupts = <18>; >> + clocks = <&rcc 0 168>; >> + clock-names = "adc"; >> + st,max-clk-rate-hz = <36000000>; >> + vdda-supply = <&vdda>; >> + vref-supply = <&vref>; >> + interrupt-controller; >> + #interrupt-cells = <1>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + adc@0 { >> + compatible = "st,stm32f4-adc"; >> + #io-channel-cells = <1>; >> + reg = <0x0>; >> + clocks = <&rcc 0 168>; >> + interrupt-parent = <&adc123>; >> + interrupts = <0>; >> + st,adc-channels = <8>; >> + dmas = <&dma2 0 0 0x400 0x0>; >> + dma-names = "rx"; >> + assigned-resolution-bits = <8>; >> + }; >> + // ... >> + // other adc child nodes follow... >> + }; >> + >> + - | >> + // Example 2: with stm32mp157c to setup ADC1 with: >> + // - channel 1 as single-ended >> + // - channels 2 & 3 as differential (with resp. 6 & 7 negative inputs) >> + #include >> + #include >> + adc12: adc@48003000 { >> + compatible = "st,stm32mp1-adc-core"; >> + reg = <0x48003000 0x400>; >> + interrupts = , >> + ; >> + clocks = <&rcc ADC12>, <&rcc ADC12_K>; >> + clock-names = "bus", "adc"; >> + booster-supply = <&booster>; >> + vdd-supply = <&vdd>; >> + vdda-supply = <&vdda>; >> + vref-supply = <&vref>; >> + st,syscfg = <&syscfg>; >> + interrupt-controller; >> + #interrupt-cells = <1>; >> + #address-cells = <1>; >> + #size-cells = <0>; >> + adc@0 { >> + compatible = "st,stm32mp1-adc"; >> + #io-channel-cells = <1>; >> + reg = <0x0>; >> + interrupt-parent = <&adc12>; >> + interrupts = <0>; >> + st,adc-channels = <1>; >> + st,adc-diff-channels = <2 6>, <3 7>; >> + st,min-sample-time-nsecs = <5000>; >> + dmas = <&dmamux1 9 0x400 0x05>; >> + dma-names = "rx"; >> + }; >> + // ... >> + // other adc child node follow... >> + }; >> + >> +... >> -- >> 2.7.4 >> _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel