public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
From: Chris Ball <cjb@laptop.org>
To: Dirk Behme <dirk.behme@de.bosch.com>
Cc: linux-mmc@vger.kernel.org, linaro-dev@lists.linaro.org,
	linux-embedded@vger.kernel.org,
	Jassi Brar <jaswinder.singh@linaro.org>
Subject: Re: [PATCH 1/2] mmc: block: mmcblkN: use slot index instead of dynamic name index
Date: Mon, 06 Aug 2012 11:31:17 -0400	[thread overview]
Message-ID: <8739402g1m.fsf@octavius.laptop.org> (raw)
In-Reply-To: <1344237900-14815-1-git-send-email-dirk.behme@de.bosch.com> (Dirk Behme's message of "Mon, 6 Aug 2012 09:24:59 +0200")

Hi,

On Mon, Aug 06 2012, Dirk Behme wrote:
> On embedded devices, often there is a combination of removable mmc
> devices (e.g. MMC/SD cards) and hard wired ones (e.g. eMMC).
> Depending on the hardware configuration, the 'mmcblkN' node might
> change if the removable device is available or not at boot time.
>
> E.g. if the removable device is attached at boot time, it might
> become mmxblk0. And the hard wired one mmcblk1. But if the removable
> device isn't there at boot time, the hard wired one will become
> mmcblk0. This makes it somehow difficult to hard code the root device
> to the non-removable device and boot fast.
>
> This change does simply associate 'N' of 'mmcblkN' with the slot index
> instead of the dynamic name index. The slot index is always the same,
> ensuring that the non-removable mmc device is associated always
> with the same mmcblkN. Independent of the availability of the removable
> one.

I like this change in principle, but doesn't it break boot for everyone
currently using e.g. root=/dev/mmcblk0p1 on slot index 2?  That doesn't
sound like an acceptable regression.

Thanks,

- Chris.
-- 
Chris Ball   <cjb@laptop.org>   <http://printf.net/>
One Laptop Per Child

  parent reply	other threads:[~2012-08-06 15:31 UTC|newest]

Thread overview: 5+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-06  7:24 [PATCH 1/2] mmc: block: mmcblkN: use slot index instead of dynamic name index Dirk Behme
     [not found] ` <1344237900-14815-1-git-send-email-dirk.behme-V5te9oGctAVWk0Htik3J/w@public.gmane.org>
2012-08-06  7:25   ` [PATCH 2/2] mmc: block: remove unused name_idx Dirk Behme
2012-08-06 15:31 ` Chris Ball [this message]
2012-08-06 16:15   ` [PATCH 1/2] mmc: block: mmcblkN: use slot index instead of dynamic name index Jassi Brar
2012-08-08  6:12   ` Dirk Behme

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=8739402g1m.fsf@octavius.laptop.org \
    --to=cjb@laptop.org \
    --cc=dirk.behme@de.bosch.com \
    --cc=jaswinder.singh@linaro.org \
    --cc=linaro-dev@lists.linaro.org \
    --cc=linux-embedded@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox