[U-Boot-Users] [PATCH] mpc83xx: UEC: add support for Broadcom BCM5481 Gigabit PHY

[Kim Phillips]

On Tue, 08 Jan 2008 13:04:50 -0500
Ben Warren <biggerbadderben at gmail.com> wrote:

> Kim Phillips wrote:
> > On Tue, 25 Dec 2007 20:01:26 +0300
> > Anton Vorontsov <avorontsov at ru.mvista.com> wrote:
<snip>
> >> There are no openly available specifications for that PHY.
> >>
> >> Signed-off-by: Anton Vorontsov <avorontsov at ru.mvista.com>
> >>     
> >
> > Also, shouldn't networking patches go through Ben Warren?
> >   
> Sorry Kim, but I didn't take it because it was in 83xx code. These 
> multi-jurisdictional patches are pretty common, and there doesn't seem 
> to be a well-defined way of handling them. In this case, it's pretty 
> clearly network code, but I'd like to at least have your 'acked-by' in 
> the paper trail. Does this sound like the right way to do it, or am I 
> making things too bureaucratic?

I could easily be accused of that myself ;)

I could be wrong, but I thought gbit_read_status() could have been
merged with genmii_read_status(), and the aneg wait code go into
genmii_update_link() (J.Tjerlund and I were working on such a patch
before the holidays) and the CONFIG_MPC8360ERDK ifdef be eliminated.

Kim