linux-tegra.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Stephen Warren <swarren-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
To: Joseph Lo <josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
	linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org
Subject: Re: [PATCH V5 7/9] ARM: dts: tegra: add non-removable and keep-power-in-suspend property for MMC
Date: Tue, 02 Apr 2013 13:58:41 -0600	[thread overview]
Message-ID: <515B3871.50801@wwwdotorg.org> (raw)
In-Reply-To: <1364901613-25080-8-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>

On 04/02/2013 05:20 AM, Joseph Lo wrote:
> This patch adds "non-removable" property of MMC host where the eMMC device
> is for Tegra platform.
> 
> And the "keep-power-in-suspend" property was used for the SDIO device that
> need this to go into suspend mode (e.g. BRCM43xx series).

>  arch/arm/boot/dts/tegra20-paz00.dts      | 1 +
>  arch/arm/boot/dts/tegra20-seaboard.dts   | 2 ++
>  arch/arm/boot/dts/tegra20-ventana.dts    | 2 ++
>  arch/arm/boot/dts/tegra20-whistler.dts   | 1 +
>  arch/arm/boot/dts/tegra30-beaver.dts     | 1 +
>  arch/arm/boot/dts/tegra30-cardhu-a02.dts | 1 +
>  arch/arm/boot/dts/tegra30-cardhu-a04.dts | 1 +
>  arch/arm/boot/dts/tegra30-cardhu.dtsi    | 1 +

I think there are a bunch of other boards that need a similar change,
right? There are certainly eMMC devices on other boards, and shouldn't
they be marked with non-removable? In particular, Beaver, Dalmore, and
Pluto, and likely some/all of the Colibri and Avionic Design boards. I
think it should be obvious which SDHCI devices need this, since there
won't be any cd-gpios property and the bus-width will likely be 8.

  parent reply	other threads:[~2013-04-02 19:58 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-04-02 11:20 [PATCH V5 0/9] ARM: tegra: add platform suspend support Joseph Lo
     [not found] ` <1364901613-25080-1-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-04-02 11:20   ` [PATCH V5 1/9] gpio: tegra: add gpio wakeup source handling Joseph Lo
     [not found]     ` <1364901613-25080-2-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-04-03 16:29       ` Linus Walleij
     [not found]         ` <CACRpkdYOLuUyBu7ER7hqLGZnAoYS0Z7XJ6LC2=ec3vxKzF1rqA-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2013-04-03 16:52           ` Stephen Warren
     [not found]             ` <515C5E43.1080904-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-04-03 21:20               ` Stephen Warren
2013-04-02 11:20   ` [PATCH V5 2/9] ARM: tegra: irq: add wake up handling Joseph Lo
2013-04-02 11:20   ` [PATCH V5 3/9] ARM: dt: tegra: add bindings of power management configurations for PMC Joseph Lo
2013-04-02 11:20   ` [PATCH V5 4/9] ARM: tegra: pm: add platform suspend support Joseph Lo
2013-04-02 11:20   ` [PATCH V5 5/9] ARM: dts: tegra: add power gpio keys to DT Joseph Lo
2013-04-02 11:20   ` [PATCH V5 6/9] ARM: dts: tegra: whistler: add wakeup source for KBC Joseph Lo
2013-04-02 11:20   ` [PATCH V5 7/9] ARM: dts: tegra: add non-removable and keep-power-in-suspend property for MMC Joseph Lo
     [not found]     ` <1364901613-25080-8-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-04-02 19:58       ` Stephen Warren [this message]
     [not found]         ` <515B3871.50801-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-04-03  5:45           ` Thierry Reding
     [not found]             ` <20130403054549.GA6764-RM9K5IK7kjIQXX3q8xo1gnVAuStQJXxyR5q1nwbD4aMs9pC9oP6+/A@public.gmane.org>
2013-04-03  6:28               ` Joseph Lo
2013-04-03  6:59           ` Joseph Lo
2013-04-02 11:20   ` [PATCH V5 8/9] ARM: tegra: config: defconfig update Joseph Lo
2013-04-02 11:20   ` [PATCH V5 9/9] ARM: dts: tegra: add the PM configurations of PMC Joseph Lo
     [not found]     ` <1364901613-25080-10-git-send-email-josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2013-04-02 20:00       ` Stephen Warren
     [not found]         ` <515B38F7.5050606-3lzwWm7+Weoh9ZMKESR00Q@public.gmane.org>
2013-04-03  5:59           ` Thierry Reding
     [not found]             ` <20130403055935.GB6764-RM9K5IK7kjIQXX3q8xo1gnVAuStQJXxyR5q1nwbD4aMs9pC9oP6+/A@public.gmane.org>
2013-04-03  6:34               ` Joseph Lo
     [not found]                 ` <1364970865.16957.21.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2013-04-03  7:04                   ` Thierry Reding
     [not found]                     ` <20130403070423.GB22867-RM9K5IK7kjIQXX3q8xo1gnVAuStQJXxyR5q1nwbD4aMs9pC9oP6+/A@public.gmane.org>
2013-04-03  7:11                       ` Joseph Lo
     [not found]                         ` <1364973077.16957.32.camel-yx3yKKdKkHfc7b1ADBJPm0n48jw8i0AO@public.gmane.org>
2013-04-03  7:19                           ` Thierry Reding
     [not found]                             ` <20130403071932.GA23157-RM9K5IK7kjIQXX3q8xo1gnVAuStQJXxyR5q1nwbD4aMs9pC9oP6+/A@public.gmane.org>
2013-04-03  7:33                               ` Joseph Lo
2013-04-02 19:39   ` [PATCH V5 0/9] ARM: tegra: add platform suspend support Stephen Warren

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=515B3871.50801@wwwdotorg.org \
    --to=swarren-3lzwwm7+weoh9zmkesr00q@public.gmane.org \
    --cc=josephl-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org \
    --cc=linux-arm-kernel-IAPFreCvJWM7uuMidbF8XUB+6BGkLq7r@public.gmane.org \
    --cc=linux-tegra-u79uwXL29TY76Z2rM5mHXA@public.gmane.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;
as well as URLs for NNTP newsgroup(s).