[{"id":1772881,"web_url":"http://patchwork.ozlabs.org/comment/1772881/","msgid":"<20170921154621.0000328d@huawei.com>","list_archive_url":null,"date":"2017-09-21T14:46:21","subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","submitter":{"id":71988,"url":"http://patchwork.ozlabs.org/api/people/71988/","name":"Jonathan Cameron","email":"Jonathan.Cameron@huawei.com"},"content":"On Wed, 20 Sep 2017 23:18:07 +0800\nIcenowy Zheng wrote:\n\n> The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power inputs:\n> AC, USB and Battery.\n> \n> This patchset adds support for the AC and Battery supplies, which is useful\n> for the boards from Pine64 (Pine64, SoPine w/ baseboard model A, Pinebook).\n> \n> The USB supply is not yet supported in this patchset because it's not\n> present on Pine series boards.\n> \n> In order to enable battery monitoring the ADC for battery is also enabled\n> for AXs.\n> \n> In order to enable battery monitoring the ADC for battery is also enabled\n> for AXP803.\n\nI'll go with the obvious question...\n\nWhy an RFC rather than a standard patch submission? I'm not immediately\nseeing what is controversial!\n\nJonathan\n\n> \n> Icenowy Zheng (7):\n> dt-bindings: add compatibles for AXP803 Battery/USB power supplies\n> iio: adc: axp20x-adc: allow to skip ADC rate setup now\n> iio: adc: axp20x-adc: add support for AXP803\n> power: supply: axp20x-battery: support AXP803\n> mfd: axp20x: add cells for AXP803 ADC/AC Power/Battery\n> arm64: allwinner: a64: add power supply nodes in AXP803 DTSI\n> arm64: allwinner: a64: enable AC and Battery for Pine64\n> \n> .../bindings/power/supply/axp20x_battery.txt | 1 +\n> .../bindings/power/supply/axp20x_usb_power.txt | 1 +\n> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++\n> .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 8 ++\n> drivers/iio/adc/axp20x_adc.c | 114 ++++++++++++++++++++-\n> drivers/mfd/axp20x.c | 11 ++\n> drivers/power/supply/axp20x_battery.c | 88 ++++++++++++++--\n> 7 files changed, 226 insertions(+), 12 deletions(-)\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"UZoqfB4x\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xyfbV6zL6z9sP1\n\tfor ;\n\tFri, 22 Sep 2017 00:47:32 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dv2lD-0006ST-3F; Thu, 21 Sep 2017 14:47:27 +0000","from szxga05-in.huawei.com ([45.249.212.191])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dv2l8-0006O7-IH for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 14:47:24 +0000","from 172.30.72.60 (EHLO DGGEMS412-HUB.china.huawei.com)\n\t([172.30.72.60])\n\tby dggrg05-dlp.huawei.com (MOS 4.4.6-GA FastPath queued)\n\twith ESMTP id DHT69867; Thu, 21 Sep 2017 22:46:41 +0800 (CST)","from localhost (10.206.48.115) by DGGEMS412-HUB.china.huawei.com\n\t(10.3.19.212) with Microsoft SMTP Server (TLS) id 14.3.301.0;\n\tThu, 21 Sep 2017 22:46:35 +0800"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=mofCfy0vT1LdTvDxE1jfAjIyEowwjZjyd26P329EbDY=;\n\tb=UZoqfB4xj1Nnvd\n\tcympoFLOdRXjirAKWiqGZm6ISjWSQRQ5egIUbAUGN6yCM/8+IFijqzflbC4YW6DEQPDi8oQ3cPtpd\n\tF8K5GxUqcgz48BSvglljg58BdIJLrrV1gXIobT9RZ8T621q0odq20XaRh4y3dU7g16jQm+NIlElvI\n\tMcfz4yWBn9+QRTiKY0b/sByO979dMogEKnCq2fhIWI3rZEw4v8WIZTbl7LXwxDtS1Wu9RJQ2lldi1\n\tTxAYtixuap6rTBvVvbRTQC7XNID/V++GUScrozxMH/XOViE+LchLmUkluug2L7iJJjhamniAuqbUO\n\t8ajrmFEOp1dgvpVQpsQQ==;","Date":"Thu, 21 Sep 2017 15:46:21 +0100","From":"Jonathan Cameron ","To":"Icenowy Zheng ","Subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","Message-ID":"<20170921154621.0000328d@huawei.com>","In-Reply-To":"<20170920151814.22461-1-icenowy@aosc.io>","References":"<20170920151814.22461-1-icenowy@aosc.io>","Organization":"Huawei","X-Mailer":"Claws Mail 3.15.0 (GTK+ 2.24.31; x86_64-w64-mingw32)","MIME-Version":"1.0","X-Originating-IP":"[10.206.48.115]","X-CFilter-Loop":"Reflected","X-Mirapoint-Virus-RAPID-Raw":"score=unknown(0),\n\trefid=str=0001.0A020201.59C3D0D3.0058, ss=1, re=0.000, recu=0.000,\n\treip=0.000, cl=1, cld=1, fgs=0, ip=0.0.0.0,\n\tso=2014-11-16 11:51:01, dmn=2013-03-21 17:37:32","X-Mirapoint-Loop-Id":"45ae5d313cc1ccfada3cb2743d239cd3","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_074723_109071_5882A468 ","X-CRM114-Status":"GOOD ( 12.97 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-sunxi@googlegroups.com,\n\tlinux-kernel@vger.kernel.org,\n\tQuentin Schulz ,\n\tChen-Yu Tsai , linux-arm-kernel@lists.infradead.org,\n\tMaxime Ripard ,\n\tLee Jones , Jonathan Cameron ","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1772916,"web_url":"http://patchwork.ozlabs.org/comment/1772916/","msgid":"","list_archive_url":null,"date":"2017-09-21T15:20:11","subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","submitter":{"id":71295,"url":"http://patchwork.ozlabs.org/api/people/71295/","name":"Icenowy Zheng","email":"icenowy@aosc.io"},"content":"于 2017年9月21日 GMT+08:00 下午10:46:21, Jonathan Cameron 写到:\n>On Wed, 20 Sep 2017 23:18:07 +0800\n>Icenowy Zheng wrote:\n>\n>> The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power\n>inputs:\n>> AC, USB and Battery.\n>> \n>> This patchset adds support for the AC and Battery supplies, which is\n>useful\n>> for the boards from Pine64 (Pine64, SoPine w/ baseboard model A,\n>Pinebook).\n>> \n>> The USB supply is not yet supported in this patchset because it's not\n>> present on Pine series boards.\n>> \n>> In order to enable battery monitoring the ADC for battery is also\n>enabled\n>> for AXs.\n>> \n>> In order to enable battery monitoring the ADC for battery is also\n>enabled\n>> for AXP803.\n>\n>I'll go with the obvious question...\n>\n>Why an RFC rather than a standard patch submission? I'm not immediately\n>seeing what is controversial!\n\nOh I am just not confident about this patchset,\nespecially the IIO part.\n\n>\n>Jonathan\n>\n>> \n>> Icenowy Zheng (7):\n>> dt-bindings: add compatibles for AXP803 Battery/USB power supplies\n>> iio: adc: axp20x-adc: allow to skip ADC rate setup now\n>> iio: adc: axp20x-adc: add support for AXP803\n>> power: supply: axp20x-battery: support AXP803\n>> mfd: axp20x: add cells for AXP803 ADC/AC Power/Battery\n>> arm64: allwinner: a64: add power supply nodes in AXP803 DTSI\n>> arm64: allwinner: a64: enable AC and Battery for Pine64\n>> \n>> .../bindings/power/supply/axp20x_battery.txt | 1 +\n>> .../bindings/power/supply/axp20x_usb_power.txt | 1 +\n>> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++\n>> .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 8 ++\n>> drivers/iio/adc/axp20x_adc.c | 114\n>++++++++++++++++++++-\n>> drivers/mfd/axp20x.c | 11 ++\n>> drivers/power/supply/axp20x_battery.c | 88\n>++++++++++++++--\n>> 7 files changed, 226 insertions(+), 12 deletions(-)\n>> \n>\n>\n>_______________________________________________\n>linux-arm-kernel mailing list\n>linux-arm-kernel@lists.infradead.org\n>http://lists.infradead.org/mailman/listinfo/linux-arm-kernel","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"PGRfdxhQ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3xygLK41gKz9t3w\n\tfor ;\n\tFri, 22 Sep 2017 01:21:13 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dv3Hq-0008Ic-3s; Thu, 21 Sep 2017 15:21:10 +0000","from catfish.maple.relay.mailchannels.net ([23.83.214.32])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dv3Hk-0008F9-SZ for linux-arm-kernel@lists.infradead.org;\n\tThu, 21 Sep 2017 15:21:08 +0000","from relay.mailchannels.net (localhost [127.0.0.1])\n\tby relay.mailchannels.net (Postfix) with ESMTP id 9A781208919;\n\tThu, 21 Sep 2017 15:20:37 +0000 (UTC)","from hermes.aosc.io (unknown [100.96.138.121])\n\t(Authenticated sender: lmn-TZDUIOWCRQMW)\n\tby relay.mailchannels.net (Postfix) with ESMTPA id 6133D206AC4;\n\tThu, 21 Sep 2017 15:20:36 +0000 (UTC)","from hermes.aosc.io (hermes.aosc.io [172.20.107.195])\n\t(using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384)\n\tby 0.0.0.0:2500 (trex/5.9.14); Thu, 21 Sep 2017 15:20:37 +0000","from localhost (localhost [127.0.0.1]) (Authenticated sender:\n\ticenowy@aosc.io)\n\tby hermes.aosc.io (Postfix) with ESMTPSA id 2E2E74787D;\n\tThu, 21 Sep 2017 15:20:22 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Message-ID:From:To:Subject:MIME-Version\n\t:References:In-Reply-To:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=l80ozVBZ92oyMI+WDBHBD3CrX+xXE6kdIQOWedpVZ3w=;\n\tb=PGRfdxhQLfPApW\n\t5OwAWwIoEp5piF+we4v7lKDZe9YJXTjwtyZ0iFcp79t6pRtSBjrET+QsT0a63STPoQtJUdYMANpoN\n\t1iAnP7ny6eojDItLCZefK7zSrhZqxcCXrO2NVkU9FUOSLeIPjJANuWuvdvvQ1821pkCda5nQDYI4o\n\t2NkjxSdKGSZAnjRIvc5uVuXudIyqpn1yJ8aeWsQzHibqshAAVy62JIS5yIfYESn/uBik9oopKM0Vj\n\t0cFfuFyfc5sVwtuUKjQe4VJS+nsF38QdhrNSHe0q0sO8R0UON9LsR0zCgBK+OZwttFABClh67fpof\n\twSMbn4/g/nR/Bm0yb8wg==;","X-Sender-Id":["lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io"],"X-MC-Relay":"Neutral","X-MailChannels-SenderId":"lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","X-MailChannels-Auth-Id":"lmn-TZDUIOWCRQMW","X-Society-Continue":"0496d34d01af5fb4_1506007237202_855898276","X-MC-Loop-Signature":"1506007237201:2366339737","X-MC-Ingress-Time":"1506007237201","Date":"Thu, 21 Sep 2017 23:20:11 +0800","In-Reply-To":"<20170921154621.0000328d@huawei.com>","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170921154621.0000328d@huawei.com>","MIME-Version":"1.0","Subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","To":"linux-arm-kernel@lists.infradead.org,\n\tJonathan Cameron ","From":"Icenowy Zheng ","Message-ID":"","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170921_082105_042944_19AF0904 ","X-CRM114-Status":"GOOD ( 10.19 )","X-Spam-Score":"-4.7 (----)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-4.7 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [23.83.214.32 listed in list.dnswl.org]\n\t-2.8 RCVD_IN_MSPIKE_H2 RBL: Average reputation (+2)\n\t[23.83.214.32 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tQuentin Schulz ,\n\tlinux-sunxi@googlegroups.com,\n\tMaxime Ripard , \n\tChen-Yu Tsai , Lee Jones ,\n\tJonathan Cameron ","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774216,"web_url":"http://patchwork.ozlabs.org/comment/1774216/","msgid":"<20170924153233.14eb8964@archlinux>","list_archive_url":null,"date":"2017-09-24T14:32:33","subject":"Re: [RFC PATCH 3/7] iio: adc: axp20x-adc: add support for AXP803","submitter":{"id":10151,"url":"http://patchwork.ozlabs.org/api/people/10151/","name":"Jonathan Cameron","email":"jic23@kernel.org"},"content":"On Wed, 20 Sep 2017 23:18:10 +0800\nIcenowy Zheng wrote:\n\n> AXP803 SoC features an ADC part including these channels: GPADC (GPIO0)\n> and TS pins, PMIC internal temperature sensor, battery voltage, battery\n> charge/discharge current.\n> \n> Add support for the battery-related channels and internal temperature\n> channel in order to allow battery monitoring. The TS and GPADC channels\n> are complex and will be support after more investigation.\n> \n> Signed-off-by: Icenowy Zheng \n\nA few comments inline but this looks good to me.\n\nI will want to leave plenty of time for others to comment however, particularly\nQuentin.\n\nThanks,\n\nJonathan\n\n> ---\n> drivers/iio/adc/axp20x_adc.c | 108 +++++++++++++++++++++++++++++++++++++++++++\n> 1 file changed, 108 insertions(+)\n> \n> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c\n> index 93dd6b80059e..4f0cd98cf6ea 100644\n> --- a/drivers/iio/adc/axp20x_adc.c\n> +++ b/drivers/iio/adc/axp20x_adc.c\n> @@ -28,6 +28,8 @@\n> \n> #define AXP20X_ADC_EN2_MASK\t\t\t(GENMASK(3, 2) | BIT(7))\n> #define AXP22X_ADC_EN1_MASK\t\t\t(GENMASK(7, 5) | BIT(0))\n> +/* TODO: Enable TS and GPADC when supporting them */\n> +#define AXP803_ADC_EN1_MASK\t\t\tGENMASK(7, 5)\n> \n> #define AXP20X_GPIO10_IN_RANGE_GPIO0\t\tBIT(0)\n> #define AXP20X_GPIO10_IN_RANGE_GPIO1\t\tBIT(1)\n> @@ -95,6 +97,17 @@ enum axp22x_adc_channel_i {\n> \tAXP22X_BATT_DISCHRG_I,\n> };\n> \n> +enum axp803_adc_channel_v {\n> +\tAXP803_TS_IN = 0,\n> +\tAXP803_GPADC_IN,\n> +\tAXP803_BATT_V,\n> +};\n> +\n> +enum axp803_adc_channel_i {\n> +\tAXP803_BATT_CHRG_I = 2,\n> +\tAXP803_BATT_DISCHRG_I,\n> +};\n> +\n> static struct iio_map axp20x_maps[] = {\n> \t{\n> \t\t.consumer_dev_name = \"axp20x-usb-power-supply\",\n> @@ -144,6 +157,11 @@ static struct iio_map axp22x_maps[] = {\n> };\n> \n> /*\n> + * AXP803 shares the same consumer map with AXP22x, as it has no ADC for\n> + * VBUS and ACIN inputs either.\n> + */\n> +\n> +/*\n> * Channels are mapped by physical system. Their channels share the same index.\n> * i.e. acin_i is in_current0_raw and acin_v is in_voltage0_raw.\n> * The only exception is for the battery. batt_v will be in_voltage6_raw and\n> @@ -197,6 +215,23 @@ static const struct iio_chan_spec axp22x_adc_channels[] = {\n> \t\t\t AXP20X_BATT_DISCHRG_I_H),\n> };\n> \n> +static const struct iio_chan_spec axp803_adc_channels[] = {\n> +\t{\n> +\t\t.type = IIO_TEMP,\n> +\t\t.address = AXP288_PMIC_ADC_H,\n> +\t\t.info_mask_separate = BIT(IIO_CHAN_INFO_RAW) |\n> +\t\t\t\t BIT(IIO_CHAN_INFO_SCALE) |\n> +\t\t\t\t BIT(IIO_CHAN_INFO_OFFSET),\n> +\t\t.datasheet_name = \"pmic_temp\",\n> +\t},\n> +\tAXP20X_ADC_CHANNEL(AXP803_BATT_V, \"batt_v\", IIO_VOLTAGE,\n> +\t\t\t AXP20X_BATT_V_H),\n> +\tAXP20X_ADC_CHANNEL(AXP803_BATT_CHRG_I, \"batt_chrg_i\", IIO_CURRENT,\n> +\t\t\t AXP20X_BATT_CHRG_I_H),\n> +\tAXP20X_ADC_CHANNEL(AXP803_BATT_DISCHRG_I, \"batt_dischrg_i\", IIO_CURRENT,\n> +\t\t\t AXP20X_BATT_DISCHRG_I_H),\n> +};\n> +\n> static int axp20x_adc_raw(struct iio_dev *indio_dev,\n> \t\t\t struct iio_chan_spec const *chan, int *val)\n> {\n> @@ -243,6 +278,19 @@ static int axp22x_adc_raw(struct iio_dev *indio_dev,\n> \treturn IIO_VAL_INT;\n> }\n> \n> +static int axp803_adc_raw(struct iio_dev *indio_dev,\n> +\t\t\t struct iio_chan_spec const *chan, int *val)\n> +{\n> +\tstruct axp20x_adc_iio *info = iio_priv(indio_dev);\n> +\n> +\t/* All channels on AXP803 are stored on 12 bits. */\n> +\t*val = axp20x_read_variable_width(info->regmap, chan->address, 12);\n> +\tif (*val < 0)\n> +\t\treturn *val;\n> +\n> +\treturn IIO_VAL_INT;\n> +}\n> +\n> static int axp20x_adc_scale_voltage(int channel, int *val, int *val2)\n> {\n> \tswitch (channel) {\n> @@ -342,6 +390,31 @@ static int axp22x_adc_scale(struct iio_chan_spec const *chan, int *val,\n> \t}\n> }\n> \n> +static int axp803_adc_scale(struct iio_chan_spec const *chan, int *val,\n> +\t\t\t int *val2)\n> +{\n> +\tswitch (chan->type) {\n> +\tcase IIO_VOLTAGE:\n> +\t\tif (chan->channel != AXP803_BATT_V)\n> +\t\t\treturn -EINVAL;\n> +\n> +\t\t*val = 1;\n> +\t\t*val2 = 100000;\n> +\t\treturn IIO_VAL_INT_PLUS_MICRO;\n> +\n> +\tcase IIO_CURRENT:\n> +\t\t*val = 1;\n\nA scale of 1 is assumed so you could drop providing this attribute.\nHowever, given there are scales for all other channels I guess that\nwould feel weird. There is nothing in our ABI saying you can't\nspecify things that are the default so it makes sense to me to keep\nthis here.\n\n> +\t\treturn IIO_VAL_INT;\n> +\n> +\tcase IIO_TEMP:\n> +\t\t*val = 106;\n> +\t\treturn IIO_VAL_INT;\n> +\n> +\tdefault:\n> +\t\treturn -EINVAL;\n> +\t}\n> +}\n> +\n> static int axp20x_adc_offset_voltage(struct iio_dev *indio_dev, int channel,\n> \t\t\t\t int *val)\n> {\n> @@ -425,6 +498,26 @@ static int axp22x_read_raw(struct iio_dev *indio_dev,\n> \t}\n> }\n> \n> +static int axp803_read_raw(struct iio_dev *indio_dev,\n> +\t\t\t struct iio_chan_spec const *chan, int *val,\n> +\t\t\t int *val2, long mask)\n> +{\n> +\tswitch (mask) {\n> +\tcase IIO_CHAN_INFO_OFFSET:\n\nI know it is impossible to get here unless we have a temperature channel,\nbut it still feels like this should be made apparent here.\n\nPerhaps a comment rather than an explicit check in the code?\n\n> +\t\t*val = -2525;\n> +\t\treturn IIO_VAL_INT;\n> +\n> +\tcase IIO_CHAN_INFO_SCALE:\n> +\t\treturn axp803_adc_scale(chan, val, val2);\n> +\n> +\tcase IIO_CHAN_INFO_RAW:\n> +\t\treturn axp803_adc_raw(indio_dev, chan, val);\n> +\n> +\tdefault:\n> +\t\treturn -EINVAL;\n> +\t}\n> +}\n> +\n> static int axp20x_write_raw(struct iio_dev *indio_dev,\n> \t\t\t struct iio_chan_spec const *chan, int val, int val2,\n> \t\t\t long mask)\n> @@ -472,6 +565,11 @@ static const struct iio_info axp22x_adc_iio_info = {\n> \t.driver_module = THIS_MODULE,\n> };\n> \n> +static const struct iio_info axp803_adc_iio_info = {\n> +\t.read_raw = axp803_read_raw,\n> +\t.driver_module = THIS_MODULE,\n\n.driver_module is now gone from this structure, but as it hasn't gone\nupstream from my tree yet I'll clean these up if they are still there\nonce we get to the point of applying this patch.\n\n> +};\n> +\n> static int axp20x_adc_rate(int rate)\n> {\n> \treturn AXP20X_ADC_RATE_HZ(rate);\n> @@ -512,9 +610,19 @@ static const struct axp_data axp22x_data = {\n> \t.maps = axp22x_maps,\n> };\n> \n> +static const struct axp_data axp803_data = {\n> +\t.iio_info = &axp803_adc_iio_info,\n> +\t.num_channels = ARRAY_SIZE(axp803_adc_channels),\n> +\t.channels = axp803_adc_channels,\n> +\t.adc_en1_mask = AXP803_ADC_EN1_MASK,\n> +\t.adc_en2 = false,\n> +\t.maps = axp22x_maps,\n> +};\n> +\n> static const struct platform_device_id axp20x_adc_id_match[] = {\n> \t{ .name = \"axp20x-adc\", .driver_data = (kernel_ulong_t)&axp20x_data, },\n> \t{ .name = \"axp22x-adc\", .driver_data = (kernel_ulong_t)&axp22x_data, },\n> +\t{ .name = \"axp803-adc\", .driver_data = (kernel_ulong_t)&axp803_data, },\n> \t{ /* sentinel */ },\n> };\n> MODULE_DEVICE_TABLE(platform, axp20x_adc_id_match);","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"iMoBEKJU\"; dkim-atps=neutral","mail.kernel.org;\n\tdmarc=none (p=none dis=none) header.from=kernel.org","mail.kernel.org;\n\tspf=none smtp.mailfrom=jic23@kernel.org"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0V7T0Fspz9sPs\n\tfor ;\n\tMon, 25 Sep 2017 00:33:08 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw7xv-0005nX-EA; Sun, 24 Sep 2017 14:33:03 +0000","from mail.kernel.org ([198.145.29.99])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw7xq-0005dN-Es for linux-arm-kernel@lists.infradead.org;\n\tSun, 24 Sep 2017 14:33:01 +0000","from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net\n\t[81.96.234.148])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.kernel.org (Postfix) with ESMTPSA id B52CF2170C;\n\tSun, 24 Sep 2017 14:32:35 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=vM6uf+gLuw43ufSmnyO4fdR8NT8ZHVBmPT/WRVY1twU=;\n\tb=iMoBEKJUHvTeFc\n\tUnxJkxdQu37iYGzuRdNZgPQlYrYXpXl7PZIHGoCl42rTVwrxYz+qlgTVLrhaC2+El5unNNwWIwmEM\n\tvP9InZgQSeNg3eoiYJZcraO7PAtcdDrcFxRRRunX1UhHSjze04dExZ/vXzbhbJNF/KC/eh0Ji/wey\n\tTZpjXsxK9bxoQSs9LHk4hwNFlQ5xFB3I2JnEFHdB5TEV05+qQnbXZBmC2mto++i7r2WYeLL+SGEsp\n\tCVQF1xiniATLg/MNYlluZj9+e1psoHrz+qy11JXKCuRkqBiAaqR0NhRaxMtdo9lf3KJ2qwjEoNOwz\n\tIS2QZ07biKZ/8ZF+rLcA==;","DMARC-Filter":"OpenDMARC Filter v1.3.2 mail.kernel.org B52CF2170C","Date":"Sun, 24 Sep 2017 15:32:33 +0100","From":"Jonathan Cameron ","To":"Icenowy Zheng ","Subject":"Re: [RFC PATCH 3/7] iio: adc: axp20x-adc: add support for AXP803","Message-ID":"<20170924153233.14eb8964@archlinux>","In-Reply-To":"<20170920151814.22461-4-icenowy@aosc.io>","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-4-icenowy@aosc.io>","X-Mailer":"Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170924_073258_713754_1232E17A ","X-CRM114-Status":"GOOD ( 22.94 )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [198.145.29.99 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-sunxi@googlegroups.com,\n\tlinux-kernel@vger.kernel.org,\n\tQuentin Schulz ,\n\tChen-Yu Tsai ,\n\tMaxime Ripard , \n\tLee Jones , linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774218,"web_url":"http://patchwork.ozlabs.org/comment/1774218/","msgid":"<20170924153436.570b60ea@archlinux>","list_archive_url":null,"date":"2017-09-24T14:34:36","subject":"Re: [RFC PATCH 4/7] power: supply: axp20x-battery: support AXP803","submitter":{"id":10151,"url":"http://patchwork.ozlabs.org/api/people/10151/","name":"Jonathan Cameron","email":"jic23@kernel.org"},"content":"On Wed, 20 Sep 2017 23:18:11 +0800\nIcenowy Zheng wrote:\n\n> The AXP803 PMIC has battery support like other AXP PMICs, but with\n> different definition of max target charging voltage and constant\n> charging current.\n> \n> Add support for AXP803 battery in axp20x-battery driver.\n> \n> Signed-off-by: Icenowy Zheng \n\nThis looks fine to me, but I haven't dived into the precise values\netc.\n\nJonathan\n\n> ---\n> drivers/power/supply/axp20x_battery.c | 88 +++++++++++++++++++++++++++++++----\n> 1 file changed, 78 insertions(+), 10 deletions(-)\n> \n> diff --git a/drivers/power/supply/axp20x_battery.c b/drivers/power/supply/axp20x_battery.c\n> index 7494f0f0eadb..c9a9fb320c92 100644\n> --- a/drivers/power/supply/axp20x_battery.c\n> +++ b/drivers/power/supply/axp20x_battery.c\n> @@ -49,6 +49,8 @@\n> #define AXP22X_CHRG_CTRL1_TGT_4_22V\t(1 << 5)\n> #define AXP22X_CHRG_CTRL1_TGT_4_24V\t(3 << 5)\n> \n> +#define AXP803_CHRG_CTRL1_TGT_4_35V\t(3 << 5)\n> +\n> #define AXP20X_CHRG_CTRL1_TGT_CURR\tGENMASK(3, 0)\n> \n> #define AXP20X_V_OFF_MASK\t\tGENMASK(2, 0)\n> @@ -123,20 +125,71 @@ static int axp22x_battery_get_max_voltage(struct axp20x_batt_ps *axp20x_batt,\n> \treturn 0;\n> }\n> \n> +static int axp803_battery_get_max_voltage(struct axp20x_batt_ps *axp20x_batt,\n> +\t\t\t\t\t int *val)\n> +{\n> +\tint ret, reg;\n> +\n> +\tret = regmap_read(axp20x_batt->regmap, AXP20X_CHRG_CTRL1, ®);\n> +\tif (ret)\n> +\t\treturn ret;\n> +\n> +\tswitch (reg & AXP20X_CHRG_CTRL1_TGT_VOLT) {\n> +\tcase AXP20X_CHRG_CTRL1_TGT_4_1V:\n> +\t\t*val = 4100000;\n> +\t\tbreak;\n> +\tcase AXP20X_CHRG_CTRL1_TGT_4_15V:\n> +\t\t*val = 4150000;\n> +\t\tbreak;\n> +\tcase AXP20X_CHRG_CTRL1_TGT_4_2V:\n> +\t\t*val = 4200000;\n> +\t\tbreak;\n> +\tcase AXP803_CHRG_CTRL1_TGT_4_35V:\n> +\t\t*val = 4350000;\n> +\t\tbreak;\n> +\tdefault:\n> +\t\treturn -EINVAL;\n> +\t}\n> +\n> +\treturn 0;\n> +}\n> +\n> static void raw_to_constant_charge_current(struct axp20x_batt_ps *axp, int *val)\n> {\n> -\tif (axp->axp_id == AXP209_ID)\n> +\tswitch (axp->axp_id) {\n> +\tcase AXP209_ID:\n> \t\t*val = *val * 100000 + 300000;\n> -\telse\n> +\t\tbreak;\n> +\tcase AXP221_ID:\n> \t\t*val = *val * 150000 + 300000;\n> +\t\tbreak;\n> +\tcase AXP803_ID:\n> +\t\t*val = *val * 200000 + 200000;\n> +\t\tbreak;\n> +\t}\n> }\n> \n> static void constant_charge_current_to_raw(struct axp20x_batt_ps *axp, int *val)\n> {\n> -\tif (axp->axp_id == AXP209_ID)\n> +\tswitch (axp->axp_id) {\n> +\tcase AXP209_ID:\n> \t\t*val = (*val - 300000) / 100000;\n> -\telse\n> +\t\tbreak;\n> +\tcase AXP221_ID:\n> \t\t*val = (*val - 300000) / 150000;\n> +\t\tbreak;\n> +\tcase AXP803_ID:\n> +\t\t*val = (*val - 200000) / 200000;\n> +\t\t/*\n> +\t\t * The maximum charge current on AXP803 is 2.8A, and the\n> +\t\t * datasheet says \"1110-1111 reserved\" in this part.\n> +\t\t * So we return an invalid value -1 in this situation,\n> +\t\t * which will be dealed by the caller of this function,\n> +\t\t */\n> +\t\tif (*val > 13)\n> +\t\t\t*val = -1;\n> +\t\tbreak;\n> +\t}\n> }\n> \n> static int axp20x_get_constant_charge_current(struct axp20x_batt_ps *axp,\n> @@ -269,9 +322,13 @@ static int axp20x_battery_get_prop(struct power_supply *psy,\n> \t\tif (ret)\n> \t\t\treturn ret;\n> \n> -\t\tif (axp20x_batt->axp_id == AXP221_ID &&\n> -\t\t !(reg & AXP22X_FG_VALID))\n> -\t\t\treturn -EINVAL;\n> +\t\tswitch (axp20x_batt->axp_id) {\n> +\t\tcase AXP221_ID:\n> +\t\tcase AXP803_ID:\n> +\t\t\tif (!(reg & AXP22X_FG_VALID))\n> +\t\t\t\treturn -EINVAL;\n> +\t\t\tbreak;\n> +\t\t};\n> \n> \t\t/*\n> \t\t * Fuel Gauge data takes 7 bits but the stored value seems to be\n> @@ -281,11 +338,19 @@ static int axp20x_battery_get_prop(struct power_supply *psy,\n> \t\tbreak;\n> \n> \tcase POWER_SUPPLY_PROP_VOLTAGE_MAX_DESIGN:\n> -\t\tif (axp20x_batt->axp_id == AXP209_ID)\n> +\t\tswitch (axp20x_batt->axp_id) {\n> +\t\tcase AXP209_ID:\n> \t\t\treturn axp20x_battery_get_max_voltage(axp20x_batt,\n> \t\t\t\t\t\t\t &val->intval);\n> -\t\treturn axp22x_battery_get_max_voltage(axp20x_batt,\n> -\t\t\t\t\t\t &val->intval);\n> +\t\tcase AXP221_ID:\n> +\t\t\treturn axp22x_battery_get_max_voltage(axp20x_batt,\n> +\t\t\t\t\t\t\t &val->intval);\n> +\t\tcase AXP803_ID:\n> +\t\t\treturn axp803_battery_get_max_voltage(axp20x_batt,\n> +\t\t\t\t\t\t\t &val->intval);\n> +\t\tdefault:\n> +\t\t\treturn -EINVAL;\n> +\t\t}\n> \n> \tcase POWER_SUPPLY_PROP_VOLTAGE_MIN_DESIGN:\n> \t\tret = regmap_read(axp20x_batt->regmap, AXP20X_V_OFF, ®);\n> @@ -467,6 +532,9 @@ static const struct of_device_id axp20x_battery_ps_id[] = {\n> \t}, {\n> \t\t.compatible = \"x-powers,axp221-battery-power-supply\",\n> \t\t.data = (void *)AXP221_ID,\n> +\t}, {\n> +\t\t.compatible = \"x-powers,axp803-battery-power-supply\",\n> +\t\t.data = (void *)AXP803_ID,\n> \t}, { /* sentinel */ },\n> };\n> MODULE_DEVICE_TABLE(of, axp20x_battery_ps_id);","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"OpexT9/D\"; dkim-atps=neutral","mail.kernel.org;\n\tdmarc=none (p=none dis=none) header.from=kernel.org","mail.kernel.org;\n\tspf=none smtp.mailfrom=jic23@kernel.org"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0V9t4JFlz9sPs\n\tfor ;\n\tMon, 25 Sep 2017 00:35:14 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw7zx-0007GY-Et; Sun, 24 Sep 2017 14:35:09 +0000","from mail.kernel.org ([198.145.29.99])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw7zp-0006jX-Q9 for linux-arm-kernel@lists.infradead.org;\n\tSun, 24 Sep 2017 14:35:05 +0000","from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net\n\t[81.96.234.148])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.kernel.org (Postfix) with ESMTPSA id 5E2532148C;\n\tSun, 24 Sep 2017 14:34:39 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=r22Bt50QxOkANf0Guq3OlhrCFt7IcEVSHofDX81ORbM=;\n\tb=OpexT9/D9GHCsk\n\t0xD4lvqttN94t5fI7kcPfgZg5IY/gfPjo05qtTpiuXpLFHDbRmnbFq/fSRKfZE5w3OLhRE0S+DWi8\n\tSXEKhKFd85s9Za41MdRbqAdMBtLlEqqLThGlAkD0Frt6cgYPXuOl/CssvsboMtjJi/hxPWB1VBc03\n\t1WgZ1QkgBEj2WNpeXWSlkvWnZuKd0yLoxISK32X2qqRDrYFDxxr85hhm/C149CEyAfj3aKlO/iCd1\n\tOWLC0ubhcMMkjgi1AoZw46C2ARMqiwUUbmzWzSKiJqGiLtMQU2EuH1Ur/g0NhZchXL81uiQZeUuGz\n\tsfJu9Hc/wFRJ4fe/0FZA==;","DMARC-Filter":"OpenDMARC Filter v1.3.2 mail.kernel.org 5E2532148C","Date":"Sun, 24 Sep 2017 15:34:36 +0100","From":"Jonathan Cameron ","To":"Icenowy Zheng ","Subject":"Re: [RFC PATCH 4/7] power: supply: axp20x-battery: support AXP803","Message-ID":"<20170924153436.570b60ea@archlinux>","In-Reply-To":"<20170920151814.22461-5-icenowy@aosc.io>","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-5-icenowy@aosc.io>","X-Mailer":"Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170924_073502_155837_22974FDE ","X-CRM114-Status":"GOOD ( 19.81 )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [198.145.29.99 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-sunxi@googlegroups.com,\n\tlinux-kernel@vger.kernel.org,\n\tQuentin Schulz ,\n\tChen-Yu Tsai ,\n\tMaxime Ripard , \n\tLee Jones , linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774220,"web_url":"http://patchwork.ozlabs.org/comment/1774220/","msgid":"<20170924153647.1d82e83c@archlinux>","list_archive_url":null,"date":"2017-09-24T14:36:47","subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","submitter":{"id":10151,"url":"http://patchwork.ozlabs.org/api/people/10151/","name":"Jonathan Cameron","email":"jic23@kernel.org"},"content":"On Thu, 21 Sep 2017 23:20:11 +0800\nIcenowy Zheng wrote:\n\n> 于 2017年9月21日 GMT+08:00 下午10:46:21, Jonathan Cameron 写到:\n> >On Wed, 20 Sep 2017 23:18:07 +0800\n> >Icenowy Zheng wrote:\n> > \n> >> The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power \n> >inputs: \n> >> AC, USB and Battery.\n> >> \n> >> This patchset adds support for the AC and Battery supplies, which is \n> >useful \n> >> for the boards from Pine64 (Pine64, SoPine w/ baseboard model A, \n> >Pinebook). \n> >> \n> >> The USB supply is not yet supported in this patchset because it's not\n> >> present on Pine series boards.\n> >> \n> >> In order to enable battery monitoring the ADC for battery is also \n> >enabled \n> >> for AXs.\n> >> \n> >> In order to enable battery monitoring the ADC for battery is also \n> >enabled \n> >> for AXP803. \n> >\n> >I'll go with the obvious question...\n> >\n> >Why an RFC rather than a standard patch submission? I'm not immediately\n> >seeing what is controversial! \n> \n> Oh I am just not confident about this patchset,\n> especially the IIO part.\n\nIt all looks fine to me. I would imagine that, once everyone is\nhappy, this will go through the mfd tree, but Lee may have other ideas!\n\nJonathan\n> \n> >\n> >Jonathan\n> > \n> >> \n> >> Icenowy Zheng (7):\n> >> dt-bindings: add compatibles for AXP803 Battery/USB power supplies\n> >> iio: adc: axp20x-adc: allow to skip ADC rate setup now\n> >> iio: adc: axp20x-adc: add support for AXP803\n> >> power: supply: axp20x-battery: support AXP803\n> >> mfd: axp20x: add cells for AXP803 ADC/AC Power/Battery\n> >> arm64: allwinner: a64: add power supply nodes in AXP803 DTSI\n> >> arm64: allwinner: a64: enable AC and Battery for Pine64\n> >> \n> >> .../bindings/power/supply/axp20x_battery.txt | 1 +\n> >> .../bindings/power/supply/axp20x_usb_power.txt | 1 +\n> >> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++\n> >> .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 8 ++\n> >> drivers/iio/adc/axp20x_adc.c | 114 \n> >++++++++++++++++++++- \n> >> drivers/mfd/axp20x.c | 11 ++\n> >> drivers/power/supply/axp20x_battery.c | 88 \n> >++++++++++++++-- \n> >> 7 files changed, 226 insertions(+), 12 deletions(-)\n> >> \n> >\n> >\n> >_______________________________________________\n> >linux-arm-kernel mailing list\n> >linux-arm-kernel@lists.infradead.org\n> >http://lists.infradead.org/mailman/listinfo/linux-arm-kernel \n> --\n> To unsubscribe from this list: send the line \"unsubscribe linux-iio\" in\n> the body of a message to majordomo@vger.kernel.org\n> More majordomo info at http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"nJ7g0gtt\"; dkim-atps=neutral","mail.kernel.org;\n\tdmarc=none (p=none dis=none) header.from=kernel.org","mail.kernel.org;\n\tspf=none smtp.mailfrom=jic23@kernel.org"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0VDJ3kJTz9sPs\n\tfor ;\n\tMon, 25 Sep 2017 00:37:20 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw821-0000Wj-MB; Sun, 24 Sep 2017 14:37:17 +0000","from mail.kernel.org ([198.145.29.99])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dw81w-0000QE-8t for linux-arm-kernel@lists.infradead.org;\n\tSun, 24 Sep 2017 14:37:14 +0000","from archlinux (cpc91196-cmbg18-2-0-cust659.5-4.cable.virginm.net\n\t[81.96.234.148])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby mail.kernel.org (Postfix) with ESMTPSA id E964D2148C;\n\tSun, 24 Sep 2017 14:36:49 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:MIME-Version:References:In-Reply-To:\n\tMessage-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=K0p3h6gqeD84dRCANrgiDNI5QVAyfG7BEHYgwXDW2hU=;\n\tb=nJ7g0gttaLvxrY\n\t/Px2qK9p+bzjWTmKZRDCyTBeC4RvaFiKrbQlFvZrunFoTHyM/kE9GyhQKVJ/iRm37K5xjuQ2wfJNQ\n\tjV6CyedC3EEhI0RCTEeA9jVdvffvxkpyg6IuQ8ePVeveqLH+xIgduhLcr3wUT2YRJoc/m6o73tefI\n\tH5tqwu6BfoXcap9MxvgQLytMuvFgDMzbCFPLJv+UUl0cniWRAmGAWvgNKHRCvsc44DhMaDcWBgGi6\n\tshqylCn1STV4xbulssH8I2hlqPB7IH9xZ9RE/GtvnVw+1rPXF62KMgbbVmQRm7pxfhAEs1Zk2F7bF\n\tiL8mv7X8rx0fYURK8e8w==;","DMARC-Filter":"OpenDMARC Filter v1.3.2 mail.kernel.org E964D2148C","Date":"Sun, 24 Sep 2017 15:36:47 +0100","From":"Jonathan Cameron ","To":"Icenowy Zheng ","Subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","Message-ID":"<20170924153647.1d82e83c@archlinux>","In-Reply-To":"","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170921154621.0000328d@huawei.com>\n\t","X-Mailer":"Claws Mail 3.15.1-dirty (GTK+ 2.24.31; x86_64-pc-linux-gnu)","MIME-Version":"1.0","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170924_073712_515439_BD5E1D68 ","X-CRM114-Status":"GOOD ( 19.08 )","X-Spam-Score":"-6.9 (------)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-6.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-5.0 RCVD_IN_DNSWL_HI RBL: Sender listed at http://www.dnswl.org/,\n\thigh trust [198.145.29.99 listed in list.dnswl.org]\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tQuentin Schulz ,\n\tlinux-sunxi@googlegroups.com,\n\tJonathan Cameron , \n\tMaxime Ripard ,\n\tChen-Yu Tsai , \n\tLee Jones , linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774466,"web_url":"http://patchwork.ozlabs.org/comment/1774466/","msgid":"<300803eb-4526-52c4-e74b-92625602a6da@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T07:20:02","subject":"Re: [RFC PATCH 2/7] iio: adc: axp20x-adc: allow to skip ADC rate\n\tsetup now","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 20/09/2017 17:18, Icenowy Zheng wrote:\n> The ADC rate setup on AXP803 is more complex than AXP20x/22x.\n> \n\nCan you elaborate?\n\nI can see two rate settings in register 0x85.\n\nBits 7-6 => TS/GPIO ADC speed setting (25, 50, 100, 200 Hz) ((ilog2(x) /\n25) << 6, same as AXP20X).\nBits 5-4 => Vol/Cur ADC speed setting (100, 200, 400, 800 Hz) ((ilog2(x)\n/ 100) << 4, same as AXP22X except the bit shift).\n\nJust set both to 100 or 200 by default (common rate). If someone wants\nto add a more specific rate setting, he could do it in the future.\n\nSetting a default is safer as we don't know if anything before the\nkernel does anything to this register.\n\nThanks,\nQuentin\n\n> As it's not a necessary setup, allow it to be skipped, to allow simpler\n> AXP803 support now.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> drivers/iio/adc/axp20x_adc.c | 6 ++++--\n> 1 file changed, 4 insertions(+), 2 deletions(-)\n> \n> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c\n> index 11e177180ea0..93dd6b80059e 100644\n> --- a/drivers/iio/adc/axp20x_adc.c\n> +++ b/drivers/iio/adc/axp20x_adc.c\n> @@ -556,8 +556,10 @@ static int axp20x_probe(struct platform_device *pdev)\n> \t\t\t\t AXP20X_ADC_EN2_MASK, AXP20X_ADC_EN2_MASK);\n> \n> \t/* Configure ADCs rate */\n> -\tregmap_update_bits(info->regmap, AXP20X_ADC_RATE, AXP20X_ADC_RATE_MASK,\n> -\t\t\t info->data->adc_rate(100));\n> +\tif (info->data->adc_rate)\n> +\t\tregmap_update_bits(info->regmap, AXP20X_ADC_RATE,\n> +\t\t\t\t AXP20X_ADC_RATE_MASK,\n> +\t\t\t\t info->data->adc_rate(100));\n> \n> \tret = iio_map_array_register(indio_dev, info->data->maps);\n> \tif (ret < 0) {\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"QisRPg8F\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0wV80srQz9tXG\n\tfor ;\n\tMon, 25 Sep 2017 17:20:48 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwNh4-0001Fr-G2; Mon, 25 Sep 2017 07:20:42 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwNgy-0000Ym-Bl for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 07:20:39 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 03AEA20878; Mon, 25 Sep 2017 09:20:12 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id A88B22085B;\n\tMon, 25 Sep 2017 09:20:01 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=ufxZFIzvWHQSgWXYOJQhXFArxld6h1czU8O+CJFXsPM=;\n\tb=QisRPg8FNTRn9e\n\tl/NpjdK/0nTNBBCEvhrVzA8OCMxxjXMLnQj+8VnhHBXhNao2j7f+mMgrjfU9S5mWi/oxJyLxOQHVZ\n\tdnfTcW6XZ7GRYJqVRoKd6dEMGrFdbEsDjXD5izd+q+oYBRh++O0Qb7dw30+GONA081AWDS0BkSmYa\n\tHwmVxHt+ckPyGZf5j8f3olDcuZkrMIkxUEJdrdkZtTmVhkWh2T7AVaBHWMQC7y/nzohYfwOCAohIC\n\tTfFhDXN7qY2xTWb9YEv2fnuHhU1fcg3cN0wWG1O5bVgAh3YlXcazRWRS6pmw6BI5wkni5xxcHcfpM\n\tXtsZdnjvRS/EU41u3lxQ==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 2/7] iio: adc: axp20x-adc: allow to skip ADC rate\n\tsetup now","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-3-icenowy@aosc.io>","From":"Quentin Schulz ","Message-ID":"<300803eb-4526-52c4-e74b-92625602a6da@free-electrons.com>","Date":"Mon, 25 Sep 2017 09:20:02 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170920151814.22461-3-icenowy@aosc.io>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_002036_742150_9E7277ED ","X-CRM114-Status":"GOOD ( 16.70 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774526,"web_url":"http://patchwork.ozlabs.org/comment/1774526/","msgid":"<1ba65180-9f13-0ef4-9213-1b20f15b5bdb@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T08:48:54","subject":"Re: [RFC PATCH 3/7] iio: adc: axp20x-adc: add support for AXP803","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 20/09/2017 17:18, Icenowy Zheng wrote:\n> AXP803 SoC features an ADC part including these channels: GPADC (GPIO0)\n> and TS pins, PMIC internal temperature sensor, battery voltage, battery\n> charge/discharge current.\n> \n> Add support for the battery-related channels and internal temperature\n> channel in order to allow battery monitoring. The TS and GPADC channels\n> are complex and will be support after more investigation.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> drivers/iio/adc/axp20x_adc.c | 108 +++++++++++++++++++++++++++++++++++++++++++\n> 1 file changed, 108 insertions(+)\n> \n> diff --git a/drivers/iio/adc/axp20x_adc.c b/drivers/iio/adc/axp20x_adc.c\n> index 93dd6b80059e..4f0cd98cf6ea 100644\n> --- a/drivers/iio/adc/axp20x_adc.c\n> +++ b/drivers/iio/adc/axp20x_adc.c\n> @@ -28,6 +28,8 @@\n[...]> /*\n> + * AXP803 shares the same consumer map with AXP22x, as it has no ADC for\n> + * VBUS and ACIN inputs either.\n> + */\n> +\n> +/*\n\nPut that in the commit log?\nIf we add a comment for each newly supported PMIC we will end up with\nmore comments than code :)\n\n[...]>\n> +static const struct axp_data axp803_data = {\n> +\t.iio_info = &axp803_adc_iio_info,\n> +\t.num_channels = ARRAY_SIZE(axp803_adc_channels),\n> +\t.channels = axp803_adc_channels,\n> +\t.adc_en1_mask = AXP803_ADC_EN1_MASK,\n> +\t.adc_en2 = false,\n\nNot required I guess, by default it is false.\n\nThanks,\nQuentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"BQvUV9fB\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0ySZ21Gmz9s76\n\tfor ;\n\tMon, 25 Sep 2017 18:49:34 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwP4z-0003ao-N0; Mon, 25 Sep 2017 08:49:29 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwP4v-0003IV-S3 for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 08:49:27 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 193F9208C5; Mon, 25 Sep 2017 10:49:04 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id B9ECA208E3;\n\tMon, 25 Sep 2017 10:48:53 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=16weRXA7WMvLP+ZAM8eD7azhAbY7DoJ+VR+HKyVhNOI=;\n\tb=BQvUV9fBkDPqHm\n\t32v28kiSFoeiAOHqSZ4WywDtqn6eMycDZCDyCcB//UgVF4D+ka2gC3TUbWhNgSztIIIMymVI1wxqM\n\tSc9w9r1B+DsUF6oQKYDxH9XHVfMOymjhmvk+sq63J17DzgIh5lGzNOyKqHr3vn1niL3z/f+34Z+Ck\n\t9Gp1mH7fo/UXq5NScwv8ZPvZeRF0GtSj7g+1Ah4AUKcq/RAYlQaNW/hR9xCRU0/7linlkEuPkFY8z\n\ts5MWpAh6+zVEMxtuLNKYxUeJ0WTyMiSxBETdciiTvQEK0p7pYj6w39eeTWWFbOKJs0UTS68S0OUdI\n\tY0hsEI1TRDPlDHAl9KYA==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 3/7] iio: adc: axp20x-adc: add support for AXP803","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-4-icenowy@aosc.io>","From":"Quentin Schulz ","Message-ID":"<1ba65180-9f13-0ef4-9213-1b20f15b5bdb@free-electrons.com>","Date":"Mon, 25 Sep 2017 10:48:54 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170920151814.22461-4-icenowy@aosc.io>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_014926_077152_9133A037 ","X-CRM114-Status":"GOOD ( 15.28 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774538,"web_url":"http://patchwork.ozlabs.org/comment/1774538/","msgid":"<9b72b982-85b2-22e0-076a-15c066619006@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T08:58:46","subject":"Re: [RFC PATCH 4/7] power: supply: axp20x-battery: support AXP803","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 20/09/2017 17:18, Icenowy Zheng wrote:\n> The AXP803 PMIC has battery support like other AXP PMICs, but with\n> different definition of max target charging voltage and constant\n> charging current.\n> \n> Add support for AXP803 battery in axp20x-battery driver.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> drivers/power/supply/axp20x_battery.c | 88 +++++++++++++++++++++++++++++++----\n> 1 file changed, 78 insertions(+), 10 deletions(-)\n> \n> diff --git a/drivers/power/supply/axp20x_battery.c b/drivers/power/supply/axp20x_battery.c\n> index 7494f0f0eadb..c9a9fb320c92 100644\n> --- a/drivers/power/supply/axp20x_battery.c\n> +++ b/drivers/power/supply/axp20x_battery.c\n> @@ -49,6 +49,8 @@\n[...]\n> static void constant_charge_current_to_raw(struct axp20x_batt_ps *axp, int *val)\n> {\n> -\tif (axp->axp_id == AXP209_ID)\n> +\tswitch (axp->axp_id) {\n> +\tcase AXP209_ID:\n> \t\t*val = (*val - 300000) / 100000;\n> -\telse\n> +\t\tbreak;\n> +\tcase AXP221_ID:\n> \t\t*val = (*val - 300000) / 150000;\n> +\t\tbreak;\n> +\tcase AXP803_ID:\n> +\t\t*val = (*val - 200000) / 200000;\n> +\t\t/*\n> +\t\t * The maximum charge current on AXP803 is 2.8A, and the\n> +\t\t * datasheet says \"1110-1111 reserved\" in this part.\n> +\t\t * So we return an invalid value -1 in this situation,\n> +\t\t * which will be dealed by the caller of this function,\n> +\t\t */\n\nGood, we could do that for the two others compatible as well. They are\nnot explicitly marked as reserved but it stops at 1100 for AXP223/AXP221\nfor example.\n\n> +\t\tif (*val > 13)\n> +\t\t\t*val = -1;\n> +\t\tbreak;\n> +\t}\n> }\n> \n> static int axp20x_get_constant_charge_current(struct axp20x_batt_ps *axp,\n> @@ -269,9 +322,13 @@ static int axp20x_battery_get_prop(struct power_supply *psy,\n> \t\tif (ret)\n> \t\t\treturn ret;\n> \n> -\t\tif (axp20x_batt->axp_id == AXP221_ID &&\n> -\t\t !(reg & AXP22X_FG_VALID))\n> -\t\t\treturn -EINVAL;\n> +\t\tswitch (axp20x_batt->axp_id) {\n> +\t\tcase AXP221_ID:\n> +\t\tcase AXP803_ID:\n> +\t\t\tif (!(reg & AXP22X_FG_VALID))\n> +\t\t\t\treturn -EINVAL;\n> +\t\t\tbreak;\n> +\t\t};\n\nLooks weird to me.\n\nif (axp20x_batt->axp_id != AXP209_ID && !(reg & AXP22X_FG_VALID))\n\nwould be a better match?\n[...]\n\nThanks,\nQuentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"Xw9sIMbQ\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0yh33QMfz9s4s\n\tfor ;\n\tMon, 25 Sep 2017 18:59:31 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPEb-0003i8-Vr; Mon, 25 Sep 2017 08:59:26 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPEU-0003Ut-AZ for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 08:59:21 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 97FB5208A2; Mon, 25 Sep 2017 10:58:56 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 0E85220832;\n\tMon, 25 Sep 2017 10:58:46 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=DbVM4rYpc2Ho9NNgsvaI/CokQDFXGEEVuufoB83y31U=;\n\tb=Xw9sIMbQEem5ls\n\tFMlxwWadzO4MhWJbJoA+yncDiyvGkPNpHVr0/uHWAtqdULqzys74w1xPH2a2jY0X9IEv3yrtMXiTL\n\tFnz6bti+0iDg3wKCJxGhPapbPa6C3TEGzZtPDoUYzFsA8iqaaPXmGptlFwGJY1Dz7woBavG42uhiW\n\tPcYg7NRP2G11cA5ThPZfeWhYYfWWU/RQp1ZgDs76xoVPNVCz5FS2kfs4iZVc0mOqD6UVkF1sla3Z2\n\t0lUnKvxvSropNBqrqaNMst7Wi0ICG8QEUDKcV2RoaAow1p0Y2j/bc6m7a08a722lgCRdMbVoGM77L\n\tFoZBTy742m7g8JPBZ4Iw==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 4/7] power: supply: axp20x-battery: support AXP803","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-5-icenowy@aosc.io>","From":"Quentin Schulz ","Message-ID":"<9b72b982-85b2-22e0-076a-15c066619006@free-electrons.com>","Date":"Mon, 25 Sep 2017 10:58:46 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170920151814.22461-5-icenowy@aosc.io>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_015918_772362_2159040C ","X-CRM114-Status":"GOOD ( 19.77 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774557,"web_url":"http://patchwork.ozlabs.org/comment/1774557/","msgid":"<8cf69d3d-0004-b5dc-16e0-ce1a4f8c313b@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T09:14:42","subject":"Re: [RFC PATCH 1/7] dt-bindings: add compatibles for AXP803\n\tBattery/USB power supplies","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 20/09/2017 17:18, Icenowy Zheng wrote:\n> The AXP803 PMIC has different Battery and USB power supplies than the\n> AXP series PMICs already supported by the kernel, but the AC power\n> supply is the same as AXP22x (as it can only detect the present/online\n> state of the AC power supply on both AXP22x and AXP803).\n> \n> Add compatible strings for the AXP803 Battery/USB power supplies. For AC\n> power supply the one on AXP803 is compatible with the one on AXP22x.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> Documentation/devicetree/bindings/power/supply/axp20x_battery.txt | 1 +\n> Documentation/devicetree/bindings/power/supply/axp20x_usb_power.txt | 1 +\n> 2 files changed, 2 insertions(+)\n> \n> diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt\n> index c24886676a60..091e5471a8c6 100644\n> --- a/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt\n> +++ b/Documentation/devicetree/bindings/power/supply/axp20x_battery.txt\n> @@ -4,6 +4,7 @@ Required Properties:\n> - compatible, one of:\n> \t\t\t\"x-powers,axp209-battery-power-supply\"\n> \t\t\t\"x-powers,axp221-battery-power-supply\"\n> +\t\t\t\"x-powers,axp803-battery-power-supply\"\n> \n> This node is a subnode of the axp20x/axp22x PMIC.\n> \n> diff --git a/Documentation/devicetree/bindings/power/supply/axp20x_usb_power.txt b/Documentation/devicetree/bindings/power/supply/axp20x_usb_power.txt\n> index ba8d35f66cbe..f30e3bf8d23f 100644\n> --- a/Documentation/devicetree/bindings/power/supply/axp20x_usb_power.txt\n> +++ b/Documentation/devicetree/bindings/power/supply/axp20x_usb_power.txt\n> @@ -4,6 +4,7 @@ Required Properties:\n> -compatible: One of: \"x-powers,axp202-usb-power-supply\"\n> \"x-powers,axp221-usb-power-supply\"\n> \"x-powers,axp223-usb-power-supply\"\n> + \"x-powers,axp803-usb-power-supply\"\n\nNo. You are adding support for AC and battery power supply in this\npatchset, not for USB.\n\nQuentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"k54tDet2\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0z2N3WcQz9t30\n\tfor ;\n\tMon, 25 Sep 2017 19:15:24 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPU0-000214-Sh; Mon, 25 Sep 2017 09:15:20 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPTv-0000aX-25 for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:15:17 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 3A74420884; Mon, 25 Sep 2017 11:14:53 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 6BE6820848;\n\tMon, 25 Sep 2017 11:14:42 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=S6LR8D3CIz08Xds7tCOSymeQDKOY56hZr0o4MaRcEEU=;\n\tb=k54tDet2P+PqEV\n\tyUQjc4gl3KaAkFDPHABhdijZEzwUqzwjQKazLiCUzzzuNHbdj6s/GV9yQ+7DTgHqAX+IdOVWrV6Nk\n\tZDhEH+AV9mPVCn96HsL+WhVeHChJm0OPv1Nt73wMJe2gp4cl/ARMr+altgBa6hoknttUpwBe+tuo7\n\tsPXqWVwzOvyqwIavSs6VXRHDzvkDFDPsLVv/I+TAvN6YFH+850uKR8i0mLzCfaBI+kimMl+993S/m\n\t760GaYDyRkRe1dko1ziirmHGMYOeemdk/D7oNTpEOpTHd3zX5oNpBkqjye3aZyO8jUBZpw85nNhQv\n\tLctT99jl12s7g6VIq+Wg==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT\n\tshortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 1/7] dt-bindings: add compatibles for AXP803\n\tBattery/USB power supplies","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-2-icenowy@aosc.io>","From":"Quentin Schulz ","Message-ID":"<8cf69d3d-0004-b5dc-16e0-ce1a4f8c313b@free-electrons.com>","Date":"Mon, 25 Sep 2017 11:14:42 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170920151814.22461-2-icenowy@aosc.io>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_021515_346306_AB7821D7 ","X-CRM114-Status":"GOOD ( 15.08 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774559,"web_url":"http://patchwork.ozlabs.org/comment/1774559/","msgid":"","list_archive_url":null,"date":"2017-09-25T09:14:34","subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","submitter":{"id":71295,"url":"http://patchwork.ozlabs.org/api/people/71295/","name":"Icenowy Zheng","email":"icenowy@aosc.io"},"content":"于 2017年9月25日 GMT+08:00 下午5:11:57, Quentin Schulz 写到:\n>Hi Icenowy,\n>\n>On 20/09/2017 17:18, Icenowy Zheng wrote:\n>> AXP803 PMIC features AC/USB/Battery power supplies.\n>> \n>> As we have now the device tree bindings for them, add device tree\n>> nodes for them.\n>> \n>> Signed-off-by: Icenowy Zheng \n>> ---\n>> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++++++++++++++\n>> 1 file changed, 15 insertions(+)\n>> \n>> diff --git a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>> index ff8af52743ff..3a8615231b7c 100644\n>> --- a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>> +++ b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>> @@ -49,6 +49,16 @@\n>> \tinterrupt-controller;\n>> \t#interrupt-cells = <1>;\n>> \n>> +\tac_power_supply: ac-power-supply {\n>> +\t\tcompatible = \"x-powers,axp221-ac-power-supply\";\n>> +\t\tstatus = \"disabled\";\n>> +\t};\n>> +\n>> +\tbattery_power_supply: battery-power-supply {\n>> +\t\tcompatible = \"x-powers,axp803-battery-power-supply\";\n>> +\t\tstatus = \"disabled\";\n>> +\t};\n>> +\n>> \tregulators {\n>> \t\t/* Default work frequency for buck regulators */\n>> \t\tx-powers,dcdc-freq = <3000>;\n>> @@ -147,4 +157,9 @@\n>> \t\t\tregulator-name = \"rtc-ldo\";\n>> \t\t};\n>> \t};\n>> +\n>> +\tusb_power_supply: usb_power_supply {\n>> +\t\tcompatible = \"x-powers,axp803-usb-power-supply\";\n>> +\t\tstatus = \"disabled\";\n>> +\t};\n>\n>No. You have added support for the AC and battery power supply drivers\n>in this patchset, not for USB.\n\nBut I added its device tree binding.\n\n>\n>Quentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"tt8jp0hg\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0z3w5p6Vz9t30\n\tfor ;\n\tMon, 25 Sep 2017 19:16:44 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPVH-00032H-DT; Mon, 25 Sep 2017 09:16:39 +0000","from butterfly.ash.relay.mailchannels.net ([23.83.222.27])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPVA-0002j6-9Z for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:16:36 +0000","from relay.mailchannels.net (localhost [127.0.0.1])\n\tby relay.mailchannels.net (Postfix) with ESMTP id EB4955C77B9;\n\tMon, 25 Sep 2017 09:16:05 +0000 (UTC)","from hermes.aosc.io (unknown [100.96.136.222])\n\t(Authenticated sender: lmn-TZDUIOWCRQMW)\n\tby relay.mailchannels.net (Postfix) with ESMTPA id BFB445C7E7B;\n\tMon, 25 Sep 2017 09:16:04 +0000 (UTC)","from hermes.aosc.io (hermes.aosc.io [172.20.96.210])\n\t(using TLSv1.2 with cipher DHE-RSA-AES256-GCM-SHA384)\n\tby 0.0.0.0:2500 (trex/5.9.14); Mon, 25 Sep 2017 09:16:05 +0000","from localhost (localhost [127.0.0.1]) (Authenticated sender:\n\ticenowy@aosc.io)\n\tby hermes.aosc.io (Postfix) with ESMTPSA id 8C12048103;\n\tMon, 25 Sep 2017 09:15:56 +0000 (UTC)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:Message-ID:From:To:Subject:MIME-Version\n\t:References:In-Reply-To:Date:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=AVTqdfNnbbwm6SjaS5FqlSgKNN/DhZWjN9sNfKhSyjE=;\n\tb=tt8jp0hgD/rAuM\n\tbXROXCjQO/2Cwkf6/cbPDOh0sNENWqRwQDpuHQ8RTMyIl+dPXJetGDJvX6yJot5mgqLTmhVCxvpSE\n\twuVtS2JALMAqtduW61lgQYhUfDMvRpV09aoU0qABwXTIJ/AZQwxBdOmBbZ+nCV1JmUcnvrwaOfBrz\n\tfWHjxYcBkqgdAS9iySfz2xlT+bZ7LsA4C4IDRwTKxdequG64pVpE1jIv/skaxXLwIrbuiZ3a5C1Mt\n\tB+I6UmFiJPwaACY1M5y4eOhPvUXaP1Ussi928S87HOsaqcBziqNfaASCUELan7+If1fmISAMsnKkz\n\t+XNjdqcdnFg1DUdJS0PA==;","X-Sender-Id":["lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io"],"X-MC-Relay":"Neutral","X-MailChannels-SenderId":"lmn-tzduiowcrqmw|x-authsender|icenowy@aosc.io","X-MailChannels-Auth-Id":"lmn-TZDUIOWCRQMW","X-Tasty-Hysterical":"0f13c17a5d7de740_1506330965487_134956776","X-MC-Loop-Signature":"1506330965487:2499623858","X-MC-Ingress-Time":"1506330965486","Date":"Mon, 25 Sep 2017 17:14:34 +0800","In-Reply-To":"<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-7-icenowy@aosc.io>\n\t<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>","MIME-Version":"1.0","Subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","To":"Quentin Schulz ,\n\tChen-Yu Tsai ,\n\tMaxime Ripard , \n\tJonathan Cameron , Lee Jones ","From":"Icenowy Zheng ","Message-ID":"","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_021633_176461_172F72DB ","X-CRM114-Status":"UNSURE ( 6.60 )","X-CRM114-Notice":"Please train this message.","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 RCVD_IN_DNSWL_NONE RBL: Sender listed at http://www.dnswl.org/,\n\tno trust [23.83.222.27 listed in list.dnswl.org]\n\t-0.0 RCVD_IN_MSPIKE_H3 RBL: Good reputation (+3)\n\t[23.83.222.27 listed in wl.mailspike.net]\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]\n\t-0.0 RCVD_IN_MSPIKE_WL Mailspike good senders","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774562,"web_url":"http://patchwork.ozlabs.org/comment/1774562/","msgid":"","list_archive_url":null,"date":"2017-09-25T09:19:07","subject":"Re: [linux-sunxi] Re: [RFC PATCH 6/7] arm64: allwinner: a64: add\n\tpower supply nodes in AXP803 DTSI","submitter":{"id":47154,"url":"http://patchwork.ozlabs.org/api/people/47154/","name":"Chen-Yu Tsai","email":"wens@csie.org"},"content":"On Mon, Sep 25, 2017 at 5:14 PM, Icenowy Zheng wrote:\n>\n>\n> 于 2017年9月25日 GMT+08:00 下午5:11:57, Quentin Schulz 写到:\n>>Hi Icenowy,\n>>\n>>On 20/09/2017 17:18, Icenowy Zheng wrote:\n>>> AXP803 PMIC features AC/USB/Battery power supplies.\n>>>\n>>> As we have now the device tree bindings for them, add device tree\n>>> nodes for them.\n>>>\n>>> Signed-off-by: Icenowy Zheng \n>>> ---\n>>> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++++++++++++++\n>>> 1 file changed, 15 insertions(+)\n>>>\n>>> diff --git a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> index ff8af52743ff..3a8615231b7c 100644\n>>> --- a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> +++ b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> @@ -49,6 +49,16 @@\n>>> interrupt-controller;\n>>> #interrupt-cells = <1>;\n>>>\n>>> + ac_power_supply: ac-power-supply {\n>>> + compatible = \"x-powers,axp221-ac-power-supply\";\n>>> + status = \"disabled\";\n>>> + };\n>>> +\n>>> + battery_power_supply: battery-power-supply {\n>>> + compatible = \"x-powers,axp803-battery-power-supply\";\n>>> + status = \"disabled\";\n>>> + };\n>>> +\n>>> regulators {\n>>> /* Default work frequency for buck regulators */\n>>> x-powers,dcdc-freq = <3000>;\n>>> @@ -147,4 +157,9 @@\n>>> regulator-name = \"rtc-ldo\";\n>>> };\n>>> };\n>>> +\n>>> + usb_power_supply: usb_power_supply {\n>>> + compatible = \"x-powers,axp803-usb-power-supply\";\n>>> + status = \"disabled\";\n>>> + };\n>>\n>>No. You have added support for the AC and battery power supply drivers\n>>in this patchset, not for USB.\n>\n> But I added its device tree binding.\n\nPlease do both at the same time. If you only add the binding without\nthe driver, how can you be sure the binding would be a proper fit?\nMoreover, no one can actually test it.\n\nChenYu","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"i4iJSm8k\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0z896F4yz9s7F\n\tfor ;\n\tMon, 25 Sep 2017 19:20:25 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPYa-0005U3-Sr; Mon, 25 Sep 2017 09:20:04 +0000","from smtp.csie.ntu.edu.tw ([140.112.30.61])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPYW-00053L-Ah for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:20:03 +0000","from mail-wr0-f169.google.com (mail-wr0-f169.google.com\n\t[209.85.128.169])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits))\n\t(No client certificate requested) (Authenticated sender: b93043)\n\tby smtp.csie.ntu.edu.tw (Postfix) with ESMTPSA id 86D6C20404\n\tfor ;\n\tMon, 25 Sep 2017 17:19:31 +0800 (CST)","by mail-wr0-f169.google.com with SMTP id u96so6585714wrb.6\n\tfor ;\n\tMon, 25 Sep 2017 02:19:31 -0700 (PDT)","by 10.223.196.226 with HTTP; Mon, 25 Sep 2017 02:19:07 -0700 (PDT)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:To:Subject:Message-ID:Date:From:\n\tReferences:In-Reply-To:MIME-Version:Reply-To:Content-ID:Content-Description:\n\tResent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=nC8Gd8G09C096mWZhUF/atjKjVkNEDyVrK91c3sk1hs=;\n\tb=i4iJSm8kuGLxsx\n\tkKahxV/LFAdfbxM692x+dRX8GCy+7fve6s5gndmAUy82Jssz0RdI5mvf9CjzTWvQCMtpuKKl8L7cF\n\tSosCO3j/D0kBFHOs4yYHN40XZyRL3qrn6MJbiMwYgcITHhZRGtPrhVg9Td/P5VXtaDTfIxfcjMktf\n\tky7GljL38uRZCxRzSxufCXH+RIJbe5t0SkbxVRWfiuCB1H89vvTZiw6veLB72dr7R8w1VXzwDWxv6\n\tZXS8IMAyDFKR/IVxIeneaYQt5ta8Nydg0racw/xMi18UIKyXkgZeyTGMZLmQocxRwOpC6V+iIBdqL\n\tdtb+9axNwkscB5baHQrQ==;","X-Gm-Message-State":"AHPjjUgzoczbLFjokuZkPlbsDXbyUOJnHGHEgMKIm3NHFhm7E9URxB5B\n\t7xfCDpqKi8COpTaLLtrL5egImSAKUs7B/HcFjdc=","X-Google-Smtp-Source":"AOwi7QDUTFyy2L6YO/Du2yps2VwOGwZZcqayiPs8pYWv3Lf4f0HDMWgiCbDHy4hewEUulZ91ZuepY+3eFKgcsOCpafw=","X-Received":"by 10.223.187.129 with SMTP id q1mr5960279wrg.239.1506331168125; \n\tMon, 25 Sep 2017 02:19:28 -0700 (PDT)","MIME-Version":"1.0","In-Reply-To":"","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-7-icenowy@aosc.io>\n\t<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>\n\t","From":"Chen-Yu Tsai ","Date":"Mon, 25 Sep 2017 17:19:07 +0800","X-Gmail-Original-Message-ID":"","Message-ID":"","Subject":"Re: [linux-sunxi] Re: [RFC PATCH 6/7] arm64: allwinner: a64: add\n\tpower supply nodes in AXP803 DTSI","To":"Icenowy Zheng ","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_022000_827270_EF9D5305 ","X-CRM114-Status":"GOOD ( 10.19 )","X-Spam-Score":"-2.3 (--)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-2.3 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-2.3 RCVD_IN_DNSWL_MED RBL: Sender listed at http://www.dnswl.org/,\n\tmedium trust [140.112.30.61 listed in list.dnswl.org]\n\t0.5 RCVD_IN_SORBS_SPAM RBL: SORBS: sender is a spam source\n\t[209.85.128.169 listed in dnsbl.sorbs.net]\n\t1.4 RCVD_IN_BRBL_LASTEXT RBL: No description available.\n\t[140.112.30.61 listed in bb.barracudacentral.org]\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree ,\n\t\"open list:THERMAL\" ,\n\tlinux-iio@vger.kernel.org, linux-sunxi ,\n\tlinux-kernel ,\n\tQuentin Schulz ,\n\tChen-Yu Tsai ,\n\tlinux-arm-kernel ,\n\tMaxime Ripard ,\n\tLee Jones , Jonathan Cameron ","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774565,"web_url":"http://patchwork.ozlabs.org/comment/1774565/","msgid":"","list_archive_url":null,"date":"2017-09-25T09:22:16","subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 24/09/2017 16:36, Jonathan Cameron wrote:\n> On Thu, 21 Sep 2017 23:20:11 +0800\n> Icenowy Zheng wrote:\n> \n>> 于 2017年9月21日 GMT+08:00 下午10:46:21, Jonathan Cameron 写到:\n>>> On Wed, 20 Sep 2017 23:18:07 +0800\n>>> Icenowy Zheng wrote:\n>>> \n>>>> The AXP803 PMIC, used by most Allwinner A64 boards, features 3 power \n>>> inputs: \n>>>> AC, USB and Battery.\n>>>>\n>>>> This patchset adds support for the AC and Battery supplies, which is \n>>> useful \n>>>> for the boards from Pine64 (Pine64, SoPine w/ baseboard model A, \n>>> Pinebook). \n>>>>\n>>>> The USB supply is not yet supported in this patchset because it's not\n>>>> present on Pine series boards.\n>>>>\n>>>> In order to enable battery monitoring the ADC for battery is also \n>>> enabled \n>>>> for AXs.\n>>>>\n>>>> In order to enable battery monitoring the ADC for battery is also \n>>> enabled \n>>>> for AXP803. \n>>>\n>>> I'll go with the obvious question...\n>>>\n>>> Why an RFC rather than a standard patch submission? I'm not immediately\n>>> seeing what is controversial! \n>>\n>> Oh I am just not confident about this patchset,\n>> especially the IIO part.\n> \n> It all looks fine to me. I would imagine that, once everyone is\n> happy, this will go through the mfd tree, but Lee may have other ideas!\n> \n\nSmall modifications to make but I definitely agree with Jonathan that\nyou did not need to send an RFC.\n\nQuentin\n\n> Jonathan\n>>\n>>>\n>>> Jonathan\n>>> \n>>>>\n>>>> Icenowy Zheng (7):\n>>>> dt-bindings: add compatibles for AXP803 Battery/USB power supplies\n>>>> iio: adc: axp20x-adc: allow to skip ADC rate setup now\n>>>> iio: adc: axp20x-adc: add support for AXP803\n>>>> power: supply: axp20x-battery: support AXP803\n>>>> mfd: axp20x: add cells for AXP803 ADC/AC Power/Battery\n>>>> arm64: allwinner: a64: add power supply nodes in AXP803 DTSI\n>>>> arm64: allwinner: a64: enable AC and Battery for Pine64\n>>>>\n>>>> .../bindings/power/supply/axp20x_battery.txt | 1 +\n>>>> .../bindings/power/supply/axp20x_usb_power.txt | 1 +\n>>>> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++\n>>>> .../arm64/boot/dts/allwinner/sun50i-a64-pine64.dts | 8 ++\n>>>> drivers/iio/adc/axp20x_adc.c | 114 \n>>> ++++++++++++++++++++- \n>>>> drivers/mfd/axp20x.c | 11 ++\n>>>> drivers/power/supply/axp20x_battery.c | 88 \n>>> ++++++++++++++-- \n>>>> 7 files changed, 226 insertions(+), 12 deletions(-)\n>>>> \n>>>\n>>>\n>>> _______________________________________________\n>>> linux-arm-kernel mailing list\n>>> linux-arm-kernel@lists.infradead.org\n>>> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel \n>> --\n>> To unsubscribe from this list: send the line \"unsubscribe linux-iio\" in\n>> the body of a message to majordomo@vger.kernel.org\n>> More majordomo info at http://vger.kernel.org/majordomo-info.html\n> \n> \n> _______________________________________________\n> linux-arm-kernel mailing list\n> linux-arm-kernel@lists.infradead.org\n> http://lists.infradead.org/mailman/listinfo/linux-arm-kernel\n>","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"B1Ydp5eE\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0zC42FgRz9s7F\n\tfor ;\n\tMon, 25 Sep 2017 19:22:56 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPbI-0008IZ-P9; Mon, 25 Sep 2017 09:22:52 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPbD-00082w-LM for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:22:50 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 256D320851; Mon, 25 Sep 2017 11:22:26 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 9F22020848;\n\tMon, 25 Sep 2017 11:22:15 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=kAyv2JpBEqcZ/QgP2ALlfHpsw7ulgyDOLjaJpVj0LOc=;\n\tb=B1Ydp5eEkCpAO/\n\tzIYnqKBEf72vNj2y8iQzHL4BEApM6MqN24RnuELJoOkSMqEdRFxs3BG1LoMCj+rUFVpD/SkBat97M\n\tuITNhuJHs6cvALG5CSTe/fuxT3I/23JqXwXyfaY3ifi09jZzqYiiQVMlCR+CCpJ8XwdCm95Q6pM/N\n\t7NlW1rEnbLplV177MFfWcvUpFKolERA6/Ksf2Uz5rJiAqx25SHn8cl5JuB0PgMhlRUjCSKXy5NxR+\n\tIrHJq03MzYLyn4gML5gLINihJXrimeD0mEnpC2/5hKMocit0UMgTTjCYHx+6cm+eN4wkVfgX8v8De\n\taYE+p+DxFl88Trn9oVHg==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 0/7] AXP803 AC/Battery support","To":"Jonathan Cameron , Icenowy Zheng ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170921154621.0000328d@huawei.com>\n\t\n\t<20170924153647.1d82e83c@archlinux>","From":"Quentin Schulz ","Message-ID":"","Date":"Mon, 25 Sep 2017 11:22:16 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170924153647.1d82e83c@archlinux>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_022248_005831_C0AA7751 ","X-CRM114-Status":"GOOD ( 18.31 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com,\n\tJonathan Cameron , \n\tMaxime Ripard ,\n\tChen-Yu Tsai , \n\tLee Jones , linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774568,"web_url":"http://patchwork.ozlabs.org/comment/1774568/","msgid":"<39833379-66a4-4657-3d12-26bc51c88815@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T09:24:43","subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 25/09/2017 11:14, Icenowy Zheng wrote:\n> \n> \n> 于 2017年9月25日 GMT+08:00 下午5:11:57, Quentin Schulz 写到:\n>> Hi Icenowy,\n>>\n>> On 20/09/2017 17:18, Icenowy Zheng wrote:\n>>> AXP803 PMIC features AC/USB/Battery power supplies.\n>>>\n>>> As we have now the device tree bindings for them, add device tree\n>>> nodes for them.\n>>>\n>>> Signed-off-by: Icenowy Zheng \n>>> ---\n>>> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++++++++++++++\n>>> 1 file changed, 15 insertions(+)\n>>>\n>>> diff --git a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>> b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> index ff8af52743ff..3a8615231b7c 100644\n>>> --- a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> +++ b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n>>> @@ -49,6 +49,16 @@\n>>> \tinterrupt-controller;\n>>> \t#interrupt-cells = <1>;\n>>> \n>>> +\tac_power_supply: ac-power-supply {\n>>> +\t\tcompatible = \"x-powers,axp221-ac-power-supply\";\n>>> +\t\tstatus = \"disabled\";\n>>> +\t};\n>>> +\n>>> +\tbattery_power_supply: battery-power-supply {\n>>> +\t\tcompatible = \"x-powers,axp803-battery-power-supply\";\n>>> +\t\tstatus = \"disabled\";\n>>> +\t};\n>>> +\n>>> \tregulators {\n>>> \t\t/* Default work frequency for buck regulators */\n>>> \t\tx-powers,dcdc-freq = <3000>;\n>>> @@ -147,4 +157,9 @@\n>>> \t\t\tregulator-name = \"rtc-ldo\";\n>>> \t\t};\n>>> \t};\n>>> +\n>>> +\tusb_power_supply: usb_power_supply {\n>>> +\t\tcompatible = \"x-powers,axp803-usb-power-supply\";\n>>> +\t\tstatus = \"disabled\";\n>>> +\t};\n>>\n>> No. You have added support for the AC and battery power supply drivers\n>> in this patchset, not for USB.\n> \n> But I added its device tree binding.\n\nYes and that is wrong. That would mislead users into thinking the usb\npower supply is supported (since the dt binding is here) while it's not.\n\nI would add the dt binding and the DT node only once it is supported.\n\nQuentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org\n\theader.b=\"FePMqw7p\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0zFy3TFrz9sNr\n\tfor ;\n\tMon, 25 Sep 2017 19:25:26 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPdf-000314-OH; Mon, 25 Sep 2017 09:25:19 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPdb-0001Qs-3i for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:25:17 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 9F7B420848; Mon, 25 Sep 2017 11:24:53 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id 5085A207FD;\n\tMon, 25 Sep 2017 11:24:43 +0200 (CEST)"],"DKIM-Signature":"v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=uP/L9nmLhzOa/Ch+Vn4/D1lgAp3HXoQvsppLE+Ubz30=;\n\tb=FePMqw7pI3ymCJ\n\tMxRf5goDUMFpAM98NhjAfPR1L+vX7a4BclGI5cL9iF/JTX/fhhDhVYVN/n4MlZLf1R0iKvb71a9ce\n\txB0uUzstSTDMJQjxvAvqSC0uhkYqmcISIy9qbO40byDEdZZtY2F/ogWyJ8IqGxBZYpZcOKJkxHzZZ\n\tI7Ft7BXlW3S/MXmE+pnJH9u6lZoOaNb6Dyv33tXISGFmhcgbTA+vpb2SVzdhdhbVab2xjFylzOAdR\n\tDv79WWECu3xSGB9hmbvLZOhFi0qDk5hAqJYDZep8UzeTrJS4gFm4cdsE4NLpP5HrKvVTszqlMqIs6\n\tKcuvi8TKBovcKdguLbIQ==;","X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-7-icenowy@aosc.io>\n\t<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>\n\t","From":"Quentin Schulz ","Message-ID":"<39833379-66a4-4657-3d12-26bc51c88815@free-electrons.com>","Date":"Mon, 25 Sep 2017 11:24:43 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_022515_442752_327161F4 ","X-CRM114-Status":"GOOD ( 12.47 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on bombadil.infradead.org summary:\n\tContent analysis details: (-1.9 points)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"utf-8\"","Content-Transfer-Encoding":"base64","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}},{"id":1774569,"web_url":"http://patchwork.ozlabs.org/comment/1774569/","msgid":"<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>","list_archive_url":null,"date":"2017-09-25T09:11:57","subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","submitter":{"id":69366,"url":"http://patchwork.ozlabs.org/api/people/69366/","name":"Quentin Schulz","email":"quentin.schulz@free-electrons.com"},"content":"Hi Icenowy,\n\nOn 20/09/2017 17:18, Icenowy Zheng wrote:\n> AXP803 PMIC features AC/USB/Battery power supplies.\n> \n> As we have now the device tree bindings for them, add device tree\n> nodes for them.\n> \n> Signed-off-by: Icenowy Zheng \n> ---\n> arch/arm64/boot/dts/allwinner/axp803.dtsi | 15 +++++++++++++++\n> 1 file changed, 15 insertions(+)\n> \n> diff --git a/arch/arm64/boot/dts/allwinner/axp803.dtsi b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n> index ff8af52743ff..3a8615231b7c 100644\n> --- a/arch/arm64/boot/dts/allwinner/axp803.dtsi\n> +++ b/arch/arm64/boot/dts/allwinner/axp803.dtsi\n> @@ -49,6 +49,16 @@\n> \tinterrupt-controller;\n> \t#interrupt-cells = <1>;\n> \n> +\tac_power_supply: ac-power-supply {\n> +\t\tcompatible = \"x-powers,axp221-ac-power-supply\";\n> +\t\tstatus = \"disabled\";\n> +\t};\n> +\n> +\tbattery_power_supply: battery-power-supply {\n> +\t\tcompatible = \"x-powers,axp803-battery-power-supply\";\n> +\t\tstatus = \"disabled\";\n> +\t};\n> +\n> \tregulators {\n> \t\t/* Default work frequency for buck regulators */\n> \t\tx-powers,dcdc-freq = <3000>;\n> @@ -147,4 +157,9 @@\n> \t\t\tregulator-name = \"rtc-ldo\";\n> \t\t};\n> \t};\n> +\n> +\tusb_power_supply: usb_power_supply {\n> +\t\tcompatible = \"x-powers,axp803-usb-power-supply\";\n> +\t\tstatus = \"disabled\";\n> +\t};\n\nNo. You have added support for the AC and battery power supply drivers\nin this patchset, not for USB.\n\nQuentin","headers":{"Return-Path":"","X-Original-To":"incoming-imx@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming-imx@bilbo.ozlabs.org","Authentication-Results":["ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=lists.infradead.org\n\t(client-ip=65.50.211.133; helo=bombadil.infradead.org;\n\tenvelope-from=linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org;\n\treceiver=)","ozlabs.org; dkim=pass (2048-bit key;\n\tunprotected) header.d=lists.infradead.org\n\theader.i=@lists.infradead.org header.b=\"OLn8fL8u\"; \n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n\tunprotected) header.d=infradead.org header.i=@infradead.org\n\theader.b=\"Bw3+6no0\"; dkim-atps=neutral"],"Received":["from bombadil.infradead.org (bombadil.infradead.org\n\t[65.50.211.133])\n\t(using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 3y0zGx23Fxz9sNr\n\tfor ;\n\tMon, 25 Sep 2017 19:26:17 +1000 (AEST)","from localhost ([127.0.0.1] helo=bombadil.infradead.org)\n\tby bombadil.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPeX-0003w6-Ir; Mon, 25 Sep 2017 09:26:13 +0000","from casper.infradead.org ([2001:8b0:10b:1236::1])\n\tby bombadil.infradead.org with esmtps (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPeU-0002v6-44 for linux-arm-kernel@bombadil.infradead.org;\n\tMon, 25 Sep 2017 09:26:10 +0000","from mail.free-electrons.com ([62.4.15.54])\n\tby casper.infradead.org with esmtp (Exim 4.87 #1 (Red Hat Linux))\n\tid 1dwPRF-00054G-0b for linux-arm-kernel@lists.infradead.org;\n\tMon, 25 Sep 2017 09:12:30 +0000","by mail.free-electrons.com (Postfix, from userid 110)\n\tid 6823A209C8; Mon, 25 Sep 2017 11:12:07 +0200 (CEST)","from [192.168.0.13] (LStLambert-657-1-97-87.w90-63.abo.wanadoo.fr\n\t[90.63.216.87])\n\tby mail.free-electrons.com (Postfix) with ESMTPSA id C42D920884;\n\tMon, 25 Sep 2017 11:11:56 +0200 (CEST)"],"DKIM-Signature":["v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=lists.infradead.org; s=bombadil.20170209; h=Sender:\n\tContent-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post:\n\tList-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:Date:\n\tMessage-ID:From:References:To:Subject:Reply-To:Content-ID:Content-Description\n\t:Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:\n\tList-Owner; bh=crY+4NNP77Jf0U+gKRxDUQcTBP9wwUCU1Yg8gxebzIw=;\n\tb=OLn8fL8uavOz3Y\n\t2CyTtfhNs5USWvR07iO4lTJIFbwaX79RJAlrDlRG4YURqhsB3H3p4BE++0pgFUD07miINhBvwiEWm\n\t3M44bYBRNYyc4DXyrA5xCCp+ybfmNuGbXTEo8oPjq0hD+npGJz3rDm5IIQx8SQkif0yzH4lDR0QGO\n\t/5ky/30QRMMTc9SLNMecf0T60kCoSy/1+IglSjcgEOz7kMkNvcMGZabxH/TsnGcOZ1YRQB2TTFWvC\n\t4U08jLY6I9CY+BA/PVuJs/4b/QVKnisYE6EMARx6eqROPfM0EgtRxp+6eir86r2ZQ3FUgczRhCnVB\n\tJSAju/hBi2P8+Z9AW4pA==;","v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed;\n\td=infradead.org; s=casper.20170209;\n\th=Content-Transfer-Encoding:Content-Type:\n\tIn-Reply-To:MIME-Version:Date:Message-ID:From:References:Cc:To:Subject:Sender\n\t:Reply-To:Content-ID:Content-Description:Resent-Date:Resent-From:\n\tResent-Sender:Resent-To:Resent-Cc:Resent-Message-ID:List-Id:List-Help:\n\tList-Unsubscribe:List-Subscribe:List-Post:List-Owner:List-Archive;\n\tbh=cnV1vldXJuEyARSaSWGmIjOeaA38H46mMWMvpPuYm0s=;\n\tb=Bw3+6no0EdJQmUGVXBenhIplJZ\n\ttIyep6GtS6P9UuFII+VRYoV7Xn3tt2ZQAfsJ1tTA5XP8N+i4VOPPyp5qNtefiLZm7sJr99Jvv2CXb\n\tBO+h+F6jX7S3tg2LatNZuWzAxv5U+iniAFp7HjfBEM/uV5rz2fe/U5vEGlMR4g/5d9eYUsTKTcc0M\n\tB4y12rQlgBvS1bBDeYPOyCLbjrmr0/veVoYH///n/vj5o0NztalGq/7EmhqcXP/3M3Pr3lo6JXJgV\n\tKuGPN1ryhR9lKLsIcPr3MDGDRkYHJOqIHAMboPgo6Vd8RZNxj5csaR/0Hq4BxGpWZgTM+uip3dhgG\n\tjuwHehCQ==;"],"X-Spam-Checker-Version":"SpamAssassin 3.4.0 (2014-02-07) on\n\tmail.free-electrons.com","X-Spam-Level":"","X-Spam-Status":"No, score=-1.0 required=5.0 tests=ALL_TRUSTED,SHORTCIRCUIT,\n\tURIBL_BLOCKED shortcircuit=ham autolearn=disabled version=3.4.0","Subject":"Re: [RFC PATCH 6/7] arm64: allwinner: a64: add power supply nodes in\n\tAXP803 DTSI","To":"Icenowy Zheng , Chen-Yu Tsai ,\n\tMaxime Ripard ,\n\tJonathan Cameron , Lee Jones ","References":"<20170920151814.22461-1-icenowy@aosc.io>\n\t<20170920151814.22461-7-icenowy@aosc.io>","From":"Quentin Schulz ","Message-ID":"<7ce5c53e-70ba-456e-ecf7-5132ef2c2c64@free-electrons.com>","Date":"Mon, 25 Sep 2017 11:11:57 +0200","User-Agent":"Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101\n\tThunderbird/52.3.0","MIME-Version":"1.0","In-Reply-To":"<20170920151814.22461-7-icenowy@aosc.io>","Content-Language":"en-US","X-CRM114-Version":"20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 ","X-CRM114-CacheID":"sfid-20170925_101229_068079_7D753595 ","X-CRM114-Status":"GOOD ( 16.80 )","X-Spam-Score":"-1.9 (-)","X-Spam-Report":"SpamAssassin version 3.4.1 on casper.infradead.org summary:\n\tContent analysis details: (-1.9 points, 5.0 required)\n\tpts rule name description\n\t---- ----------------------\n\t--------------------------------------------------\n\t-0.0 SPF_PASS SPF: sender matches SPF record\n\t-0.0 RP_MATCHES_RCVD Envelope sender domain matches handover relay\n\tdomain\n\t-1.9 BAYES_00 BODY: Bayes spam probability is 0 to 1%\n\t[score: 0.0000]","X-BeenThere":"linux-arm-kernel@lists.infradead.org","X-Mailman-Version":"2.1.21","Precedence":"list","List-Unsubscribe":",\n\t","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n\t","Cc":"devicetree@vger.kernel.org, linux-pm@vger.kernel.org,\n\tlinux-iio@vger.kernel.org, linux-kernel@vger.kernel.org,\n\tlinux-sunxi@googlegroups.com, linux-arm-kernel@lists.infradead.org","Content-Type":"text/plain; charset=\"us-ascii\"","Content-Transfer-Encoding":"7bit","Sender":"\"linux-arm-kernel\" ","Errors-To":"linux-arm-kernel-bounces+incoming-imx=patchwork.ozlabs.org@lists.infradead.org","List-Id":"linux-imx-kernel.lists.patchwork.ozlabs.org"}}]