[PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O

Raag Jadav posted 2 patches 1 month, 1 week ago
MAINTAINERS                             |  7 ++
drivers/gpio/Kconfig                    |  2 +-
drivers/gpio/gpio-elkhartlake.c         | 36 ++++++-----
drivers/platform/x86/intel/Kconfig      | 13 ++++
drivers/platform/x86/intel/Makefile     |  1 +
drivers/platform/x86/intel/ehl_pse_io.c | 86 +++++++++++++++++++++++++
include/linux/ehl_pse_io_aux.h          | 24 +++++++
7 files changed, 151 insertions(+), 18 deletions(-)
create mode 100644 drivers/platform/x86/intel/ehl_pse_io.c
create mode 100644 include/linux/ehl_pse_io_aux.h
[PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O
Posted by Raag Jadav 1 month, 1 week ago
This series adds Intel Elkhart Lake PSE I/O driver which enumerates the
PCI parent device and splits two child I/O devices (GPIO and Timed I/O
which are available as a single PCI function through shared MMIO) to their
respective I/O drivers.

In spirit, it is a continuation of PSE TIO series[1] which received
objection from Greg for abusing platform device and has now been reworked
to use auxiliary device instead.

Currently TIO driver[2] falls under PPS subsystem supporting generator
functionality and will be coming up in a separate follow-up series for
its independent design changes as per below roadmap.

=> Extend TIO driver[2] to support PPS client functionality.
=> Develop a PPS common driver which hooks to both generator and client
   counterparts.
=> Develop an auxiliary glue driver for PPS common driver.

[1] https://lore.kernel.org/r/20250307052231.551737-1-raag.jadav@intel.com
[2] https://lore.kernel.org/r/20250219040618.70962-1-subramanian.mohan@intel.com

v2:
- Consolidate OFFSET and SIZE macros (Andy)
- Make child device objects parent managed (Andy)
- Fix double free on error path (Andy)
- Shorten child device names to fit id string length (Andy)

v3:
- Use auxiliary_device_create() (Andy)

Raag Jadav (2):
  platform/x86/intel: Introduce Intel Elkhart Lake PSE I/O
  gpio: elkhartlake: Convert to auxiliary driver

 MAINTAINERS                             |  7 ++
 drivers/gpio/Kconfig                    |  2 +-
 drivers/gpio/gpio-elkhartlake.c         | 36 ++++++-----
 drivers/platform/x86/intel/Kconfig      | 13 ++++
 drivers/platform/x86/intel/Makefile     |  1 +
 drivers/platform/x86/intel/ehl_pse_io.c | 86 +++++++++++++++++++++++++
 include/linux/ehl_pse_io_aux.h          | 24 +++++++
 7 files changed, 151 insertions(+), 18 deletions(-)
 create mode 100644 drivers/platform/x86/intel/ehl_pse_io.c
 create mode 100644 include/linux/ehl_pse_io_aux.h

-- 
2.43.0
Re: [PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O
Posted by Bartosz Golaszewski 1 month ago
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>


On Wed, 12 Nov 2025 09:10:09 +0530, Raag Jadav wrote:
> This series adds Intel Elkhart Lake PSE I/O driver which enumerates the
> PCI parent device and splits two child I/O devices (GPIO and Timed I/O
> which are available as a single PCI function through shared MMIO) to their
> respective I/O drivers.
> 
> In spirit, it is a continuation of PSE TIO series[1] which received
> objection from Greg for abusing platform device and has now been reworked
> to use auxiliary device instead.
> 
> [...]

Applied, thanks!

[1/2] platform/x86/intel: Introduce Intel Elkhart Lake PSE I/O
      https://git.kernel.org/brgl/linux/c/a0c83150eea5807dbedf786f55cd49b14af118a8
[2/2] gpio: elkhartlake: Convert to auxiliary driver
      https://git.kernel.org/brgl/linux/c/10c15296906952016a84e1e45d8dc361f35afbd8

Best regards,
-- 
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
Re: [PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O
Posted by Bartosz Golaszewski 1 month, 1 week ago
On Wed, Nov 12, 2025 at 4:41 AM Raag Jadav <raag.jadav@intel.com> wrote:
>
> This series adds Intel Elkhart Lake PSE I/O driver which enumerates the
> PCI parent device and splits two child I/O devices (GPIO and Timed I/O
> which are available as a single PCI function through shared MMIO) to their
> respective I/O drivers.
>
> In spirit, it is a continuation of PSE TIO series[1] which received
> objection from Greg for abusing platform device and has now been reworked
> to use auxiliary device instead.
>
> Currently TIO driver[2] falls under PPS subsystem supporting generator
> functionality and will be coming up in a separate follow-up series for
> its independent design changes as per below roadmap.
>
> => Extend TIO driver[2] to support PPS client functionality.
> => Develop a PPS common driver which hooks to both generator and client
>    counterparts.
> => Develop an auxiliary glue driver for PPS common driver.
>
> [1] https://lore.kernel.org/r/20250307052231.551737-1-raag.jadav@intel.com
> [2] https://lore.kernel.org/r/20250219040618.70962-1-subramanian.mohan@intel.com
>
> v2:
> - Consolidate OFFSET and SIZE macros (Andy)
> - Make child device objects parent managed (Andy)
> - Fix double free on error path (Andy)
> - Shorten child device names to fit id string length (Andy)
>
> v3:
> - Use auxiliary_device_create() (Andy)
>
> Raag Jadav (2):
>   platform/x86/intel: Introduce Intel Elkhart Lake PSE I/O
>   gpio: elkhartlake: Convert to auxiliary driver
>
>  MAINTAINERS                             |  7 ++
>  drivers/gpio/Kconfig                    |  2 +-
>  drivers/gpio/gpio-elkhartlake.c         | 36 ++++++-----
>  drivers/platform/x86/intel/Kconfig      | 13 ++++
>  drivers/platform/x86/intel/Makefile     |  1 +
>  drivers/platform/x86/intel/ehl_pse_io.c | 86 +++++++++++++++++++++++++
>  include/linux/ehl_pse_io_aux.h          | 24 +++++++
>  7 files changed, 151 insertions(+), 18 deletions(-)
>  create mode 100644 drivers/platform/x86/intel/ehl_pse_io.c
>  create mode 100644 include/linux/ehl_pse_io_aux.h
>
> --
> 2.43.0
>

When this goes into the x86 tree, can you make it available on an
immutable branch for me to pull into the GPIO tree? Either just patch
1/2 or both of them with my Ack.

Bart
Re: [PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O
Posted by Raag Jadav 1 month ago
On Wed, Nov 12, 2025 at 04:14:11PM +0100, Bartosz Golaszewski wrote:

...

> When this goes into the x86 tree, can you make it available on an
> immutable branch for me to pull into the GPIO tree? Either just patch
> 1/2 or both of them with my Ack.

Anything I can do to move this forward?

Raag
Re: [PATCH v3 0/2] Introduce Intel Elkhart Lake PSE I/O
Posted by Bartosz Golaszewski 1 month ago
On Wed, Nov 19, 2025 at 7:40 AM Raag Jadav <raag.jadav@intel.com> wrote:
>
> On Wed, Nov 12, 2025 at 04:14:11PM +0100, Bartosz Golaszewski wrote:
>
> ...
>
> > When this goes into the x86 tree, can you make it available on an
> > immutable branch for me to pull into the GPIO tree? Either just patch
> > 1/2 or both of them with my Ack.
>
> Anything I can do to move this forward?
>
> Raag

With an Ack from Ilpo I can take it through my tree.

Bart