Hi : Ok, I will remove the Mixed case variables. Thanks.
Jesse ----- Original Message ----- From: "Francois Romieu" <[EMAIL PROTECTED]> To: "Jesse Huang" <[EMAIL PROTECTED]> Cc: <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]>; <[EMAIL PROTECTED]>; <linux-kernel@vger.kernel.org>; <netdev@vger.kernel.org>; <[EMAIL PROTECTED]> Sent: Saturday, August 19, 2006 6:17 PM Subject: Re: [PATCH 5/7] ip1000: Modify coding style of ipg_config_autoneg() Jesse Huang <[EMAIL PROTECTED]> : > From: Jesse Huang <[EMAIL PROTECTED]> > > This is only coding style modify for ipg_config_autoneg(). Thanks for the > suggestion form Francois. > > Change Logs: > Modify coding style of ipg_config_autoneg() > > --- > > drivers/net/ipg.c | 17 ++++++++++------- > 1 files changed, 10 insertions(+), 7 deletions(-) > > 737498ca620437d8179e21be4d5220333066cbbd > diff --git a/drivers/net/ipg.c b/drivers/net/ipg.c > index f859107..be96f93 100644 > --- a/drivers/net/ipg.c > +++ b/drivers/net/ipg.c > @@ -491,11 +491,13 @@ static int ipg_config_autoneg(struct net > int fullduplex; > int txflowcontrol; > int rxflowcontrol; > + long MacCtrlValue; Mixed case variables are not exactly welcome. -- Ueimor - 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