2023-03-26 07:07:14

by Sumitra Sharma

[permalink] [raw]
Subject: [PATCH v4] staging: greybus: Inline pwm_chip_to_gb_pwm_chip()

Convert 'pwm_chip_to_gb_pwm_chip' from a macro to a static
inline function, to make the relevant types apparent in the
definition and to benefit from the type checking performed by
the compiler at call sites.

Signed-off-by: Sumitra Sharma <[email protected]>
---

Changes in v2: Change patch subjects, noted by Alison Schofield
<[email protected]>

Changes in v3: Change patch subjects to length 80 chars,
noted by Alison Schofield <[email protected]>

Changes in v4: Apply change to fresh git tree, noted by
Greg KH <[email protected]>

drivers/staging/greybus/pwm.c | 7 +++++--
1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/drivers/staging/greybus/pwm.c b/drivers/staging/greybus/pwm.c
index 26d39e08c3b6..88da1d796f13 100644
--- a/drivers/staging/greybus/pwm.c
+++ b/drivers/staging/greybus/pwm.c
@@ -21,8 +21,11 @@ struct gb_pwm_chip {
struct pwm_chip chip;
struct pwm_chip *pwm;
};
-#define pwm_chip_to_gb_pwm_chip(chip) \
- container_of(chip, struct gb_pwm_chip, chip)
+
+static inline struct gb_pwm_chip *pwm_chip_to_gb_pwm_chip(struct pwm_chip *chip)
+{
+ return container_of(chip, struct gb_pwm_chip, chip);
+}

static int gb_pwm_count_operation(struct gb_pwm_chip *pwmc)
{
--
2.25.1