From: Yisen Zhuang <yisen.zhu...@huawei.com> Date: Mon, 27 Jun 2016 17:54:15 +0800
> @@ -361,9 +371,10 @@ static int hns_mdio_reset(struct mii_bus *bus) > return -ENODEV; > } > > + sc_reg = &mdio_dev->sc_reg; > /* 1. reset req, and read reset st check */ > - ret = mdio_sc_cfg_reg_write(mdio_dev, MDIO_SC_RESET_REQ, 0x1, > - MDIO_SC_RESET_ST, 0x1, > + ret = mdio_sc_cfg_reg_write(mdio_dev, sc_reg->mdio_reset_req, 0x1, > + sc_reg->mdio_reset_st, 0x1, > MDIO_CHECK_SET_ST); > if (ret) { > dev_err(&bus->dev, "MDIO reset fail\n"); What in the world are you doing to the indentation here? Please read your patches before actually sending them, such things will be quite obvious by simple visual inspection.