mbox series

[0/3] pinctrl: da9062: don't pull in internal GPIOLIB headers

Message ID 20230901112926.13216-1-brgl@bgdev.pl
Headers show
Series pinctrl: da9062: don't pull in internal GPIOLIB headers | expand

Message

Bartosz Golaszewski Sept. 1, 2023, 11:29 a.m. UTC
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>

I'm removing instances of users of internal GPIOLIB headers which should
not be included by anyone but GPIO core code. The pinctrl-da9062 driver
uses gpiochip_get_desc() which we should put into the gpio/driver.h
header as it does sometimes make sense for GPIO providers to get its own
descriptors without having to go through gpiochip_request_own_desc().

Bartosz Golaszewski (3):
  pinctrl: da9062: add missing include
  gpiolib: make gpiochip_get_desc() public
  pinctrl: da9062: don't include private GPIOLIB header

 drivers/gpio/gpiolib.h           | 2 --
 drivers/pinctrl/pinctrl-da9062.c | 7 +------
 include/linux/gpio/driver.h      | 2 ++
 3 files changed, 3 insertions(+), 8 deletions(-)

Comments

Bartosz Golaszewski Sept. 11, 2023, 7:18 a.m. UTC | #1
On Fri, Sep 1, 2023 at 1:29 PM Bartosz Golaszewski <brgl@bgdev.pl> wrote:
>
> From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
>
> I'm removing instances of users of internal GPIOLIB headers which should
> not be included by anyone but GPIO core code. The pinctrl-da9062 driver
> uses gpiochip_get_desc() which we should put into the gpio/driver.h
> header as it does sometimes make sense for GPIO providers to get its own
> descriptors without having to go through gpiochip_request_own_desc().
>
> Bartosz Golaszewski (3):
>   pinctrl: da9062: add missing include
>   gpiolib: make gpiochip_get_desc() public
>   pinctrl: da9062: don't include private GPIOLIB header
>
>  drivers/gpio/gpiolib.h           | 2 --
>  drivers/pinctrl/pinctrl-da9062.c | 7 +------
>  include/linux/gpio/driver.h      | 2 ++
>  3 files changed, 3 insertions(+), 8 deletions(-)
>
> --
> 2.39.2
>

Linus,

Do you mind me taking these through the GPIO tree or do you prefer to
apply them yourself in which case I'll apply patch 2/3 first and
provide you with an immutable tag?

Bartosz