LKML Archive on
help / color / mirror / Atom feed
From: Paul Kocialkowski <>
To: Russell King - ARM Linux <>
Cc:,,, Shawn Guo <>,
	Sascha Hauer <>,
	Fabio Estevam <>,
	Rob Herring <>,
	Mark Rutland <>
Subject: Re: [PATCH] ARM: dts: imx6qdl-cubox-i: Move card-detect GPIO to 1.5 SOM devices only
Date: Sun, 22 Apr 2018 19:22:14 +0200	[thread overview]
Message-ID: <> (raw)
In-Reply-To: <>

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


Le dimanche 22 avril 2018 à 15:39 +0100, Russell King - ARM Linux a
écrit :
> 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.

It appears that I was also confused about the different models in the
CuBox-i line. I had assumed that the models had been updated to a new
revision (carrying that 1.5 SOM) since the time I bought mine (which was
thus some sort of first generation). With this scenario in mind, it made
sense to me that a CD GPIO required only for the second generation would
have been applied by mistake to all the cubox-i devices by having it in
the common dtsi.

I checked on the leaflet inside the box and models did not change, only
the 4x4 model was added in the meantime. 

> 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.)

The one I have definitely does not have a spring-loaded mechanism, so I
would call it a push/pull static slot. I mentioned a hinged slot because
that's the term used to describe the layout in the CuBox-i-Lower
schematics, SD/MMC Connector part: "U3 is an internal hinged micro SD".
Looking at pictures of what hinged slots look like, that's definitely
not what I have here.

Are all CuBox-i units that have ever been sold supposed to have a
connector with a CD line? I find it hard to believe that it's broken
specifically on mine.

If that's really needed, I could open up the device, check whether the
R8 pull is in place and check the voltage there.


Paul Kocialkowski,

developer of free digital technology and hardware support.

Coding blog:
Git repositories:

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

  reply	other threads:[~2018-04-22 17:22 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 [this message]
2018-04-29 13:47     ` Paul Kocialkowski
2018-04-29 14:09       ` Jon Nettleton
2018-04-29 15:37         ` Paul Kocialkowski
     [not found]   ` <>
2018-04-22 17:23     ` Paul Kocialkowski

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:

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

  git send-email \ \ \ \ \ \ \ \ \ \ \ \
    --subject='Re: [PATCH] ARM: dts: imx6qdl-cubox-i: Move card-detect GPIO to 1.5 SOM devices only' \

* 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).