lkml.org 
[lkml]   [2024]   [Apr]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
From
Subject[PATCH] Documentation: Add additional missing arg to dev_gpiod_get examples
Date
in 39b2bbe3d715 (gpio: add flags argument to gpiod_get*() functions) a
third argument was added to the function, apply that to examples found
in the tree with a simple regex: gpiod_get\([^(,]*,[^(,]*\)

Signed-off-by: Marco Sirabella <marco@sirabella.org>
---
Documentation/driver-api/pin-control.rst | 2 +-
Documentation/firmware-guide/acpi/enumeration.rst | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/Documentation/driver-api/pin-control.rst b/Documentation/driver-api/pin-control.rst
index 4639912dc9cc..bdb0a17d092b 100644
--- a/Documentation/driver-api/pin-control.rst
+++ b/Documentation/driver-api/pin-control.rst
@@ -1293,7 +1293,7 @@ So say that your driver is fetching its resources like this:
struct gpio_desc *gpio;

pinctrl = devm_pinctrl_get_select_default(&dev);
- gpio = devm_gpiod_get(&dev, "foo");
+ gpio = devm_gpiod_get(&dev, "foo", GPIOD_ASIS);

Here we first request a certain pin state and then request GPIO "foo" to be
used. If you're using the subsystems orthogonally like this, you should
diff --git a/Documentation/firmware-guide/acpi/enumeration.rst b/Documentation/firmware-guide/acpi/enumeration.rst
index 0165b09c0957..f9dcf15f7346 100644
--- a/Documentation/firmware-guide/acpi/enumeration.rst
+++ b/Documentation/firmware-guide/acpi/enumeration.rst
@@ -416,11 +416,11 @@ a code like this::

struct gpio_desc *irq_desc, *power_desc;

- irq_desc = gpiod_get(dev, "irq");
+ irq_desc = gpiod_get(dev, "irq", GPIOD_IN);
if (IS_ERR(irq_desc))
/* handle error */

- power_desc = gpiod_get(dev, "power");
+ power_desc = gpiod_get(dev, "power", GPIOD_IN);
if (IS_ERR(power_desc))
/* handle error */

--
2.43.2

\
 
 \ /
  Last update: 2024-05-27 16:40    [W:0.024 / U:0.536 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site