[PATCH v2 28/34] staging: mt7621-mmc: Remove unnecessary setting of dev.platform_data

Christian Lütke-Stetzkamp christian at lkamp.de
Wed Apr 4 20:15:37 UTC 2018


In the probe function, pdev->dev.platform_data is set to &msdc0_hw,
but the only get access to that is also in the probe function. So that
assignment is removed. The msdc0_hw structure is still available to the
driver as part of the msdc_host structure.

Signed-off-by: Christian Lütke-Stetzkamp <christian at lkamp.de>
---
 drivers/staging/mt7621-mmc/sd.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/drivers/staging/mt7621-mmc/sd.c b/drivers/staging/mt7621-mmc/sd.c
index 9e507cd2506f..e8846b7c545b 100644
--- a/drivers/staging/mt7621-mmc/sd.c
+++ b/drivers/staging/mt7621-mmc/sd.c
@@ -2701,7 +2701,7 @@ static int msdc_drv_probe(struct platform_device *pdev)
 	struct msdc_hw *hw;
 	int ret, irq;
 
-	pdev->dev.platform_data = &msdc0_hw;
+	hw = &msdc0_hw;
 
 	if (of_property_read_bool(pdev->dev.of_node, "mtk,wp-en"))
 		msdc0_hw.flags |= MSDC_WP_PIN_EN;
@@ -2711,7 +2711,6 @@ static int msdc_drv_probe(struct platform_device *pdev)
 	if (!mmc)
 		return -ENOMEM;
 
-	hw   = (struct msdc_hw *)pdev->dev.platform_data;
 	irq  = platform_get_irq(pdev, 0);
 
 	//BUG_ON((!hw) || (!mem) || (irq < 0)); /* --- by chhung */
-- 
2.16.1



More information about the devel mailing list