LKML Archive on lore.kernel.org
help / color / mirror / Atom feed
From: Paul Kocialkowski <contact@paulk.fr>
To: Jon Nettleton <jon@solid-run.com>,
	Russell King - ARM Linux <linux@armlinux.org.uk>
Cc: linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org,
	linux-kernel@vger.kernel.org, Shawn Guo <shawnguo@kernel.org>,
	Sascha Hauer <kernel@pengutronix.de>,
	Fabio Estevam <fabio.estevam@nxp.com>,
	Rob Herring <robh+dt@kernel.org>,
	Mark Rutland <mark.rutland@arm.com>
Subject: Re: [PATCH] ARM: dts: imx6qdl-cubox-i: Move card-detect GPIO to 1.5 SOM devices only
Date: Sun, 22 Apr 2018 19:23:42 +0200	[thread overview]
Message-ID: <12bc46186bb643203601165b7b0136d96e0613c9.camel@paulk.fr> (raw)
In-Reply-To: <CABdtJHuE05ea1EPSR916vSEz6gtCirLF77c_PW2N2ofGnzN-bw@mail.gmail.com>

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

Hi,

Le dimanche 22 avril 2018 à 15:02 +0000, Jon Nettleton a écrit :
> On Sun, Apr 22, 2018 at 4:39 PM Russell King - ARM Linux <linux@armlin
> ux.org.uk> wrote:
> > On Sun, Apr 22, 2018 at 04:21:51PM +0200, Paul Kocialkowski wrote:
> > > The Solid-Run CuBox-i lower board used in the first generation of
> > > CuBox-i devices feature a hinged micro SD card slot, that does not
> > have
> > > card-detect capability. Since the card-detect GPIO was specified
> > in the
> > > common cubox-i dtsi, it is moved to each device using the 1.5 SOM
> > and is
> > > thus removed from the imx6q-cubox-i dts.
> > 
> > This seems incorrect.
> > 
> > The 1.5 SOM is a relatively recent thing, and I have one of the
> > first
> > Cubox-i's that were produced which is not hinged, and does not have
> > a
> > 1.5 SOM.  There is _no_ correlation between the 1.5 SOM and the SD
> > card slot.
> > 
> > Talking to folk at SolidRun, we're all confused about your
> > assertions.
> > 
> > We're also confused about "hinged micro SD card slot" - Cubox-i's
> > come with either a push-push slot or a static (push-in, pull-out)
> > slot.
> > (Prototypes were the latter.)
> > 
> > Maybe someone's modified yours?
> > 
> Regardless of the physical mechanics of the device, the CD-GPIO is a
> mechanism of the carrier and not the SOM. I have one of the few Cubox-
> i's that have been physically modified to work with the rev 1.5 som
> and have no issues (the heat spreader needs additional cutouts).

There was some confusion about the relationship with the 1.5 SOM. It is
now clear that the issue is not linked to it at all.

> Now if you are still using the old 2013 u-boot then there is possibly
> confusion between the eMMC and SDHC slots which can cause
> strangeness.  Please use the 2018.01 branch on our github and that
> will handle this properly.

I'm using U-Boot 2018.03, so I think I'm safe on that side.

Cheers,

-- 
Paul Kocialkowski,

developer of free digital technology and hardware support.

Website: https://www.paulk.fr/
Coding blog: https://code.paulk.fr/
Git repositories: https://git.paulk.fr/ https://git.code.paulk.fr/

[-- Attachment #2: This is a digitally signed message part --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

      parent reply	other threads:[~2018-04-22 17:23 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-04-22 14:21 Paul Kocialkowski
2018-04-22 14:39 ` Russell King - ARM Linux
2018-04-22 17:22   ` Paul Kocialkowski
2018-04-29 13:47     ` Paul Kocialkowski
2018-04-29 14:09       ` Jon Nettleton
2018-04-29 15:37         ` Paul Kocialkowski
     [not found]   ` <CABdtJHuE05ea1EPSR916vSEz6gtCirLF77c_PW2N2ofGnzN-bw@mail.gmail.com>
2018-04-22 17:23     ` Paul Kocialkowski [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=12bc46186bb643203601165b7b0136d96e0613c9.camel@paulk.fr \
    --to=contact@paulk.fr \
    --cc=devicetree@vger.kernel.org \
    --cc=fabio.estevam@nxp.com \
    --cc=jon@solid-run.com \
    --cc=kernel@pengutronix.de \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux@armlinux.org.uk \
    --cc=mark.rutland@arm.com \
    --cc=robh+dt@kernel.org \
    --cc=shawnguo@kernel.org \
    --subject='Re: [PATCH] ARM: dts: imx6qdl-cubox-i: Move card-detect GPIO to 1.5 SOM devices only' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).