linux-gpio.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [GIT PULL] intel-pinctrl for 6.1-1
@ 2022-10-20 14:49 Andy Shevchenko
  2022-10-20 17:31 ` Bartosz Golaszewski
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-20 14:49 UTC (permalink / raw)
  To: Linux pin control, Bartosz Golaszewski
  Cc: Linus Walleij, Mika Westerberg, Andy Shevchenko

Hi Linux pin control  maintainers,

A bunch of cleanups of the pin control and GPIO headers. It has been a couple
of days in Linux Next without any major issue reported, a few fixes are already
incorporated in the respective patches. The idea is that pin control and GPIO
subsystems take it now and we will continue from this for all new code.

Linus, Bart, please pull to your trees.
The tag is signed and can be considered as immutable.

Thanks,

With Best Regards,
Andy Shevchenko

The following changes since commit 9abf2313adc1ca1b6180c508c25f22f9395cc780:

  Linux 6.1-rc1 (2022-10-16 15:36:24 -0700)

are available in the Git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/pinctrl/intel.git tags/intel-pinctrl-v6.1-1

for you to fetch changes up to 1053f6a58766ac8eadcb72630262a5a454048479:

  pinctrl: Clean up headers (2022-10-19 18:08:45 +0300)

----------------------------------------------------------------
intel-pinctrl for v6.1-1

* Add missing and remove unused headers in pin control and GPIO drivers
* Revise the pin control and GPIO headers

----------------------------------------------------------------
Andy Shevchenko (50):
      gpio: aspeed: Add missing header(s)
      gpio: arizona: Remove unused header(s)
      gpio: da9052: Remove unused header(s)
      gpio: mockup: Add missing header(s)
      gpio: pca953x: Add missing header(s)
      gpio: pl061: Add missing header(s)
      gpio: reg: Add missing header(s)
      gpio: wm8350: Remove unused header(s)
      gpio: tegra186: Add missing header(s)
      gpiolib: cdev: Add missing header(s)
      gpiolib: Clean up headers
      media: c8sectpfe: Add missing header(s)
      pinctrl: actions: Add missing header(s)
      pinctrl: aspeed: Add missing header(s)
      pinctrl: at91: Add missing header(s)
      pinctrl: axp209: Add missing header(s)
      pinctrl: bcm: Add missing header(s)
      pinctrl: bm1880: Add missing header(s)
      pinctrl: cygnus-mux: Add missing header(s)
      pinctrl: gemini: Add missing header(s)
      pinctrl: imx: Add missing header(s)
      pinctrl: ingenic: Add missing header(s)
      pinctrl: k210: Add missing header(s)
      pinctrl: lochnagar: Add missing header(s)
      pinctrl: lpc18xx: Add missing header(s)
      pinctrl: mediatek: Add missing header(s)
      pinctrl: microchip-sgpio: Add missing header(s)
      pinctrl: mvebu: Add missing header(s)
      pinctrl: npcm7xx: Add missing header(s)
      pinctrl: ocelot: Add missing header(s)
      pinctrl: qcom: Add missing header(s)
      pinctrl: renesas: Add missing header(s)
      pinctrl: samsung: Add missing header(s)
      pinctrl: single: Add missing header(s)
      pinctrl: spear: Add missing header(s)
      pinctrl: sprd: Add missing header(s)
      pinctrl: st: Add missing header(s)
      pinctrl: starfive: Add missing header(s)
      pinctrl: stm32: Add missing header(s)
      pinctrl: stmfx: Add missing header(s)
      pinctrl: sunxi: Add missing header(s)
      pinctrl: tegra: Add missing header(s)
      pinctrl: ti-iodelay: Add missing header(s)
      pinctrl: uniphier: Add missing header(s)
      pinctrl: zynqmp: Add missing header(s)
      pinctrl: cherryview: Add missing header(s)
      pinctrl: lynxpoint: Add missing header(s)
      pinctrl: merrifield: Add missing header(s)
      pinctrl: intel: Add missing header(s)
      pinctrl: Clean up headers

 drivers/gpio/gpio-arizona.c                        |  5 ++--
 drivers/gpio/gpio-aspeed.c                         |  5 ++--
 drivers/gpio/gpio-da9052.c                         |  9 +++----
 drivers/gpio/gpio-mockup.c                         |  1 +
 drivers/gpio/gpio-pca953x.c                        |  3 ++-
 drivers/gpio/gpio-pl061.c                          | 15 ++++++-----
 drivers/gpio/gpio-tegra186.c                       |  3 ++-
 drivers/gpio/gpio-wm8350.c                         |  7 +++--
 drivers/gpio/gpiolib-acpi.h                        | 12 +++++++++
 drivers/gpio/gpiolib-cdev.c                        |  4 ++-
 drivers/gpio/gpiolib-of.h                          | 11 +++++++-
 drivers/gpio/gpiolib-sysfs.h                       |  2 ++
 .../platform/st/sti/c8sectpfe/c8sectpfe-core.c     |  8 +++---
 drivers/pinctrl/actions/pinctrl-owl.c              | 10 ++++---
 drivers/pinctrl/aspeed/pinctrl-aspeed.c            |  1 +
 drivers/pinctrl/bcm/pinctrl-bcm281xx.c             | 13 +++++----
 drivers/pinctrl/bcm/pinctrl-cygnus-mux.c           |  9 ++++---
 drivers/pinctrl/bcm/pinctrl-iproc-gpio.c           | 12 +++++----
 drivers/pinctrl/bcm/pinctrl-ns2-mux.c              |  8 +++---
 drivers/pinctrl/bcm/pinctrl-nsp-mux.c              |  8 +++---
 drivers/pinctrl/cirrus/pinctrl-lochnagar.c         |  6 +++--
 drivers/pinctrl/core.c                             | 19 ++++++-------
 drivers/pinctrl/core.h                             | 12 ++++++++-
 drivers/pinctrl/devicetree.h                       |  6 +++++
 drivers/pinctrl/freescale/pinctrl-imx.c            |  8 +++---
 drivers/pinctrl/freescale/pinctrl-imx1-core.c      |  4 ++-
 drivers/pinctrl/intel/pinctrl-cherryview.c         |  6 +++--
 drivers/pinctrl/intel/pinctrl-intel.c              |  6 +++--
 drivers/pinctrl/intel/pinctrl-lynxpoint.c          |  6 +++--
 drivers/pinctrl/intel/pinctrl-merrifield.c         |  4 ++-
 drivers/pinctrl/mediatek/pinctrl-moore.c           |  3 +++
 drivers/pinctrl/mediatek/pinctrl-paris.c           |  5 ++++
 drivers/pinctrl/mvebu/pinctrl-mvebu.c              | 14 +++++-----
 drivers/pinctrl/nuvoton/pinctrl-npcm7xx.c          | 11 +++++---
 drivers/pinctrl/pinconf.h                          | 10 +++++++
 drivers/pinctrl/pinctrl-at91-pio4.c                | 10 ++++---
 drivers/pinctrl/pinctrl-at91.c                     | 16 ++++++-----
 drivers/pinctrl/pinctrl-axp209.c                   |  8 +++---
 drivers/pinctrl/pinctrl-bm1880.c                   |  6 +++--
 drivers/pinctrl/pinctrl-gemini.c                   | 12 +++++----
 drivers/pinctrl/pinctrl-ingenic.c                  | 10 ++++---
 drivers/pinctrl/pinctrl-k210.c                     | 12 +++++----
 drivers/pinctrl/pinctrl-lpc18xx.c                  |  6 +++--
 drivers/pinctrl/pinctrl-microchip-sgpio.c          |  4 ++-
 drivers/pinctrl/pinctrl-ocelot.c                   | 10 ++++---
 drivers/pinctrl/pinctrl-single.c                   |  6 ++---
 drivers/pinctrl/pinctrl-st.c                       | 21 +++++++++------
 drivers/pinctrl/pinctrl-stmfx.c                    |  2 ++
 drivers/pinctrl/pinctrl-utils.h                    |  5 ++++
 drivers/pinctrl/pinctrl-zynqmp.c                   |  5 +++-
 drivers/pinctrl/pinmux.c                           | 17 +++++++-----
 drivers/pinctrl/pinmux.h                           | 11 ++++++++
 drivers/pinctrl/qcom/pinctrl-spmi-gpio.c           |  8 +++---
 drivers/pinctrl/qcom/pinctrl-spmi-mpp.c            |  8 +++---
 drivers/pinctrl/renesas/pinctrl-rzg2l.c            |  7 +++--
 drivers/pinctrl/renesas/pinctrl-rzn1.c             |  8 ++++--
 drivers/pinctrl/renesas/pinctrl-rzv2m.c            |  4 ++-
 drivers/pinctrl/renesas/pinctrl.c                  |  8 +++---
 drivers/pinctrl/samsung/pinctrl-samsung.c          | 11 ++++----
 drivers/pinctrl/spear/pinctrl-spear.c              |  6 +++--
 drivers/pinctrl/sprd/pinctrl-sprd.c                |  6 +++--
 drivers/pinctrl/starfive/pinctrl-starfive-jh7100.c |  3 +++
 drivers/pinctrl/stm32/pinctrl-stm32.c              | 16 ++++++-----
 drivers/pinctrl/sunxi/pinctrl-sunxi.c              | 18 +++++++------
 drivers/pinctrl/tegra/pinctrl-tegra-xusb.c         |  6 +++--
 drivers/pinctrl/tegra/pinctrl-tegra.c              |  6 +++--
 drivers/pinctrl/ti/pinctrl-ti-iodelay.c            |  8 +++---
 drivers/pinctrl/uniphier/pinctrl-uniphier-core.c   |  8 +++---
 include/linux/gpio.h                               |  2 +-
 include/linux/gpio/aspeed.h                        |  4 +++
 include/linux/gpio/driver.h                        |  2 +-
 include/linux/gpio/gpio-reg.h                      |  4 +++
 include/linux/gpio/machine.h                       |  1 -
 include/linux/pinctrl/consumer.h                   | 31 ++++++++++------------
 include/linux/pinctrl/devinfo.h                    |  6 +++--
 include/linux/pinctrl/machine.h                    |  8 +++---
 include/linux/pinctrl/pinconf-generic.h            | 23 +++++++++-------
 include/linux/pinctrl/pinctrl.h                    | 18 ++++++-------
 include/linux/pinctrl/pinmux.h                     |  5 ++--
 79 files changed, 428 insertions(+), 234 deletions(-)

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-20 14:49 [GIT PULL] intel-pinctrl for 6.1-1 Andy Shevchenko
@ 2022-10-20 17:31 ` Bartosz Golaszewski
  2022-10-20 17:43   ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Bartosz Golaszewski @ 2022-10-20 17:31 UTC (permalink / raw)
  To: Andy Shevchenko; +Cc: Linux pin control, Linus Walleij, Mika Westerberg

On Thu, Oct 20, 2022 at 4:58 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
>
> Hi Linux pin control  maintainers,
>
> A bunch of cleanups of the pin control and GPIO headers. It has been a couple
> of days in Linux Next without any major issue reported, a few fixes are already
> incorporated in the respective patches. The idea is that pin control and GPIO
> subsystems take it now and we will continue from this for all new code.
>
> Linus, Bart, please pull to your trees.
> The tag is signed and can be considered as immutable.
>
> Thanks,
>
> With Best Regards,
> Andy Shevchenko
>
> The following changes since commit 9abf2313adc1ca1b6180c508c25f22f9395cc780:
>
>   Linux 6.1-rc1 (2022-10-16 15:36:24 -0700)
>
> are available in the Git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/pinctrl/intel.git tags/intel-pinctrl-v6.1-1
>
> for you to fetch changes up to 1053f6a58766ac8eadcb72630262a5a454048479:
>
>   pinctrl: Clean up headers (2022-10-19 18:08:45 +0300)
>
> ----------------------------------------------------------------
> intel-pinctrl for v6.1-1
>
> * Add missing and remove unused headers in pin control and GPIO drivers
> * Revise the pin control and GPIO headers
>
> ----------------------------------------------------------------
> Andy Shevchenko (50):
>       gpio: aspeed: Add missing header(s)
>       gpio: arizona: Remove unused header(s)
>       gpio: da9052: Remove unused header(s)
>       gpio: mockup: Add missing header(s)
>       gpio: pca953x: Add missing header(s)
>       gpio: pl061: Add missing header(s)
>       gpio: reg: Add missing header(s)
>       gpio: wm8350: Remove unused header(s)
>       gpio: tegra186: Add missing header(s)
>       gpiolib: cdev: Add missing header(s)
>       gpiolib: Clean up headers

Can you send the GPIO changes separately? This way I don't need to
pull all those pinctrl patches into the GPIO PR for the next merge
window.

Bartosz

>       media: c8sectpfe: Add missing header(s)
>       pinctrl: actions: Add missing header(s)
>       pinctrl: aspeed: Add missing header(s)
>       pinctrl: at91: Add missing header(s)
>       pinctrl: axp209: Add missing header(s)
>       pinctrl: bcm: Add missing header(s)
>       pinctrl: bm1880: Add missing header(s)
>       pinctrl: cygnus-mux: Add missing header(s)
>       pinctrl: gemini: Add missing header(s)
>       pinctrl: imx: Add missing header(s)
>       pinctrl: ingenic: Add missing header(s)
>       pinctrl: k210: Add missing header(s)
>       pinctrl: lochnagar: Add missing header(s)
>       pinctrl: lpc18xx: Add missing header(s)
>       pinctrl: mediatek: Add missing header(s)
>       pinctrl: microchip-sgpio: Add missing header(s)
>       pinctrl: mvebu: Add missing header(s)
>       pinctrl: npcm7xx: Add missing header(s)
>       pinctrl: ocelot: Add missing header(s)
>       pinctrl: qcom: Add missing header(s)
>       pinctrl: renesas: Add missing header(s)
>       pinctrl: samsung: Add missing header(s)
>       pinctrl: single: Add missing header(s)
>       pinctrl: spear: Add missing header(s)
>       pinctrl: sprd: Add missing header(s)
>       pinctrl: st: Add missing header(s)
>       pinctrl: starfive: Add missing header(s)
>       pinctrl: stm32: Add missing header(s)
>       pinctrl: stmfx: Add missing header(s)
>       pinctrl: sunxi: Add missing header(s)
>       pinctrl: tegra: Add missing header(s)
>       pinctrl: ti-iodelay: Add missing header(s)
>       pinctrl: uniphier: Add missing header(s)
>       pinctrl: zynqmp: Add missing header(s)
>       pinctrl: cherryview: Add missing header(s)
>       pinctrl: lynxpoint: Add missing header(s)
>       pinctrl: merrifield: Add missing header(s)
>       pinctrl: intel: Add missing header(s)
>       pinctrl: Clean up headers
>
>  drivers/gpio/gpio-arizona.c                        |  5 ++--
>  drivers/gpio/gpio-aspeed.c                         |  5 ++--
>  drivers/gpio/gpio-da9052.c                         |  9 +++----
>  drivers/gpio/gpio-mockup.c                         |  1 +
>  drivers/gpio/gpio-pca953x.c                        |  3 ++-
>  drivers/gpio/gpio-pl061.c                          | 15 ++++++-----
>  drivers/gpio/gpio-tegra186.c                       |  3 ++-
>  drivers/gpio/gpio-wm8350.c                         |  7 +++--
>  drivers/gpio/gpiolib-acpi.h                        | 12 +++++++++
>  drivers/gpio/gpiolib-cdev.c                        |  4 ++-
>  drivers/gpio/gpiolib-of.h                          | 11 +++++++-
>  drivers/gpio/gpiolib-sysfs.h                       |  2 ++
>  .../platform/st/sti/c8sectpfe/c8sectpfe-core.c     |  8 +++---
>  drivers/pinctrl/actions/pinctrl-owl.c              | 10 ++++---
>  drivers/pinctrl/aspeed/pinctrl-aspeed.c            |  1 +
>  drivers/pinctrl/bcm/pinctrl-bcm281xx.c             | 13 +++++----
>  drivers/pinctrl/bcm/pinctrl-cygnus-mux.c           |  9 ++++---
>  drivers/pinctrl/bcm/pinctrl-iproc-gpio.c           | 12 +++++----
>  drivers/pinctrl/bcm/pinctrl-ns2-mux.c              |  8 +++---
>  drivers/pinctrl/bcm/pinctrl-nsp-mux.c              |  8 +++---
>  drivers/pinctrl/cirrus/pinctrl-lochnagar.c         |  6 +++--
>  drivers/pinctrl/core.c                             | 19 ++++++-------
>  drivers/pinctrl/core.h                             | 12 ++++++++-
>  drivers/pinctrl/devicetree.h                       |  6 +++++
>  drivers/pinctrl/freescale/pinctrl-imx.c            |  8 +++---
>  drivers/pinctrl/freescale/pinctrl-imx1-core.c      |  4 ++-
>  drivers/pinctrl/intel/pinctrl-cherryview.c         |  6 +++--
>  drivers/pinctrl/intel/pinctrl-intel.c              |  6 +++--
>  drivers/pinctrl/intel/pinctrl-lynxpoint.c          |  6 +++--
>  drivers/pinctrl/intel/pinctrl-merrifield.c         |  4 ++-
>  drivers/pinctrl/mediatek/pinctrl-moore.c           |  3 +++
>  drivers/pinctrl/mediatek/pinctrl-paris.c           |  5 ++++
>  drivers/pinctrl/mvebu/pinctrl-mvebu.c              | 14 +++++-----
>  drivers/pinctrl/nuvoton/pinctrl-npcm7xx.c          | 11 +++++---
>  drivers/pinctrl/pinconf.h                          | 10 +++++++
>  drivers/pinctrl/pinctrl-at91-pio4.c                | 10 ++++---
>  drivers/pinctrl/pinctrl-at91.c                     | 16 ++++++-----
>  drivers/pinctrl/pinctrl-axp209.c                   |  8 +++---
>  drivers/pinctrl/pinctrl-bm1880.c                   |  6 +++--
>  drivers/pinctrl/pinctrl-gemini.c                   | 12 +++++----
>  drivers/pinctrl/pinctrl-ingenic.c                  | 10 ++++---
>  drivers/pinctrl/pinctrl-k210.c                     | 12 +++++----
>  drivers/pinctrl/pinctrl-lpc18xx.c                  |  6 +++--
>  drivers/pinctrl/pinctrl-microchip-sgpio.c          |  4 ++-
>  drivers/pinctrl/pinctrl-ocelot.c                   | 10 ++++---
>  drivers/pinctrl/pinctrl-single.c                   |  6 ++---
>  drivers/pinctrl/pinctrl-st.c                       | 21 +++++++++------
>  drivers/pinctrl/pinctrl-stmfx.c                    |  2 ++
>  drivers/pinctrl/pinctrl-utils.h                    |  5 ++++
>  drivers/pinctrl/pinctrl-zynqmp.c                   |  5 +++-
>  drivers/pinctrl/pinmux.c                           | 17 +++++++-----
>  drivers/pinctrl/pinmux.h                           | 11 ++++++++
>  drivers/pinctrl/qcom/pinctrl-spmi-gpio.c           |  8 +++---
>  drivers/pinctrl/qcom/pinctrl-spmi-mpp.c            |  8 +++---
>  drivers/pinctrl/renesas/pinctrl-rzg2l.c            |  7 +++--
>  drivers/pinctrl/renesas/pinctrl-rzn1.c             |  8 ++++--
>  drivers/pinctrl/renesas/pinctrl-rzv2m.c            |  4 ++-
>  drivers/pinctrl/renesas/pinctrl.c                  |  8 +++---
>  drivers/pinctrl/samsung/pinctrl-samsung.c          | 11 ++++----
>  drivers/pinctrl/spear/pinctrl-spear.c              |  6 +++--
>  drivers/pinctrl/sprd/pinctrl-sprd.c                |  6 +++--
>  drivers/pinctrl/starfive/pinctrl-starfive-jh7100.c |  3 +++
>  drivers/pinctrl/stm32/pinctrl-stm32.c              | 16 ++++++-----
>  drivers/pinctrl/sunxi/pinctrl-sunxi.c              | 18 +++++++------
>  drivers/pinctrl/tegra/pinctrl-tegra-xusb.c         |  6 +++--
>  drivers/pinctrl/tegra/pinctrl-tegra.c              |  6 +++--
>  drivers/pinctrl/ti/pinctrl-ti-iodelay.c            |  8 +++---
>  drivers/pinctrl/uniphier/pinctrl-uniphier-core.c   |  8 +++---
>  include/linux/gpio.h                               |  2 +-
>  include/linux/gpio/aspeed.h                        |  4 +++
>  include/linux/gpio/driver.h                        |  2 +-
>  include/linux/gpio/gpio-reg.h                      |  4 +++
>  include/linux/gpio/machine.h                       |  1 -
>  include/linux/pinctrl/consumer.h                   | 31 ++++++++++------------
>  include/linux/pinctrl/devinfo.h                    |  6 +++--
>  include/linux/pinctrl/machine.h                    |  8 +++---
>  include/linux/pinctrl/pinconf-generic.h            | 23 +++++++++-------
>  include/linux/pinctrl/pinctrl.h                    | 18 ++++++-------
>  include/linux/pinctrl/pinmux.h                     |  5 ++--
>  79 files changed, 428 insertions(+), 234 deletions(-)
>
> --
> With Best Regards,
> Andy Shevchenko
>
>

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-20 17:31 ` Bartosz Golaszewski
@ 2022-10-20 17:43   ` Andy Shevchenko
  2022-10-20 17:44     ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-20 17:43 UTC (permalink / raw)
  To: Bartosz Golaszewski; +Cc: Linux pin control, Linus Walleij, Mika Westerberg

On Thu, Oct 20, 2022 at 07:31:23PM +0200, Bartosz Golaszewski wrote:
> On Thu, Oct 20, 2022 at 4:58 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> >
> > Hi Linux pin control  maintainers,
> >
> > A bunch of cleanups of the pin control and GPIO headers. It has been a couple
> > of days in Linux Next without any major issue reported, a few fixes are already
> > incorporated in the respective patches. The idea is that pin control and GPIO
> > subsystems take it now and we will continue from this for all new code.
> >
> > Linus, Bart, please pull to your trees.
> > The tag is signed and can be considered as immutable.
> >
> > Thanks,
> >
> > With Best Regards,
> > Andy Shevchenko
> >
> > The following changes since commit 9abf2313adc1ca1b6180c508c25f22f9395cc780:
> >
> >   Linux 6.1-rc1 (2022-10-16 15:36:24 -0700)
> >
> > are available in the Git repository at:
> >
> >   git://git.kernel.org/pub/scm/linux/kernel/git/pinctrl/intel.git tags/intel-pinctrl-v6.1-1
> >
> > for you to fetch changes up to 1053f6a58766ac8eadcb72630262a5a454048479:
> >
> >   pinctrl: Clean up headers (2022-10-19 18:08:45 +0300)
> >
> > ----------------------------------------------------------------
> > intel-pinctrl for v6.1-1
> >
> > * Add missing and remove unused headers in pin control and GPIO drivers
> > * Revise the pin control and GPIO headers
> >
> > ----------------------------------------------------------------
> > Andy Shevchenko (50):
> >       gpio: aspeed: Add missing header(s)
> >       gpio: arizona: Remove unused header(s)
> >       gpio: da9052: Remove unused header(s)
> >       gpio: mockup: Add missing header(s)
> >       gpio: pca953x: Add missing header(s)
> >       gpio: pl061: Add missing header(s)
> >       gpio: reg: Add missing header(s)
> >       gpio: wm8350: Remove unused header(s)
> >       gpio: tegra186: Add missing header(s)
> >       gpiolib: cdev: Add missing header(s)
> >       gpiolib: Clean up headers
> 
> Can you send the GPIO changes separately? This way I don't need to
> pull all those pinctrl patches into the GPIO PR for the next merge
> window.

Some of them, but not all, if that what you wish.
I.o.w. a couple of the GPIO changes must be part of pin control series.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-20 17:43   ` Andy Shevchenko
@ 2022-10-20 17:44     ` Andy Shevchenko
  2022-10-21  8:02       ` Linus Walleij
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-20 17:44 UTC (permalink / raw)
  To: Bartosz Golaszewski; +Cc: Linux pin control, Linus Walleij, Mika Westerberg

On Thu, Oct 20, 2022 at 08:43:18PM +0300, Andy Shevchenko wrote:
> On Thu, Oct 20, 2022 at 07:31:23PM +0200, Bartosz Golaszewski wrote:
> > On Thu, Oct 20, 2022 at 4:58 PM Andy Shevchenko
> > <andriy.shevchenko@linux.intel.com> wrote:

...

> > Can you send the GPIO changes separately? This way I don't need to
> > pull all those pinctrl patches into the GPIO PR for the next merge
> > window.
> 
> Some of them, but not all, if that what you wish.
> I.o.w. a couple of the GPIO changes must be part of pin control series.

And I just realized that if any of new GPIO code will appear with the wrong
headers, there will be an inconsistent state. That said, I prefer this PR is
go as is.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-20 17:44     ` Andy Shevchenko
@ 2022-10-21  8:02       ` Linus Walleij
  2022-10-23 20:04         ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Linus Walleij @ 2022-10-21  8:02 UTC (permalink / raw)
  To: Andy Shevchenko; +Cc: Bartosz Golaszewski, Linux pin control, Mika Westerberg

On Thu, Oct 20, 2022 at 7:44 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> On Thu, Oct 20, 2022 at 08:43:18PM +0300, Andy Shevchenko wrote:
> > On Thu, Oct 20, 2022 at 07:31:23PM +0200, Bartosz Golaszewski wrote:
> > > On Thu, Oct 20, 2022 at 4:58 PM Andy Shevchenko
> > > <andriy.shevchenko@linux.intel.com> wrote:
>
> ...
>
> > > Can you send the GPIO changes separately? This way I don't need to
> > > pull all those pinctrl patches into the GPIO PR for the next merge
> > > window.
> >
> > Some of them, but not all, if that what you wish.
> > I.o.w. a couple of the GPIO changes must be part of pin control series.
>
> And I just realized that if any of new GPIO code will appear with the wrong
> headers, there will be an inconsistent state. That said, I prefer this PR is
> go as is.

Bart are you fine with pulling this as-is? (I am.)

Last merge window I pulled in a big I2C cleanup (remove to return void on
6 million drivers) and it was fine, my diffstat looked horrible but Torvalds
accepted it anyway I just mentioned it to him.

I think there is a way to shave off the irrelevant stuff from the diffstat,
and I think Andy even told me how to do this but can't find/remember the
git method used.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-21  8:02       ` Linus Walleij
@ 2022-10-23 20:04         ` Andy Shevchenko
  2022-10-24  7:16           ` Linus Walleij
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-23 20:04 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Bartosz Golaszewski, Linux pin control, Mika Westerberg

On Fri, Oct 21, 2022 at 10:02:35AM +0200, Linus Walleij wrote:
> On Thu, Oct 20, 2022 at 7:44 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Thu, Oct 20, 2022 at 08:43:18PM +0300, Andy Shevchenko wrote:
> > > On Thu, Oct 20, 2022 at 07:31:23PM +0200, Bartosz Golaszewski wrote:
> > > > On Thu, Oct 20, 2022 at 4:58 PM Andy Shevchenko
> > > > <andriy.shevchenko@linux.intel.com> wrote:

...

> > > > Can you send the GPIO changes separately? This way I don't need to
> > > > pull all those pinctrl patches into the GPIO PR for the next merge
> > > > window.
> > >
> > > Some of them, but not all, if that what you wish.
> > > I.o.w. a couple of the GPIO changes must be part of pin control series.
> >
> > And I just realized that if any of new GPIO code will appear with the wrong
> > headers, there will be an inconsistent state. That said, I prefer this PR is
> > go as is.
> 
> Bart are you fine with pulling this as-is? (I am.)

Hold on a bit, I have to rebuild a whole kernel to test if I missed something
(it's already clear I missed lantiq patch, now in my branch, but obviously not
 in that tag, since I called it immutable).

> Last merge window I pulled in a big I2C cleanup (remove to return void on
> 6 million drivers) and it was fine, my diffstat looked horrible but Torvalds
> accepted it anyway I just mentioned it to him.
> 
> I think there is a way to shave off the irrelevant stuff from the diffstat,
> and I think Andy even told me how to do this but can't find/remember the
> git method used.

IIRC when you create a pull request, do it from the certain base, like
origin/master..HEAD~0 and it will drop the applied changes automatically.
But I don't remember this by heart, so need to refresh my memories as well.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-23 20:04         ` Andy Shevchenko
@ 2022-10-24  7:16           ` Linus Walleij
  2022-10-24 15:06             ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Linus Walleij @ 2022-10-24  7:16 UTC (permalink / raw)
  To: Andy Shevchenko; +Cc: Bartosz Golaszewski, Linux pin control, Mika Westerberg

On Sun, Oct 23, 2022 at 10:04 PM Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> On Fri, Oct 21, 2022 at 10:02:35AM +0200, Linus Walleij wrote:

> > Bart are you fine with pulling this as-is? (I am.)
>
> Hold on a bit, I have to rebuild a whole kernel to test if I missed something
> (it's already clear I missed lantiq patch, now in my branch, but obviously not
>  in that tag, since I called it immutable).

OK I hold my horses waiting for your confirmation, I expect to
pull the immutable branch once it is confirmed stable.

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-24  7:16           ` Linus Walleij
@ 2022-10-24 15:06             ` Andy Shevchenko
  2022-10-25 13:34               ` Andy Shevchenko
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-24 15:06 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Bartosz Golaszewski, Linux pin control, Mika Westerberg

On Mon, Oct 24, 2022 at 09:16:28AM +0200, Linus Walleij wrote:
> On Sun, Oct 23, 2022 at 10:04 PM Andy Shevchenko
> <andriy.shevchenko@linux.intel.com> wrote:
> > On Fri, Oct 21, 2022 at 10:02:35AM +0200, Linus Walleij wrote:
> 
> > > Bart are you fine with pulling this as-is? (I am.)
> >
> > Hold on a bit, I have to rebuild a whole kernel to test if I missed something
> > (it's already clear I missed lantiq patch, now in my branch, but obviously not
> >  in that tag, since I called it immutable).
> 
> OK I hold my horses waiting for your confirmation, I expect to
> pull the immutable branch once it is confirmed stable.

I hope I found all places under drivers/pinctrl to fix, but let's wait
a couple of days so Linux Next and CIs may have a chance to exercise it
a bit.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [GIT PULL] intel-pinctrl for 6.1-1
  2022-10-24 15:06             ` Andy Shevchenko
@ 2022-10-25 13:34               ` Andy Shevchenko
  0 siblings, 0 replies; 9+ messages in thread
From: Andy Shevchenko @ 2022-10-25 13:34 UTC (permalink / raw)
  To: Linus Walleij; +Cc: Bartosz Golaszewski, Linux pin control, Mika Westerberg

On Mon, Oct 24, 2022 at 06:06:52PM +0300, Andy Shevchenko wrote:
> On Mon, Oct 24, 2022 at 09:16:28AM +0200, Linus Walleij wrote:
> > On Sun, Oct 23, 2022 at 10:04 PM Andy Shevchenko
> > <andriy.shevchenko@linux.intel.com> wrote:
> > > On Fri, Oct 21, 2022 at 10:02:35AM +0200, Linus Walleij wrote:

...

> > > > Bart are you fine with pulling this as-is? (I am.)
> > >
> > > Hold on a bit, I have to rebuild a whole kernel to test if I missed something
> > > (it's already clear I missed lantiq patch, now in my branch, but obviously not
> > >  in that tag, since I called it immutable).
> > 
> > OK I hold my horses waiting for your confirmation, I expect to
> > pull the immutable branch once it is confirmed stable.
> 
> I hope I found all places under drivers/pinctrl to fix, but let's wait
> a couple of days so Linux Next and CIs may have a chance to exercise it
> a bit.

New PR has been sent.

-- 
With Best Regards,
Andy Shevchenko



^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2022-10-25 13:35 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-10-20 14:49 [GIT PULL] intel-pinctrl for 6.1-1 Andy Shevchenko
2022-10-20 17:31 ` Bartosz Golaszewski
2022-10-20 17:43   ` Andy Shevchenko
2022-10-20 17:44     ` Andy Shevchenko
2022-10-21  8:02       ` Linus Walleij
2022-10-23 20:04         ` Andy Shevchenko
2022-10-24  7:16           ` Linus Walleij
2022-10-24 15:06             ` Andy Shevchenko
2022-10-25 13:34               ` Andy Shevchenko

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).