lkml.org 
[lkml]   [2022]   [May]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH v3] drivers: gpio: zevio: drop of_gpio.h header
On Thu, May 12, 2022 at 9:14 AM Moses Christopher Bollavarapu
<mosescb.dev@gmail.com> wrote:
>
> Remove of_gpio.h header file, replace of_* functions and structs
> with appropriate alternatives.

LGTM,
Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>

> Signed-off-by: Moses Christopher Bollavarapu <mosescb.dev@gmail.com>
> ---
> V2 -> V3: Add missing return in front of dev_error_probe
> V1 -> V2: Move gpio_chip member to top of the struct
> Use dev_error_probe instead of dev_err
> Minor style fixes
>
> drivers/gpio/gpio-zevio.c | 25 ++++++++++++++-----------
> 1 file changed, 14 insertions(+), 11 deletions(-)
>
> diff --git a/drivers/gpio/gpio-zevio.c b/drivers/gpio/gpio-zevio.c
> index f6f8a541348f..ce9d1282165c 100644
> --- a/drivers/gpio/gpio-zevio.c
> +++ b/drivers/gpio/gpio-zevio.c
> @@ -11,7 +11,6 @@
> #include <linux/bitops.h>
> #include <linux/io.h>
> #include <linux/of_device.h>
> -#include <linux/of_gpio.h>
> #include <linux/slab.h>
> #include <linux/gpio/driver.h>
>
> @@ -53,22 +52,23 @@
> #define ZEVIO_GPIO_BIT(gpio) (gpio&7)
>
> struct zevio_gpio {
> + struct gpio_chip chip;
> spinlock_t lock;
> - struct of_mm_gpio_chip chip;
> + void __iomem *regs;
> };
>
> static inline u32 zevio_gpio_port_get(struct zevio_gpio *c, unsigned pin,
> unsigned port_offset)
> {
> unsigned section_offset = ((pin >> 3) & 3)*ZEVIO_GPIO_SECTION_SIZE;
> - return readl(IOMEM(c->chip.regs + section_offset + port_offset));
> + return readl(IOMEM(c->regs + section_offset + port_offset));
> }
>
> static inline void zevio_gpio_port_set(struct zevio_gpio *c, unsigned pin,
> unsigned port_offset, u32 val)
> {
> unsigned section_offset = ((pin >> 3) & 3)*ZEVIO_GPIO_SECTION_SIZE;
> - writel(val, IOMEM(c->chip.regs + section_offset + port_offset));
> + writel(val, IOMEM(c->regs + section_offset + port_offset));
> }
>
> /* Functions for struct gpio_chip */
> @@ -178,12 +178,15 @@ static int zevio_gpio_probe(struct platform_device *pdev)
> platform_set_drvdata(pdev, controller);
>
> /* Copy our reference */
> - controller->chip.gc = zevio_gpio_chip;
> - controller->chip.gc.parent = &pdev->dev;
> + controller->chip = zevio_gpio_chip;
> + controller->chip.parent = &pdev->dev;
>
> - status = of_mm_gpiochip_add_data(pdev->dev.of_node,
> - &(controller->chip),
> - controller);
> + controller->regs = devm_platform_ioremap_resource(pdev, 0);
> + if (IS_ERR(controller->regs))
> + return dev_err_probe(&pdev->dev, PTR_ERR(controller->regs),
> + "failed to ioremap memory resource\n");
> +
> + status = devm_gpiochip_add_data(&pdev->dev, &controller->chip, controller);
> if (status) {
> dev_err(&pdev->dev, "failed to add gpiochip: %d\n", status);
> return status;
> @@ -192,10 +195,10 @@ static int zevio_gpio_probe(struct platform_device *pdev)
> spin_lock_init(&controller->lock);
>
> /* Disable interrupts, they only cause errors */
> - for (i = 0; i < controller->chip.gc.ngpio; i += 8)
> + for (i = 0; i < controller->chip.ngpio; i += 8)
> zevio_gpio_port_set(controller, i, ZEVIO_GPIO_INT_MASK, 0xFF);
>
> - dev_dbg(controller->chip.gc.parent, "ZEVIO GPIO controller set up!\n");
> + dev_dbg(controller->chip.parent, "ZEVIO GPIO controller set up!\n");
>
> return 0;
> }
> --
> 2.30.2
>


--
With Best Regards,
Andy Shevchenko

\
 
 \ /
  Last update: 2022-05-12 12:39    [W:0.904 / U:0.852 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site