Received: by 2002:a25:ab43:0:0:0:0:0 with SMTP id u61csp986058ybi; Fri, 24 May 2019 15:01:42 -0700 (PDT) X-Google-Smtp-Source: APXvYqy0fCibVf6LXj+eHMCTPE+7QL2QwbghOrjYb2aCc2+TsvDgqMVkCfrxJ7k2NQCCt5MTbDmj X-Received: by 2002:a17:902:9a84:: with SMTP id w4mr48156888plp.241.1558735302370; Fri, 24 May 2019 15:01:42 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558735302; cv=none; d=google.com; s=arc-20160816; b=TJFXLCAhqc6yUAABu4d56gajK8yYHiZdkR1tOnuT7G39XjaJK9r1/az2f0p6YoC1vT atku5DsENSQOvSEKbVwq1f8xSFT3iOeJsPT51cki6iRkx9NA8TezShkQVes8yAcz0jyI lGPTzSc6YdmigxzWO+jdan1L0QVPqI0qxDiMXE1RSEOgCylcof28JKlim9iK9d0PNjOC 6BaE7mJ3ArVf11x7W9VjkBvJP6GdJGzWOMLubCDdf6kKcH9pfKELoVgg/ZN8vwmjray9 eKmTIrwqfU0P273jpo83AGDKDBm1aXvlpv9MLpeq1oXu/5jDdzzJTQbhP+osLAkaf1U+ thPw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:user-agent:message-id:references :in-reply-to:cc:from:date:content-transfer-encoding:mime-version :subject:to:dkim-signature; bh=Ql0cD5JfK/yoXltjS6jKt/CQqtSPm7QpPWra93u8UTM=; b=tWK9vjLsnUz11lwUZ3wqKBXcdU9/7BfKMCUi3/JzbMxObn3Qx+HZxUtn+ZhUs1zMSa Tioe4KPLJD68CThZIJzyTx872dbTf6PjrlNWEmH2hJd8l8PhmmZNqkn49IpKNCho8KE7 e7Oio/aRN5Xeb80gD+PdUimyRQEHkBB0CI45SIHpeTonnjcTl2d5RuWafRBkFwIxoWqp hcBEnrYRDLJlTc1XW7dJyE6KZ9RTPokeALkkBkfpFfF4lfyFz/mkj1n47jcTVvwiOnvj aaGRTrK9Bkgh1x+j++lBcfl85eVz+mzgrhCinzirL1fmk7fmBWplr9HAxcqlR4ZzqsYm roTQ== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@akkea.ca header.s=mail header.b=t0f08gxx; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id t188si6614398pgc.228.2019.05.24.15.01.26; Fri, 24 May 2019 15:01:42 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@akkea.ca header.s=mail header.b=t0f08gxx; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2404371AbfEXWAG (ORCPT + 99 others); Fri, 24 May 2019 18:00:06 -0400 Received: from node.akkea.ca ([192.155.83.177]:57728 "EHLO node.akkea.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2404303AbfEXWAF (ORCPT ); Fri, 24 May 2019 18:00:05 -0400 Received: by node.akkea.ca (Postfix, from userid 33) id 1542B4E204B; Fri, 24 May 2019 22:00:05 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=akkea.ca; s=mail; t=1558735205; bh=Ql0cD5JfK/yoXltjS6jKt/CQqtSPm7QpPWra93u8UTM=; h=To:Subject:Date:From:Cc:In-Reply-To:References; b=t0f08gxxwNFLW0NebIk/BaI1njSOhzERZPR6qldRKUWUdgWZr/nykIdgrEZ5tXBx7 H/TK0bpZRHlrxlMR8DjeKiBl7istEYv722fQclR3fBGZkHuHuKDwsu8ZKne6+Oo3e3 +G3RuN67uhpEN5rb9mt05M8PUKLCAl5kairFbl/4= To: Pavel Machek Subject: Re: [PATCH v13 2/4] arm64: dts: fsl: librem5: Add a device tree for the Librem5 devkit X-PHP-Originating-Script: 1000:rcube.php MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII; format=flowed Content-Transfer-Encoding: 7bit Date: Fri, 24 May 2019 15:00:05 -0700 From: Angus Ainslie Cc: angus.ainslie@puri.sm, Rob Herring , Mark Rutland , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , NXP Linux Team , devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org In-Reply-To: <20190523191926.GB3803@xo-6d-61-c0.localdomain> References: <20190520142330.3556-1-angus@akkea.ca> <20190520142330.3556-3-angus@akkea.ca> <20190523191926.GB3803@xo-6d-61-c0.localdomain> Message-ID: <1e6c0664949f38452b8f14b901bff513@www.akkea.ca> X-Sender: angus@akkea.ca User-Agent: Roundcube Webmail/1.1.3 Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Pavel, On 2019-05-23 12:19, Pavel Machek wrote: > Hi! > >> - LEDs >> - gyro >> - magnetometer > >> + leds { >> + compatible = "gpio-leds"; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_gpio_leds>; >> + >> + led1 { >> + label = "LED 1"; > > So, what kind of LED do you have, and what color is it? label should > probably be something like > notify:green. > As we don't have a specific use for these yet does it really matter if there is a colour or a number associated with them ? >> + charger@6b { /* bq25896 */ >> + compatible = "ti,bq25890"; >> + reg = <0x6b>; >> + pinctrl-names = "default"; >> + pinctrl-0 = <&pinctrl_charger>; >> + interrupt-parent = <&gpio3>; >> + interrupts = <25 IRQ_TYPE_EDGE_FALLING>; >> + ti,battery-regulation-voltage = <4192000>; /* 4.192V */ >> + ti,charge-current = <1600000>; /* 1.6 A */ > > No space before A, for consistency. > >> + ti,termination-current = <66000>; /* 66mA */ >> + ti,precharge-current = <1300000>; /* 1.3A */ > > I thought precharge is usually something low, because you are not yet > sure of battery health...? > I think I put that in incorrectly. The intention was 130mA. >> + ti,minimum-sys-voltage = <2750000>; /* 2.75V */ > > Are you sure? Normally systems shut down at 3.2V, 3V or so. Li-ion > batteries don't > really like to be discharged _this_ deep. You are correct. I'll fix it for the next version. Thanks Angus > > Pavel