linux-kernel - Re: [PATCH v9 5/8] gpio: Initial support for ROHM bd70528 GPIO block

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 14 Feb 2019 10:29:01 +0200
From:   Matti Vaittinen <matti.vaittinen@...rohmeurope.com>
To:     Linus Walleij <linus.walleij@...aro.org>
Cc:     Matti Vaittinen <mazziesaccount@...il.com>,
        Guenter Roeck <linux@...ck-us.net>,
        heikki.haikola@...rohmeurope.com, mikko.mutanen@...rohmeurope.com,
        Lee Jones <lee.jones@...aro.org>,
        Rob Herring <robh+dt@...nel.org>,
        Mark Rutland <mark.rutland@....com>,
        Mark Brown <broonie@...nel.org>,
        Greg KH <gregkh@...uxfoundation.org>,
        "Rafael J. Wysocki" <rafael@...nel.org>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        Bartosz Golaszewski <bgolaszewski@...libre.com>,
        Sebastian Reichel <sre@...nel.org>,
        Liam Girdwood <lgirdwood@...il.com>,
        Alessandro Zummo <a.zummo@...ertech.it>,
        Alexandre Belloni <alexandre.belloni@...tlin.com>,
        Wim Van Sebroeck <wim@...ux-watchdog.org>,
        "open list:OPEN FIRMWARE AND FLATTENED DEVICE TREE BINDINGS" 
        <devicetree@...r.kernel.org>,
        "linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
        linux-clk <linux-clk@...r.kernel.org>,
        "open list:GPIO SUBSYSTEM" <linux-gpio@...r.kernel.org>,
        Linux PM list <linux-pm@...r.kernel.org>,
        linux-rtc@...r.kernel.org,
        LINUXWATCHDOG <linux-watchdog@...r.kernel.org>,
        bogdan-petru.chircu-mare@....com
Subject: Re: [PATCH v9 5/8] gpio: Initial support for ROHM bd70528 GPIO block

Thanks a bunch Linus,

On Thu, Feb 14, 2019 at 09:00:33AM +0100, Linus Walleij wrote:
> On Wed, Feb 13, 2019 at 2:34 PM Matti Vaittinen
> <matti.vaittinen@...rohmeurope.com> wrote:
> 
> > ROHM BD70528 PMIC has 4 GPIO pins. Allow them to be
> > controlled by GPIO framework.
> >
> > IRQs are handled by regmap-irq and GPIO driver is not
> > aware of the irq usage.
> >
> > Signed-off-by: Matti Vaittinen <matti.vaittinen@...rohmeurope.com>
> > Reviewed-by: Linus Walleij <linus.walleij@...aro.org>
> 
> (...)
> 
> Just spotted this:
> 
> > +#include <linux/gpio/driver.h>
> > +#include <linux/gpio.h>
> 
> A driver should only need <linux/gpio/driver.h>
> the <linux/gpio.h> should not be used at all in new code,
> it is a legacy header.

Allright. I'll do v10 and drop the header. Thanks for pointing this out

> > +static int bd70528_gpio_set_config(struct gpio_chip *chip, unsigned int offset,
> > +                                  unsigned long config)
> > +{
> > +       struct bd70528_gpio *bdgpio = gpiochip_get_data(chip);
> > +
> > +       switch (pinconf_to_config_param(config)) {
> > +       case PIN_CONFIG_DRIVE_OPEN_DRAIN:
> > +               return regmap_update_bits(bdgpio->chip.regmap,
> > +                                         GPIO_OUT_REG(offset),
> > +                                         BD70528_GPIO_DRIVE_MASK,
> > +                                         BD70528_GPIO_OPEN_DRAIN);
> > +               break;
> > +       case PIN_CONFIG_DRIVE_PUSH_PULL:
> > +               return regmap_update_bits(bdgpio->chip.regmap,
> > +                                         GPIO_OUT_REG(offset),
> > +                                         BD70528_GPIO_DRIVE_MASK,
> > +                                         BD70528_GPIO_PUSH_PULL);
> > +               break;
> > +       case PIN_CONFIG_INPUT_DEBOUNCE:
> > +               return bd70528_set_debounce(bdgpio, offset,
> > +                                           pinconf_to_config_argument(config));
> > +               break;
> > +       default:
> > +               break;
> > +       }
> > +       return -ENOTSUPP;
> > +}
> 
> BTW I just merged code from Thomas Petazzoni that make it possible to
> also support pull-up and pull-down from GPIO here if you want to enable
> it.

Thanks for heads up Linus! I'll check it out although I don't want to
add up more changes in this series... But my initial feeling is that
this will deserve another patch (set) when initial support for BD70528
is available. (This answer is not locked, it may be I will fit the
change in this series if I am not buried under other duties ;] )

Br,
	Matti Vaittinen


-- 
Matti Vaittinen, Linux device drivers
ROHM Semiconductors, Finland SWDC
Kiviharjunlenkki 1E
90220 OULU
FINLAND

~~~ "I don't think so," said Rene Descartes.  Just then, he vanished ~~~