* [PATCH v5 01/11] mmc: tmio: enhance illegal sequence handling
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 02/11] mmc: tmio: document mandatory and optional callbacks Simon Horman
` (9 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Ai Kyuse, Simon Horman
From: Ai Kyuse <ai.kyuse.uw@renesas.com>
An illegal sequence command error may occur if there is a stopbit or
cmd_index error as well as a CRC error. The correct course of action
is to re-enable IRQs
An illegal sequence data error may occur if there is a CRC or stopbit
error, or underrun. In this case set data->error correctly.
This is in preparation for enabling tuning support which relies on
differentiating between illegal sequence and other errors.
Signed-off-by: Ai Kyuse <ai.kyuse.uw@renesas.com>
[simon: broken out of a larger patch]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
---
v4
* Added Ack from Wolfram Sang
v3
* No change
v2
* Broken out of a larger patch (but forgot to post)
---
drivers/mmc/host/tmio_mmc_pio.c | 15 +++++++++------
1 file changed, 9 insertions(+), 6 deletions(-)
diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
index 192817850dff..806308ac93e7 100644
--- a/drivers/mmc/host/tmio_mmc_pio.c
+++ b/drivers/mmc/host/tmio_mmc_pio.c
@@ -520,7 +520,7 @@ void tmio_mmc_do_data_irq(struct tmio_mmc_host *host)
schedule_work(&host->done);
}
-static void tmio_mmc_data_irq(struct tmio_mmc_host *host)
+static void tmio_mmc_data_irq(struct tmio_mmc_host *host, unsigned int stat)
{
struct mmc_data *data;
spin_lock(&host->lock);
@@ -529,6 +529,9 @@ static void tmio_mmc_data_irq(struct tmio_mmc_host *host)
if (!data)
goto out;
+ if (stat & TMIO_STAT_CRCFAIL || stat & TMIO_STAT_STOPBIT_ERR ||
+ stat & TMIO_STAT_TXUNDERRUN)
+ data->error = -EILSEQ;
if (host->chan_tx && (data->flags & MMC_DATA_WRITE) && !host->force_pio) {
u32 status = sd_ctrl_read16_and_16_as_32(host, CTL_STATUS);
bool done = false;
@@ -577,8 +580,6 @@ static void tmio_mmc_cmd_irq(struct tmio_mmc_host *host,
goto out;
}
- host->cmd = NULL;
-
/* This controller is sicker than the PXA one. Not only do we need to
* drop the top 8 bits of the first response word, we also need to
* modify the order of the response for short response command types.
@@ -598,14 +599,16 @@ static void tmio_mmc_cmd_irq(struct tmio_mmc_host *host,
if (stat & TMIO_STAT_CMDTIMEOUT)
cmd->error = -ETIMEDOUT;
- else if (stat & TMIO_STAT_CRCFAIL && cmd->flags & MMC_RSP_CRC)
+ else if ((stat & TMIO_STAT_CRCFAIL && cmd->flags & MMC_RSP_CRC) ||
+ stat & TMIO_STAT_STOPBIT_ERR ||
+ stat & TMIO_STAT_CMD_IDX_ERR)
cmd->error = -EILSEQ;
/* If there is data to handle we enable data IRQs here, and
* we will ultimatley finish the request in the data_end handler.
* If theres no data or we encountered an error, finish now.
*/
- if (host->data && !cmd->error) {
+ if (host->data && (!cmd->error || cmd->error == -EILSEQ)) {
if (host->data->flags & MMC_DATA_READ) {
if (host->force_pio || !host->chan_rx)
tmio_mmc_enable_mmc_irqs(host, TMIO_MASK_READOP);
@@ -666,7 +669,7 @@ static bool __tmio_mmc_sdcard_irq(struct tmio_mmc_host *host,
/* Data transfer completion */
if (ireg & TMIO_STAT_DATAEND) {
tmio_mmc_ack_mmc_irqs(host, TMIO_STAT_DATAEND);
- tmio_mmc_data_irq(host);
+ tmio_mmc_data_irq(host, status);
return true;
}
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 02/11] mmc: tmio: document mandatory and optional callbacks
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
2016-08-30 21:09 ` [PATCH v5 01/11] mmc: tmio: enhance illegal sequence handling Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 03/11] mmc: tmio: Add hw reset support Simon Horman
` (8 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* New patch
---
drivers/mmc/host/tmio_mmc.h | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
index 1f1cb2628f67..4b36cb5c2d9c 100644
--- a/drivers/mmc/host/tmio_mmc.h
+++ b/drivers/mmc/host/tmio_mmc.h
@@ -151,8 +151,10 @@ struct tmio_mmc_host {
bool native_hotplug;
bool sdio_irq_enabled;
- int (*write16_hook)(struct tmio_mmc_host *host, int addr);
+ /* Mandatory callback */
int (*clk_enable)(struct tmio_mmc_host *host);
+
+ /* Optional callbacks */
unsigned int (*clk_update)(struct tmio_mmc_host *host,
unsigned int new_clock);
void (*clk_disable)(struct tmio_mmc_host *host);
@@ -161,6 +163,7 @@ struct tmio_mmc_host {
int (*card_busy)(struct mmc_host *mmc);
int (*start_signal_voltage_switch)(struct mmc_host *mmc,
struct mmc_ios *ios);
+ int (*write16_hook)(struct tmio_mmc_host *host, int addr);
};
struct tmio_mmc_host *tmio_mmc_host_alloc(struct platform_device *pdev);
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 03/11] mmc: tmio: Add hw reset support
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
2016-08-30 21:09 ` [PATCH v5 01/11] mmc: tmio: enhance illegal sequence handling Simon Horman
2016-08-30 21:09 ` [PATCH v5 02/11] mmc: tmio: document mandatory and optional callbacks Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-31 7:50 ` Ulf Hansson
2016-08-30 21:09 ` [PATCH v5 04/11] mmc: tmio: Add tuning support Simon Horman
` (7 subsequent siblings)
10 siblings, 1 reply; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Ai Kyuse, Simon Horman
From: Ai Kyuse <ai.kyuse.uw@renesas.com>
Add hw reset support.
Signed-off-by: Ai Kyuse <ai.kyuse.uw@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
This is required by tuning support which will
be introduced by follow-up patches.
v5 [Simon Horman]
* As suggested by Ulf Hansson
- Broke out of a larger patch
---
drivers/mmc/host/tmio_mmc.h | 1 +
drivers/mmc/host/tmio_mmc_pio.c | 12 ++++++++++++
2 files changed, 13 insertions(+)
diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
index 4b36cb5c2d9c..4b71f31fba63 100644
--- a/drivers/mmc/host/tmio_mmc.h
+++ b/drivers/mmc/host/tmio_mmc.h
@@ -164,6 +164,7 @@ struct tmio_mmc_host {
int (*start_signal_voltage_switch)(struct mmc_host *mmc,
struct mmc_ios *ios);
int (*write16_hook)(struct tmio_mmc_host *host, int addr);
+ void (*hw_reset)(struct tmio_mmc_host *host);
};
struct tmio_mmc_host *tmio_mmc_host_alloc(struct platform_device *pdev);
diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
index 806308ac93e7..90758647bae6 100644
--- a/drivers/mmc/host/tmio_mmc_pio.c
+++ b/drivers/mmc/host/tmio_mmc_pio.c
@@ -756,6 +756,17 @@ static int tmio_mmc_start_data(struct tmio_mmc_host *host,
return 0;
}
+static void tmio_mmc_hw_reset(struct mmc_host *mmc)
+{
+ struct tmio_mmc_host *host = mmc_priv(mmc);
+
+ if (host->hw_reset)
+ host->hw_reset(host);
+
+ mmc_retune_timer_stop(host->mmc);
+ mmc_retune_needed(host->mmc);
+}
+
/* Process requests from the MMC layer */
static void tmio_mmc_request(struct mmc_host *mmc, struct mmc_request *mrq)
{
@@ -970,6 +981,7 @@ static struct mmc_host_ops tmio_mmc_ops = {
.get_cd = mmc_gpio_get_cd,
.enable_sdio_irq = tmio_mmc_enable_sdio_irq,
.multi_io_quirk = tmio_multi_io_quirk,
+ .hw_reset = tmio_mmc_hw_reset,
};
static int tmio_mmc_init_ocr(struct tmio_mmc_host *host)
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v5 03/11] mmc: tmio: Add hw reset support
2016-08-30 21:09 ` [PATCH v5 03/11] mmc: tmio: Add hw reset support Simon Horman
@ 2016-08-31 7:50 ` Ulf Hansson
2016-09-01 6:45 ` Simon Horman
0 siblings, 1 reply; 23+ messages in thread
From: Ulf Hansson @ 2016-08-31 7:50 UTC (permalink / raw)
To: Simon Horman
Cc: Wolfram Sang, Magnus Damm, linux-mmc, Linux-Renesas, Ai Kyuse
On 30 August 2016 at 23:09, Simon Horman <horms+renesas@verge.net.au> wrote:
> From: Ai Kyuse <ai.kyuse.uw@renesas.com>
>
> Add hw reset support.
>
> Signed-off-by: Ai Kyuse <ai.kyuse.uw@renesas.com>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> This is required by tuning support which will
> be introduced by follow-up patches.
>
> v5 [Simon Horman]
> * As suggested by Ulf Hansson
> - Broke out of a larger patch
> ---
> drivers/mmc/host/tmio_mmc.h | 1 +
> drivers/mmc/host/tmio_mmc_pio.c | 12 ++++++++++++
> 2 files changed, 13 insertions(+)
>
> diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
> index 4b36cb5c2d9c..4b71f31fba63 100644
> --- a/drivers/mmc/host/tmio_mmc.h
> +++ b/drivers/mmc/host/tmio_mmc.h
> @@ -164,6 +164,7 @@ struct tmio_mmc_host {
> int (*start_signal_voltage_switch)(struct mmc_host *mmc,
> struct mmc_ios *ios);
> int (*write16_hook)(struct tmio_mmc_host *host, int addr);
> + void (*hw_reset)(struct tmio_mmc_host *host);
> };
>
> struct tmio_mmc_host *tmio_mmc_host_alloc(struct platform_device *pdev);
> diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
> index 806308ac93e7..90758647bae6 100644
> --- a/drivers/mmc/host/tmio_mmc_pio.c
> +++ b/drivers/mmc/host/tmio_mmc_pio.c
> @@ -756,6 +756,17 @@ static int tmio_mmc_start_data(struct tmio_mmc_host *host,
> return 0;
> }
>
> +static void tmio_mmc_hw_reset(struct mmc_host *mmc)
> +{
> + struct tmio_mmc_host *host = mmc_priv(mmc);
> +
> + if (host->hw_reset)
> + host->hw_reset(host);
> +
> + mmc_retune_timer_stop(host->mmc);
> + mmc_retune_needed(host->mmc);
Both the above tuning calls are completely pointless as the mmc core
will run a reinitialization of the card when a ->hw_reset() host ops
is invoked.
That means a regular tuning will happen as part of the initialization
of the card, so you don't need to trigger it from here as well.
So if that's the only reason to why you need to add the hw_reset()
support, I think you should drop it instead.
> +}
> +
> /* Process requests from the MMC layer */
> static void tmio_mmc_request(struct mmc_host *mmc, struct mmc_request *mrq)
> {
> @@ -970,6 +981,7 @@ static struct mmc_host_ops tmio_mmc_ops = {
> .get_cd = mmc_gpio_get_cd,
> .enable_sdio_irq = tmio_mmc_enable_sdio_irq,
> .multi_io_quirk = tmio_multi_io_quirk,
> + .hw_reset = tmio_mmc_hw_reset,
> };
>
> static int tmio_mmc_init_ocr(struct tmio_mmc_host *host)
> --
> 2.7.0.rc3.207.g0ac5344
>
Kind regards
Uffe
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v5 03/11] mmc: tmio: Add hw reset support
2016-08-31 7:50 ` Ulf Hansson
@ 2016-09-01 6:45 ` Simon Horman
0 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-09-01 6:45 UTC (permalink / raw)
To: Ulf Hansson; +Cc: Wolfram Sang, Magnus Damm, linux-mmc, Linux-Renesas, Ai Kyuse
On Wed, Aug 31, 2016 at 09:50:53AM +0200, Ulf Hansson wrote:
> On 30 August 2016 at 23:09, Simon Horman <horms+renesas@verge.net.au> wrote:
> > From: Ai Kyuse <ai.kyuse.uw@renesas.com>
> >
> > Add hw reset support.
> >
> > Signed-off-by: Ai Kyuse <ai.kyuse.uw@renesas.com>
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > This is required by tuning support which will
> > be introduced by follow-up patches.
> >
> > v5 [Simon Horman]
> > * As suggested by Ulf Hansson
> > - Broke out of a larger patch
> > ---
> > drivers/mmc/host/tmio_mmc.h | 1 +
> > drivers/mmc/host/tmio_mmc_pio.c | 12 ++++++++++++
> > 2 files changed, 13 insertions(+)
> >
> > diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
> > index 4b36cb5c2d9c..4b71f31fba63 100644
> > --- a/drivers/mmc/host/tmio_mmc.h
> > +++ b/drivers/mmc/host/tmio_mmc.h
> > @@ -164,6 +164,7 @@ struct tmio_mmc_host {
> > int (*start_signal_voltage_switch)(struct mmc_host *mmc,
> > struct mmc_ios *ios);
> > int (*write16_hook)(struct tmio_mmc_host *host, int addr);
> > + void (*hw_reset)(struct tmio_mmc_host *host);
> > };
> >
> > struct tmio_mmc_host *tmio_mmc_host_alloc(struct platform_device *pdev);
> > diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
> > index 806308ac93e7..90758647bae6 100644
> > --- a/drivers/mmc/host/tmio_mmc_pio.c
> > +++ b/drivers/mmc/host/tmio_mmc_pio.c
> > @@ -756,6 +756,17 @@ static int tmio_mmc_start_data(struct tmio_mmc_host *host,
> > return 0;
> > }
> >
> > +static void tmio_mmc_hw_reset(struct mmc_host *mmc)
> > +{
> > + struct tmio_mmc_host *host = mmc_priv(mmc);
> > +
> > + if (host->hw_reset)
> > + host->hw_reset(host);
> > +
> > + mmc_retune_timer_stop(host->mmc);
> > + mmc_retune_needed(host->mmc);
>
> Both the above tuning calls are completely pointless as the mmc core
> will run a reinitialization of the card when a ->hw_reset() host ops
> is invoked.
> That means a regular tuning will happen as part of the initialization
> of the card, so you don't need to trigger it from here as well.
>
> So if that's the only reason to why you need to add the hw_reset()
> support, I think you should drop it instead.
Yes, sorry for leaving that in.
I will drop it.
> > +}
> > +
> > /* Process requests from the MMC layer */
> > static void tmio_mmc_request(struct mmc_host *mmc, struct mmc_request *mrq)
> > {
> > @@ -970,6 +981,7 @@ static struct mmc_host_ops tmio_mmc_ops = {
> > .get_cd = mmc_gpio_get_cd,
> > .enable_sdio_irq = tmio_mmc_enable_sdio_irq,
> > .multi_io_quirk = tmio_multi_io_quirk,
> > + .hw_reset = tmio_mmc_hw_reset,
> > };
> >
> > static int tmio_mmc_init_ocr(struct tmio_mmc_host *host)
> > --
> > 2.7.0.rc3.207.g0ac5344
> >
>
> Kind regards
> Uffe
>
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v5 04/11] mmc: tmio: Add tuning support
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (2 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 03/11] mmc: tmio: Add hw reset support Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 05/11] mmc: sh_mobile_sdhi: " Simon Horman
` (6 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Ai Kyuse, Simon Horman
From: Ai Kyuse <ai.kyuse.uw@renesas.com>
Add tuning support for use with SDR104 mode
Signed-off-by: Ai Kyuse <ai.kyuse.uw@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5 [Simon Horman]
* As suggested by Ulf Hansson:
- Move hw reset support into a separate patch
- Reply on core to retune on hw reset
- Use more descriptive name for callback to check for SSC error
- Rely on core to retune in case of -EILSEQ
- Document mandatory and optional callbacks
v4 [Simon Horman]
* As suggested by Wolfram Sang:
- Do not perform tuning if host->select_tuning is not set:
it seems to make little sense to do so and moreover there is currently
no such use-case
- Do not add mrc->sbc handling from tmio_mmc_request,
this is a hang-over from earlier versions of this patchset which
did not use core infrastructure for retuning
- Tidy up local variable usage
* Correct index passed to prepare_tuning(): this seems to have
been the last piece of resolving the timeouts during tuning puzzle
* Further cleanups to tmio_mmc_execute_tuning():
- Ensure tap is sized proportionally to its members
- Remove stray '*' in comment
- Use mmc rather than host->mmc, these are equivalent but
the former seems tidier
- Correct inverted logic in setting tap values
* Re-introduce retuning support. This was removed in v3.
v3 [Simon Horman]
* As suggested by Kuninori Morimoto:
- Do not add unused retuning callback to struct tmio_mmc_host
- Change return type of prepare_tuning callback to void
- Add tap_size parameter to select_tuning callback
v2 [Simon Horman]
* As suggested by Kuninori Morimoto:
- Actually remove unnecessary TMIO_MMC_HAS_UHS_SCC define
* As suggested by Wolfram Sang:
- Rely on core to call tuning. This simplifies things somewhat.
- Use mmc_send_tuning()
- A side affect of this appears to be that we now see some recoverable
errors logged during tuning. These are typically corrected by
subsequent tuning. It is the logging that is the apparent side effect
of this change.
e.g.
sh_mobile_sdhi ee100000.sd: timeout waiting for hardware interrupt (CMD19)
sh_mobile_sdhi ee100000.sd: Tuning procedure failed
* Use bool rather than unsigned long to pass test status
to select_tuning() callback
* Do not retune if init_tuning callback is not present or
indicates that there are no taps present
* Retune on hardware reset
v1 [Simon Horman]
* Omit start_signal_voltage_switch and tmio_mmc_card_busy changes which are
already present in mainline in a different form
* Return num from init_tuning rather than passing an extra parameter
to hold the return value
* Only call host->init_tuning if it is non-NULL
* Place tmio_mmc_execute_tuning() such that no new forward declarations are
required
* Remove unused TMIO_MMC_HAS_UHS_SCC define
v0 [Ai Kyuse]
---
drivers/mmc/host/tmio_mmc.h | 9 ++++++
drivers/mmc/host/tmio_mmc_pio.c | 62 +++++++++++++++++++++++++++++++++++++++--
2 files changed, 69 insertions(+), 2 deletions(-)
diff --git a/drivers/mmc/host/tmio_mmc.h b/drivers/mmc/host/tmio_mmc.h
index 4b71f31fba63..8c0d204563d0 100644
--- a/drivers/mmc/host/tmio_mmc.h
+++ b/drivers/mmc/host/tmio_mmc.h
@@ -150,6 +150,7 @@ struct tmio_mmc_host {
struct mutex ios_lock; /* protect set_ios() context */
bool native_hotplug;
bool sdio_irq_enabled;
+ u32 scc_tappos;
/* Mandatory callback */
int (*clk_enable)(struct tmio_mmc_host *host);
@@ -165,6 +166,14 @@ struct tmio_mmc_host {
struct mmc_ios *ios);
int (*write16_hook)(struct tmio_mmc_host *host, int addr);
void (*hw_reset)(struct tmio_mmc_host *host);
+ void (*prepare_tuning)(struct tmio_mmc_host *host, unsigned long tap);
+ bool (*check_scc_error)(struct tmio_mmc_host *host);
+
+ /* Mandatory callback for tuning to occur which is
+ * optional for SDR50 and mandatory for SDR104 */
+ unsigned int (*init_tuning)(struct tmio_mmc_host *host);
+ int (*select_tuning)(struct tmio_mmc_host *host, bool *tap,
+ int tap_size);
};
struct tmio_mmc_host *tmio_mmc_host_alloc(struct platform_device *pdev);
diff --git a/drivers/mmc/host/tmio_mmc_pio.c b/drivers/mmc/host/tmio_mmc_pio.c
index 90758647bae6..c85ffd83ebb9 100644
--- a/drivers/mmc/host/tmio_mmc_pio.c
+++ b/drivers/mmc/host/tmio_mmc_pio.c
@@ -36,6 +36,7 @@
#include <linux/io.h>
#include <linux/irq.h>
#include <linux/mfd/tmio.h>
+#include <linux/mmc/card.h>
#include <linux/mmc/host.h>
#include <linux/mmc/mmc.h>
#include <linux/mmc/slot-gpio.h>
@@ -298,6 +299,11 @@ static void tmio_mmc_finish_request(struct tmio_mmc_host *host)
if (mrq->cmd->error || (mrq->data && mrq->data->error))
tmio_mmc_abort_dma(host);
+ if (host->check_scc_error && host->check_scc_error(host)) {
+ mmc_retune_timer_stop(host->mmc);
+ mmc_retune_needed(host->mmc);
+ }
+
mmc_request_done(host->mmc, mrq);
}
@@ -762,9 +768,57 @@ static void tmio_mmc_hw_reset(struct mmc_host *mmc)
if (host->hw_reset)
host->hw_reset(host);
+}
+
+static int tmio_mmc_execute_tuning(struct mmc_host *mmc, u32 opcode)
+{
+ struct tmio_mmc_host *host = mmc_priv(mmc);
+ unsigned int num;
+ int i, ret = 0;
+ bool *tap;
+
+ if (!host->init_tuning || !host->select_tuning)
+ /* Tuning is not supported */
+ goto out;
+
+ num = host->init_tuning(host);
+ if (!num)
+ /* Tuning is not supported */
+ goto out;
+
+ tap = kmalloc(num * 2 * sizeof(*tap), GFP_KERNEL);
+ if (!tap) {
+ ret = -ENOMEM;
+ goto out;
+ }
+
+ /* Issue CMD19 twice for each tap */
+ for (i = 0; i < 2 * num; i++) {
+ if (host->prepare_tuning)
+ host->prepare_tuning(host, i % num);
+
+ ret = mmc_send_tuning(mmc, opcode, NULL);
+ if (ret && ret != -EILSEQ)
+ goto err_free;
+ tap[i] = (ret != 0);
+
+ mdelay(1);
+ }
+
+ ret = host->select_tuning(host, tap, num * 2);
+
+err_free:
+ kfree(tap);
+out:
+ if (ret < 0) {
+ dev_warn(&host->pdev->dev, "Tuning procedure failed\n");
+ mmc_hw_reset(mmc);
+ } else {
+ host->mmc->retune_period = 0;
+ }
+
+ return ret;
- mmc_retune_timer_stop(host->mmc);
- mmc_retune_needed(host->mmc);
}
/* Process requests from the MMC layer */
@@ -982,6 +1036,7 @@ static struct mmc_host_ops tmio_mmc_ops = {
.enable_sdio_irq = tmio_mmc_enable_sdio_irq,
.multi_io_quirk = tmio_multi_io_quirk,
.hw_reset = tmio_mmc_hw_reset,
+ .execute_tuning = tmio_mmc_execute_tuning,
};
static int tmio_mmc_init_ocr(struct tmio_mmc_host *host)
@@ -1207,6 +1262,9 @@ int tmio_mmc_host_runtime_suspend(struct device *dev)
struct mmc_host *mmc = dev_get_drvdata(dev);
struct tmio_mmc_host *host = mmc_priv(mmc);
+ mmc_retune_timer_stop(host->mmc);
+ mmc_retune_needed(host->mmc);
+
tmio_mmc_disable_mmc_irqs(host, TMIO_MASK_ALL);
if (host->clk_cache)
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 05/11] mmc: sh_mobile_sdhi: Add tuning support
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (3 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 04/11] mmc: tmio: Add tuning support Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 06/11] ARM: dts: lager: Enable UHS-I SDR-104 Simon Horman
` (5 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman, Ai Kyuse
Add tuning support for use with SDR104 mode
This includes adding support for the sampling clock controller (SCC).
Based on work by Ai Kyuse.
Cc: Ai Kyuse <ai.kyuse.uw@renesas.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5 [Simon Horman]
* As suggested by Ulf Hansson
- Use more descriptive name for callback to check for SSC error
* Correct return statement in sh_mobile_sdhi_hw_reset
* Reinstate Gen3 tuning support
v4 [Simon Horman]
As suggested by Geert Uytterhoeven:
* guard SDR104 specific portion of probe with host->mmc->caps &
MMC_CAP_UHS_SDR104
* As suggested by Wolfram Sang:
- Pass priv to sd_scc_{read,write}32 to save host_to_priv access
for each call
- Use 0x0 instead of other representations of 0 in hex
- Use CLK_CTL_SCLKEN
- Do not add unused SH_MOBILE_SDHI_MAX_TAP
- Use ternary operator in sh_mobile_sdhi_select_tuning
- Do include as yet unsupported HS200 in error string
* Reintroduce retuning support: This was removed in v3.
* Revert to algorithm in v1 patchset, on further reading of the documentation
it appears to be correct.
v3 [Simon Horman]
* As suggested by Kuninori Morimoto:
- Do not add unused retuning callback to struct tmio_mmc_host
- Change return type of prepare_tuning callback to void
- Add tap_size parameter to select_tuning callback
v2 [Simon Horman]
* As suggested by Kuninori Morimoto
- Use host->mmc->caps & MMC_CAP_UHS_SDR104 instead of
pdata->flags & TMIO_MMC_HAS_UHS_SCC to avoid needing the
MMC_CAP_UHS_SDR104 flag at all.
N.B: Morimoto-san suggested using but this flag is not actually
set there in by current probe come.
- Simplify logic in sh_mobile_sdhi_inquiry_tuning
* As suggested by Wolfram Sang
- Use clk_rate instead of clk for field in struct sh_mobile_sdhi_scc
- Remove inquiry_tuning callback which is now unnecessary as calling
of tuning is handled by the core
- Removed unused sh_mobile_sdhi_set_clk_div callback
- Save sci_base address rather than calculating it on each read and write
* Update selection logic in sh_mobile_sdhi_select_tuning to match spec
* Use bool instead of long for taps parameter of
sh_mobile_sdhi_select_tuning()
* Return 0 from sh_mobile_sdhi_init_tuning() if the
SDR104 capability is not set and thus tuning should not
be performed because it is not supported by the hardware
v1 [Simon Horman]
* Rebase
* Always use value of 0x8 for TAPNUM field of DTCNTL register
rather than reading value from DT property. There does not
seem to be a need to expose this in DT at this point.
* Do not include tmio_mmc_start_signal_voltage_switch changes which
are already in mainline in a different form
* Do not add renesas,clk-rate property as the max-frequency property, which
is now present in mainline, seems to provide the needed rate
* Omit Gen3 specific changes
* Do not provide renesas,mmc-scc-tappos DT property.
Instead, always use taps provided in driver.
* Do not parse sd-uhs-sdr50 and sd-uhs-sdr104 properties.
This is handled by the core.
v0 [Ai Kyuse]
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
drivers/mmc/host/sh_mobile_sdhi.c | 275 +++++++++++++++++++++++++++++++++++++-
1 file changed, 274 insertions(+), 1 deletion(-)
diff --git a/drivers/mmc/host/sh_mobile_sdhi.c b/drivers/mmc/host/sh_mobile_sdhi.c
index d679c8a533b6..b5f3f0752104 100644
--- a/drivers/mmc/host/sh_mobile_sdhi.c
+++ b/drivers/mmc/host/sh_mobile_sdhi.c
@@ -47,6 +47,11 @@
#define host_to_priv(host) container_of((host)->pdata, struct sh_mobile_sdhi, mmc_data)
+struct sh_mobile_sdhi_scc {
+ unsigned long clk_rate; /* clock rate for SDR104 */
+ u32 tap; /* sampling clock position for SDR104 */
+};
+
struct sh_mobile_sdhi_of_data {
unsigned long tmio_flags;
unsigned long capabilities;
@@ -54,6 +59,9 @@ struct sh_mobile_sdhi_of_data {
enum dma_slave_buswidth dma_buswidth;
dma_addr_t dma_rx_offset;
unsigned bus_shift;
+ int scc_offset;
+ struct sh_mobile_sdhi_scc *taps;
+ int taps_num;
};
static const struct sh_mobile_sdhi_of_data of_default_cfg = {
@@ -66,12 +74,35 @@ static const struct sh_mobile_sdhi_of_data of_rcar_gen1_compatible = {
.capabilities = MMC_CAP_SD_HIGHSPEED | MMC_CAP_SDIO_IRQ,
};
+/* Definitions for sampling clocks */
+static struct sh_mobile_sdhi_scc rcar_gen2_scc_taps[] = {
+ {
+ .clk_rate = 156000000,
+ .tap = 0x00000703,
+ },
+ {
+ .clk_rate = 0,
+ .tap = 0x00000300,
+ },
+};
+
static const struct sh_mobile_sdhi_of_data of_rcar_gen2_compatible = {
.tmio_flags = TMIO_MMC_HAS_IDLE_WAIT | TMIO_MMC_WRPROTECT_DISABLE |
TMIO_MMC_CLK_ACTUAL | TMIO_MMC_MIN_RCAR2,
.capabilities = MMC_CAP_SD_HIGHSPEED | MMC_CAP_SDIO_IRQ,
.dma_buswidth = DMA_SLAVE_BUSWIDTH_4_BYTES,
.dma_rx_offset = 0x2000,
+ .scc_offset = 0x0300,
+ .taps = rcar_gen2_scc_taps,
+ .taps_num = ARRAY_SIZE(rcar_gen2_scc_taps),
+};
+
+/* Definitions for sampling clocks */
+static struct sh_mobile_sdhi_scc rcar_gen3_scc_taps[] = {
+ {
+ .clk_rate = 0,
+ .tap = 0x00000300,
+ },
};
static const struct sh_mobile_sdhi_of_data of_rcar_gen3_compatible = {
@@ -79,6 +110,9 @@ static const struct sh_mobile_sdhi_of_data of_rcar_gen3_compatible = {
TMIO_MMC_CLK_ACTUAL | TMIO_MMC_MIN_RCAR2,
.capabilities = MMC_CAP_SD_HIGHSPEED | MMC_CAP_SDIO_IRQ,
.bus_shift = 2,
+ .scc_offset = 0x1000,
+ .taps = rcar_gen3_scc_taps,
+ .taps_num = ARRAY_SIZE(rcar_gen3_scc_taps),
};
static const struct of_device_id sh_mobile_sdhi_of_match[] = {
@@ -104,6 +138,7 @@ struct sh_mobile_sdhi {
struct tmio_mmc_dma dma_priv;
struct pinctrl *pinctrl;
struct pinctrl_state *pins_default, *pins_uhs;
+ void __iomem *scc_ctl;
};
static void sh_mobile_sdhi_sdbuf_width(struct tmio_mmc_host *host, int width)
@@ -254,6 +289,211 @@ static int sh_mobile_sdhi_start_signal_voltage_switch(struct mmc_host *mmc,
return pinctrl_select_state(priv->pinctrl, pin_state);
}
+/* SCC registers */
+#define SH_MOBILE_SDHI_SCC_DTCNTL 0x000
+#define SH_MOBILE_SDHI_SCC_TAPSET 0x002
+#define SH_MOBILE_SDHI_SCC_DT2FF 0x004
+#define SH_MOBILE_SDHI_SCC_CKSEL 0x006
+#define SH_MOBILE_SDHI_SCC_RVSCNTL 0x008
+#define SH_MOBILE_SDHI_SCC_RVSREQ 0x00A
+
+/* Definitions for values the SH_MOBILE_SDHI_SCC_DTCNTL register */
+#define SH_MOBILE_SDHI_SCC_DTCNTL_TAPEN BIT(0)
+#define SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_SHIFT 16
+#define SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_MASK 0xff
+
+/* Definitions for values the SH_MOBILE_SDHI_SCC_CKSEL register */
+#define SH_MOBILE_SDHI_SCC_CKSEL_DTSEL BIT(0)
+/* Definitions for values the SH_MOBILE_SDHI_SCC_RVSCNTL register */
+#define SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN BIT(1)
+/* Definitions for values the SH_MOBILE_SDHI_SCC_RVSREQ register */
+#define SH_MOBILE_SDHI_SCC_RVSREQ_RVSERR BIT(2)
+
+static inline u32 sd_scc_read32(struct tmio_mmc_host *host,
+ struct sh_mobile_sdhi *priv, int addr)
+{
+ return readl(priv->scc_ctl + (addr << host->bus_shift));
+}
+
+static inline void sd_scc_write32(struct tmio_mmc_host *host,
+ struct sh_mobile_sdhi *priv,
+ int addr, u32 val)
+{
+ writel(val, priv->scc_ctl + (addr << host->bus_shift));
+}
+
+static unsigned int sh_mobile_sdhi_init_tuning(struct tmio_mmc_host *host)
+{
+ struct sh_mobile_sdhi *priv;
+
+ if (!(host->mmc->caps & MMC_CAP_UHS_SDR104))
+ return 0;
+
+ priv = host_to_priv(host);
+
+ /* set sampling clock selection range */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_DTCNTL,
+ 0x8 << SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_SHIFT);
+
+ /* Initialize SCC */
+ sd_ctrl_write32_as_16_and_16(host, CTL_STATUS, 0x0);
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_DTCNTL,
+ SH_MOBILE_SDHI_SCC_DTCNTL_TAPEN |
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_DTCNTL));
+
+ sd_ctrl_write16(host, CTL_SD_CARD_CLK_CTL, ~CLK_CTL_SCLKEN &
+ sd_ctrl_read16(host, CTL_SD_CARD_CLK_CTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_CKSEL,
+ SH_MOBILE_SDHI_SCC_CKSEL_DTSEL |
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_CKSEL));
+
+ sd_ctrl_write16(host, CTL_SD_CARD_CLK_CTL, CLK_CTL_SCLKEN |
+ sd_ctrl_read16(host, CTL_SD_CARD_CLK_CTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL,
+ ~SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN &
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_DT2FF, host->scc_tappos);
+
+ /* Read TAPNUM */
+ return (sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_DTCNTL) >>
+ SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_SHIFT) &
+ SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_MASK;
+}
+
+static void sh_mobile_sdhi_prepare_tuning(struct tmio_mmc_host *host,
+ unsigned long tap)
+{
+ struct sh_mobile_sdhi *priv = host_to_priv(host);
+
+ /* Set sampling clock position */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_TAPSET, tap);
+}
+
+#define SH_MOBILE_SDHI_MAX_TAP 3
+
+static int sh_mobile_sdhi_select_tuning(struct tmio_mmc_host *host,
+ bool *tap, int tap_size)
+{
+ struct sh_mobile_sdhi *priv = host_to_priv(host);
+ unsigned long tap_num; /* total number of taps */
+ unsigned long tap_cnt; /* counter of tuning success */
+ unsigned long tap_set; /* tap position */
+ unsigned long tap_start;/* start position of tuning success */
+ unsigned long tap_end; /* end position of tuning success */
+ unsigned long ntap; /* temporary counter of tuning success */
+ unsigned long i;
+
+ /* Clear SCC_RVSREQ */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSREQ, 0);
+
+ /* Select SCC */
+ tap_num = (sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_DTCNTL) >>
+ SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_SHIFT) &
+ SH_MOBILE_SDHI_SCC_DTCNTL_TAPNUM_MASK;
+
+ if (tap_num * 2 != tap_size)
+ return -EINVAL;
+
+ /*
+ * Find the longest consecutive run of successful probes. If that
+ * is more than SH_MOBILE_SDHI_MAX_TAP probes long then use the
+ * center index as the tap.
+ */
+ tap_cnt = 0;
+ ntap = 0;
+ tap_start = 0;
+ tap_end = 0;
+ for (i = 0; i < tap_num * 2; i++) {
+ if (tap[i] == 0)
+ ntap++;
+ else {
+ if (ntap > tap_cnt) {
+ tap_start = i - ntap;
+ tap_end = i - 1;
+ tap_cnt = ntap;
+ }
+ ntap = 0;
+ }
+ }
+
+ if (ntap > tap_cnt) {
+ tap_start = i - ntap;
+ tap_end = i - 1;
+ tap_cnt = ntap;
+ }
+
+ if (tap_cnt >= SH_MOBILE_SDHI_MAX_TAP)
+ tap_set = (tap_start + tap_end) / 2 % tap_num;
+ else
+ return -EIO;
+
+ /* Set SCC */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_TAPSET, tap_set);
+
+ /* Enable auto re-tuning */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL,
+ SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN |
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL));
+
+ return 0;
+}
+
+
+static bool sh_mobile_sdhi_check_scc_error(struct tmio_mmc_host *host)
+{
+ struct sh_mobile_sdhi *priv;
+
+ if (!(host->mmc->caps & MMC_CAP_UHS_SDR104))
+ return 0;
+
+ priv = host_to_priv(host);
+
+ /* Check SCC error */
+ if (sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL) &
+ SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN &&
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSREQ) &
+ SH_MOBILE_SDHI_SCC_RVSREQ_RVSERR) {
+ /* Clear SCC error */
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSREQ, 0);
+ return true;
+ }
+
+ return false;
+}
+
+static void sh_mobile_sdhi_hw_reset(struct tmio_mmc_host *host)
+{
+ struct sh_mobile_sdhi *priv;
+
+ if (host->mmc->caps & MMC_CAP_UHS_SDR104)
+ return;
+
+ priv = host_to_priv(host);
+
+ /* Reset SCC */
+ sd_ctrl_write16(host, CTL_SD_CARD_CLK_CTL, ~CLK_CTL_SCLKEN &
+ sd_ctrl_read16(host, CTL_SD_CARD_CLK_CTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_CKSEL,
+ ~SH_MOBILE_SDHI_SCC_CKSEL_DTSEL &
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_CKSEL));
+
+ sd_ctrl_write16(host, CTL_SD_CARD_CLK_CTL, CLK_CTL_SCLKEN |
+ sd_ctrl_read16(host, CTL_SD_CARD_CLK_CTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL,
+ ~SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN &
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL));
+
+ sd_scc_write32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL,
+ ~SH_MOBILE_SDHI_SCC_RVSCNTL_RVSEN &
+ sd_scc_read32(host, priv, SH_MOBILE_SDHI_SCC_RVSCNTL));
+}
+
static int sh_mobile_sdhi_wait_idle(struct tmio_mmc_host *host)
{
int timeout = 1000;
@@ -324,7 +564,7 @@ static int sh_mobile_sdhi_probe(struct platform_device *pdev)
struct tmio_mmc_data *mmd = pdev->dev.platform_data;
struct tmio_mmc_host *host;
struct resource *res;
- int irq, ret, i = 0;
+ int irq, ret, i;
struct tmio_mmc_dma *dma_priv;
res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
@@ -383,6 +623,11 @@ static int sh_mobile_sdhi_probe(struct platform_device *pdev)
host->card_busy = sh_mobile_sdhi_card_busy;
host->start_signal_voltage_switch =
sh_mobile_sdhi_start_signal_voltage_switch;
+ host->init_tuning = sh_mobile_sdhi_init_tuning;
+ host->prepare_tuning = sh_mobile_sdhi_prepare_tuning;
+ host->select_tuning = sh_mobile_sdhi_select_tuning;
+ host->check_scc_error = sh_mobile_sdhi_check_scc_error;
+ host->hw_reset = sh_mobile_sdhi_hw_reset;
}
/* Orginally registers were 16 bit apart, could be 32 or 64 nowadays */
@@ -423,6 +668,34 @@ static int sh_mobile_sdhi_probe(struct platform_device *pdev)
if (ret < 0)
goto efree;
+ if (host->mmc->caps & MMC_CAP_UHS_SDR104) {
+ host->mmc->caps |= MMC_CAP_HW_RESET;
+
+ if (of_id && of_id->data) {
+ const struct sh_mobile_sdhi_of_data *of_data;
+ const struct sh_mobile_sdhi_scc *taps;
+ bool hit = false;
+
+ of_data = of_id->data;
+ taps = of_data->taps;
+
+ for (i = 0; i < of_data->taps_num; i++) {
+ if (taps[i].clk_rate == 0 ||
+ taps[i].clk_rate == host->mmc->f_max) {
+ host->scc_tappos = taps->tap;
+ hit = true;
+ break;
+ }
+ }
+
+ if (!hit)
+ dev_warn(&host->pdev->dev, "Unknown clock rate for SDR104\n");
+
+ priv->scc_ctl = host->ctl + of_data->scc_offset;
+ }
+ }
+
+ i = 0;
while (1) {
irq = platform_get_irq(pdev, i);
if (irq < 0)
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 06/11] ARM: dts: lager: Enable UHS-I SDR-104
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (4 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 05/11] mmc: sh_mobile_sdhi: " Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property Simon Horman
` (4 subsequent siblings)
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
Add the sd-uhs-sdr104 property to SDHI0.
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
arch/arm/boot/dts/r8a7790-lager.dts | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/arm/boot/dts/r8a7790-lager.dts b/arch/arm/boot/dts/r8a7790-lager.dts
index 52b56fcaddf2..5e43763cec65 100644
--- a/arch/arm/boot/dts/r8a7790-lager.dts
+++ b/arch/arm/boot/dts/r8a7790-lager.dts
@@ -559,6 +559,7 @@
vqmmc-supply = <&vccq_sdhi0>;
cd-gpios = <&gpio3 6 GPIO_ACTIVE_LOW>;
sd-uhs-sdr50;
+ sd-uhs-sdr104;
status = "okay";
};
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (5 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 06/11] ARM: dts: lager: Enable UHS-I SDR-104 Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-09-01 14:32 ` Simon Horman
2016-09-16 10:29 ` Wolfram Sang
2016-08-30 21:09 ` [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks Simon Horman
` (3 subsequent siblings)
10 siblings, 2 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
Remove cap-mmc-highspeed property from SDHI2 and SDHI3.
This property is unnecessary as the driver automatically sets
the highspeed capability. Furthermore its use is inconsistent with SDHI0
and SDHI1 which are also highspeed capable but do not have this property
present.
Found by inspection.
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* New patch
---
arch/arm64/boot/dts/renesas/r8a7795.dtsi | 2 --
1 file changed, 2 deletions(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index b902356873c2..70949f4d50a2 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -1117,7 +1117,6 @@
interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 312>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
- cap-mmc-highspeed;
status = "disabled";
};
@@ -1127,7 +1126,6 @@
interrupts = <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 311>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
- cap-mmc-highspeed;
status = "disabled";
};
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property
2016-08-30 21:09 ` [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property Simon Horman
@ 2016-09-01 14:32 ` Simon Horman
2016-09-16 10:29 ` Wolfram Sang
1 sibling, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-09-01 14:32 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson; +Cc: Magnus Damm, linux-mmc, linux-renesas-soc
On Tue, Aug 30, 2016 at 11:09:55PM +0200, Simon Horman wrote:
> Remove cap-mmc-highspeed property from SDHI2 and SDHI3.
>
> This property is unnecessary as the driver automatically sets
> the highspeed capability. Furthermore its use is inconsistent with SDHI0
> and SDHI1 which are also highspeed capable but do not have this property
> present.
>
> Found by inspection.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
I have queued this up.
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property
2016-08-30 21:09 ` [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property Simon Horman
2016-09-01 14:32 ` Simon Horman
@ 2016-09-16 10:29 ` Wolfram Sang
1 sibling, 0 replies; 23+ messages in thread
From: Wolfram Sang @ 2016-09-16 10:29 UTC (permalink / raw)
To: Simon Horman
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 534 bytes --]
On Tue, Aug 30, 2016 at 11:09:55PM +0200, Simon Horman wrote:
> Remove cap-mmc-highspeed property from SDHI2 and SDHI3.
>
> This property is unnecessary as the driver automatically sets
> the highspeed capability. Furthermore its use is inconsistent with SDHI0
> and SDHI1 which are also highspeed capable but do not have this property
> present.
>
> Found by inspection.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
For completeness:
Reviewed-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (6 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 07/11] arm64: dts: r8a7795: remove unnecessary cap-mmc-highspeed property Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-31 8:44 ` Wolfram Sang
2016-08-30 21:09 ` [PATCH v5 09/11] arm64: dts: r8a7795: salvator: enable UHS for SDHI 0 & 3 Simon Horman
` (2 subsequent siblings)
10 siblings, 1 reply; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* New patch
---
arch/arm64/boot/dts/renesas/r8a7795.dtsi | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a7795.dtsi b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
index 70949f4d50a2..f83b87e9366e 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795.dtsi
+++ b/arch/arm64/boot/dts/renesas/r8a7795.dtsi
@@ -1099,6 +1099,7 @@
interrupts = <GIC_SPI 165 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 314>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
+ max-frequency = <200000000>;
status = "disabled";
};
@@ -1108,6 +1109,7 @@
interrupts = <GIC_SPI 166 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 313>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
+ max-frequency = <200000000>;
status = "disabled";
};
@@ -1117,6 +1119,7 @@
interrupts = <GIC_SPI 167 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 312>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
+ max-frequency = <200000000>;
status = "disabled";
};
@@ -1126,6 +1129,7 @@
interrupts = <GIC_SPI 168 IRQ_TYPE_LEVEL_HIGH>;
clocks = <&cpg CPG_MOD 311>;
power-domains = <&sysc R8A7795_PD_ALWAYS_ON>;
+ max-frequency = <200000000>;
status = "disabled";
};
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks
2016-08-30 21:09 ` [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks Simon Horman
@ 2016-08-31 8:44 ` Wolfram Sang
2016-09-01 14:24 ` Simon Horman
0 siblings, 1 reply; 23+ messages in thread
From: Wolfram Sang @ 2016-08-31 8:44 UTC (permalink / raw)
To: Simon Horman
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 298 bytes --]
On Tue, Aug 30, 2016 at 11:09:56PM +0200, Simon Horman wrote:
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> ---
> v5
> * New patch
Huh? There is already 721e9a08aa514c ("arm64: dts: r8a7795: set maximum
frequency for SDHI clocks") in renesas/drivers? Why didn't you pick this
one?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread* Re: [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks
2016-08-31 8:44 ` Wolfram Sang
@ 2016-09-01 14:24 ` Simon Horman
0 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-09-01 14:24 UTC (permalink / raw)
To: Wolfram Sang
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
On Wed, Aug 31, 2016 at 10:44:49AM +0200, Wolfram Sang wrote:
> On Tue, Aug 30, 2016 at 11:09:56PM +0200, Simon Horman wrote:
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > ---
> > v5
> > * New patch
>
> Huh? There is already 721e9a08aa514c ("arm64: dts: r8a7795: set maximum
> frequency for SDHI clocks") in renesas/drivers? Why didn't you pick this
> one?
Sorry, I was working aganst mmc/next and forgot to check
against the renesas tree for the patches that should go/be there.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v5 09/11] arm64: dts: r8a7795: salvator: enable UHS for SDHI 0 & 3
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (7 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 08/11] arm64: r8a7795: Set maximum frequencies for SDHI clocks Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-08-30 21:09 ` [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104 Simon Horman
2016-08-30 21:09 ` [PATCH v5 11/11] arm64: defconfig: Enable SDHI and GPIO_REGULATOR Simon Horman
10 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
From: Wolfram Sang <wsa+renesas@sang-engineering.com>
Reviewed-by: Geert Uytterhoeven <geert+renesas@glider.be>
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* Added to series, previously posted by Wolfram independently
---
arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 22 ++++++++++++++++++++--
1 file changed, 20 insertions(+), 2 deletions(-)
diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index 98f02631a0f0..54540a223942 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -175,11 +175,25 @@
sdhi0_pins: sd0 {
groups = "sdhi0_data4", "sdhi0_ctrl";
function = "sdhi0";
+ power-source = <3300>;
+ };
+
+ sdhi0_pins_uhs: sd0_uhs {
+ groups = "sdhi0_data4", "sdhi0_ctrl";
+ function = "sdhi0";
+ power-source = <1800>;
};
sdhi3_pins: sd3 {
groups = "sdhi3_data4", "sdhi3_ctrl";
function = "sdhi3";
+ power-source = <3300>;
+ };
+
+ sdhi3_pins_uhs: sd3_uhs {
+ groups = "sdhi3_data4", "sdhi3_ctrl";
+ function = "sdhi3";
+ power-source = <1800>;
};
sound_pins: sound {
@@ -304,25 +318,29 @@
&sdhi0 {
pinctrl-0 = <&sdhi0_pins>;
- pinctrl-names = "default";
+ pinctrl-1 = <&sdhi0_pins_uhs>;
+ pinctrl-names = "default", "state_uhs";
vmmc-supply = <&vcc_sdhi0>;
vqmmc-supply = <&vccq_sdhi0>;
cd-gpios = <&gpio3 12 GPIO_ACTIVE_LOW>;
wp-gpios = <&gpio3 13 GPIO_ACTIVE_HIGH>;
bus-width = <4>;
+ sd-uhs-sdr50;
status = "okay";
};
&sdhi3 {
pinctrl-0 = <&sdhi3_pins>;
- pinctrl-names = "default";
+ pinctrl-1 = <&sdhi3_pins_uhs>;
+ pinctrl-names = "default", "state_uhs";
vmmc-supply = <&vcc_sdhi3>;
vqmmc-supply = <&vccq_sdhi3>;
cd-gpios = <&gpio4 15 GPIO_ACTIVE_LOW>;
wp-gpios = <&gpio4 16 GPIO_ACTIVE_HIGH>;
bus-width = <4>;
+ sd-uhs-sdr50;
status = "okay";
};
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (8 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 09/11] arm64: dts: r8a7795: salvator: enable UHS for SDHI 0 & 3 Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-09-01 7:23 ` Wolfram Sang
2016-08-30 21:09 ` [PATCH v5 11/11] arm64: defconfig: Enable SDHI and GPIO_REGULATOR Simon Horman
10 siblings, 1 reply; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
Add the sd-uhs-sdr104 property to SDHI0 and SDHI1.
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* New patch
---
arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
index 54540a223942..d3ad5ddcc95c 100644
--- a/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
+++ b/arch/arm64/boot/dts/renesas/r8a7795-salvator-x.dts
@@ -327,6 +327,7 @@
wp-gpios = <&gpio3 13 GPIO_ACTIVE_HIGH>;
bus-width = <4>;
sd-uhs-sdr50;
+ sd-uhs-sdr104;
status = "okay";
};
@@ -341,6 +342,7 @@
wp-gpios = <&gpio4 16 GPIO_ACTIVE_HIGH>;
bus-width = <4>;
sd-uhs-sdr50;
+ sd-uhs-sdr104;
status = "okay";
};
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread* Re: [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104
2016-08-30 21:09 ` [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104 Simon Horman
@ 2016-09-01 7:23 ` Wolfram Sang
2016-09-01 14:25 ` Simon Horman
0 siblings, 1 reply; 23+ messages in thread
From: Wolfram Sang @ 2016-09-01 7:23 UTC (permalink / raw)
To: Simon Horman
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 370 bytes --]
On Tue, Aug 30, 2016 at 11:09:58PM +0200, Simon Horman wrote:
> Add the sd-uhs-sdr104 property to SDHI0 and SDHI1.
>
> Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
After removing a SDR104 card from the slot, the interface is stalled
here. No new cards (even non-SDR104) can be detected anymore, sadly.
Happens for both slots on my H3 Salvator-X.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104
2016-09-01 7:23 ` Wolfram Sang
@ 2016-09-01 14:25 ` Simon Horman
2016-09-01 14:29 ` Wolfram Sang
0 siblings, 1 reply; 23+ messages in thread
From: Simon Horman @ 2016-09-01 14:25 UTC (permalink / raw)
To: Wolfram Sang
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
On Thu, Sep 01, 2016 at 09:23:54AM +0200, Wolfram Sang wrote:
> On Tue, Aug 30, 2016 at 11:09:58PM +0200, Simon Horman wrote:
> > Add the sd-uhs-sdr104 property to SDHI0 and SDHI1.
> >
> > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
>
> After removing a SDR104 card from the slot, the interface is stalled
> here. No new cards (even non-SDR104) can be detected anymore, sadly.
> Happens for both slots on my H3 Salvator-X.
Bother. That is rather difficult for me to exercise.
Does this occur without sdr104 enabled (e.g. in DT)?
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104
2016-09-01 14:25 ` Simon Horman
@ 2016-09-01 14:29 ` Wolfram Sang
2016-09-01 14:37 ` Simon Horman
0 siblings, 1 reply; 23+ messages in thread
From: Wolfram Sang @ 2016-09-01 14:29 UTC (permalink / raw)
To: Simon Horman
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
[-- Attachment #1: Type: text/plain, Size: 751 bytes --]
On Thu, Sep 01, 2016 at 04:25:40PM +0200, Simon Horman wrote:
> On Thu, Sep 01, 2016 at 09:23:54AM +0200, Wolfram Sang wrote:
> > On Tue, Aug 30, 2016 at 11:09:58PM +0200, Simon Horman wrote:
> > > Add the sd-uhs-sdr104 property to SDHI0 and SDHI1.
> > >
> > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> >
> > After removing a SDR104 card from the slot, the interface is stalled
> > here. No new cards (even non-SDR104) can be detected anymore, sadly.
> > Happens for both slots on my H3 Salvator-X.
>
> Bother. That is rather difficult for me to exercise.
> Does this occur without sdr104 enabled (e.g. in DT)?
No, then it works fine.
I'll try to get you more information. Might be not before Monday,
though.
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 819 bytes --]
^ permalink raw reply [flat|nested] 23+ messages in thread
* Re: [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104
2016-09-01 14:29 ` Wolfram Sang
@ 2016-09-01 14:37 ` Simon Horman
0 siblings, 0 replies; 23+ messages in thread
From: Simon Horman @ 2016-09-01 14:37 UTC (permalink / raw)
To: Wolfram Sang
Cc: Wolfram Sang, Ulf Hansson, Magnus Damm, linux-mmc,
linux-renesas-soc
On Thu, Sep 01, 2016 at 04:29:43PM +0200, Wolfram Sang wrote:
> On Thu, Sep 01, 2016 at 04:25:40PM +0200, Simon Horman wrote:
> > On Thu, Sep 01, 2016 at 09:23:54AM +0200, Wolfram Sang wrote:
> > > On Tue, Aug 30, 2016 at 11:09:58PM +0200, Simon Horman wrote:
> > > > Add the sd-uhs-sdr104 property to SDHI0 and SDHI1.
> > > >
> > > > Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
> > >
> > > After removing a SDR104 card from the slot, the interface is stalled
> > > here. No new cards (even non-SDR104) can be detected anymore, sadly.
> > > Happens for both slots on my H3 Salvator-X.
> >
> > Bother. That is rather difficult for me to exercise.
> > Does this occur without sdr104 enabled (e.g. in DT)?
>
> No, then it works fine.
>
> I'll try to get you more information. Might be not before Monday,
> though.
Thanks. I'd be grateful if we could coordinate some way to get to the
bottom of this.
^ permalink raw reply [flat|nested] 23+ messages in thread
* [PATCH v5 11/11] arm64: defconfig: Enable SDHI and GPIO_REGULATOR
2016-08-30 21:09 [PATCH v5 00/11] UHS-I SDR-104 support for sh_mobile_sdhi Simon Horman
` (9 preceding siblings ...)
2016-08-30 21:09 ` [PATCH v5 10/11] arm64: dts: r8a7795: salvator-x: Enable UHS-I SDR-104 Simon Horman
@ 2016-08-30 21:09 ` Simon Horman
2016-09-01 14:33 ` Simon Horman
10 siblings, 1 reply; 23+ messages in thread
From: Simon Horman @ 2016-08-30 21:09 UTC (permalink / raw)
To: Wolfram Sang, Ulf Hansson
Cc: Magnus Damm, linux-mmc, linux-renesas-soc, Simon Horman
This allows use of the SDHI SD/SDIO controller present on R-Car Gen3 SoCs
and already enabled in the DT of the r8a7795/salvator-x (H3).
Signed-off-by: Simon Horman <horms+renesas@verge.net.au>
---
v5
* New patch
---
arch/arm64/configs/defconfig | 2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm64/configs/defconfig b/arch/arm64/configs/defconfig
index eadf4855ad2d..e9243f7772c2 100644
--- a/arch/arm64/configs/defconfig
+++ b/arch/arm64/configs/defconfig
@@ -264,6 +264,7 @@ CONFIG_REGULATOR=y
CONFIG_MFD_CROS_EC=y
CONFIG_MFD_CROS_EC_I2C=y
CONFIG_REGULATOR_FIXED_VOLTAGE=y
+CONFIG_REGULATOR_GPIO=y
CONFIG_REGULATOR_HI655X=y
CONFIG_REGULATOR_MAX77620=y
CONFIG_REGULATOR_PWM=y
@@ -320,6 +321,7 @@ CONFIG_MMC_SDHCI_OF_ESDHC=y
CONFIG_MMC_SDHCI_TEGRA=y
CONFIG_MMC_SDHCI_MSM=y
CONFIG_MMC_SPI=y
+CONFIG_MMC_SDHI=y
CONFIG_MMC_DW=y
CONFIG_MMC_DW_EXYNOS=y
CONFIG_MMC_DW_K3=y
--
2.7.0.rc3.207.g0ac5344
^ permalink raw reply related [flat|nested] 23+ messages in thread