[PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling via B4 Relay]

From: Aaron Kling <webgeek1234@gmail.com>

The p3509 carrier board does not connect the id gpio. Prior to this, the
gpio role switch driver could not detect the mode of the otg port.

Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
---
 arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
 1 file changed, 1 deletion(-)

diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
--- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
+++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
@@ -669,7 +669,6 @@ connector {
 					vbus-gpios = <&gpio
 						      TEGRA186_MAIN_GPIO(L, 4)
 						      GPIO_ACTIVE_LOW>;
-					id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
 				};
 			};
 

---
base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
change-id: 20250513-tx2nx-role-switch-37ec55d25189

Best regards,
-- 
Aaron Kling <webgeek1234@gmail.com>

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling]

On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
<devnull+webgeek1234.gmail.com@kernel.org> wrote:
>
> From: Aaron Kling <webgeek1234@gmail.com>
>
> The p3509 carrier board does not connect the id gpio. Prior to this, the
> gpio role switch driver could not detect the mode of the otg port.
>
> Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> ---
>  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> @@ -669,7 +669,6 @@ connector {
>                                         vbus-gpios = <&gpio
>                                                       TEGRA186_MAIN_GPIO(L, 4)
>                                                       GPIO_ACTIVE_LOW>;
> -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
>                                 };
>                         };
>
>
> ---
> base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> change-id: 20250513-tx2nx-role-switch-37ec55d25189
>
> Best regards,
> --
> Aaron Kling <webgeek1234@gmail.com>
>
>

Friendly reminder about this patch.

Sincerely,
Aaron

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling]

On Wed, May 28, 2025 at 12:42 PM Aaron Kling <webgeek1234@gmail.com> wrote:
>
> On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
> <devnull+webgeek1234.gmail.com@kernel.org> wrote:
> >
> > From: Aaron Kling <webgeek1234@gmail.com>
> >
> > The p3509 carrier board does not connect the id gpio. Prior to this, the
> > gpio role switch driver could not detect the mode of the otg port.
> >
> > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > ---
> >  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
> >  1 file changed, 1 deletion(-)
> >
> > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > @@ -669,7 +669,6 @@ connector {
> >                                         vbus-gpios = <&gpio
> >                                                       TEGRA186_MAIN_GPIO(L, 4)
> >                                                       GPIO_ACTIVE_LOW>;
> > -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
> >                                 };
> >                         };
> >
> >
> > ---
> > base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> > change-id: 20250513-tx2nx-role-switch-37ec55d25189
> >
> > Best regards,
> > --
> > Aaron Kling <webgeek1234@gmail.com>
> >
> >
>
> Friendly reminder about this patch.

Re-reminder about this patch.

Aaron

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling]

On Mon, Jun 30, 2025 at 2:27 PM Aaron Kling <webgeek1234@gmail.com> wrote:
>
> On Wed, May 28, 2025 at 12:42 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> >
> > On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
> > <devnull+webgeek1234.gmail.com@kernel.org> wrote:
> > >
> > > From: Aaron Kling <webgeek1234@gmail.com>
> > >
> > > The p3509 carrier board does not connect the id gpio. Prior to this, the
> > > gpio role switch driver could not detect the mode of the otg port.
> > >
> > > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > > ---
> > >  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
> > >  1 file changed, 1 deletion(-)
> > >
> > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> > > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > @@ -669,7 +669,6 @@ connector {
> > >                                         vbus-gpios = <&gpio
> > >                                                       TEGRA186_MAIN_GPIO(L, 4)
> > >                                                       GPIO_ACTIVE_LOW>;
> > > -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
> > >                                 };
> > >                         };
> > >
> > >
> > > ---
> > > base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> > > change-id: 20250513-tx2nx-role-switch-37ec55d25189
> > >
> > > Best regards,
> > > --
> > > Aaron Kling <webgeek1234@gmail.com>
> > >
> > >
> >
> > Friendly reminder about this patch.
>
> Re-reminder about this patch.

Yet another reminder about this patch. It's been over two months
without a response and many other patches have been pulled in the
meantime.

Aaron

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling]

On Mon, Jul 14, 2025 at 12:35 AM Aaron Kling <webgeek1234@gmail.com> wrote:
>
> On Mon, Jun 30, 2025 at 2:27 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> >
> > On Wed, May 28, 2025 at 12:42 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> > >
> > > On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
> > > <devnull+webgeek1234.gmail.com@kernel.org> wrote:
> > > >
> > > > From: Aaron Kling <webgeek1234@gmail.com>
> > > >
> > > > The p3509 carrier board does not connect the id gpio. Prior to this, the
> > > > gpio role switch driver could not detect the mode of the otg port.
> > > >
> > > > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > > > ---
> > > >  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
> > > >  1 file changed, 1 deletion(-)
> > > >
> > > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> > > > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > @@ -669,7 +669,6 @@ connector {
> > > >                                         vbus-gpios = <&gpio
> > > >                                                       TEGRA186_MAIN_GPIO(L, 4)
> > > >                                                       GPIO_ACTIVE_LOW>;
> > > > -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
> > > >                                 };
> > > >                         };
> > > >
> > > >
> > > > ---
> > > > base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> > > > change-id: 20250513-tx2nx-role-switch-37ec55d25189
> > > >
> > > > Best regards,
> > > > --
> > > > Aaron Kling <webgeek1234@gmail.com>
> > > >
> > > >
> > >
> > > Friendly reminder about this patch.
> >
> > Re-reminder about this patch.
>
> Yet another reminder about this patch. It's been over two months
> without a response and many other patches have been pulled in the
> meantime.

Reminder yet again about this patch. It's now been two and a half
months without even an acknowledgement from the maintainers.

This one is getting annoying. What does it take to get a response from
the tegra subsystem maintainers? Does time have to be pre-allocated by
the company to look at patches that aren't from @nvidia.com's? Are
there certain times during a development cycle? When responses happen,
it seems like there's a lot of activity. But then everything goes
silent again for months. I've not seen any pattern to it so far and
it's becoming extremely frustrating.

Aaron

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Thierry Reding]

[-- Attachment #1: Type: text/plain, Size: 3071 bytes --]

On Thu, Jul 31, 2025 at 04:36:17PM -0500, Aaron Kling wrote:
> On Mon, Jul 14, 2025 at 12:35 AM Aaron Kling <webgeek1234@gmail.com> wrote:
> >
> > On Mon, Jun 30, 2025 at 2:27 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> > >
> > > On Wed, May 28, 2025 at 12:42 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> > > >
> > > > On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
> > > > <devnull+webgeek1234.gmail.com@kernel.org> wrote:
> > > > >
> > > > > From: Aaron Kling <webgeek1234@gmail.com>
> > > > >
> > > > > The p3509 carrier board does not connect the id gpio. Prior to this, the
> > > > > gpio role switch driver could not detect the mode of the otg port.
> > > > >
> > > > > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > > > > ---
> > > > >  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
> > > > >  1 file changed, 1 deletion(-)
> > > > >
> > > > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> > > > > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > @@ -669,7 +669,6 @@ connector {
> > > > >                                         vbus-gpios = <&gpio
> > > > >                                                       TEGRA186_MAIN_GPIO(L, 4)
> > > > >                                                       GPIO_ACTIVE_LOW>;
> > > > > -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
> > > > >                                 };
> > > > >                         };
> > > > >
> > > > >
> > > > > ---
> > > > > base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> > > > > change-id: 20250513-tx2nx-role-switch-37ec55d25189
> > > > >
> > > > > Best regards,
> > > > > --
> > > > > Aaron Kling <webgeek1234@gmail.com>
> > > > >
> > > > >
> > > >
> > > > Friendly reminder about this patch.
> > >
> > > Re-reminder about this patch.
> >
> > Yet another reminder about this patch. It's been over two months
> > without a response and many other patches have been pulled in the
> > meantime.
> 
> Reminder yet again about this patch. It's now been two and a half
> months without even an acknowledgement from the maintainers.
> 
> This one is getting annoying. What does it take to get a response from
> the tegra subsystem maintainers? Does time have to be pre-allocated by
> the company to look at patches that aren't from @nvidia.com's? Are
> there certain times during a development cycle? When responses happen,
> it seems like there's a lot of activity. But then everything goes
> silent again for months. I've not seen any pattern to it so far and
> it's becoming extremely frustrating.

Just people being busy. Nothing more, nothing less. I'll pick this up
once the merge window closes.

Thanks,
Thierry

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

Re: [PATCH] arm64: tegra: Remove otg id gpio from Jetson TX2 NX

[Aaron Kling]

On Fri, Aug 1, 2025 at 4:43 AM Thierry Reding <thierry.reding@gmail.com> wrote:
>
> On Thu, Jul 31, 2025 at 04:36:17PM -0500, Aaron Kling wrote:
> > On Mon, Jul 14, 2025 at 12:35 AM Aaron Kling <webgeek1234@gmail.com> wrote:
> > >
> > > On Mon, Jun 30, 2025 at 2:27 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> > > >
> > > > On Wed, May 28, 2025 at 12:42 PM Aaron Kling <webgeek1234@gmail.com> wrote:
> > > > >
> > > > > On Tue, May 13, 2025 at 4:10 PM Aaron Kling via B4 Relay
> > > > > <devnull+webgeek1234.gmail.com@kernel.org> wrote:
> > > > > >
> > > > > > From: Aaron Kling <webgeek1234@gmail.com>
> > > > > >
> > > > > > The p3509 carrier board does not connect the id gpio. Prior to this, the
> > > > > > gpio role switch driver could not detect the mode of the otg port.
> > > > > >
> > > > > > Signed-off-by: Aaron Kling <webgeek1234@gmail.com>
> > > > > > ---
> > > > > >  arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts | 1 -
> > > > > >  1 file changed, 1 deletion(-)
> > > > > >
> > > > > > diff --git a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > > index 26f71651933d1d8ef32bbd1645cac1820bd2e104..81f204e456409df355bbcb691ef99b0d0c9d504e 100644
> > > > > > --- a/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > > +++ b/arch/arm64/boot/dts/nvidia/tegra186-p3509-0000+p3636-0001.dts
> > > > > > @@ -669,7 +669,6 @@ connector {
> > > > > >                                         vbus-gpios = <&gpio
> > > > > >                                                       TEGRA186_MAIN_GPIO(L, 4)
> > > > > >                                                       GPIO_ACTIVE_LOW>;
> > > > > > -                                       id-gpios = <&pmic 0 GPIO_ACTIVE_HIGH>;
> > > > > >                                 };
> > > > > >                         };
> > > > > >
> > > > > >
> > > > > > ---
> > > > > > base-commit: 405e6c37c89ef0df2bfc7a988820a3df22dacb1b
> > > > > > change-id: 20250513-tx2nx-role-switch-37ec55d25189
> > > > > >
> > > > > > Best regards,
> > > > > > --
> > > > > > Aaron Kling <webgeek1234@gmail.com>
> > > > > >
> > > > > >
> > > > >
> > > > > Friendly reminder about this patch.
> > > >
> > > > Re-reminder about this patch.
> > >
> > > Yet another reminder about this patch. It's been over two months
> > > without a response and many other patches have been pulled in the
> > > meantime.
> >
> > Reminder yet again about this patch. It's now been two and a half
> > months without even an acknowledgement from the maintainers.
> >
> > This one is getting annoying. What does it take to get a response from
> > the tegra subsystem maintainers? Does time have to be pre-allocated by
> > the company to look at patches that aren't from @nvidia.com's? Are
> > there certain times during a development cycle? When responses happen,
> > it seems like there's a lot of activity. But then everything goes
> > silent again for months. I've not seen any pattern to it so far and
> > it's becoming extremely frustrating.
>
> Just people being busy. Nothing more, nothing less. I'll pick this up
> once the merge window closes.

If I understand correctly, the merge window closed on August 10th. And
I still haven't seen any movement on open patches. If everything
unrelated is on hold until after Tegra264 launch or something like
that, it would be nice to at least set that expectation instead of
everything being held arbitrarily in suspense.

Aaron