All of lore.kernel.org
 help / color / mirror / Atom feed
From: Grant Likely <grant.likely@secretlab.ca>
To: Shawn Guo <shawn.guo@linaro.org>, linux-mmc@vger.kernel.org
Cc: Sascha Hauer <s.hauer@pengutronix.de>,
	Chris Ball <cjb@laptop.org>,
	devicetree-discuss@lists.ozlabs.org
Subject: Re: [PATCH 6/6] mmc: dt: bus-width can be an optional property
Date: Fri, 08 Feb 2013 23:02:58 +0000	[thread overview]
Message-ID: <20130208230259.05F933E2C27@localhost> (raw)
In-Reply-To: <1358766149-9534-7-git-send-email-shawn.guo@linaro.org>

On Mon, 21 Jan 2013 19:02:29 +0800, Shawn Guo <shawn.guo@linaro.org> wrote:
> None of mmc drivers implements bus-width as a required device tree
> property.  Instead, some drivers like atmel-mci, dw_mmc, sdhci-s3c
> implement it as an optional one, and will force bus width to be 1
> when the property is absent.  Let's change the common binding to
> reflect what the drivers are usually doing.
> 
> Signed-off-by: Shawn Guo <shawn.guo@linaro.org>
> Cc: devicetree-discuss@lists.ozlabs.org

Acked-by: Grant Likely <grant.likely@secretlab.ca>

> ---
>  Documentation/devicetree/bindings/mmc/mmc.txt |    5 ++---
>  1 file changed, 2 insertions(+), 3 deletions(-)
> 
> diff --git a/Documentation/devicetree/bindings/mmc/mmc.txt b/Documentation/devicetree/bindings/mmc/mmc.txt
> index a591c67..cef217d 100644
> --- a/Documentation/devicetree/bindings/mmc/mmc.txt
> +++ b/Documentation/devicetree/bindings/mmc/mmc.txt
> @@ -6,9 +6,6 @@ Interpreted by the OF core:
>  - reg: Registers location and length.
>  - interrupts: Interrupts used by the MMC controller.
>  
> -Required properties:
> -- bus-width: Number of data lines, can be <1>, <4>, or <8>
> -
>  Card detection:
>  If no property below is supplied, standard SDHCI card detect is used.
>  Only one of the properties in this section should be supplied:
> @@ -17,6 +14,8 @@ Only one of the properties in this section should be supplied:
>    - non-removable: non-removable slot (like eMMC); assume always present.
>  
>  Optional properties:
> +- bus-width: Number of data lines, can be <1>, <4>, or <8>.  The default
> +  will be <1> if the property is absent.
>  - wp-gpios: Specify GPIOs for write protection, see gpio binding
>  - cd-inverted: when present, polarity on the cd gpio line is inverted
>  - wp-inverted: when present, polarity on the wp gpio line is inverted
> -- 
> 1.7.9.5
> 
> 
> _______________________________________________
> devicetree-discuss mailing list
> devicetree-discuss@lists.ozlabs.org
> https://lists.ozlabs.org/listinfo/devicetree-discuss

-- 
Grant Likely, B.Sc, P.Eng.
Secret Lab Technologies, Ltd.

  reply	other threads:[~2013-02-08 23:03 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-01-21 11:02 [PATCH 0/6] sdhci-esdhc-imx: 8bit mmc support Shawn Guo
2013-01-21 11:02 ` [PATCH 1/6] mmc: sdhci-esdhc-imx: separate transfer mode from command write for usdhc Shawn Guo
2013-01-21 11:02 ` [PATCH 2/6] mmc: sdhci-esdhc-imx: manually reset MIX_CTRL " Shawn Guo
2013-01-21 11:02 ` [PATCH 3/6] mmc: sdhci-esdhc-imx: Auto CMD23 support " Shawn Guo
2013-01-21 11:02 ` [PATCH 4/6] mmc: sdhci: rename platform_8bit_width to platform_bus_width Shawn Guo
2013-01-21 11:02 ` [PATCH 5/6] mmc: sdhci-esdhc-imx: support 8bit mode Shawn Guo
2013-01-21 11:02 ` [PATCH 6/6] mmc: dt: bus-width can be an optional property Shawn Guo
2013-02-08 23:02   ` Grant Likely [this message]
2013-01-25  6:47 ` [PATCH 0/6] sdhci-esdhc-imx: 8bit mmc support Dirk Behme
2013-01-28 21:50 ` Chris Ball

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=20130208230259.05F933E2C27@localhost \
    --to=grant.likely@secretlab.ca \
    --cc=cjb@laptop.org \
    --cc=devicetree-discuss@lists.ozlabs.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=s.hauer@pengutronix.de \
    --cc=shawn.guo@linaro.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.