All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jisheng Zhang <jszhang@marvell.com>
To: Ziji Hu <huziji@marvell.com>
Cc: Ulf Hansson <ulf.hansson@linaro.org>,
	Gregory CLEMENT <gregory.clement@free-electrons.com>,
	Adrian Hunter <adrian.hunter@intel.com>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	Jason Cooper <jason@lakedaemon.net>,
	"Andrew Lunn" <andrew@lunn.ch>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	Thomas Petazzoni <thomas.petazzoni@free-electrons.com>,
	"linux-arm-kernel@lists.infradead.org" 
	<linux-arm-kernel@lists.infradead.org>,
	Mike Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@codeaurora.org>,
	linux-clk <linux-clk@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Rob Herring <robh+dt@kernel.org>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Jimmy Xu <zmxu@marvell.com>, Nadav Haklai <nadavh@marvell.com>,
	Ryan Gao <ygao@marvell.com>, Doug Jones <dougj@marvell.com>,
	Victor Gu <xigu@marvell.com>, "Wei(SOCP) Liu" <liuw@marvell.com>,
	Wilson Ding <dingwei@marvell.com>,
	Yehuda Yitschak <yehuday@marvell.com>,
	Marcin Wojtas <mw@semihalf.com>, Hanna Hawa <hannah@marvell.com>,
	Kostya Porotchkin <kostap@marvell.com>
Subject: Re: [PATCH v6 08/14] mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality
Date: Thu, 16 Mar 2017 10:24:17 +0800	[thread overview]
Message-ID: <20170316102417.0909bccc@xhacker> (raw)
In-Reply-To: <10855817-e693-e007-9499-c4e97936fadb@marvell.com>

Hi Ziji

On Wed, 15 Mar 2017 21:58:43 +0800 Ziji Hu wrote:

> Hi Ulf,
> 
> On 2017/3/15 21:11, Ulf Hansson wrote:
> > On 14 February 2017 at 18:01, Gregory CLEMENT
> > <gregory.clement@free-electrons.com> wrote:  
> >> From: Hu Ziji <huziji@marvell.com>  
> <snip>
> >> +config MMC_SDHCI_XENON
> >> +       tristate "Marvell Xenon eMMC/SD/SDIO SDHCI driver"
> >> +       depends on MMC_SDHCI_PLTFM
> >> +       help
> >> +         This selects Marvell Xenon eMMC/SD/SDIO SDHCI.
> >> +         If you have a machine with integrated Marvell Xenon SDHC IP,  
> > 
> > /s/SDHC/SDHCI
> >   
> 
> 	Sorry. I don't get your point.
> 	Could you please give more detailed comments?

Ulf want you to s/SDHC/SDHCI in the line "If you have .....", I.E
change the line to:
If you have a machine with integrated Marvell Xenon SDHCI IP,

Thanks,
Jisheng

WARNING: multiple messages have this Message-ID (diff)
From: Jisheng Zhang <jszhang@marvell.com>
To: Ziji Hu <huziji@marvell.com>
Cc: Jimmy Xu <zmxu@marvell.com>, Andrew Lunn <andrew@lunn.ch>,
	Ulf Hansson <ulf.hansson@linaro.org>,
	"linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>,
	Mike Turquette <mturquette@baylibre.com>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	Nadav Haklai <nadavh@marvell.com>, Victor Gu <xigu@marvell.com>,
	Doug Jones <dougj@marvell.com>,
	linux-clk <linux-clk@vger.kernel.org>,
	Yehuda Yitschak <yehuday@marvell.com>,
	Marcin Wojtas <mw@semihalf.com>,
	Kostya Porotchkin <kostap@marvell.com>,
	Hanna Hawa <hannah@marvell.com>,
	Sebastian Hesselbarth <sebastian.hesselbarth@gmail.com>,
	"devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
	Jason Cooper <jason@lakedaemon.net>,
	Rob Herring <robh+dt@kernel.org>, Ryan Gao <ygao@marvell.com>,
	Gregory CLEMENT <gregory.clement@free-electrons.com>,
	"Wei(SOCP) Liu" <liuw@marvell.com>,
	linux-arm-kernel@lists.infradead
Subject: Re: [PATCH v6 08/14] mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality
Date: Thu, 16 Mar 2017 10:24:17 +0800	[thread overview]
Message-ID: <20170316102417.0909bccc@xhacker> (raw)
In-Reply-To: <10855817-e693-e007-9499-c4e97936fadb@marvell.com>

Hi Ziji

On Wed, 15 Mar 2017 21:58:43 +0800 Ziji Hu wrote:

> Hi Ulf,
> 
> On 2017/3/15 21:11, Ulf Hansson wrote:
> > On 14 February 2017 at 18:01, Gregory CLEMENT
> > <gregory.clement@free-electrons.com> wrote:  
> >> From: Hu Ziji <huziji@marvell.com>  
> <snip>
> >> +config MMC_SDHCI_XENON
> >> +       tristate "Marvell Xenon eMMC/SD/SDIO SDHCI driver"
> >> +       depends on MMC_SDHCI_PLTFM
> >> +       help
> >> +         This selects Marvell Xenon eMMC/SD/SDIO SDHCI.
> >> +         If you have a machine with integrated Marvell Xenon SDHC IP,  
> > 
> > /s/SDHC/SDHCI
> >   
> 
> 	Sorry. I don't get your point.
> 	Could you please give more detailed comments?

Ulf want you to s/SDHC/SDHCI in the line "If you have .....", I.E
change the line to:
If you have a machine with integrated Marvell Xenon SDHCI IP,

Thanks,
Jisheng

WARNING: multiple messages have this Message-ID (diff)
From: jszhang@marvell.com (Jisheng Zhang)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v6 08/14] mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality
Date: Thu, 16 Mar 2017 10:24:17 +0800	[thread overview]
Message-ID: <20170316102417.0909bccc@xhacker> (raw)
In-Reply-To: <10855817-e693-e007-9499-c4e97936fadb@marvell.com>

Hi Ziji

On Wed, 15 Mar 2017 21:58:43 +0800 Ziji Hu wrote:

> Hi Ulf,
> 
> On 2017/3/15 21:11, Ulf Hansson wrote:
> > On 14 February 2017 at 18:01, Gregory CLEMENT
> > <gregory.clement@free-electrons.com> wrote:  
> >> From: Hu Ziji <huziji@marvell.com>  
> <snip>
> >> +config MMC_SDHCI_XENON
> >> +       tristate "Marvell Xenon eMMC/SD/SDIO SDHCI driver"
> >> +       depends on MMC_SDHCI_PLTFM
> >> +       help
> >> +         This selects Marvell Xenon eMMC/SD/SDIO SDHCI.
> >> +         If you have a machine with integrated Marvell Xenon SDHC IP,  
> > 
> > /s/SDHC/SDHCI
> >   
> 
> 	Sorry. I don't get your point.
> 	Could you please give more detailed comments?

Ulf want you to s/SDHC/SDHCI in the line "If you have .....", I.E
change the line to:
If you have a machine with integrated Marvell Xenon SDHCI IP,

Thanks,
Jisheng

  reply	other threads:[~2017-03-16  2:29 UTC|newest]

Thread overview: 125+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-14 17:01 [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller Gregory CLEMENT
2017-02-14 17:01 ` Gregory CLEMENT
2017-02-14 17:01 ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 01/14] clk: apn806: Add eMMC clock to system controller driver Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 02/14] clk: apn806: Turn the eMMC clock as optional for dts backwards compatible Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 03/14] mmc: core: Add mmc-card dt sub-node parse in core layer Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-15 12:43   ` Ulf Hansson
2017-03-15 12:43     ` Ulf Hansson
2017-03-15 12:43     ` Ulf Hansson
2017-03-15 12:43     ` Ulf Hansson
2017-03-15 13:46     ` Ziji Hu
2017-03-15 13:46       ` Ziji Hu
2017-03-15 13:46       ` Ziji Hu
2017-03-15 13:46       ` Ziji Hu
2017-03-22 17:11     ` Gregory CLEMENT
2017-03-22 17:11       ` Gregory CLEMENT
2017-03-22 17:11       ` Gregory CLEMENT
2017-03-22 17:11       ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 04/14] mmc: sdhci: Export sdhci_set_ios() from sdhci.c Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-22 14:48   ` Adrian Hunter
2017-03-22 14:48     ` Adrian Hunter
2017-03-22 14:48     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 05/14] mmc: sdhci: Export sdhci_start_signal_voltage_switch() in sdhci.c Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-22 14:49   ` Adrian Hunter
2017-03-22 14:49     ` Adrian Hunter
2017-03-22 14:49     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 06/14] mmc: sdhci: Export sdhci_enable_sdio_irq() from sdhci.c Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-22 14:49   ` Adrian Hunter
2017-03-22 14:49     ` Adrian Hunter
2017-03-22 14:49     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 07/14] dt: bindings: Add bindings for Marvell Xenon SD Host Controller Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-22 21:44   ` Rob Herring
2017-02-22 21:44     ` Rob Herring
2017-02-22 21:44     ` Rob Herring
2017-03-15 12:48   ` Ulf Hansson
2017-03-15 12:48     ` Ulf Hansson
2017-03-15 12:48     ` Ulf Hansson
2017-03-15 12:48     ` Ulf Hansson
2017-03-23  5:32     ` [EXT] " Ziji Hu
2017-03-23  5:32       ` Ziji Hu
2017-03-23  5:32       ` Ziji Hu
2017-03-23  5:32       ` Ziji Hu
2017-02-14 17:01 ` [PATCH v6 08/14] mmc: sdhci-xenon: Add Marvell Xenon SDHC core functionality Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-15 13:11   ` Ulf Hansson
2017-03-15 13:11     ` Ulf Hansson
2017-03-15 13:11     ` Ulf Hansson
2017-03-15 13:11     ` Ulf Hansson
2017-03-15 13:58     ` Ziji Hu
2017-03-15 13:58       ` Ziji Hu
2017-03-15 13:58       ` Ziji Hu
2017-03-15 13:58       ` Ziji Hu
2017-03-16  2:24       ` Jisheng Zhang [this message]
2017-03-16  2:24         ` Jisheng Zhang
2017-03-16  2:24         ` Jisheng Zhang
2017-03-16  2:24         ` Jisheng Zhang
2017-03-22 14:32   ` Adrian Hunter
2017-03-22 14:32     ` Adrian Hunter
2017-03-22 14:32     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 09/14] mmc: sdhci-xenon: Add support to PHYs of Marvell Xenon SDHC Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-15 13:39   ` Ulf Hansson
2017-03-15 13:39     ` Ulf Hansson
2017-03-15 13:39     ` Ulf Hansson
2017-03-15 13:39     ` Ulf Hansson
2017-03-15 14:31     ` Ziji Hu
2017-03-15 14:31       ` Ziji Hu
2017-03-15 14:31       ` Ziji Hu
2017-03-15 14:31       ` Ziji Hu
2017-03-22 14:33   ` Adrian Hunter
2017-03-22 14:33     ` Adrian Hunter
2017-03-22 14:33     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 10/14] mmc: sdhci-xenon: Add SoC PHY PAD voltage control Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-03-22 14:47   ` Adrian Hunter
2017-03-22 14:47     ` Adrian Hunter
2017-03-22 14:47     ` Adrian Hunter
2017-02-14 17:01 ` [PATCH v6 11/14] MAINTAINERS: add entry for Marvell Xenon MMC Host Controller drivers Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 12/14] arm64: dts: marvell: add eMMC support for Armada 37xx Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 13/14] arm64: dts: marvell: add sdhci support for Armada 7K/8K Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01 ` [PATCH v6 14/14] arm64: configs: enable SDHCI driver for Xenon Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-14 17:01   ` Gregory CLEMENT
2017-02-20 16:59 ` [PATCH v6 00/14] mmc: Add support to Marvell Xenon SD Host Controller Gregory CLEMENT
2017-02-20 16:59   ` Gregory CLEMENT
2017-02-20 16:59   ` Gregory CLEMENT
2017-02-20 16:59   ` Gregory CLEMENT
2017-02-20 19:32   ` Ulf Hansson
2017-02-20 19:32     ` Ulf Hansson
2017-02-20 19:32     ` Ulf Hansson
2017-02-20 19:32     ` Ulf Hansson
2017-02-27  9:41     ` Ziji Hu
2017-02-27  9:41       ` Ziji Hu
2017-02-27  9:41       ` Ziji Hu
2017-02-27  9:41       ` Ziji Hu
2017-03-10 10:15     ` Gregory CLEMENT
2017-03-10 10:15       ` Gregory CLEMENT
2017-03-10 10:15       ` Gregory CLEMENT
2017-03-10 10:15       ` Gregory CLEMENT
2017-02-20 17:07 ` Russell King - ARM Linux
2017-02-20 17:07   ` Russell King - ARM Linux
2017-02-20 17:07   ` Russell King - ARM Linux

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=20170316102417.0909bccc@xhacker \
    --to=jszhang@marvell.com \
    --cc=adrian.hunter@intel.com \
    --cc=andrew@lunn.ch \
    --cc=devicetree@vger.kernel.org \
    --cc=dingwei@marvell.com \
    --cc=dougj@marvell.com \
    --cc=gregory.clement@free-electrons.com \
    --cc=hannah@marvell.com \
    --cc=huziji@marvell.com \
    --cc=jason@lakedaemon.net \
    --cc=kostap@marvell.com \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=liuw@marvell.com \
    --cc=mturquette@baylibre.com \
    --cc=mw@semihalf.com \
    --cc=nadavh@marvell.com \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@codeaurora.org \
    --cc=sebastian.hesselbarth@gmail.com \
    --cc=thomas.petazzoni@free-electrons.com \
    --cc=ulf.hansson@linaro.org \
    --cc=xigu@marvell.com \
    --cc=yehuday@marvell.com \
    --cc=ygao@marvell.com \
    --cc=zmxu@marvell.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 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.