Re: [PATCH 1/2] arm64: tegra: fix p3767 card detect polarity

[Thierry Reding <thierry.reding@gmail.com>]

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

On Tue, Aug 15, 2023 at 01:37:40PM +0000, Brad Griffis wrote:
> On Fri, Jul 21, 2023 at 6:22 AM +0000, Thierry Reding wrote:
> > On Mon, Jun 26, 2023 at 06:09:19PM +0000, Brad Griffis wrote:
> > > The card detect pin on Orin Nano SKU5 (p3767-0005) is active-low.
> > >
> > > Fixes: 13b0aca303e9 ("arm64: tegra: Support Jetson Orin NX")
> > > Signed-off-by: Brad Griffis <bgriffis@nvidia.com>
> > > ---
> > >  arch/arm64/boot/dts/nvidia/tegra234-p3767.dtsi | 2 +-
> > >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > This DTS include is also used by Jetson Orin NX, but I assume that the same
> > polarity applies to that as well?
> 
> The polarity is the same for all Orin NX/Nano modules that have SD card.
> 
> > Actually, looking at the Orin NX, I don't see an SD card slot and judging by the
> > documentation that I was able to find it's not documented for Orin Nano
> > either. Or is it only certain variants that were equipped with the SD slot?
> 
> UEFI dynamically disables this mmc instance on SKUs that do not support it.

Okay, good. I've adjusted the commit message a tiny little bit to
clarify that this applies to all NX/Nano SKUs that have an SD card slot
equipped.

Applied, thanks.
Thierry

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