diff mbox

[2/3] mmc: tmio: move tmio_mmc_hw_reset()

Message ID 20180606232252.31583-3-niklas.soderlund+renesas@ragnatech.se (mailing list archive)
State New, archived
Headers show

Commit Message

Niklas Söderlund June 6, 2018, 11:22 p.m. UTC 
Avoid having to use forward declaration when in a later patch fixing the
reset operation by moving tmio_mmc_hw_reset() earlier in the source
file. No functional change in this change.

Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>
---
 drivers/mmc/host/tmio_mmc_core.c | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

Comments

Wolfram Sang June 9, 2018, 6:17 p.m. UTC | #1 
On Thu, Jun 07, 2018 at 01:22:51AM +0200, Niklas Söderlund wrote:
> Avoid having to use forward declaration when in a later patch fixing the
> reset operation by moving tmio_mmc_hw_reset() earlier in the source
> file. No functional change in this change.
> 
> Signed-off-by: Niklas Söderlund <niklas.soderlund+renesas@ragnatech.se>

I'd merge patches 2+3, but I'll leave that to Ulf...
diff mbox

Patch

diff --git a/drivers/mmc/host/tmio_mmc_core.c b/drivers/mmc/host/tmio_mmc_core.c
index 2ede81476daf76ce..9f5793407552a23a 100644
--- a/drivers/mmc/host/tmio_mmc_core.c
+++ b/drivers/mmc/host/tmio_mmc_core.c
@@ -243,6 +243,14 @@  static void tmio_mmc_reset(struct tmio_mmc_host *host)
 
 }
 
+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);
+}
+
 static void tmio_mmc_reset_work(struct work_struct *work)
 {
 	struct tmio_mmc_host *host = container_of(work, struct tmio_mmc_host,
@@ -769,14 +777,6 @@  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);
-}
-
 static int tmio_mmc_execute_tuning(struct mmc_host *mmc, u32 opcode)
 {
 	struct tmio_mmc_host *host = mmc_priv(mmc);