Received-SPF: pass (google.com: domain of linux-kernel+bounces-153325-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45d1:ec00::1 as permitted sender) client-ip=2604:1380:45d1:ec00::1;
Received-SPF: Pass (protection.outlook.com: domain of topic.nl designates
 40.69.19.60 as permitted sender) receiver=protection.outlook.com;
 client-ip=40.69.19.60; helo=westeu100-emailsignatures-cloud.codetwo.com; pr=C
Authentication-Results-Original: dkim=none (message not signed)
 header.d=none;dmarc=none action=none header.from=topic.nl;
Message-ID: <e910a7cf-a5a3-4272-8695-138b6aeb6137@topic.nl>
Date: Mon, 22 Apr 2024 14:12:45 +0200
User-Agent: Mozilla Thunderbird
From: Mike Looijmans <mike.looijmans@topic.nl>
Subject: Re: [PATCH v4 1/5] dt-bindings: power: Extend battery chemistry with
 capacitor
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 linux-pm@vger.kernel.org
CC: Conor Dooley <conor+dt@kernel.org>,
 Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
 Rob Herring <robh@kernel.org>, Sebastian Reichel <sre@kernel.org>,
 devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
References: <1b153bce-a66a-45ee-a5c6-963ea6fb1c82.949ef384-8293-46b8-903f-40a477c056ae.b2a893bc-f00b-47cf-ae07-b37ec1bace22@emailsignatures365.codetwo.com>
 <20240415081305.316107-1-mike.looijmans@topic.nl>
 <1f94ee56-7f8e-453b-ab86-f640b298d81c@linaro.org>
 <16ff3a55-3d04-46e6-b78c-c1964c062329@topic.nl>
 <eab45cdd-a0b5-4766-b62c-00c1a92a3618@linaro.org>
Content-Language: en-US, nl
Organization: Topic
In-Reply-To: <eab45cdd-a0b5-4766-b62c-00c1a92a3618@linaro.org>
Content-Type: text/plain; charset="UTF-8"; format=flowed
Content-Transfer-Encoding: quoted-printable
Precedence: bulk
MIME-Version: 1.0
X-Microsoft-Antispam-Message-Info-Original:
 =?utf-8?B?UUdoUUhkYlM5YWNuY2ZFckN2S2hXTzB4QmQ5aGtkWlMyYW5FQmp4cG1XSFV1?=
 =?utf-8?B?TFgvbHAxenVXb25zY3FJQXJjaEZVaDIxcHp0RkpGSkdnc1J3dmJXeGt1Zk1h?=
 =?utf-8?B?czJRNUNNaEVWQkV1Y2RndUhiVmFNeno0UlAxZXN1RG5xS0t2M3J0V0h1YWxq?=
 =?utf-8?B?ei81blR6WGp1c2VDdkxZdjFJaTkzYWxoU0I4VlJHTU40QUNkbW92T01HRUU1?=
 =?utf-8?B?OHhxeVZtbExXbHRuTWJtWlltYjBnd3RIZ3NDcnNTblZXTXd2d2xYb3U5SjNK?=
 =?utf-8?B?NGVXQ1ZXWElMa3JzcTdEU3hNWVVXVmhWQ0c0MGRXRllhZU1IOGE2cDI5MTBn?=
 =?utf-8?B?VGcwT3hWek52WXlEQ1pTcDhBanJSMHdxdFRsbGhPZW9NRS95VWc5cHJuWjVR?=
 =?utf-8?B?NGJhMXR5Szh6VEdmZ3FwaW9KRzNib3hoT3cyamJXK3p1bE0zU0k0cStrUWx5?=
 =?utf-8?B?Unl5TDZaOXF2Qy91NmJEeTBUdUxuV0NVa3lWei9Zb2VGdGZZUGRycVNvM1I4?=
 =?utf-8?B?R1VpR3d1V2hIYlcwdGtCeUJmaW8rUmNKYStYSHk2ZEorbU9jV05ueHU4SFdS?=
 =?utf-8?B?VGpCZTdadlQrd0tGbXNCS3BCU1puWWlzbjRGaXk5ZmJmd2U3amZCTStPbXlm?=
 =?utf-8?B?S29MTUZWUmFDNnNmUlMwWHJpcmFHZnBrejhiMlEwUGlkeExySVA2bFdaYnZI?=
 =?utf-8?B?M3Z0ZEhtcTRkcTVyR1Y0VTZIU1YwMlUwYUhEb2R2bnRRWE1HMVA1YzZhdSsv?=
 =?utf-8?B?RlkzNmFLN21mbnBWak4rUU9XNS9EZzVmL21iYVR6M1h3YzkybGFQSlhCa252?=
 =?utf-8?B?WWpwa0wveW96NzQzZzlyN3NiS0hmejBjSi95VERpUGxXdjhzbEVHRjgrT1NN?=
 =?utf-8?B?dUhuc0Rha2o5UlcyWmE5NURTWUUxdG9pcUQrSkdkbWlMZHh4TkRqZWZGZmxx?=
 =?utf-8?B?aCtKaE9VaUR2bjZ4NERMS2s3cVVERmVQc1piY1VMUnJSWTZiZFdidlpWQ0Z0?=
 =?utf-8?B?TklsMTFtRFNIckF0akVqN3crY3RVQ1prNXBWWXd2WEdic2dwQ3Y2OS9NTXU5?=
 =?utf-8?B?eGFURGFGWG5DUGQ0ZXc4VURFMTZZdDNYb1FOa0tOcjBkSkNJRnZja1dSWkUv?=
 =?utf-8?B?MTMyM0EzWktPb3hyU3lOclVUMmhLb204TkJWNGhEUGRJcGliT24zR2k2Z1FV?=
 =?utf-8?B?ajZMMUpoL1hoOUY3WU5aV0laWk9zUm1nZlJScUMyZlR2S0g0VkJOWTRKMURI?=
 =?utf-8?B?bTgveTVjUnNDVGdlT2NzeGhjRVpScmJkM2U3Z3EyVGFCM2hoME5Cc0lqTEE1?=
 =?utf-8?B?aVpvRzhkdjFMQk9pWkVvUWEzc1RSaUh6bUdHWmdmd3NvWG82djhJZWpFNjdH?=
 =?utf-8?B?RTBMY25kTlhCWDN3YmFEeC9uR1NQaWs3Nml5WnAyWUdYQ3dyeGNHdk9CS0Nq?=
 =?utf-8?B?amRQTTJPOExNYVRTWHBDSW8raXdFd3F0MFZubkFINXp5YlZxd2djZ01sMG45?=
 =?utf-8?B?cnE2YTkzYy9lMDhCR0s2bldCYjNpdGpoQWNHaWt2RUF1THNPL2RadmJFN25C?=
 =?utf-8?B?dmpkUCt2bEZ6czhEa1lBTnFLeGpTYTJlV2FyblBUOWtzUDQrZk81VXh5aGMx?=
 =?utf-8?B?bGdFeXZBenh2eTlkT1BnMHZsUzViMnphN0RSQzY3NGo1UTdCWjJ5ZW1LaEpO?=
 =?utf-8?B?ZXhlcjA2K2NRejZ1YnFUY2tveDhkaWNLWExaeXJDa2xJN1AzTHNBTDNBPT0=?=
X-Forefront-Antispam-Report-Untrusted:
 CIP:255.255.255.255;CTRY:;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:AS8PR04MB8644.eurprd04.prod.outlook.com;PTR:;CAT:NONE;SFS:(13230031)(366007)(52116005)(1800799015)(376005)(38350700005);DIR:OUT;SFP:1102;
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR04MB6804
X-CodeTwo-MessageID: 8923e685-922e-431a-a4b6-a49dfaa04aef.20240422121249@westeu100-emailsignatures-cloud.codetwo.com
X-CodeTwoProcessed: true
X-EOPAttributedMessage: 0
X-MS-Exchange-Transport-CrossTenantHeadersStripped:
 DB1PEPF000509F1.eurprd03.prod.outlook.com
X-MS-PublicTrafficType: Email
X-MS-Office365-Filtering-Correlation-Id-Prvs:
	d62f6644-d0b9-4542-a412-08dc62c58550
X-Microsoft-Antispam: BCL:0;
X-Microsoft-Antispam-Message-Info:
	=?utf-8?B?YmwzUGFBK2xpTlN3dmQ5Y1QvY3ZkZEE2eUl5SDJEVGxJNVAwZzdDYXZoMUhw?=
 =?utf-8?B?QWlIdkV2YUE4em9Rait6YjZKVlY0ZTF5d3h6NnNmNzJYL20rR1Ivb29rRW1N?=
 =?utf-8?B?dDZjTFZVeUZFWDVoY0k5RjdrcFJvRG5vNTJLVmY5dDJ4SUZxeE93TUROM3Rv?=
 =?utf-8?B?MnRtZGdwZFpiVE5yaWdEdEtkOUR6MUp3QXFZUkxQYk5Pc2dlZkxVbUNCSHNC?=
 =?utf-8?B?cnlKQWhiOEJ1dVRqSlVLdXRRL2hxS3pONEw2QUE1U0lQUW44ampKeW1TNTZw?=
 =?utf-8?B?M0ZHeGRKZmJUd2ZTUlQ5VHV0OEtWNDVQMWUvMmp2MmplbzM4WHlkZUoxNjVS?=
 =?utf-8?B?TjZFSUYzV1dxVkE2Vjl2Y2haMDBjV2JsWDErS0xsaEliN0pXWVEvVVlUMUJC?=
 =?utf-8?B?SDRrY1VYcE0wdE5uWVJpQ1hvVzAzbUNpLy9IeHhxenBTOG45Znk0K3h6WUlr?=
 =?utf-8?B?cDVkVXUweDJkYk0xeWMzazArdnhER2h2azRmVUtuMTJmWjJJV0dUb1I5TnFq?=
 =?utf-8?B?Tzl6SklJeTlxa1ViOGlhU0llbzdqOWEvdVpXcklQNS90MFdWUElEdnpHTmRP?=
 =?utf-8?B?L1FiZFBVVUhwZmxlOVlXandpejlQbnpnOWVzY3p5Z2t5akdzaXBjZkM2bmtL?=
 =?utf-8?B?aWF4emJSVHBJT1pHcFdhVGdsS2JPNlNiMldaTDN0elJEUGx5TDk4TzdXbVdY?=
 =?utf-8?B?UFhpV253T0c0Qit4VGpGOEZBM0VnenFkNGRMcGxiM1V4T3JkVEhCc3VXYXNO?=
 =?utf-8?B?Z0VTQ0FveUtRZFZ3cnduU2hzaHRBZExhTzk0YU1XbzdvZVpGWktDRk5MR0o1?=
 =?utf-8?B?d295R3oyYVFMQTM2SWF2a1JVQXRzRy84WFBvU2lXdmxRMDJHRnhVVkhvbWhI?=
 =?utf-8?B?SXRueUxQU0xIMjJsYklaTlUzVVJ5Ty9FZFA4SWdSM3ErWFU4aEdUSGVJanNS?=
 =?utf-8?B?bmpYVGJTbGNnbEhFRXdZVGhiTmkwNzMxOFg1QWszMWNLS083ZlpVMVR1Y2c3?=
 =?utf-8?B?RHFGc1RZc2g0YjB5THU1T1ZZQjFlWm1uRG9rSVVqMnpkWDZ5M0M4a2lwUTQ3?=
 =?utf-8?B?VEtNTG90S000U0RTMzd1NUxISVdkc3IrSE9VazFmbjh4MkhJQnlZZkM0YmNO?=
 =?utf-8?B?cjFITittNXdJdmtneHk3MUIyR3c0a2UxWnZ6U2w4N3VyNFBoWVFIRVVhb0ZK?=
 =?utf-8?B?Ni8wclorVHNrbVVrd1hIUDNia05jaytMSDJmdlN2N1crc0U3clp0T1JXaDdH?=
 =?utf-8?B?dFZyMVBGekFCZXdkRHdtdXhMaWgrYVRaS1JScEdLODk1NUxuWEd1ZjloaitK?=
 =?utf-8?B?SFpDWGR5Z1hwWFF0SDlKZjB0eUpoQkNXVFlhQWtZTkRwUHVaVHRFUUtvRVBN?=
 =?utf-8?B?VHZwYkFIb3V1MzJtY2VuVWJZT1p6bnBQcFovN2pWQjAydFlYemJiRFFQR2hW?=
 =?utf-8?B?bmN1VGcveFJIRWpaaFk4QnlDdHNqcGdiTHFxTStLbXAzS3RLenJpTTRkRkVo?=
 =?utf-8?B?aEtZRVhLZWpycjRPVm9yL1pjMURhdjIraGNZRTRwMUJ2SW1qcy9pek14Ry9o?=
 =?utf-8?B?K3gzcmFXQ01TOG5mcjhadWxnS1VaWGU4S0dqdkwvMWhMZjQ2ME5UNVdZL2do?=
 =?utf-8?B?c21oVDl5SmZZM2FXSyt1cGo2eldud2ZhSHZ0OFdZS3dneUc3OFk1Q2dmMmlI?=
 =?utf-8?B?OE05Nko4SW9Cc0ZPL1o0SE5UWFBoZi8rMjZ6TlBaM2U5Umh1MnVGNEQxUU1K?=
 =?utf-8?Q?mtsCwOj99gSuGHqdzoEEpz77DCHNcdLeYHkaJ6F?=
X-Forefront-Antispam-Report:
	CIP:40.69.19.60;CTRY:IE;LANG:en;SCL:1;SRV:;IPV:NLI;SFV:NSPM;H:westeu100-emailsignatures-cloud.codetwo.com;PTR:westeu100-emailsignatures-cloud.codetwo.com;CAT:NONE;SFS:(13230031)(376005)(82310400014)(36860700004)(1800799015);DIR:OUT;SFP:1102;
X-OriginatorOrg: topic.nl
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Apr 2024 12:12:50.6447
 (UTC)
X-MS-Exchange-CrossTenant-Network-Message-Id: 270038f3-c777-4ddf-722c-08dc62c587d5
X-MS-Exchange-CrossTenant-Id: 449607a5-3517-482d-8d16-41dd868cbda3
X-MS-Exchange-CrossTenant-OriginalAttributedTenantConnectingIp: TenantId=449607a5-3517-482d-8d16-41dd868cbda3;Ip=[40.69.19.60];Helo=[westeu100-emailsignatures-cloud.codetwo.com]
X-MS-Exchange-CrossTenant-AuthSource:
	DB1PEPF000509F1.eurprd03.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Anonymous
X-MS-Exchange-CrossTenant-FromEntityHeader: HybridOnPrem
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB8PR04MB6972

On 22-04-2024 13:56, Krzysztof Kozlowski wrote:
> On 22/04/2024 09:50, Mike Looijmans wrote:
>> On 15-04-2024 17:24, Krzysztof Kozlowski wrote:
>>> On 15/04/2024 10:13, Mike Looijmans wrote:
>>>> Another technology to store energy is a (super)capacitor.
>>>>
>>>> Signed-off-by: Mike Looijmans <mike.looijmans@topic.nl>
>>>> ---
>>>>
>>>> (no changes since v1)
>>>>
>>>>    Documentation/devicetree/bindings/power/supply/battery.yaml | 1 +
>>>>    1 file changed, 1 insertion(+)
>>>>
>>>> diff --git a/Documentation/devicetree/bindings/power/supply/battery.ya=
ml b/Documentation/devicetree/bindings/power/supply/battery.yaml
>>>> index 491488e7b970..a22c97dfad88 100644
>>>> --- a/Documentation/devicetree/bindings/power/supply/battery.yaml
>>>> +++ b/Documentation/devicetree/bindings/power/supply/battery.yaml
>>>> @@ -44,6 +44,7 @@ properties:
>>>>          - const: lithium-ion-polymer
>>>>          - const: lithium-ion-iron-phosphate
>>>>          - const: lithium-ion-manganese-oxide
>>>> +      - const: capacitor
>>> Please keep some sort of order... everyone insists on adding to the end
>>> of the lists...
>> Could make it alphabetical, but that requires re-ordering the existing
>> ones so "lithium" sorts before "nickel".
> So just put it at beginning, less sorting later.

Ok. I'll wait a bit for comments on the other parts, before sending a v6=20
of the series.


>
>> I think the intention here was that the order is the same as the
>> matching defines in power-supply.h
> There is no such binding as power-supply.h. And other headers do not
> matter, they are not bindings.

Clear, thanks.


--=20
Mike Looijmans
System Expert

TOPIC Embedded Products B.V.
Materiaalweg 4, 5681 RJ Best
The Netherlands

T: +31 (0) 499 33 69 69
E: mike.looijmans@topic.nl
W: www.topic.nl