Hello,
I am working on an 8360e platform, bringing up a RTBI interface on UCC1
connected as a FIXED_PHY_PORT (smi addr 8) to a switch chip port.
Question 1a
===========
In uec.c::uec_startup, there is this logic
if (enet_interface == ENET_1000_TBI ||
enet_interface == ENET_1000_RTBI) {
utbipar |= (uec_info->phy_address + uec_info->uf_info.ucc_num)
<< UTBIPAR_PHY_ADDRESS_SHIFT;
} else {
utbipar |= (0x10 + uec_info->uf_info.ucc_num)
<< UTBIPAR_PHY_ADDRESS_SHIFT;
}
With UCC1 (devnum = 0) I am hitting a bug here I think, because I am using UCC1
(devnum 0) my utbipar is set to (8+0)
which is the same as my fixed-port phy. The tbi controller (see section 30.11.7
on the 8360 reference manual) returns 0x0000
on its ID registers (register offset 2 and 3). This prevents the fixed-phy-port
detection to finish and I got detected as a generic MII instead.
If the intent here is to make utibipar and the actual phy address different,
can the logic be:
utbipar |= (uec_info->phy_address + uec_info->uf_info.ucc_num + 1) <<
UTBIPAR_PHY_ADDRESS_SHIFT;
to handle the case when ucc_num = 0 -----------------------------^
Question 1b
===========
on the else cause, when the interface is NOT (R)TBI, why is utbipar set at 0x10
offset from the UCC_num ?
Since the tbi controller is internal per UEC and responds in-precedent to any
external PHY, keeping it at 0x1F (the default)
will make sure it stays out of everyone's way and not impose additional PHY
address selection restrictions (other than NOT being 0x1F)
on the HW design, no?
Question 2
==========
I got my RTBI interface working after modifying the TBICON register to match
our hardware, now I am looking for a way to add support for it.
Is this a good approach?:
in uccf.h, make:
=================================
typedef struct ucc_tbireg {
unsigned short cr,
unsigned short sr,
...
} ucc_tbireg_t;
=================================
then add it as a member for typedef struct uec_info in uec.h
=================================
then fill it in during the initialization of the static UEC instantiation:
static uec_info_t eth2_uec_info = {
#if (CONFIG_SYS_UEC2_INTERFACE_MODE == ENET_1000_RTBI) ||
(CONFIG_SYS_UEC2_INTERFACE_MODE == ENET_1000_TBI)
.tbi_info = {
.cr = CONFIG_SYS_UEC2_TBI_CR,
.ana = CONFIG_SYS_UEC2_TBI_ANA,
....
},
#endif
=================================
Then process it during uec_startup, with values of "0x0" meaning do-not-modify
(we can even add bitmasks to mpc83xx.h a'la DDRCDR or so).
=================================
If any of these have a hope of being mainlined, I can work out a patch (I can
only test for RTBI though, since that is the only connection I got)
but the TBI registers should apply regardless.
Thanks very much for everyone's time!
- Richard.
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
