Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <ab469af7-22e1-c3cb-6374-26c99f30a900@axis.com>
Date:   Mon, 8 May 2023 08:51:47 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.6.0
Subject: Re: [PATCH v2 2/2] regulator: Add support for TI TPS6287x regulators
To:     Mark Brown <broonie@kernel.org>
CC:     Liam Girdwood <lgirdwood@gmail.com>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
        <linux-kernel@vger.kernel.org>, <devicetree@vger.kernel.org>,
        <kernel@axis.com>
References: <20230502-tps6287x-driver-v2-0-fb5419d46c49@axis.com>
 <20230502-tps6287x-driver-v2-2-fb5419d46c49@axis.com>
 <ZFOfX+PTsmA35TsC@finisterre.sirena.org.uk>
 <e02589b8-118d-0feb-d32a-bdf39d1b88f6@axis.com>
 <ZFTuAZvnvY2sSXrm@finisterre.sirena.org.uk>
Content-Language: en-US
From:   =?UTF-8?Q?M=c3=a5rten_Lindahl?= <martenli@axis.com>
In-Reply-To: <ZFTuAZvnvY2sSXrm@finisterre.sirena.org.uk>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?a3Y1dFRpaVlaekY0RE5XeHVPL1AwRkplakpUbDUxR3ZGcmZ0WWVDTEtSUnk2?=
 =?utf-8?B?cEpuODljTU8rS3J6SURoS1l3S0RoOUtYaVpQaCtGRGNCUk9icGE2Z0U1Njh4?=
 =?utf-8?B?ek4yYVd4bFFGQkRRTWZ3bVB1R1VhYTUxd2N5YXFPbWJiMURyb2Nvb2svMThC?=
 =?utf-8?B?dFVIRnhtMTlYN1Uxellja3dQOUZhMElwM1lSemVqTnJMY1hDRVFJbXhKNi95?=
 =?utf-8?B?TUF6cWlDQVRHWTE1UGZWVk1IY09SWEVESnkrQnNSQTlKLy9VajFYL0ZjQlNH?=
 =?utf-8?B?VFVPNCs0YXI3Y1FhOE9URE1sUndBUUMxdjA5Z2dMTnRKNVpNTUl6Qi9Fdlhx?=
 =?utf-8?B?YldCOUs1YlZNVjJNZHpkZ2VJUG1yVHBpQXJCbWhBZCtnSlZIK2tYUHJRQlB0?=
 =?utf-8?B?NzZQZktjQXBmQ0VIaUJoeXhqaDlHUm95bE9yOEFIaGNTSjBWNGJidnY3WkJZ?=
 =?utf-8?B?Rk9ndk5rbXNwK1Y4Z0FGK3hIUXJJMUV1SE1ZYjlady94d3FpcmxvL2VrcHV4?=
 =?utf-8?B?OFduWWxNYmllcDhyaGtMZk1NdWVFSEoyS05vOTlMWm5zWWlOcmJKdWh5b2pK?=
 =?utf-8?B?UFBkbnQvTVY5dmhvN2hQTVhhUFB5VmFBVVcxUFR2dDRCN1JsT3QxSWJNOVE0?=
 =?utf-8?B?SjFmWlJSNUkvQWYva3cvSWN1UkJzc1QrU2VpRGcwMUxpVkQxamlaamtLZEYr?=
 =?utf-8?B?OE1Va3NGblA3Q2oycENxMHpOZWlOWkFkeHk1WFVSTWd6UTliRU9YQU5ncEZI?=
 =?utf-8?B?MnVwdStvaHkxL1VvV3pSdUhVYS9NV09pZDQ5eDVybzFjR2YrbUlYejRZSW1m?=
 =?utf-8?B?MndUYjVZR1I0Uk1QYytDY2pTYy9jTzVvZkxEcERTemRrdGllQU1veWUxeUQr?=
 =?utf-8?B?Sk9LT0tCSFpRV1M1SEZvSnF3Tm9IR005aGRJRkNlN3U3WHZqVWpXUHUrMlJD?=
 =?utf-8?B?L1hueDZsK3VRRHFMMSsybkllSFI2NUNScDlQMW9ld0FHMVNIMUhqa3REbS9z?=
 =?utf-8?B?U3FNU2t5aXpyMFo5K25qWWNWd0dkWVFBQlBNTmNRZ3FYMUdGVFFub0VoWHg1?=
 =?utf-8?B?RHVTSXZhSDZxbDdJRXJKT3BSTXAzcFFjWldqVzd4V04yRjBPRzNCNm1Qd0ZT?=
 =?utf-8?B?ZjZKUm5LOXJPQ1BEeFd3bExkY0lQQ2dtSzJENnpzYy9DZHRWQWdXNlZnOVFC?=
 =?utf-8?B?cE5VbFdjWFFWNjFpOU5uSmZYdWwvaVJkN3VreUppTHV5WXkwcXNBNVI4RExv?=
 =?utf-8?B?K0ZyMVQxSkpnV09yeTRic3RDSGdBdEdwWmEyYTFaUjJvSzd6dTY0NEx5bzVu?=
 =?utf-8?B?cmRoT2tPMXhUbUkrRHpHTmV3SWFudytwa1dBazFtMW5uTXpYcVlCREtJSVly?=
 =?utf-8?B?K3dnUGNpQncwSXF4L0JCN2h0blZsUmlrQk10ZFlFSHVSb3JPTkVxa2l1bWRn?=
 =?utf-8?B?OVJDSTlWT1RhMjdUbkNmekc1MlcrMnQybmJQRUhidzhST2VPeWJ1alIyTzc0?=
 =?utf-8?B?UHRtQkhwK1hJRi81NUhqdkFkTXlnSDh3UjBnUzBCM3FrRVdtaURQVEJrbGQz?=
 =?utf-8?B?clM0K2pMeVBINnBiTVE4b2JiMnhSaVZ2WTZQZ1JBNXhDVXl6cXZ2cDNzWmJ6?=
 =?utf-8?B?dXAwMWxseGxuWGxYdVJ3aTc0eXora0JEK3d0TTZkUFp0V3ZSNVJqVk4ySUVS?=
 =?utf-8?B?cGNBeXNTdkk4dTRobUtwU1Z3OUI1dmJXUTM3L3d0WngwYTBNcEF1Y3o0R3Rm?=
 =?utf-8?B?TGRVdVd0VlVYd2ZDRUlIdHo0YS96UlRmRXNZZkVweHhiSlgxdmM4MGUwVzRm?=
 =?utf-8?B?ODJXNUthQkVMcHNBUjdBNGdZeWFRYURsdndJT1ZNaW5qdlhWazkzUDdGWlNE?=
 =?utf-8?B?a0pEeWJjQk1UTFdodXZFaWw4NzJ2WkZ3cGF2cHUyUFZFT0hwNFBuQzJ6RFNB?=
 =?utf-8?B?Rm1hVVB0VnlxdEh0NzlTT3BrWDhPT3huYXN6WHR4SmNROFpzSVo5cXljOEk5?=
 =?utf-8?B?VDV2SUczalVrMkFDRDNwTllIMU1CNWlNZEkvb2dLNDhEK2NjVGc4Y00wRlkr?=
 =?utf-8?B?cTBKd0VVUmhpS3FFNFp3Zm9LMjFiV3luRStsckRFQVQvRS9MektIQzE4VGtm?=
 =?utf-8?Q?mi8M=3D?=
X-MS-Exchange-CrossTenant-Network-Message-Id: d8e86b69-d249-4c63-eaa5-08db4f90b252
X-MS-Exchange-CrossTenant-AuthSource: AS8PR02MB9529.eurprd02.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 08 May 2023 06:51:49.0466
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 78703d3c-b907-432f-b066-88f7af9ca3af
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: uGYb6oZi6rEy3pcbVmxgXfYIeG6cqmu5ttuKkT3LroW0MV59vnZ2IS3c2r7pkgcQKi9Ym97dWFqJ6v19BqgLnQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: AM0PR02MB5876
X-Spam-Status: No, score=-7.4 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_MED,
        SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 5/5/23 13:52, Mark Brown wrote:
> On Fri, May 05, 2023 at 08:46:06AM +0200, Mårten Lindahl wrote:
>
>> 2. Dynamically set the range when a new voltage is set. This way any
>>     voltage from
>>     0.4V to 3.35V could be set if the DT node has:
>>     regulator-min-microvolt = <400000>;
>>     regulator-max-microvolt = <3350000>;
> You can probably use one of the linear range mappings with a custom
> set/get which munges the range selection and voltage selection into a
> single value.

Hi Mark!

I found the *sel_pickable_regmap helpers which seems to do what I need 
for this.

Thanks!

Kind regards

Mårten