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.
next prev parent 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.