From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: ARC-Seal: i=1; a=rsa-sha256; t=1523639444; cv=none; d=google.com; s=arc-20160816; b=s9gvM1GMmZU0y/J13g0kNhJGz5Mit1vSlIbORx8LgWaKWLM36tRZSr/VTCUzXX2W1/ SQ8/gK4ik9ktr6yYcNPremhsxG6j0plPoriynnscnSCSynTfSVWSyRGO5RFEGLY0dqpg rYm/DErpJ2SaZfSM3n61TiFjYPplYOrdU92AC7ZpmHZDHamGKWT428us94D/FMQYvhP9 MVtMCpMzel1WRyJeJe1NkjIjSGcGACys/hSzDnyTtR4O2tzXZCHQBym+E/sZ3qja+nrH wUin9kQiwtPPKAUtj1vmzMcSSK+Ihw+V/a6KBRA1QtTSlPvkjKln157VkAuj3fPQ4crz iPhA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=message-id:from:cc:to:subject:content-transfer-encoding :mime-version:references:in-reply-to:user-agent:date:dkim-signature :arc-authentication-results; bh=bJRFGCKnKZzjRtny9ST74v8A6eZfUvufrmK2ax/zWo4=; b=y5/BXsSUMogER2NadaCsx4CZsykHmiI9YiK6mx4YEwERCV21KiU3qFkJm0VEMSeXv5 3+/js0P7o694zv6r9cPaL2hkNNt9xx4zp/JvJZuuZHwyehYWr4qG+5wfQTZS0GWKf7TN Q5s84D/PJ6y4t2RoMqCjKf2VLaO74GIVtyZRi1SbpKcyLD0E9Nod7Bt7alt5bdt4W/Y5 w605youugKl29+/2Wi+GZm5l0IiRFtgnDpTM1YUYoTUURX6+fgKngxRI9a0NiqqzXUVz FqajeVS4EvvJQ3tA4zk0bQsjGIRr2K7yRd3K0krjus0uE3UjpdWaDJP4qS+F0QHZXCks 4ldw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=DfLx7fIZ; spf=pass (google.com: domain of ctatlor97@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=ctatlor97@gmail.com; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com Authentication-Results: mx.google.com; dkim=pass header.i=@gmail.com header.s=20161025 header.b=DfLx7fIZ; spf=pass (google.com: domain of ctatlor97@gmail.com designates 209.85.220.65 as permitted sender) smtp.mailfrom=ctatlor97@gmail.com; dmarc=pass (p=NONE sp=QUARANTINE dis=NONE) header.from=gmail.com X-Google-Smtp-Source: AIpwx4/ECgXjtjTqIU6Gaq0XgtVdmXzWBy4hMg8Me1D/5Ep8aWgtDj/cFFkHBrw/HD+xl0iOj5X9bg== Date: Fri, 13 Apr 2018 18:10:42 +0100 User-Agent: K-9 Mail for Android In-Reply-To: <1D7FA61D-2233-45DC-A244-B2914BB1B29F@gmail.com> References: <20180407135934.26122-1-ctatlor97@gmail.com> <20180407175802.29444-1-ctatlor97@gmail.com> <20180407175802.29444-2-ctatlor97@gmail.com> <20180413163527.ludqnnc5nkj4spre@rob-hp-laptop> <1D7FA61D-2233-45DC-A244-B2914BB1B29F@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [PATCH v4 2/3] dt-bindings: power: supply: qcom_bms: Add bindings To: Rob Herring CC: linux-arm-msm@vger.kernel.org,Sebastian Reichel ,Mark Rutland ,Mauro Carvalho Chehab ,"David S. Miller" ,Greg Kroah-Hartman ,Linus Walleij ,Andrew Morton ,Randy Dunlap ,linux-pm@vger.kernel.org,devicetree@vger.kernel.org,linux-kernel@vger.kernel.org From: Craig Tatlor Message-ID: X-getmail-retrieved-from-mailbox: INBOX X-GMAIL-THRID: =?utf-8?q?1597095394982255487?= X-GMAIL-MSGID: =?utf-8?q?1597651754384759600?= X-Mailing-List: linux-kernel@vger.kernel.org List-ID: On 13 April 2018 18:08:54 BST, Craig Tatlor wrote: > > >On 13 April 2018 17:35:27 BST, Rob Herring wrote: >>On Sat, Apr 07, 2018 at 06:57:45PM +0100, Craig Tatlor wrote: >>> Add bindings for the Qualcomm Battery Monitoring system=2E >>>=20 >>> Signed-off-by: Craig Tatlor >>> --- >>> =2E=2E=2E/bindings/power/supply/qcom_bms=2Etxt | 93 >>+++++++++++++++++++ >>> 1 file changed, 93 insertions(+) >>> create mode 100644 >>Documentation/devicetree/bindings/power/supply/qcom_bms=2Etxt >>>=20 >>> diff --git >>a/Documentation/devicetree/bindings/power/supply/qcom_bms=2Etxt >>b/Documentation/devicetree/bindings/power/supply/qcom_bms=2Etxt >>> new file mode 100644 >>> index 000000000000=2E=2E6296399edc09 >>> --- /dev/null >>> +++ b/Documentation/devicetree/bindings/power/supply/qcom_bms=2Etxt >>> @@ -0,0 +1,93 @@ >>> +Qualcomm Battery Measurement System >>> + >>> +The Qualcomm Battery Measurement System is found inside of Qualcomm >>PM8941 >> >>Is it Monitoring or Measurment? Monitoring, not sure how I missed that :) >> >>> +PMICs=2E It provides OCV and coulomb counter registers that allow the >>kernel >>> +to infer a capacity level=2E >> >>s/kernel/OS/ >> >>OCV? >Open Circuit Voltage - should I expand this? >> >>> + >>> +Required properties: >>> +- compatible: Should contain "qcom,pm8941-bms"=2E >>> +- reg: Specifies the SPMI address and length >of >>the >>> + controller's registers=2E >>> +- interrupts: OCV threshold interrupt=2E >>> +- io-channels: Should contain IIO channel specifier >for >>the >>> + ADC channel that reports battery temperature=2E >>> +- io-channel-names: Should contain "temp"=2E >>> +- qcom,fcc-temp-legend: An array containing the temperature, in >>degC, >>> + for each column of the FCC lookup table=2E >> >>What's FCC? >Full charge capacity - Should i also expand this? or maybe explain in >header paragraph? >> >>> +- qcom,fcc-lut: An array of FCC values in mah, one >entry >>for each >>> + temperature defined in in qcom,fcc-temp-legend=2E >>> +- qcom,ocv-temp-legend: An array containing the temperature, in >>degC, >>> + for each column of the OCV lookup table=2E >>> +- qcom,ocv-capacity-legend: An array containing the capacity for >>each >>> + row of the OCV lookup table=2E >>> +- qcom,ocv-lut: An array of OCV values in mV, one entry >>for each >>> + capacity defined in qcom,ocv-capacity-legend=2E >> >>Need to specify the sizes of these if not 32-bit=2E >Right=2E >> >>All these seem to have units, so add unit suffixes as defined in=20 >>property-units=2Etxt=2E >Will do=2E >> >>> + >>> +Example: >>> + >>> + pm8941_vadc: vadc@3100 { >> >>adc@=2E=2E=2E >Okay=2E >> >>> + compatible =3D "qcom,spmi-vadc"; >>> + reg =3D <0x3100>; >>> + interrupts =3D <0x0 0x31 0x0 IRQ_TYPE_EDGE_RISING>; >>> + #address-cells =3D <1>; >>> + #size-cells =3D <0>; >>> + #io-channel-cells =3D <1>; >>> + >>> + bat_temp { >>> + reg =3D ; >>> + }; >>> + }; >>> + >>> + bms@4000 { >>> + compatible =3D "qcom,pm8941-bms"; >>> + reg =3D <0x4000>; >>> + interrupts =3D <0x0 0x40 0x4 IRQ_TYPE_EDGE_RISING>; >>> + interrupt-names =3D "ocv_thr"; >>> + >>> + io-channels =3D <&pm8941_vadc VADC_LR_MUX1_BAT_THERM>; >>> + io-channel-names =3D "temp"; >>> + >>> + qcom,fcc-temp-legend =3D /bits/ 8 <(-10) 0 25 50 65>; >>> + qcom,fcc-lut =3D /bits/ 16 <6010 6070 6680 6780 6670>; >>> + >>> + qcom,ocv-capacity-legend =3D /bits/ 8 <100 95 90 85 >>> + 80 75 70 65 >>> + 60 55 50 45 >>> + 40 35 30 25 >>> + 20 15 10 9 >>> + 8 7 6 5 4 >>> + 3 2 1 0>; >>> + >>> + qcom,ocv-temp-legend =3D /bits/ 8 <(-10) 0 25 50 65>; >>> + qcom,ocv-lut =3D /bits/ 16 <4288 4288 4306 4315 4315 >>> + 4261 4241 4259 4266 4246 >>> + 4201 4181 4201 4207 4187 >>> + 4153 4133 4150 4155 4135 >>> + 4105 4085 4100 4104 4084 >>> + 4058 4038 4052 4058 4038 >>> + 4012 3992 4004 4014 3994 >>> + 3970 3950 3959 3971 3951 >>> + 3931 3911 3915 3927 3907 >>> + 3899 3879 3880 3884 3864 >>> + 3873 3853 3851 3853 3833 >>> + 3848 3828 3827 3829 3809 >>> + 3829 3809 3808 3809 3789 >>> + 3815 3795 3791 3791 3771 >>> + 3801 3781 3775 3772 3752 >>> + 3785 3765 3751 3746 3726 >>> + 3767 3747 3727 3719 3699 >>> + 3750 3730 3702 3692 3672 >>> + 3728 3708 3680 3672 3652 >>> + 3720 3700 3676 3665 3645 >>> + 3712 3692 3670 3660 3645 >>> + 3695 3675 3658 3648 3633 >>> + 3662 3647 3629 3620 3610 >>> + 3620 3605 3589 3580 3570 >>> + 3562 3552 3538 3529 3519 >>> + 3490 3480 3474 3470 3465 >>> + 3403 3398 3388 3380 3375 >>> + 3320 3300 3255 3221 3206 >>> + 3000 3000 3000 3000 3000>; >>> + }; >>> + }; >>> +}; >>> --=20 >>> 2=2E17=2E0 >>>=20