Re: [RFT PATCH 0/2] regulator: richtek,rt4801: use existing ena_gpiod feature

From: Krzysztof Kozlowski
Date: Fri Apr 01 2022 - 16:36:33 EST


On 01/04/2022 18:51, cy_huang(黃啟原) wrote:
>
> 於 五,2022-04-01 於 17:42 +0200,Krzysztof Kozlowski 提到:
>> Hi,
>>
>> Not tested.
>>
>> Follow up of discussion:
>> https://lore.kernel.org/all/2d124d53-1f36-5315-3877-af8f8ad5f824@xxxxxxxxxx/#t
>> <https://lore.kernel.org/all/2d124d53-1f36-5315-3877-af8f8ad5f824@xxxxxxxxxx/#t
>> >
>
> If RT4801 enable-gpio both go from high to low, the voltage reg will be
> reset, and I2C cannot be accessibele.
> This patch can be used to individually control enable for DSVP and DSVN,
> but the above i2c behavior is not considered.
> You can check rt4801_enable/rt4801_disable for the details.

Ah, so this is why this custom enable methods is there (pus bitmap etc).
Then the core ena_gpiod cannot be used, although still we want to move
on from two GPIOs in one enable-gpios property.

Best regards,
Krzysztof