Received-SPF: pass (google.com: domain of linux-kernel+bounces-205163-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:45e3:2400::1 as permitted sender) client-ip=2604:1380:45e3:2400::1;
Message-ID: <91d65552-a5ad-4daa-be0d-7a3544ec9fec@phytec.com>
Date: Thu, 6 Jun 2024 16:38:07 -0700
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH] arm64: defconfig: Enable TI LP873X PMIC
To: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
 catalin.marinas@arm.com, will@kernel.org, quic_bjorande@quicinc.com,
 geert+renesas@glider.be, konrad.dybcio@linaro.org, shawnguo@kernel.org,
 neil.armstrong@linaro.org, arnd@arndb.de, m.szyprowski@samsung.com,
 nfraprado@collabora.com, u-kumar1@ti.com
Cc: linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org,
 upstream@lists.phytec.de, w.egorov@phytec.de
References: <20240604233836.3628063-1-nmorrisson@phytec.com>
 <1bc38042-7034-4937-a2e6-2dcfd8c2d609@linaro.org>
Content-Language: en-US
From: Nathan Morrisson <nmorrisson@phytec.com>
In-Reply-To: <1bc38042-7034-4937-a2e6-2dcfd8c2d609@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?OXlLd01DU2tBZEFwWTdrckY1aHZhZDVKRlh6SGZUMGY1ZHdNMVkzMlBPQVhY?=
 =?utf-8?B?cUdDOE13ZUd2dklsbGtCUUd5NTlBcCtsMHUvT08xbmRvWm5xWGFHeEJVQnlU?=
 =?utf-8?B?WmFDSUx3Wjg1WmM5YUI5Mllybm1DdWtvdXRIbkhLTWdvOFArK09lT29rTUo4?=
 =?utf-8?B?TlZBOUZ3K3lpVi8zaEwxRG9KMFNjN3NiL0NxM0NqSXg1aEJsWUVnTnprc0Zr?=
 =?utf-8?B?V2FpdFFNcEgzS1NMSnRRNHN0clB5NHlkeGdZSndqTFhuMEg0MjNZUkpMMFJn?=
 =?utf-8?B?WUpOMTFjOS9tUWxJTkRnREpKcjBXVGhVbkZQckhWYVQ2NlFFaUhwQ2w0KzVU?=
 =?utf-8?B?MVdkZWxMNUZRV3RtalZJcUhzWWlwbE41b3VBbVVrNnBNcFc0bFVYZ0FwcUFn?=
 =?utf-8?B?VjZuZ1ZKTUNIVlprVTRGTVdmRldJYVNsOHE2Wnc4dzhudStUSWhHa1B5Q24y?=
 =?utf-8?B?bWoybUtMY1FOL21pY29Id29yZ2lxcG5EMG9Fdi9uL2N4MDBJRy9sSUZQWm1m?=
 =?utf-8?B?eVp3OUg2SXhkVEwyeEZUcHZiTWJmSjBMUksyVkw5Z0g5Y21tMmlPek9BTWRl?=
 =?utf-8?B?TmpXc1BIMXR1dkZCd3JRUHZiVU5sWjd4YUk5OFRFWUNpMk8vWmt2S202S09t?=
 =?utf-8?B?ZWRJUytkdmI4OFlpN2s0cm1wMlJoUEwwdUtkZDFaSU1velFlTUl3LzQ0cjNF?=
 =?utf-8?B?UEdURjZsVUFFYktiZnc5YmFHNzNkMEY1VldHSk5RNXNxSkF6a01yWWgrK3hS?=
 =?utf-8?B?Q0FJb3JqUjFnajk0dEJQMllFQmNWK1NaVEFYQ0xsMlEyWXB6blo1M1EvRjFu?=
 =?utf-8?B?YktoclJ0Ulk0eFA5WDBwRzBvbnZ6TGRqcXRDWjd4SzRsZDRYaUdjVElXN2c3?=
 =?utf-8?B?SG1GMGhWSGIzdS81ZW8xZE5UQmJoZVhuRzk1U0M1dXpjcHRmWVFXZmZLTU5D?=
 =?utf-8?B?a2RrdDJlQTQ1cVM5NUxaZG1mMFFBOWdaaG4zZllXTW9Ea09JWHBSekkraGlQ?=
 =?utf-8?B?UmM2TWJtT3o5QnB3SXJlR0h1RmVHVmNIV1ptWjFJTTR5bGVrWmx0U25aTnk5?=
 =?utf-8?B?eXk2dUMwLy9BeHlZNFpwTitOcTRBbmswRkppU1RPaitwbEhjL0VrMVVkOXM3?=
 =?utf-8?B?TFZ0Y0l2d1kybnlaRmNzS3ozeXl1VW00RkoxeURQdFdickZoZnJpYmVNY0tl?=
 =?utf-8?B?UFQ1eU1GaUhXbGw0OWNabDZuYTJPZXBLdi9QTk1zWFVUS2E3UXpYUW5jUVBI?=
 =?utf-8?B?bEVwNUdTSlVwZmtJMWFVdmFPZnZ0ampoTTVhSTJrM3JBNHV0bDBhdXJTMGxD?=
 =?utf-8?B?N2xEZ0c4bGJjRDBWUWxYczhRbUsvVU5UQmR3ZlR0blg5RGdSQ1plaldYdnpT?=
 =?utf-8?B?S3llZ2xIR1Z2b2pwNlFLLzZKVjJabVNuaDhHZTkremw4WVM1Q3J1L1kyK0xV?=
 =?utf-8?B?alpIOU92NWV0YzlvUUU3d216RlhGK3FUL0MxY2MwbStGVXNmYnM0R2NucGph?=
 =?utf-8?B?cCtHeFZHcnJnbHRZbExEWGpQVGxUdXYvNXdLSFdVbnl4OE1Ja1pOYm5zTGxv?=
 =?utf-8?B?dVphL3ZBMVVtK2hTdk1FckdnZnBTZlF5bnA2V0EwdVAzRWlSS3l2RzBLNTFN?=
 =?utf-8?B?c3MyTDhpT01TUTBxN2NpblVxdUpkZmNmcWRMWGJWUFkzYWJITFEwazFSc2pM?=
 =?utf-8?B?aWp2ZWNmVXRSYW90Z01EcVZlK1NUelNyWGpmMEVGY0pKNy9CbHgzTHlwa1pu?=
 =?utf-8?B?cTdMM2Noa1FlYldWTXB1bTVZME5oaWRyUDZjNjgyc0s1a0FibUpvOUVheUkr?=
 =?utf-8?B?bEV1QUk5b0tqOEoyWnJnMGVuMUM5N0ZLZDU4bU8zWW94bHRtY09WRS9Qd01G?=
 =?utf-8?B?NGg1bFU4OTlBZmliZ2p6VXd3QnFhWUtXdmtiVk8xZU5pSEcvcnNIK2NITHow?=
 =?utf-8?B?TitDUFlRcWVKaVpjbnJsdUoxS1JJRTk1cTl4QUIxdE5xcFNEb05xMUxUanEw?=
 =?utf-8?B?anRwWjJZT0x4UFgxY21JamtQaFpPMGF0UHhiUnAzNWhJNXdzT3JrbTB6cFZZ?=
 =?utf-8?B?aTNDdm9hemUzUVc4WTNBYnZ2SHRiVzVTVTg0M0JZbk1YVnFPUVFUS28yRGd4?=
 =?utf-8?Q?kkl0JychiQ1pknIsriVa5QNmb?=
X-OriginatorOrg: phytec.com
X-MS-Exchange-CrossTenant-Network-Message-Id: ad1987f9-c553-4999-b604-08dc8681b98f
X-MS-Exchange-CrossTenant-AuthSource: SA1PR22MB5636.namprd22.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 06 Jun 2024 23:38:10.3374
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 67bcab1a-5db0-4ee8-86f4-1533d0b4b5c7
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: G6ZR++PewYpDRQ/lM9WL91XF/G6YfhaPVPYQbW3bXLjnfxoyrRFeGjYHVT4sWE/8DbrSF5N3soU14OlOLAZFYQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN4PR22MB2853


On 6/5/24 12:15 AM, Krzysztof Kozlowski wrote:
> On 05/06/2024 01:38, Nathan Morrisson wrote:
>> Enable the TI LP873X PMIC configs.
> Why? No user of this, sorry. Please explain in commit msg why this
> should be done, e.g. which upstream user needs it.

This will be used by the phytec am642-phyboard-electra, but the patch 
[1] has not been accepted yet. I will fix the other issues in a v2 as well.


[1] 
https://lore.kernel.org/lkml/aac6a41f-a384-43c5-8eb0-722cda74b4ea@phytec.de/


Regards,

Nathan

>
>> Signed-off-by: Nathan Morrisson <nmorrisson@phytec.com>
>> ---
>>   arch/arm64/configs/defconfig | 2 ++
>>   1 file changed, 2 insertions(+)
>>
>> diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
>> index 57a9abe78ee4..24ea62a8fdbf 100644
>> --- a/arch/arm64/configs/defconfig
>> +++ b/arch/arm64/configs/defconfig
>> @@ -745,6 +745,7 @@ CONFIG_MFD_SEC_CORE=y
>>   CONFIG_MFD_SL28CPLD=y
>>   CONFIG_RZ_MTU3=y
>>   CONFIG_MFD_TI_AM335X_TSCADC=m
>> +CONFIG_MFD_TI_LP873X=y
> Why this cannot be m?
>
>>   CONFIG_MFD_TPS65219=y
>>   CONFIG_MFD_TPS6594_I2C=m
>>   CONFIG_MFD_ROHM_BD718XX=y
>> @@ -760,6 +761,7 @@ CONFIG_REGULATOR_FAN53555=y
>>   CONFIG_REGULATOR_GPIO=y
>>   CONFIG_REGULATOR_HI6421V530=y
>>   CONFIG_REGULATOR_HI655X=y
>> +CONFIG_REGULATOR_LP873X=y
> Why this cannot be m?
>
>>   CONFIG_REGULATOR_MAX77620=y
>>   CONFIG_REGULATOR_MAX8973=y
>>   CONFIG_REGULATOR_MAX20411=m
> Best regards,
> Krzysztof
>