From: Jaehoon Chung <jh80.chung@samsung.com>
To: Pavel Machek <pavel@denx.de>
Cc: linux-mmc <linux-mmc@vger.kernel.org>,
Mark Rutland <mark.rutland@arm.com>,
devicetree@vger.kernel.org, Ulf Hansson <ulf.hansson@linaro.org>,
kgene.kim@samsung.com, heiko@sntech.de, tgih.jun@samsung.com,
Chris Ball <chris@printf.net>,
linux-samsung-soc@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, dinguyen@altera.com
Subject: Re: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.
Date: Mon, 25 Aug 2014 20:28:21 +0900 [thread overview]
Message-ID: <53FB1DD5.3050207@samsung.com> (raw)
In-Reply-To: <20140825112151.GA21336@amd>
On 08/25/2014 08:21 PM, Pavel Machek wrote:
> On Thu 2014-08-07 16:37:57, Jaehoon Chung wrote:
>> Since used the mmc_of_parse(), didn't parse the sub-node.
>> So we can remove the sub-node, because almost SoC used the only one card per a host.
>> And supports-highspeed can be replaced with "cap-mmc/sd-highspeed"
>> property.
>
> Would it be better to fix parsing of the device tree, and not to
> change all the device trees?
>
> Someone will want to do two slots sooner or later...
>
First, I had considered that controller can be supported the multiple slot.
But MMC maintainers and other people suggested that consider the only one card per a host.
Two slots or more don't have any benefit, (power or performance, etc).
Best Regards,
Jaehoon Chung
WARNING: multiple messages have this Message-ID (diff)
From: jh80.chung@samsung.com (Jaehoon Chung)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc.
Date: Mon, 25 Aug 2014 20:28:21 +0900 [thread overview]
Message-ID: <53FB1DD5.3050207@samsung.com> (raw)
In-Reply-To: <20140825112151.GA21336@amd>
On 08/25/2014 08:21 PM, Pavel Machek wrote:
> On Thu 2014-08-07 16:37:57, Jaehoon Chung wrote:
>> Since used the mmc_of_parse(), didn't parse the sub-node.
>> So we can remove the sub-node, because almost SoC used the only one card per a host.
>> And supports-highspeed can be replaced with "cap-mmc/sd-highspeed"
>> property.
>
> Would it be better to fix parsing of the device tree, and not to
> change all the device trees?
>
> Someone will want to do two slots sooner or later...
>
First, I had considered that controller can be supported the multiple slot.
But MMC maintainers and other people suggested that consider the only one card per a host.
Two slots or more don't have any benefit, (power or performance, etc).
Best Regards,
Jaehoon Chung
next prev parent reply other threads:[~2014-08-25 11:28 UTC|newest]
Thread overview: 48+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-08-07 7:37 [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc Jaehoon Chung
2014-08-07 7:37 ` Jaehoon Chung
2014-08-07 7:37 ` [PATCHv10 1/5] mmc: dw_mmc: Slot quirk "disable-wp" is deprecated Jaehoon Chung
2014-08-07 7:37 ` Jaehoon Chung
2014-08-11 9:45 ` Ulf Hansson
2014-08-11 9:45 ` Ulf Hansson
2014-08-07 7:37 ` [PATCHv10 2/5] mmc: dw_mmc: modify the dt-binding for removing slot-node and supports-highspeed Jaehoon Chung
2014-08-07 7:37 ` Jaehoon Chung
2014-08-11 9:46 ` Ulf Hansson
2014-08-11 9:46 ` Ulf Hansson
[not found] ` <1407397082-32495-1-git-send-email-jh80.chung-Sze3O3UU22JBDgjK7y7TUQ@public.gmane.org>
2014-08-07 7:38 ` [PATCHv10 3/5] ARM: dts: exynos: unuse the slot-node and deprecate the supports-highspeed for dw-mmc Jaehoon Chung
2014-08-07 7:38 ` Jaehoon Chung
2014-08-18 12:23 ` Jaehoon Chung
2014-08-18 12:23 ` Jaehoon Chung
2014-08-18 14:10 ` Andreas Färber
2014-08-18 14:10 ` Andreas Färber
2014-08-18 16:54 ` Kukjin Kim
2014-08-18 16:54 ` Kukjin Kim
2014-08-18 17:06 ` Andreas Färber
2014-08-18 17:06 ` Andreas Färber
2014-08-18 17:40 ` Kukjin Kim
2014-08-18 17:40 ` Kukjin Kim
2014-08-19 3:52 ` Jaehoon Chung
2014-08-19 3:52 ` Jaehoon Chung
2014-08-07 7:38 ` [PATCHv10 4/5] ARM: dts: socfpga: " Jaehoon Chung
2014-08-07 7:38 ` Jaehoon Chung
2014-08-11 21:03 ` Dinh Nguyen
2014-08-11 21:03 ` Dinh Nguyen
2014-08-07 7:38 ` [PATCHv10 5/5] ARM: dts: rockchip: " Jaehoon Chung
2014-08-07 7:38 ` Jaehoon Chung
2014-08-11 14:38 ` Heiko Stübner
2014-08-11 14:38 ` Heiko Stübner
2014-08-25 11:21 ` [PATCHv10 0/5] ARM: remove the sub-node and deprecate supports-highspeed property for dwmmc Pavel Machek
2014-08-25 11:21 ` Pavel Machek
2014-08-25 11:28 ` Jaehoon Chung [this message]
2014-08-25 11:28 ` Jaehoon Chung
2014-08-25 11:37 ` Pavel Machek
2014-08-25 11:37 ` Pavel Machek
2014-08-25 12:09 ` Jaehoon Chung
2014-08-25 12:09 ` Jaehoon Chung
2014-08-25 12:11 ` Jaehoon Chung
2014-08-25 12:11 ` Jaehoon Chung
2014-08-26 10:19 ` Pavel Machek
2014-08-26 10:19 ` Pavel Machek
2014-08-27 3:11 ` Jaehoon Chung
2014-08-27 3:11 ` Jaehoon Chung
2014-08-27 11:56 ` Pavel Machek
2014-08-27 11:56 ` Pavel Machek
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=53FB1DD5.3050207@samsung.com \
--to=jh80.chung@samsung.com \
--cc=chris@printf.net \
--cc=devicetree@vger.kernel.org \
--cc=dinguyen@altera.com \
--cc=heiko@sntech.de \
--cc=kgene.kim@samsung.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-mmc@vger.kernel.org \
--cc=linux-samsung-soc@vger.kernel.org \
--cc=mark.rutland@arm.com \
--cc=pavel@denx.de \
--cc=tgih.jun@samsung.com \
--cc=ulf.hansson@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.