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=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS autolearn=unavailable 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 19C2FC07E9B for ; Wed, 21 Jul 2021 08:58:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by mail.kernel.org (Postfix) with ESMTP id 02BE6610D2 for ; Wed, 21 Jul 2021 08:58:19 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237061AbhGUIPu convert rfc822-to-8bit (ORCPT ); Wed, 21 Jul 2021 04:15:50 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:59846 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S236784AbhGUHxK (ORCPT ); Wed, 21 Jul 2021 03:53:10 -0400 Received: from metis.ext.pengutronix.de (metis.ext.pengutronix.de [IPv6:2001:67c:670:201:290:27ff:fe1d:cc33]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 57E90C061574 for ; Wed, 21 Jul 2021 01:33:40 -0700 (PDT) Received: from lupine.hi.pengutronix.de ([2001:67c:670:100:3ad5:47ff:feaf:1a17] helo=lupine) by metis.ext.pengutronix.de with esmtps (TLS1.3:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1m67f8-0007Mj-OA; Wed, 21 Jul 2021 10:33:06 +0200 Received: from pza by lupine with local (Exim 4.92) (envelope-from ) id 1m67f0-0007mW-Ha; Wed, 21 Jul 2021 10:32:58 +0200 Message-ID: <8343dfe9d1af1ad4ab806104b74a95819c765dea.camel@pengutronix.de> Subject: Re: [PATCH] dt-bindings: Remove "status" from schema examples From: Philipp Zabel To: Rob Herring , devicetree@vger.kernel.org Cc: linux-kernel@vger.kernel.org, Maxime Ripard , Chen-Yu Tsai , Thierry Reding , Sam Ravnborg , Rui Miguel Silva , Laurent Pinchart , Mauro Carvalho Chehab , "David S. Miller" , Jakub Kicinski , Mark Brown , Robert Marko , Alessandro Zummo , Alexandre Belloni , Ramesh Shanmugasundaram , "G. Jaya Kumaran" , Linus Walleij , Oleksij Rempel , ChiYuan Huang , Wei Xu , Dilip Kota , Karol Gugala , Mateusz Holenko , Olivier Moysan , Peter Ujfalusi , dri-devel@lists.freedesktop.org, linux-media@vger.kernel.org, netdev@vger.kernel.org, linux-rtc@vger.kernel.org, alsa-devel@alsa-project.org Date: Wed, 21 Jul 2021 10:32:58 +0200 In-Reply-To: <20210720172025.363238-1-robh@kernel.org> References: <20210720172025.363238-1-robh@kernel.org> Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 8BIT User-Agent: Evolution 3.30.5-1.1 MIME-Version: 1.0 X-SA-Exim-Connect-IP: 2001:67c:670:100:3ad5:47ff:feaf:1a17 X-SA-Exim-Mail-From: p.zabel@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false X-PTX-Original-Recipient: devicetree@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org Hi Rob, On Tue, 2021-07-20 at 11:20 -0600, Rob Herring wrote: > There's no reason to have "status" properties in examples. "okay" is the > default, and "disabled" turns off some schema checks ('required' > specifically). Is this documented somewhere? If not, should it be? (Maybe in writing- schema.rst -> Schema Contents -> examples?) > Enabling qca,ar71xx causes a warning, so let's fix the node names: > > Documentation/devicetree/bindings/net/qca,ar71xx.example.dt.yaml: phy@3: '#phy-cells' is a required property > From schema: schemas/phy/phy-provider.yaml > > Cc: Maxime Ripard > Cc: Chen-Yu Tsai > Cc: Thierry Reding > Cc: Sam Ravnborg > Cc: Rui Miguel Silva > Cc: Laurent Pinchart > Cc: Mauro Carvalho Chehab > Cc: "David S. Miller" > Cc: Jakub Kicinski > Cc: Mark Brown > Cc: Robert Marko > Cc: Philipp Zabel > Cc: Alessandro Zummo > Cc: Alexandre Belloni > Cc: Ramesh Shanmugasundaram > Cc: "G. Jaya Kumaran" > Cc: Linus Walleij > Cc: Oleksij Rempel > Cc: ChiYuan Huang > Cc: Wei Xu > Cc: Dilip Kota > Cc: Karol Gugala > Cc: Mateusz Holenko > Cc: Olivier Moysan > Cc: Peter Ujfalusi > Cc: dri-devel@lists.freedesktop.org > Cc: linux-media@vger.kernel.org > Cc: netdev@vger.kernel.org > Cc: linux-rtc@vger.kernel.org > Cc: alsa-devel@alsa-project.org > Signed-off-by: Rob Herring > --- > .../display/allwinner,sun8i-a83t-dw-hdmi.yaml | 2 -- > .../display/panel/boe,tv101wum-nl6.yaml | 1 - > .../bindings/media/nxp,imx7-mipi-csi2.yaml | 2 -- > .../bindings/media/renesas,drif.yaml | 1 - > .../bindings/net/intel,dwmac-plat.yaml | 2 -- > .../bindings/net/intel,ixp4xx-ethernet.yaml | 2 -- > .../bindings/net/nfc/samsung,s3fwrn5.yaml | 3 --- > .../devicetree/bindings/net/qca,ar71xx.yaml | 25 ++++--------------- > .../regulator/richtek,rt6245-regulator.yaml | 1 - > .../regulator/vqmmc-ipq4019-regulator.yaml | 1 - > .../reset/hisilicon,hi3660-reset.yaml | 1 - > .../bindings/reset/intel,rcu-gw.yaml | 1 - [...] Reviewed-by: Philipp Zabel regards Philipp