From patchwork Mon Jun 12 16:04:00 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Tony Lindgren X-Patchwork-Id: 774699 Return-Path: X-Original-To: incoming-dt@patchwork.ozlabs.org Delivered-To: patchwork-incoming-dt@bilbo.ozlabs.org Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by ozlabs.org (Postfix) with ESMTP id 3wmd4T0TBwz9s03 for ; Tue, 13 Jun 2017 02:04:09 +1000 (AEST) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754812AbdFLQEG convert rfc822-to-8bit (ORCPT ); Mon, 12 Jun 2017 12:04:06 -0400 Received: from muru.com ([72.249.23.125]:51446 "EHLO muru.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754794AbdFLQEF (ORCPT ); Mon, 12 Jun 2017 12:04:05 -0400 Received: from atomide.com (localhost [127.0.0.1]) by muru.com (Postfix) with ESMTPS id 16AE081E6; Mon, 12 Jun 2017 16:07:47 +0000 (UTC) Date: Mon, 12 Jun 2017 09:04:00 -0700 From: Tony Lindgren To: Sebastian Reichel Cc: linux-pm@vger.kernel.org, linux-omap@vger.kernel.org, devicetree@vger.kernel.org, Marcel Partap , Michael Scott Subject: Re: [PATCH] dt-bindings: power: supply: cpcap-battery: Add power-supplies property Message-ID: <20170612160400.GY3730@atomide.com> References: <20170612072025.30033-1-tony@atomide.com> <20170612142217.tg7gidqg63uagwm4@earth> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20170612142217.tg7gidqg63uagwm4@earth> User-Agent: Mutt/1.8.2 (2017-04-18) Sender: devicetree-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org * Sebastian Reichel [170612 07:26]: > On Mon, Jun 12, 2017 at 12:20:25AM -0700, Tony Lindgren wrote: > > +- power-supplies: List of devices that supply power, typically cpcap_charger > > This is missing the info, that its about phandles. Maybe: > > - power-supplies: List of phandles for power-supplying devices, as > described in power_supply.txt. Typically a reference > to cpcap_charger. Sure, updated patch below. Regards, Tony 8< ------ >From tony Mon Sep 17 00:00:00 2001 From: Tony Lindgren Date: Mon, 12 Jun 2017 00:13:45 -0700 Subject: [PATCH] dt-bindings: power: supply: cpcap-battery: Add power-supplies property The binding for cpcap-battery is missing the standard power-supplies property as noted by Sebastian Reichel . Cc: devicetree@vger.kernel.org Cc: Marcel Partap Cc: Michael Scott Signed-off-by: Tony Lindgren Acked-by: Rob Herring --- Documentation/devicetree/bindings/power/supply/cpcap-battery.txt | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Documentation/devicetree/bindings/power/supply/cpcap-battery.txt b/Documentation/devicetree/bindings/power/supply/cpcap-battery.txt --- a/Documentation/devicetree/bindings/power/supply/cpcap-battery.txt +++ b/Documentation/devicetree/bindings/power/supply/cpcap-battery.txt @@ -8,6 +8,9 @@ Required properties: - io-channels: IIO ADC channel specifier for each name in io-channel-names - io-channel-names: Should contain the following entries: "battdetb", "battp", "chg_isense", "batti" +- power-supplies: List of phandles for power-supplying devices, as + described in power_supply.txt. Typically a reference + to cpcap_charger. Example: @@ -24,4 +27,5 @@ cpcap_battery: battery { &cpcap_adc 5 &cpcap_adc 6>; io-channel-names = "battdetb", "battp", "chg_isense", "batti"; + power-supplies = <&cpcap_charger>; };