* [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410
@ 2025-11-13 16:31 Kartik Rajput
2025-11-14 10:09 ` Jon Hunter
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: Kartik Rajput @ 2025-11-13 16:31 UTC (permalink / raw)
To: linus.walleij, brgl, thierry.reding, jonathanh, pshete, nhartman,
linux-gpio, linux-tegra, linux-kernel
Cc: kkartik
On Tegra410, Compute and System GPIOs have same port names. This
results in the same GPIO names for both Compute and System GPIOs
during initialization in `tegra186_gpio_probe()`, which results in
following warnings:
kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.00'
kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.01'
kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.02'
kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.00'
kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.01'
...
Add GPIO name prefix in the SoC data and use it to initialize the GPIO
name.
Port names remain unchanged for previous SoCs. On Tegra410, Compute
GPIOs are named COMPUTE-P<PORT>.GPIO, and System GPIOs are named
SYSTEM-P<PORT>.GPIO.
Fixes: 9631a10083d8 ("gpio: tegra186: Add support for Tegra410")
Signed-off-by: Kartik Rajput <kkartik@nvidia.com>
---
drivers/gpio/gpio-tegra186.c | 11 +++++++++--
1 file changed, 9 insertions(+), 2 deletions(-)
diff --git a/drivers/gpio/gpio-tegra186.c b/drivers/gpio/gpio-tegra186.c
index 83ecdc876985..b1498b59a921 100644
--- a/drivers/gpio/gpio-tegra186.c
+++ b/drivers/gpio/gpio-tegra186.c
@@ -109,6 +109,7 @@ struct tegra_gpio_soc {
const struct tegra_gpio_port *ports;
unsigned int num_ports;
const char *name;
+ const char *prefix;
unsigned int instance;
unsigned int num_irqs_per_bank;
@@ -940,8 +941,12 @@ static int tegra186_gpio_probe(struct platform_device *pdev)
char *name;
for (j = 0; j < port->pins; j++) {
- name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL,
- "P%s.%02x", port->name, j);
+ if (gpio->soc->prefix)
+ name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL, "%s-P%s.%02x",
+ gpio->soc->prefix, port->name, j);
+ else
+ name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL, "P%s.%02x",
+ port->name, j);
if (!name)
return -ENOMEM;
@@ -1306,6 +1311,7 @@ static const struct tegra_gpio_soc tegra410_compute_soc = {
.num_ports = ARRAY_SIZE(tegra410_compute_ports),
.ports = tegra410_compute_ports,
.name = "tegra410-gpio-compute",
+ .prefix = "COMPUTE",
.num_irqs_per_bank = 8,
.instance = 0,
};
@@ -1335,6 +1341,7 @@ static const struct tegra_gpio_soc tegra410_system_soc = {
.num_ports = ARRAY_SIZE(tegra410_system_ports),
.ports = tegra410_system_ports,
.name = "tegra410-gpio-system",
+ .prefix = "SYSTEM",
.num_irqs_per_bank = 8,
.instance = 0,
};
--
2.43.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* Re: [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410
2025-11-13 16:31 [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410 Kartik Rajput
@ 2025-11-14 10:09 ` Jon Hunter
2025-11-14 13:11 ` Thierry Reding
2025-11-17 10:04 ` Bartosz Golaszewski
2 siblings, 0 replies; 4+ messages in thread
From: Jon Hunter @ 2025-11-14 10:09 UTC (permalink / raw)
To: Kartik Rajput, linus.walleij, brgl, thierry.reding, pshete,
nhartman, linux-gpio, linux-tegra, linux-kernel
On 13/11/2025 16:31, Kartik Rajput wrote:
> On Tegra410, Compute and System GPIOs have same port names. This
> results in the same GPIO names for both Compute and System GPIOs
> during initialization in `tegra186_gpio_probe()`, which results in
> following warnings:
>
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.01'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.02'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.01'
> ...
>
> Add GPIO name prefix in the SoC data and use it to initialize the GPIO
> name.
>
> Port names remain unchanged for previous SoCs. On Tegra410, Compute
> GPIOs are named COMPUTE-P<PORT>.GPIO, and System GPIOs are named
> SYSTEM-P<PORT>.GPIO.
>
> Fixes: 9631a10083d8 ("gpio: tegra186: Add support for Tegra410")
> Signed-off-by: Kartik Rajput <kkartik@nvidia.com>
> ---
> drivers/gpio/gpio-tegra186.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/gpio/gpio-tegra186.c b/drivers/gpio/gpio-tegra186.c
> index 83ecdc876985..b1498b59a921 100644
> --- a/drivers/gpio/gpio-tegra186.c
> +++ b/drivers/gpio/gpio-tegra186.c
> @@ -109,6 +109,7 @@ struct tegra_gpio_soc {
> const struct tegra_gpio_port *ports;
> unsigned int num_ports;
> const char *name;
> + const char *prefix;
> unsigned int instance;
>
> unsigned int num_irqs_per_bank;
> @@ -940,8 +941,12 @@ static int tegra186_gpio_probe(struct platform_device *pdev)
> char *name;
>
> for (j = 0; j < port->pins; j++) {
> - name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL,
> - "P%s.%02x", port->name, j);
> + if (gpio->soc->prefix)
> + name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL, "%s-P%s.%02x",
> + gpio->soc->prefix, port->name, j);
> + else
> + name = devm_kasprintf(gpio->gpio.parent, GFP_KERNEL, "P%s.%02x",
> + port->name, j);
> if (!name)
> return -ENOMEM;
>
> @@ -1306,6 +1311,7 @@ static const struct tegra_gpio_soc tegra410_compute_soc = {
> .num_ports = ARRAY_SIZE(tegra410_compute_ports),
> .ports = tegra410_compute_ports,
> .name = "tegra410-gpio-compute",
> + .prefix = "COMPUTE",
> .num_irqs_per_bank = 8,
> .instance = 0,
> };
> @@ -1335,6 +1341,7 @@ static const struct tegra_gpio_soc tegra410_system_soc = {
> .num_ports = ARRAY_SIZE(tegra410_system_ports),
> .ports = tegra410_system_ports,
> .name = "tegra410-gpio-system",
> + .prefix = "SYSTEM",
> .num_irqs_per_bank = 8,
> .instance = 0,
> };
Looks good to me.
Reviewed-by: Jon Hunter <jonathanh@nvidia.com>
Jon
--
nvpublic
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410
2025-11-13 16:31 [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410 Kartik Rajput
2025-11-14 10:09 ` Jon Hunter
@ 2025-11-14 13:11 ` Thierry Reding
2025-11-17 10:04 ` Bartosz Golaszewski
2 siblings, 0 replies; 4+ messages in thread
From: Thierry Reding @ 2025-11-14 13:11 UTC (permalink / raw)
To: Kartik Rajput
Cc: linus.walleij, brgl, jonathanh, pshete, nhartman, linux-gpio,
linux-tegra, linux-kernel
[-- Attachment #1: Type: text/plain, Size: 1218 bytes --]
On Thu, Nov 13, 2025 at 10:01:12PM +0530, Kartik Rajput wrote:
> On Tegra410, Compute and System GPIOs have same port names. This
> results in the same GPIO names for both Compute and System GPIOs
> during initialization in `tegra186_gpio_probe()`, which results in
> following warnings:
>
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.01'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.02'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.01'
> ...
>
> Add GPIO name prefix in the SoC data and use it to initialize the GPIO
> name.
>
> Port names remain unchanged for previous SoCs. On Tegra410, Compute
> GPIOs are named COMPUTE-P<PORT>.GPIO, and System GPIOs are named
> SYSTEM-P<PORT>.GPIO.
>
> Fixes: 9631a10083d8 ("gpio: tegra186: Add support for Tegra410")
> Signed-off-by: Kartik Rajput <kkartik@nvidia.com>
> ---
> drivers/gpio/gpio-tegra186.c | 11 +++++++++--
> 1 file changed, 9 insertions(+), 2 deletions(-)
Acked-by: Thierry Reding <treding@nvidia.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* Re: [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410
2025-11-13 16:31 [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410 Kartik Rajput
2025-11-14 10:09 ` Jon Hunter
2025-11-14 13:11 ` Thierry Reding
@ 2025-11-17 10:04 ` Bartosz Golaszewski
2 siblings, 0 replies; 4+ messages in thread
From: Bartosz Golaszewski @ 2025-11-17 10:04 UTC (permalink / raw)
To: linus.walleij, brgl, thierry.reding, jonathanh, pshete, nhartman,
linux-gpio, linux-tegra, linux-kernel, Kartik Rajput
Cc: Bartosz Golaszewski
From: Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
On Thu, 13 Nov 2025 22:01:12 +0530, Kartik Rajput wrote:
> On Tegra410, Compute and System GPIOs have same port names. This
> results in the same GPIO names for both Compute and System GPIOs
> during initialization in `tegra186_gpio_probe()`, which results in
> following warnings:
>
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.01'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PA.02'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.00'
> kernel: gpio gpiochip1: Detected name collision for GPIO name 'PB.01'
> ...
>
> [...]
Applied, thanks!
[1/1] gpio: tegra186: Fix GPIO name collisions for Tegra410
https://git.kernel.org/brgl/linux/c/67f9b828d4e5e47caf3472a399c25c3c0ddc824a
Best regards,
--
Bartosz Golaszewski <bartosz.golaszewski@linaro.org>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-11-17 10:04 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-11-13 16:31 [PATCH] gpio: tegra186: Fix GPIO name collisions for Tegra410 Kartik Rajput
2025-11-14 10:09 ` Jon Hunter
2025-11-14 13:11 ` Thierry Reding
2025-11-17 10:04 ` Bartosz Golaszewski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox