public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
* [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip
@ 2026-02-13 22:40 Dmitry Torokhov
  2026-02-16  7:22 ` Bartosz Golaszewski
  2026-03-02 16:42 ` Ilpo Järvinen
  0 siblings, 2 replies; 3+ messages in thread
From: Dmitry Torokhov @ 2026-02-13 22:40 UTC (permalink / raw)
  To: Hans de Goede, Ilpo Järvinen
  Cc: Enrico Weigelt, metux IT consult, Bartosz Golaszewski,
	platform-driver-x86, linux-kernel

GPIO subsystem is switching the way it locates GPIO chip instances for
GPIO references in software nodes from matching on node names to
identity matching, which necessitates assigning firmware nodes
(software nodes) to GPIO chips.

Attach apu2_gpiochip_node to the platform device representing the GPIO
controller.

Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
---
 drivers/platform/x86/pcengines-apuv2.c | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c
index 3b086863c6ac..3f19589d1ba0 100644
--- a/drivers/platform/x86/pcengines-apuv2.c
+++ b/drivers/platform/x86/pcengines-apuv2.c
@@ -294,7 +294,8 @@ static int __init apu_board_init(void)
 	}
 
 	apu_gpio_pdev = apu_create_pdev(AMD_FCH_GPIO_DRIVER_NAME,
-					id->driver_data, sizeof(struct amd_fch_gpio_pdata), NULL);
+					id->driver_data, sizeof(struct amd_fch_gpio_pdata),
+					&apu2_gpiochip_node);
 	err = PTR_ERR_OR_ZERO(apu_gpio_pdev);
 	if (err)
 		goto err_unregister_swnodes;
-- 
2.53.0.310.g728cabbaf7-goog


-- 
Dmitry

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

* Re: [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip
  2026-02-13 22:40 [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip Dmitry Torokhov
@ 2026-02-16  7:22 ` Bartosz Golaszewski
  2026-03-02 16:42 ` Ilpo Järvinen
  1 sibling, 0 replies; 3+ messages in thread
From: Bartosz Golaszewski @ 2026-02-16  7:22 UTC (permalink / raw)
  To: Dmitry Torokhov
  Cc: Hans de Goede, Ilpo Järvinen,
	Enrico Weigelt, metux IT consult, platform-driver-x86,
	linux-kernel

On Fri, Feb 13, 2026 at 11:40 PM Dmitry Torokhov
<dmitry.torokhov@gmail.com> wrote:
>
> GPIO subsystem is switching the way it locates GPIO chip instances for
> GPIO references in software nodes from matching on node names to
> identity matching, which necessitates assigning firmware nodes
> (software nodes) to GPIO chips.
>
> Attach apu2_gpiochip_node to the platform device representing the GPIO
> controller.
>
> Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
> ---
>  drivers/platform/x86/pcengines-apuv2.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/drivers/platform/x86/pcengines-apuv2.c b/drivers/platform/x86/pcengines-apuv2.c
> index 3b086863c6ac..3f19589d1ba0 100644
> --- a/drivers/platform/x86/pcengines-apuv2.c
> +++ b/drivers/platform/x86/pcengines-apuv2.c
> @@ -294,7 +294,8 @@ static int __init apu_board_init(void)
>         }
>
>         apu_gpio_pdev = apu_create_pdev(AMD_FCH_GPIO_DRIVER_NAME,
> -                                       id->driver_data, sizeof(struct amd_fch_gpio_pdata), NULL);
> +                                       id->driver_data, sizeof(struct amd_fch_gpio_pdata),
> +                                       &apu2_gpiochip_node);
>         err = PTR_ERR_OR_ZERO(apu_gpio_pdev);
>         if (err)
>                 goto err_unregister_swnodes;
> --

Reviewed-by: Bartosz Golaszewski <bartosz.golaszewski@oss.qualcomm.com>

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

* Re: [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip
  2026-02-13 22:40 [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip Dmitry Torokhov
  2026-02-16  7:22 ` Bartosz Golaszewski
@ 2026-03-02 16:42 ` Ilpo Järvinen
  1 sibling, 0 replies; 3+ messages in thread
From: Ilpo Järvinen @ 2026-03-02 16:42 UTC (permalink / raw)
  To: Hans de Goede, Dmitry Torokhov
  Cc: Enrico Weigelt, metux IT consult, Bartosz Golaszewski,
	platform-driver-x86, linux-kernel

On Fri, 13 Feb 2026 14:40:32 -0800, Dmitry Torokhov wrote:

> GPIO subsystem is switching the way it locates GPIO chip instances for
> GPIO references in software nodes from matching on node names to
> identity matching, which necessitates assigning firmware nodes
> (software nodes) to GPIO chips.
> 
> Attach apu2_gpiochip_node to the platform device representing the GPIO
> controller.
> 
> [...]


Thank you for your contribution, it has been applied to my local
review-ilpo-next branch. Note it will show up in the public
platform-drivers-x86/review-ilpo-next branch only once I've pushed my
local branch there, which might take a while.

The list of commits applied:
[1/1] platform/x86: pcengines-apuv2: attach software node to the gpiochip
      commit: 0e59b31dbd3bfca96567e3d22518f4b6a7051a40

--
 i.


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

end of thread, other threads:[~2026-03-02 16:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-02-13 22:40 [PATCH] platform/x86: pcengines-apuv2: attach software node to the gpiochip Dmitry Torokhov
2026-02-16  7:22 ` Bartosz Golaszewski
2026-03-02 16:42 ` Ilpo Järvinen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox