We can use the wrapper API here to save some lines and remove the need
for the 'base' and 'res' local variable.
Suggested-by: Bjorn Andersson <bjorn.anders...@linaro.org>
Cc: Jeykumar Sankaran <jsa...@codeaurora.org>
Cc: Chandan Uddaraju <chand...@codeaurora.org>
Cc: Vara Reddy <va...@codeaurora.org>
Cc: Tanmay Shah <tan...@codeaurora.org>
Cc: Bjorn Andersson <bjorn.anders...@linaro.org>
Cc: Manu Gautam <mgau...@codeaurora.org>
Cc: Sandeep Maheswaram <s...@codeaurora.org>
Cc: Douglas Anderson <diand...@chromium.org>
Cc: Sean Paul <seanp...@chromium.org>
Cc: Jonathan Marek <jonat...@marek.ca>
Cc: Dmitry Baryshkov <dmitry.barysh...@linaro.org>
Cc: Rob Clark <robdcl...@chromium.org>
Signed-off-by: Stephen Boyd <swb...@chromium.org>
---
drivers/phy/qualcomm/phy-qcom-qmp.c | 11 +++--------
1 file changed, 3 insertions(+), 8 deletions(-)
diff --git a/drivers/phy/qualcomm/phy-qcom-qmp.c
b/drivers/phy/qualcomm/phy-qcom-qmp.c
index 215abd179e74..ecfcc97277f9 100644
--- a/drivers/phy/qualcomm/phy-qcom-qmp.c
+++ b/drivers/phy/qualcomm/phy-qcom-qmp.c
@@ -3235,10 +3235,8 @@ static int qcom_qmp_phy_probe(struct platform_device
*pdev)
{
struct qcom_qmp *qmp;
struct device *dev = &pdev->dev;
- struct resource *res;
struct device_node *child;
struct phy_provider *phy_provider;
- void __iomem *base;
void __iomem *serdes;
const struct qmp_phy_cfg *cfg;
int num, id;
@@ -3256,13 +3254,10 @@ static int qcom_qmp_phy_probe(struct platform_device
*pdev)
if (!cfg)
return -EINVAL;
- res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
- base = devm_ioremap_resource(dev, res);
- if (IS_ERR(base))
- return PTR_ERR(base);
-
/* per PHY serdes; usually located at base address */
- serdes = base;
+ serdes = devm_platform_ioremap_resource(pdev, 0);
+ if (IS_ERR(serdes))
+ return PTR_ERR(serdes);
/* per PHY dp_com; if PHY has dp_com control block */
if (cfg->has_phy_dp_com_ctrl) {
--
Sent by a computer, using git, on the internet
