[PATCH 3/6] staging: mt7621-gpio: change 'to_mediatek_gpio' to make just a one line return

Sergio Paracuellos sergio.paracuellos at gmail.com
Tue May 29 04:58:06 UTC 2018


Function to_mediatek_gpio can directly return without declaring
anything else in its body improving readability. Also change
pointer '*' declaration to be with return type in the upper line.

Signed-off-by: Sergio Paracuellos <sergio.paracuellos at gmail.com>
---
 drivers/staging/mt7621-gpio/gpio-mt7621.c | 10 +++-------
 1 file changed, 3 insertions(+), 7 deletions(-)

diff --git a/drivers/staging/mt7621-gpio/gpio-mt7621.c b/drivers/staging/mt7621-gpio/gpio-mt7621.c
index c96ae67..9132963 100644
--- a/drivers/staging/mt7621-gpio/gpio-mt7621.c
+++ b/drivers/staging/mt7621-gpio/gpio-mt7621.c
@@ -47,14 +47,10 @@ struct mtk_data {
 	struct mtk_gc *gc_map[MTK_BANK_CNT];
 };
 
-static inline struct mtk_gc
-*to_mediatek_gpio(struct gpio_chip *chip)
+static inline struct mtk_gc *
+to_mediatek_gpio(struct gpio_chip *chip)
 {
-	struct mtk_gc *mgc;
-
-	mgc = container_of(chip, struct mtk_gc, chip);
-
-	return mgc;
+	return container_of(chip, struct mtk_gc, chip);
 }
 
 static inline void
-- 
2.7.4



More information about the devel mailing list