From: jh80.chung@samsung•com (Jaehoon Chung)
To: linux-arm-kernel@lists•infradead.org
Subject: [RFC PATCH v4 0/9]
Date: Thu, 06 Aug 2015 16:08:02 +0900 [thread overview]
Message-ID: <55C307D2.7020501@samsung.com> (raw)
In-Reply-To: <1438843469-23807-1-git-send-email-shawn.lin@rock-chips.com>
Hi, Shawn.
I remembered that Krzysztof has mentioned "Fix the title of cover letter."
Your cover letter's title is nothing.. "[RFC PATCH v4 0/9] " ??
[RFC PATCH v4 0/9] your title...
Best Regards,
Jaehoon Chung
On 08/06/2015 03:44 PM, Shawn Lin wrote:
> Add external dma support for Synopsys MSHC
>
> Synopsys DesignWare mobile storage host controller supports three
> types of transfer mode: pio, internal dma and external dma. However,
> dw_mmc can only supports pio and internal dma now. Thus some platforms
> using dw-mshc integrated with generic dma can't work in dma mode. So we
> submit this patch to achieve it.
>
> And the config option, CONFIG_MMC_DW_IDMAC, was added by Will Newton
> (commit:f95f3850) for the first version of dw_mmc and never be touched since
> then. At that time dt-bindings hadn't been introduced into dw_mmc yet means
> we should select CONFIG_MMC_DW_IDMAC to enable internal dma mode at compile
> time. Nowadays, device-tree helps us to support a variety of boards with one
> kernel. That's why we need to remove it and decide the transfer mode by reading
> dw_mmc's HCON reg at runtime.
>
> This RFC patch needs lots of ACKs. I know it's hard, but it does need someone
> to make the running.
>
> Patch does the following things:
> - remove CONFIG_MMC_DW_IDMAC config option
> - add bindings for edmac used by synopsys-dw-mshc
> at runtime
> - add edmac support for synopsys-dw-mshc
>
> Patch is based on next of git://git.linaro.org/people/ulf.hansson/mmc
>
>
> Changes in v4:
> - remove "host->trans_mode" and use "host->use_dma" to indicate
> transfer mode.
> - remove all bt-bindings' changes since we don't need new properities.
> - check transfer mode at runtime by reading HCON reg
> - spilt defconfig changes for each sub-architecture
> - fix the title of cover letter
> - reuse some code for reducing code size
>
> Changes in v3:
> - choose transfer mode at runtime
> - remove all CONFIG_MMC_DW_IDMAC config option
> - add supports-idmac property for some platforms
>
> Changes in v2:
> - Fix typo of dev_info msg
> - remove unused dmach from declaration of dw_mci_dma_slave
>
> Shawn Lin (9):
> mmc: dw_mmc: Add external dma interface support
> Documentation: synopsys-dw-mshc: add bindings for idmac and edmac
> mips: pistachio_defconfig: remove CONFIG_MMC_DW_IDMAC
> arc: axs10x_defconfig: remove CONFIG_MMC_DW_IDMAC
> arm: exynos_defconfig: remove CONFIG_MMC_DW_IDMAC
> arm: hisi_defconfig: remove CONFIG_MMC_DW_IDMAC
> arm: lpc18xx_defconfig: remove CONFIG_MMC_DW_IDMAC
> arm: multi_v7_defconfig: remove CONFIG_MMC_DW_IDMAC
> arm: zx_defconfig: remove CONFIG_MMC_DW_IDMAC
>
> .../devicetree/bindings/mmc/synopsys-dw-mshc.txt | 25 ++
> arch/arc/configs/axs101_defconfig | 1 -
> arch/arc/configs/axs103_defconfig | 1 -
> arch/arc/configs/axs103_smp_defconfig | 1 -
> arch/arm/configs/exynos_defconfig | 1 -
> arch/arm/configs/hisi_defconfig | 1 -
> arch/arm/configs/lpc18xx_defconfig | 1 -
> arch/arm/configs/multi_v7_defconfig | 1 -
> arch/arm/configs/zx_defconfig | 1 -
> arch/mips/configs/pistachio_defconfig | 1 -
> drivers/mmc/host/Kconfig | 11 +-
> drivers/mmc/host/dw_mmc-pltfm.c | 2 +
> drivers/mmc/host/dw_mmc.c | 258 +++++++++++++++++----
> include/linux/mmc/dw_mmc.h | 27 ++-
> 14 files changed, 257 insertions(+), 75 deletions(-)
>
next parent reply other threads:[~2015-08-06 7:08 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <1438843469-23807-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-06 7:08 ` Jaehoon Chung [this message]
[not found] ` <55C30D52.6040802@rock-chips.com>
2015-08-06 7:33 ` [RFC PATCH v4 0/9] Jaehoon Chung
[not found] ` <1438843491-23853-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-06 7:08 ` [RFC PATCH v4 1/9] mmc: dw_mmc: Add external dma interface support Krzysztof Kozlowski
[not found] ` <55C30C1C.6070801@rock-chips.com>
2015-08-06 7:37 ` Krzysztof Kozlowski
2015-08-07 21:32 ` Joachim Eastwood
2015-08-07 23:37 ` Shawn Lin
2015-08-10 18:03 ` Alim Akhtar
[not found] ` <1438843539-24017-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-06 7:08 ` [RFC PATCH v4 5/9] arm: exynos_defconfig: remove CONFIG_MMC_DW_IDMAC Krzysztof Kozlowski
[not found] ` <1438843526-23976-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-07 5:48 ` [RFC PATCH v4 4/9] arc: axs10x_defconfig: " Vineet Gupta
[not found] ` <1438843590-24101-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-07 21:34 ` [RFC PATCH v4 7/9] arm: lpc18xx_defconfig: " Joachim Eastwood
[not found] ` <1438843515-23935-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-06 9:05 ` [RFC PATCH v4 3/9] mips: pistachio_defconfig: " Govindraj Raja
2015-08-06 13:17 ` Ralf Baechle
2015-08-12 22:05 ` Ralf Baechle
2015-08-13 0:08 ` Shawn Lin
[not found] ` <1438843553-24058-1-git-send-email-shawn.lin@rock-chips.com>
2015-08-13 10:18 ` [RFC PATCH v4 6/9] arm: hisi_defconfig: " Wei Xu
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=55C307D2.7020501@samsung.com \
--to=jh80.chung@samsung$(echo .)com \
--cc=linux-arm-kernel@lists$(echo .)infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
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