Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
IronPort-SDR: Uqm+6U+AXvlPr4ayZEa2D3NO9lXSc3GsgIexGuQYllymm33YchgRirKGNVHZbbWeVCOWSE8afX
 ZccmQMG6tpUg==
IronPort-SDR: ztFKjl0wJ42g1fDN9qPbeudBKkz3H6I9sjILWdWJ3tIvDoIZah7cTMUWxrZMJfCiXMaXdM0TRC
 8reOJD0KYxyg==
From:   muhammad.husaini.zulkifli@intel.com
To:     ulf.hansson@linaro.org, linux-mmc@vger.kernel.org,
        linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org
Cc:     adrian.hunter@intel.com, lakshmi.bai.raja.subramanian@intel.com,
        wan.ahmad.zainie.wan.mohamad@intel.com,
        muhammad.husaini.zulkifli@intel.com, david.e.box@linux.intel.com
Subject: [PATCH v2 1/1] mmc: sdhci-of-arasan: Fix clock registration fail for Keem Bay SOC
Date:   Wed, 18 Nov 2020 20:01:20 +0800
Message-Id: <20201118120120.24908-2-muhammad.husaini.zulkifli@intel.com>
In-Reply-To: <20201118120120.24908-1-muhammad.husaini.zulkifli@intel.com>
References: <20201118120120.24908-1-muhammad.husaini.zulkifli@intel.com>
Precedence: bulk

From: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com>

Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling")
introduces platform specific SDHCI clock operation. Each platform should
specify clock operations handlers.

Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling")
and Commit 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem Bay")
are merged about the same time. When doing rebasing during the upstream process,
Commit 16ada730a759 ("mmc: sdhci-of-arasan: Modify clock operations handling")
was not included. This will result in sd clock registration failure for
Keem Bay SOC Devices as clock operations handlers are not defined.

This patch declares the clock operation for Keem Bay SOC Devices.
Add clk_ops for SD, EMMC and SDIO operations.

Fixes: 36c6aadaae86 ("mmc: sdhci-of-arasan: Add support for Intel Keem Bay")

Signed-off-by: Muhammad Husaini Zulkifli <muhammad.husaini.zulkifli@intel.com>
Reviewed-by: Adrian Hunter <adrian.hunter@intel.com>
---
 drivers/mmc/host/sdhci-of-arasan.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/mmc/host/sdhci-of-arasan.c b/drivers/mmc/host/sdhci-of-arasan.c
index 829ccef87426..012d52e1abee 100644
--- a/drivers/mmc/host/sdhci-of-arasan.c
+++ b/drivers/mmc/host/sdhci-of-arasan.c
@@ -1199,16 +1199,19 @@ static struct sdhci_arasan_of_data sdhci_arasan_versal_data = {
 static struct sdhci_arasan_of_data intel_keembay_emmc_data = {
 	.soc_ctl_map = &intel_keembay_soc_ctl_map,
 	.pdata = &sdhci_keembay_emmc_pdata,
+	.clk_ops = &arasan_clk_ops,
 };
 
 static struct sdhci_arasan_of_data intel_keembay_sd_data = {
 	.soc_ctl_map = &intel_keembay_soc_ctl_map,
 	.pdata = &sdhci_keembay_sd_pdata,
+	.clk_ops = &arasan_clk_ops,
 };
 
 static struct sdhci_arasan_of_data intel_keembay_sdio_data = {
 	.soc_ctl_map = &intel_keembay_soc_ctl_map,
 	.pdata = &sdhci_keembay_sdio_pdata,
+	.clk_ops = &arasan_clk_ops,
 };
 
 static const struct of_device_id sdhci_arasan_of_match[] = {
-- 
2.17.1