Commit 58c6990c authored by Shawn Lin's avatar Shawn Lin Committed by Bjorn Helgaas

PCI: rockchip: Improve the deassert sequence of four reset pins

Per TRM, we need to deassert the four reset pins simultaneously.  Currently
the reset framework doesn't support that so we did it one by one.  It seems
no side effect found but it does impact the state machine of controller, so
sometimes the change speed bit is not set when sending training sequence
from recover state.  After the silicon RTL review from SoC guys, we don't
need to do the sequence recommended by TRM, and could just move the
deassert of mgmt_sticky_rst to the first place.
Signed-off-by: default avatarShawn Lin <shawn.lin@rock-chips.com>
Signed-off-by: default avatarBjorn Helgaas <bhelgaas@google.com>
parent 277743ef
...@@ -449,21 +449,25 @@ static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip) ...@@ -449,21 +449,25 @@ static int rockchip_pcie_init_port(struct rockchip_pcie *rockchip)
return err; return err;
} }
err = reset_control_deassert(rockchip->core_rst); /*
* Please don't reorder the deassert sequence of the following
* four reset pins.
*/
err = reset_control_deassert(rockchip->mgmt_sticky_rst);
if (err) { if (err) {
dev_err(dev, "deassert core_rst err %d\n", err); dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err);
return err; return err;
} }
err = reset_control_deassert(rockchip->mgmt_rst); err = reset_control_deassert(rockchip->core_rst);
if (err) { if (err) {
dev_err(dev, "deassert mgmt_rst err %d\n", err); dev_err(dev, "deassert core_rst err %d\n", err);
return err; return err;
} }
err = reset_control_deassert(rockchip->mgmt_sticky_rst); err = reset_control_deassert(rockchip->mgmt_rst);
if (err) { if (err) {
dev_err(dev, "deassert mgmt_sticky_rst err %d\n", err); dev_err(dev, "deassert mgmt_rst err %d\n", err);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment