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.

Reply via email to