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=-8.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE, SPF_PASS,URIBL_BLOCKED,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 25AC3C432C0 for ; Mon, 18 Nov 2019 12:57:14 +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 ECFB420692 for ; Mon, 18 Nov 2019 12:57:13 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="Qwuom09X"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="REU+MnKH" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org ECFB420692 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org 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-Type:Cc: List-Subscribe:List-Help:List-Post:List-Archive:List-Unsubscribe:List-Id: In-Reply-To:MIME-Version:References:Message-ID:Subject:To:From:Date:Reply-To: Content-Transfer-Encoding:Content-ID:Content-Description:Resent-Date: Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Owner; bh=EccQK7cxHRw1b0/Fvk/XBhl13bzlHp8RFQiA8lVdbEQ=; b=Qwuom09XoS2KuqEWUnsVEXKaZ ob1K/X81mQ9nhNWcdtiLgPa+gyzTwAX8HBaWdufi2naGdADbZRUZ2eN2aCUQ6Jjg973PQPZ8PEjh4 ayhRfMK95Kh+4POteLorG+TFwdPTfRjW0eqq4J6w+TxGvzQqOv5xmbXr2UF2O6oRx0TpuvjB1VgSf ajx5NyN8d6MbsGe2JDprZkZXNlmejM/649lbv0r+2DlvO74/64M3ou9C/jpJAklE3hfhRw1cK2MBT 0RrkZOKXgwGMWozuIE8bJvUd+ehpCJhn9G82phIU2dsIKATPde+Zain3/xF7qZpvKBaguX3K3lxXV F3sCF8LPw==; 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 1iWgab-00049U-Pn; Mon, 18 Nov 2019 12:57:09 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1iWgaY-00048y-4Z for linux-arm-kernel@lists.infradead.org; Mon, 18 Nov 2019 12:57:07 +0000 Received: from localhost (lfbn-1-10718-76.w90-89.abo.wanadoo.fr [90.89.68.76]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id 16E8420692; Mon, 18 Nov 2019 12:57:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1574081825; bh=D42/XnP8BTkRdLMB0sfjNYCvjVmtv3XV0wVH/ObQtWU=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=REU+MnKH/jfxkLo0UnQ+ACUSgGSvJsHkc36Fgkosi60t+lPgSNnJ3EylqBANKsD18 JOtPOTsrOh7xq4pZGrh+7YHfUTFV5SsvF9JIXJ+neHGWm0yVbFzVKLeCMWPH7cKdKz lqDfQPsr2yCymECD4z9NA08J7X595BC7WklnY5oU= Date: Mon, 18 Nov 2019 13:57:02 +0100 From: Maxime Ripard To: =?iso-8859-1?Q?Cl=E9ment_P=E9ron?= Subject: Re: [PATCH v6 1/8] dt-bindings: pwm: allwinner: Add H6 PWM description Message-ID: <20191118125702.GK4345@gilmour.lan> References: <20191118110034.19444-1-peron.clem@gmail.com> <20191118110034.19444-2-peron.clem@gmail.com> <20191118110640.GE4345@gilmour.lan> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.12.1 (2019-06-15) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20191118_045706_220053_F947B5C1 X-CRM114-Status: GOOD ( 22.47 ) 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 , linux-pwm@vger.kernel.org, Jernej Skrabec , devicetree , linux-sunxi , linux-kernel , Rob Herring , Chen-Yu Tsai , Thierry Reding , Uwe =?iso-8859-1?Q?Kleine-K=F6nig?= , Philipp Zabel , Rob Herring , linux-arm-kernel Content-Type: multipart/mixed; boundary="===============6184067595748654436==" Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org --===============6184067595748654436== Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="luUS5rQ1j20Qzq8E" Content-Disposition: inline --luUS5rQ1j20Qzq8E Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Mon, Nov 18, 2019 at 01:42:48PM +0100, Cl=E9ment P=E9ron wrote: > Hi Maxime > > On Mon, 18 Nov 2019 at 12:06, Maxime Ripard wrote: > > > > Hi, > > > > On Mon, Nov 18, 2019 at 12:00:27PM +0100, Cl=E9ment P=E9ron wrote: > > > From: Jernej Skrabec > > > > > > H6 PWM block is basically the same as A20 PWM, except that it also has > > > bus clock and reset line which needs to be handled accordingly. > > > > > > Expand Allwinner PWM binding with H6 PWM specifics. > > > > > > Signed-off-by: Jernej Skrabec > > > Reviewed-by: Rob Herring > > > Signed-off-by: Cl=E9ment P=E9ron > > > --- > > > .../bindings/pwm/allwinner,sun4i-a10-pwm.yaml | 48 +++++++++++++++++= ++ > > > 1 file changed, 48 insertions(+) > > > > > > diff --git a/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a1= 0-pwm.yaml b/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.= yaml > > > index 0ac52f83a58c..1bae446febbb 100644 > > > --- a/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.y= aml > > > +++ b/Documentation/devicetree/bindings/pwm/allwinner,sun4i-a10-pwm.y= aml > > > @@ -30,13 +30,51 @@ properties: > > > - items: > > > - const: allwinner,sun50i-h5-pwm > > > - const: allwinner,sun5i-a13-pwm > > > + - const: allwinner,sun50i-h6-pwm > > > > > > reg: > > > maxItems: 1 > > > > > > clocks: > > > + minItems: 1 > > > + maxItems: 2 > > > + items: > > > + - description: Module Clock > > > + - description: Bus Clock > > > + > > > + # Even though it only applies to subschemas under the conditionals, > > > + # not listing them here will trigger a warning because of the > > > + # additionalsProperties set to false. > > > + clock-names: true > > > + > > > + resets: > > > maxItems: 1 > > > > > > + if: > > > + properties: > > > + compatible: > > > + contains: > > > + const: allwinner,sun50i-h6-pwm > > > + > > > + then: > > > + properties: > > > + clocks: > > > + maxItems: 2 > > > + > > > + clock-names: > > > + items: > > > + - const: mod > > > + - const: bus > > > + > > > + required: > > > + - clock-names > > > + - resets > > > + > > > + else: > > > + properties: > > > + clocks: > > > + maxItems: 1 > > > + > > > > Sorry for not noticing this earlier, but this should be at the topmost > > level > > No problem, but I don't get what you want, (yaml format is new for me). > Do you mean I should put the if condition before the "resets" ? No, here if we condense a bit the file, we have something like: title: PWM properties: compatible: ... ... resets: ... if: properties: ... then: properties: ... which means that you expect that the node may contain a compatible property, a resets one, and then two properties "if" and "then", which in turn contain properties (ie, two nodes). This is obviously not what you want, what you want instead is: properties: compatible: ... ... resets: ... if: properties: ... then: properties: ... Which then describes that there's two properties, compatible and resets, and if the schema under 'if' is valid against the node we try to validate, the schema under 'then' is used to validate the node as well. I hope it's clearer, Maxime --luUS5rQ1j20Qzq8E Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iHUEABYIAB0WIQRcEzekXsqa64kGDp7j7w1vZxhRxQUCXdKVHgAKCRDj7w1vZxhR xRf+AQDIKDWmitZKrfNWWYoRdbyxBBWj5ixRoYR6aGpqSbCu9wEA5XGA6Ew4woTI rOvVvGt0j8oafbBT1xQ/xkok13nHqAo= =Y2SH -----END PGP SIGNATURE----- --luUS5rQ1j20Qzq8E-- --===============6184067595748654436== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel --===============6184067595748654436==--