Redundant code in configure_eptx(self):
in
def configure_eptx(self):
...
...
#turn on 320 MHz clocks
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK3FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK5FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK6FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK7FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK15FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK16FREQ", 0x4)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK3DRIVESTRENGTH", 0x3)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK5DRIVESTRENGTH", 0x3)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK6DRIVESTRENGTH", 0x3)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK7DRIVESTRENGTH", 0x3)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK15DRIVESTRENGTH", 0x3)
self.wr_reg("LPGBT.RWF.EPORTCLK.EPCLK16DRIVESTRENGTH", 0x3)
There is a block to configure EPCLKs..
This is redundant with the functionality in configure_clocks and in fact configures them incorrectly.
I think this is legacy code copied from ME0 :)
It should be removed.