[v3] add power-supply control to enable eeprom usage

[PATCH v3 0/2] add power-supply control to enable eeprom usage

Eliav Farber posted 2 patches 3 years, 7 months ago

Diff against v2
Download series mbox

.../devicetree/bindings/eeprom/at24.yaml      |  4 ++
drivers/misc/eeprom/at24.c                    | 40 +++++++++++++++++++
2 files changed, 44 insertions(+)

Expand all Fold all

[PATCH v3 0/2] add power-supply control to enable eeprom usage

Posted by Eliav Farber 3 years, 7 months ago

First patch describes the new binding property.
Second patch adds the functionality to the driver.

Change between v3 and v2:
- Apply on top of v6.0-rc1.

Change between v2 and v1:
- Use a gpio regulator for power-supply control.

Eliav Farber (2):
  dt-bindings: at24: add new optional power-supply property
  eeprom: at24: add support for power-supply control

 .../devicetree/bindings/eeprom/at24.yaml      |  4 ++
 drivers/misc/eeprom/at24.c                    | 40 +++++++++++++++++++
 2 files changed, 44 insertions(+)

-- 
2.37.1

Re: [PATCH v3 0/2] add power-supply control to enable eeprom usage

Posted by Bartosz Golaszewski 3 years, 7 months ago

On Sun, Aug 28, 2022 at 5:49 PM Eliav Farber <farbere@amazon.com> wrote:
>
> First patch describes the new binding property.
> Second patch adds the functionality to the driver.
>
> Change between v3 and v2:
> - Apply on top of v6.0-rc1.
>
> Change between v2 and v1:
> - Use a gpio regulator for power-supply control.
>
> Eliav Farber (2):
>   dt-bindings: at24: add new optional power-supply property
>   eeprom: at24: add support for power-supply control
>
>  .../devicetree/bindings/eeprom/at24.yaml      |  4 ++
>  drivers/misc/eeprom/at24.c                    | 40 +++++++++++++++++++
>  2 files changed, 44 insertions(+)
>
> --
> 2.37.1
>

What is the difference between this and the existing "vcc" regulator?
Why are you going through all the hoops in probe if a regular
regulator_get() will return a dummy regulator if the supply is not
found?

Bart