Skip to content

Commit

Permalink
npu2-opencapi: ODL should be in reset when enabled
Browse files Browse the repository at this point in the history
We haven't hit any problem so far, but from the ODL designer, the ODL
should be in reset when it is enabled.

The ODL remains in reset until we start a fundamental reset to
initiate link training. We still assert and deassert the ODL reset
signal as part of the normal procedure just before training the
link. Asserting is therefore useless at boot, since the ODL is already
in reset, but we keep it as it's only a scom write and it's needed
when we reset/retrain from the OS.

Signed-off-by: Frederic Barrat <fbarrat@linux.ibm.com>
Reviewed-by: Andrew Donnellan <andrew.donnellan@au1.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.ibm.com>
  • Loading branch information
fbarrat authored and stewartsmith committed Mar 14, 2019
1 parent ce9dd46 commit e4efac7
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions hw/npu2-opencapi.c
Original file line number Diff line number Diff line change
Expand Up @@ -308,6 +308,12 @@ static void enable_odl_phy_mux(uint32_t gcid, int index)
assert(false);
}

/*
* ODL must be in reset when enabling.
* It stays in reset until the link is trained
*/
assert_odl_reset(gcid, index);

/* PowerBus OLL PHY Training Config Register */
xscom_read(gcid, phy_config_scom, &reg);

Expand Down

0 comments on commit e4efac7

Please sign in to comment.