> -----Original Message----- > From: Kumar Gala [mailto:[EMAIL PROTECTED] > Sent: Thursday, July 06, 2006 9:45 PM > To: Li Yang-r58472 > Cc: Andrew Morton; [EMAIL PROTECTED]; netdev@vger.kernel.org; > [EMAIL PROTECTED] > Subject: Re: [PATCH 1/3] Freescale QE UCC gigabit ethernet driver > > Nack. Here are some high level things that should be addressed: > * There is a ton of debug code that should probably be stripped out, > such as dump_regs, etc..
The reason I left these debug code in is that, QE is very flexible. User probably needs to fine tune parameters for their own application. It will be good to give them some clue doing that. It's fine to remove them, if you do think they are too verbose. > * The phy support should use the phylib instead I'll do this when I do get some time. ;) > * convert from being a platform_device to of_device I'm not up against moving to use of_device. But why are we putting extra effort in closing the door to backward compatibility with ppc arch, and doesn't provide any new feature. ;) > > - kumar > - To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html
