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=-0.9 required=3.0 tests=DKIM_SIGNED,DKIM_VALID, DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=no 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 10E80C10DCE for ; Thu, 12 Mar 2020 10:17:35 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id DF9CE2071C for ; Thu, 12 Mar 2020 10:17:34 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="Rx+1EGWt" Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726720AbgCLKRe (ORCPT ); Thu, 12 Mar 2020 06:17:34 -0400 Received: from mail-wm1-f67.google.com ([209.85.128.67]:35213 "EHLO mail-wm1-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726023AbgCLKRe (ORCPT ); Thu, 12 Mar 2020 06:17:34 -0400 Received: by mail-wm1-f67.google.com with SMTP id m3so5588162wmi.0 for ; Thu, 12 Mar 2020 03:17:33 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=date:from:to:cc:subject:message-id:references:mime-version :content-disposition:in-reply-to; bh=JSh5SwgggaNNIP1rhJKmLIM4vESq/2xFcSyTZp+7Nus=; b=Rx+1EGWt2dGF/rxSvLT7XGg2rRWc3yl5MrGHM3fODfxN+My4zCiGC9aeKEjXVaY/C/ +LHCPygIqp9eoxc+JZrBWd7bE5FY7+BnkvLH11AHW/mdG41YmsLhn15hx0GwjkuK1X/F ewTFglT3n/TGbE1lgqU0mwv3vp++G2D3agpGYFaUIaiCmkzvLOJ1Ohco+rUjxj5WO/8d SytvKXLoXZb7b1MkmlzgHZ4b6ixTjOpbOkiacoaqMAlyG6heH8taLU8p2f7qFn5Khhr6 E7im34jcZgsPPAc4GqZdIJrpbekoOOBrxw8QTkhyx32epfHsyD4B2w6iVmJgKc36a6Ns ejHA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:date:from:to:cc:subject:message-id:references :mime-version:content-disposition:in-reply-to; bh=JSh5SwgggaNNIP1rhJKmLIM4vESq/2xFcSyTZp+7Nus=; b=EpWW5UDKXTjeA9qHNDZ/Z76cAgWoBwFdNGz+5XG/NSDCWAU/PvyrjOSJH8q8zRZyNt z0lbAzE/s+3EDhhLIa+aN9svwyEWz8pZmmP57TD/PbElWytG+af/zw28ZMmnjBWiLejE YQ9HCKRRsLaibKjJbOX8F2fawgFo5/Bn2ZaLhjrlUlMM2T/C4x+sWG2L378zJP2so4bO fuxz1T0QGpJqwAJTXkgu1Q86olQKxNfK4Hh2lNw8JqqizG8mIT3ep+dyFZYf5HOAGfTU pOiBwW0ptLZ1ler0ir70Y6CBlWeDw7/kLyacce2UdRM3mYBC0qEFboAtXosTcE8Q+b2L A2sQ== X-Gm-Message-State: ANhLgQ1ZmnW4WvEIhvKuBlDi2qSST/50jwwchrrgtjiS67Ci0my0W8k3 hXSANfad/LkzqsWmYy0Avrga0w== X-Google-Smtp-Source: ADFU+vtd22zCvQBt96HNv1FNfip491J3yg6bOcH1BHTeiIr8TJOxKRCIuR11VfvzANE3XDLrYVJOCw== X-Received: by 2002:a1c:f009:: with SMTP id a9mr4167566wmb.73.1584008252629; Thu, 12 Mar 2020 03:17:32 -0700 (PDT) Received: from holly.lan (cpc141214-aztw34-2-0-cust773.18-1.cable.virginm.net. [86.9.19.6]) by smtp.gmail.com with ESMTPSA id f4sm18901766wrt.24.2020.03.12.03.17.31 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 12 Mar 2020 03:17:31 -0700 (PDT) Date: Thu, 12 Mar 2020 10:17:29 +0000 From: Daniel Thompson To: Tobias Schramm Cc: Andy Shevchenko , Sebastian Reichel , Rob Herring , Mark Rutland , Maxime Ripard , Sam Ravnborg , Heiko Stuebner , Stephan Gerhold , Mark Brown , Mauro Carvalho Chehab , "David S. Miller" , Greg Kroah-Hartman , Jonathan Cameron , linux-pm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org Subject: Re: [PATCH v3 2/3] dt-bindings: power: supply: add cw2015_battery bindings Message-ID: <20200312101729.ppqhco2j3p66dnqz@holly.lan> References: <20200311093043.3636807-1-t.schramm@manjaro.org> <20200311093043.3636807-3-t.schramm@manjaro.org> <20200311172056.wjn3574zrfqxipw6@holly.lan> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Sender: linux-pm-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-pm@vger.kernel.org On Thu, Mar 12, 2020 at 12:17:55AM +0100, Tobias Schramm wrote: > Hi Daniel, > > thanks for reviewing. The typo will be fixed for v4. > > >> + power-supplies: > >> + description: > >> + Specifies supplies used for charging the battery connected to this gauge > >> + allOf: > >> + - $ref: /schemas/types.yaml#/definitions/phandle-array > >> + - minItems: 1 > >> + maxItems: 8 # Should be enough > > > > Is it necessary to set a maximum? power_supply.txt is still a text file > > but there is no mention of a maximum there. > > > I think so? Removing maxItems and running dtbs_check on a dts with more > than one supply phandle in the power-supplies property results in an error: > linux/arch/arm64/boot/dts/rockchip/rk3399-pinebook-pro.dt.yaml: > cw2015@62: power-supplies: [[142], [50]] is too long Interesting. I saw the "Should be enough" comment replicated in several YAML bindings (with varying degress of paranoia about how much "enough" is). There are also several that simply set minItems without setting maxItems, perhaps they have just never been any DTs that test those bindings with more than one item. Daniel.