* [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a [not found] <1450877391-3029-1-git-send-email-rsahu@apm.com> @ 2016-01-05 10:25 ` Rameshwar Sahu 2016-01-07 1:33 ` Jaehoon Chung 2016-01-27 14:15 ` Ulf Hansson 1 sibling, 1 reply; 5+ messages in thread From: Rameshwar Sahu @ 2016-01-05 10:25 UTC (permalink / raw) To: linux-arm-kernel Hi Ulf, On Wed, Dec 23, 2015 at 6:59 PM, Rameshswar Prasad Sahu <rsahu@apm•com> wrote: > From: Rameshwar Prasad Sahu <rsahu@apm•com> > > The Arason SD host controller supports set block count command (cmd23) > and high speed mode. This patch re-enable both of these features that > was disabled. For device that doesn't support high speed, it should > configure its capability register accordingly instead disables it > explicitly. > > Signed-off-by: Rameshwar Prasad Sahu <rsahu@apm•com> > --- > drivers/mmc/host/sdhci-of-arasan.c | 5 ----- > 1 files changed, 0 insertions(+), 5 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c > index 75379cb..5d9fdb3 100644 > --- a/drivers/mmc/host/sdhci-of-arasan.c > +++ b/drivers/mmc/host/sdhci-of-arasan.c > @@ -172,11 +172,6 @@ static int sdhci_arasan_probe(struct platform_device *pdev) > goto clk_disable_all; > } > > - if (of_device_is_compatible(pdev->dev.of_node, "arasan,sdhci-4.9a")) { > - host->quirks |= SDHCI_QUIRK_NO_HISPD_BIT; > - host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; > - } > - > sdhci_get_of_property(pdev); > pltfm_host = sdhci_priv(host); > pltfm_host->priv = sdhci_arasan; > -- > 1.7.1 > Any comment on this patch ?? ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a 2016-01-05 10:25 ` [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a Rameshwar Sahu @ 2016-01-07 1:33 ` Jaehoon Chung 2016-01-07 12:32 ` Rameshwar Sahu 0 siblings, 1 reply; 5+ messages in thread From: Jaehoon Chung @ 2016-01-07 1:33 UTC (permalink / raw) To: linux-arm-kernel Hi, On 01/05/2016 07:25 PM, Rameshwar Sahu wrote: > Hi Ulf, > > On Wed, Dec 23, 2015 at 6:59 PM, Rameshswar Prasad Sahu <rsahu@apm•com> wrote: >> From: Rameshwar Prasad Sahu <rsahu@apm•com> >> >> The Arason SD host controller supports set block count command (cmd23) >> and high speed mode. This patch re-enable both of these features that >> was disabled. For device that doesn't support high speed, it should >> configure its capability register accordingly instead disables it >> explicitly. >> >> Signed-off-by: Rameshwar Prasad Sahu <rsahu@apm•com> >> --- >> drivers/mmc/host/sdhci-of-arasan.c | 5 ----- >> 1 files changed, 0 insertions(+), 5 deletions(-) >> >> diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c >> index 75379cb..5d9fdb3 100644 >> --- a/drivers/mmc/host/sdhci-of-arasan.c >> +++ b/drivers/mmc/host/sdhci-of-arasan.c >> @@ -172,11 +172,6 @@ static int sdhci_arasan_probe(struct platform_device *pdev) >> goto clk_disable_all; >> } >> >> - if (of_device_is_compatible(pdev->dev.of_node, "arasan,sdhci-4.9a")) { >> - host->quirks |= SDHCI_QUIRK_NO_HISPD_BIT; >> - host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; I'm not sure what's correct since i didn't have any information for arasan. But in case of exynos, SDHCI_QUIRK_NO_HISPD_BIT is used because that bit is used as other purpose. If your SoC is not used anymore, i think this patch looks good. Best Regards, Jaehoon Chung >> - } >> - >> sdhci_get_of_property(pdev); >> pltfm_host = sdhci_priv(host); >> pltfm_host->priv = sdhci_arasan; >> -- >> 1.7.1 >> > > Any comment on this patch ?? > -- > To unsubscribe from this list: send the line "unsubscribe linux-mmc" in > the body of a message to majordomo at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a 2016-01-07 1:33 ` Jaehoon Chung @ 2016-01-07 12:32 ` Rameshwar Sahu 0 siblings, 0 replies; 5+ messages in thread From: Rameshwar Sahu @ 2016-01-07 12:32 UTC (permalink / raw) To: linux-arm-kernel Hi Jaehoon, On Thu, Jan 7, 2016 at 7:03 AM, Jaehoon Chung <jh80.chung@samsung•com> wrote: > Hi, > > On 01/05/2016 07:25 PM, Rameshwar Sahu wrote: >> Hi Ulf, >> >> On Wed, Dec 23, 2015 at 6:59 PM, Rameshswar Prasad Sahu <rsahu@apm•com> wrote: >>> From: Rameshwar Prasad Sahu <rsahu@apm•com> >>> >>> The Arason SD host controller supports set block count command (cmd23) >>> and high speed mode. This patch re-enable both of these features that >>> was disabled. For device that doesn't support high speed, it should >>> configure its capability register accordingly instead disables it >>> explicitly. >>> >>> Signed-off-by: Rameshwar Prasad Sahu <rsahu@apm•com> >>> --- >>> drivers/mmc/host/sdhci-of-arasan.c | 5 ----- >>> 1 files changed, 0 insertions(+), 5 deletions(-) >>> >>> diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c >>> index 75379cb..5d9fdb3 100644 >>> --- a/drivers/mmc/host/sdhci-of-arasan.c >>> +++ b/drivers/mmc/host/sdhci-of-arasan.c >>> @@ -172,11 +172,6 @@ static int sdhci_arasan_probe(struct platform_device *pdev) >>> goto clk_disable_all; >>> } >>> >>> - if (of_device_is_compatible(pdev->dev.of_node, "arasan,sdhci-4.9a")) { >>> - host->quirks |= SDHCI_QUIRK_NO_HISPD_BIT; >>> - host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; > > I'm not sure what's correct since i didn't have any information for arasan. > But in case of exynos, SDHCI_QUIRK_NO_HISPD_BIT is used because that bit is used as other purpose. > If your SoC is not used anymore, i think this patch looks good. Our SoC doesn't need that bit, that's why we are going to remove it. > > Best Regards, > Jaehoon Chung > >>> - } >>> - >>> sdhci_get_of_property(pdev); >>> pltfm_host = sdhci_priv(host); >>> pltfm_host->priv = sdhci_arasan; >>> -- >>> 1.7.1 >>> >> >> Any comment on this patch ?? >> -- >> To unsubscribe from this list: send the line "unsubscribe linux-mmc" in >> the body of a message to majordomo at vger.kernel.org >> More majordomo info at http://vger.kernel.org/majordomo-info.html >> >> > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a [not found] <1450877391-3029-1-git-send-email-rsahu@apm.com> 2016-01-05 10:25 ` [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a Rameshwar Sahu @ 2016-01-27 14:15 ` Ulf Hansson 1 sibling, 0 replies; 5+ messages in thread From: Ulf Hansson @ 2016-01-27 14:15 UTC (permalink / raw) To: linux-arm-kernel On 23 December 2015 at 14:29, Rameshswar Prasad Sahu <rsahu@apm•com> wrote: > From: Rameshwar Prasad Sahu <rsahu@apm•com> > > The Arason SD host controller supports set block count command (cmd23) > and high speed mode. This patch re-enable both of these features that > was disabled. For device that doesn't support high speed, it should > configure its capability register accordingly instead disables it > explicitly. > > Signed-off-by: Rameshwar Prasad Sahu <rsahu@apm•com> Thanks, applied for next! Kind regards Uffe > --- > drivers/mmc/host/sdhci-of-arasan.c | 5 ----- > 1 files changed, 0 insertions(+), 5 deletions(-) > > diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c > index 75379cb..5d9fdb3 100644 > --- a/drivers/mmc/host/sdhci-of-arasan.c > +++ b/drivers/mmc/host/sdhci-of-arasan.c > @@ -172,11 +172,6 @@ static int sdhci_arasan_probe(struct platform_device *pdev) > goto clk_disable_all; > } > > - if (of_device_is_compatible(pdev->dev.of_node, "arasan,sdhci-4.9a")) { > - host->quirks |= SDHCI_QUIRK_NO_HISPD_BIT; > - host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; > - } > - > sdhci_get_of_property(pdev); > pltfm_host = sdhci_priv(host); > pltfm_host->priv = sdhci_arasan; > -- > 1.7.1 > ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a @ 2015-12-22 13:20 Rameshwar Prasad Sahu 0 siblings, 0 replies; 5+ messages in thread From: Rameshwar Prasad Sahu @ 2015-12-22 13:20 UTC (permalink / raw) To: linux-arm-kernel The Arason SD host controller supports set block count command (cmd23) and high speed mode. This patch re-enable both of these features that was disabled. For device that doesn't support high speed, it should configure its capability register accordingly instead disables it explicitly. Signed-off-by: Rameshwar Prasad Sahu <rsahu@apm•com> --- drivers/mmc/host/sdhci-of-arasan.c | 5 ----- 1 files changed, 0 insertions(+), 5 deletions(-) diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c index 75379cb..5d9fdb3 100644 --- a/drivers/mmc/host/sdhci-of-arasan.c +++ b/drivers/mmc/host/sdhci-of-arasan.c @@ -172,11 +172,6 @@ static int sdhci_arasan_probe(struct platform_device *pdev) goto clk_disable_all; } - if (of_device_is_compatible(pdev->dev.of_node, "arasan,sdhci-4.9a")) { - host->quirks |= SDHCI_QUIRK_NO_HISPD_BIT; - host->quirks2 |= SDHCI_QUIRK2_HOST_NO_CMD23; - } - sdhci_get_of_property(pdev); pltfm_host = sdhci_priv(host); pltfm_host->priv = sdhci_arasan; -- 1.7.1 ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2016-01-27 14:15 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1450877391-3029-1-git-send-email-rsahu@apm.com>
2016-01-05 10:25 ` [PATCH] mmc: host: arasan: sdhci-of-arasan: Remove no-hispd and no-cmd23 quirks for sdhci-arasan4.9a Rameshwar Sahu
2016-01-07 1:33 ` Jaehoon Chung
2016-01-07 12:32 ` Rameshwar Sahu
2016-01-27 14:15 ` Ulf Hansson
2015-12-22 13:20 Rameshwar Prasad Sahu
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox