From: Ulf Hansson <ulf.hansson@linaro.org>
To: Sergio Prado <sergio.prado@e-labworks.com>
Cc: Rob Herring <robh+dt@kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
Ben Dooks <ben-linux@fluff.org>,
"linux-arm-kernel@lists.infradead.org"
<linux-arm-kernel@lists.infradead.org>
Subject: Re: [PATCH v4 0/2] mmc: host: s3cmci: add device tree support
Date: Thu, 16 Mar 2017 13:06:27 +0100 [thread overview]
Message-ID: <CAPDyKFqf=8aRO+=DS_adX+f5ZL3ea_nxp33+8UMvPRz0AqW_Ww@mail.gmail.com> (raw)
In-Reply-To: <1488417536-15110-1-git-send-email-sergio.prado@e-labworks.com>
On 2 March 2017 at 02:18, Sergio Prado <sergio.prado@e-labworks.com> wrote:
> This series adds support for configuring Samsung's S3C24XX MMC/SD/SDIO
> controller via device tree.
>
> Tested on FriendlyARM mini2440, based on s3c2440 SoC.
>
> Changes since v3 (as suggested by Rob Herring)
> - describing properties that wasn't documented and removing unnecessary
> comment in the device tree binding
>
> Changes since v2:
> - struct "s3cmci_drv_data" and flag "is2440" renamed
> - copying the whole driver data struct instead of only its member so
> it will be easier to extend the information in the future
> - using mmc_of_parse() to read "cd-gpios" and "wp-gpios" properties
> from device tree
>
> Changes since v1:
> - pinctrl description removed from DT binding
> - unit and label on DT binding renamed to mmc
>
> Sergio Prado (2):
> dt-bindings: mmc: add DT binding for S3C24XX MMC/SD/SDIO controller
> mmc: host: s3cmci: allow probing from device tree
>
> .../devicetree/bindings/mmc/samsung,s3cmci.txt | 42 +++
> drivers/mmc/host/s3cmci.c | 298 +++++++++++----------
> drivers/mmc/host/s3cmci.h | 3 +-
> 3 files changed, 200 insertions(+), 143 deletions(-)
> create mode 100644 Documentation/devicetree/bindings/mmc/samsung,s3cmci.txt
>
> --
> 1.9.1
>
Sergio, can you please re-spin and address the comments from Jaehoon,
then I am happy to apply this!
Kind regards
Uffe
prev parent reply other threads:[~2017-03-16 12:06 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-03-02 1:18 [PATCH v4 0/2] mmc: host: s3cmci: add device tree support Sergio Prado
2017-03-02 1:18 ` [PATCH v4 1/2] dt-bindings: mmc: add DT binding for S3C24XX MMC/SD/SDIO controller Sergio Prado
2017-03-03 2:07 ` Jaehoon Chung
2017-03-03 6:21 ` Rob Herring
2017-03-02 1:18 ` [PATCH v4 2/2] mmc: host: s3cmci: allow probing from device tree Sergio Prado
2017-03-03 2:14 ` Jaehoon Chung
2017-03-03 11:38 ` Sergio Prado
2017-03-06 3:41 ` Jaehoon Chung
2017-03-16 12:06 ` Ulf Hansson [this message]
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='CAPDyKFqf=8aRO+=DS_adX+f5ZL3ea_nxp33+8UMvPRz0AqW_Ww@mail.gmail.com' \
--to=ulf.hansson@linaro.org \
--cc=ben-linux@fluff.org \
--cc=devicetree@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-mmc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=robh+dt@kernel.org \
--cc=sergio.prado@e-labworks.com \
/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).