RE: [PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-12 Thread Jongsung Kim
Sergei Shtylyov : >> -.suspend= genphy_suspend, >> -.resume = genphy_resume, >> +.suspend= _suspend, >> +.resume = _resume, > >Contrariwise, you should have dropped & from the other functions. >It's completely

RE: [PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-12 Thread Jongsung Kim
Sergei Shtylyov sergei.shtyl...@cogentembedded.com : -.suspend= genphy_suspend, -.resume = genphy_resume, +.suspend= genphy_suspend, +.resume = genphy_resume, Contrariwise, you should have dropped from the

Re: [PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-10 Thread Sergei Shtylyov
Hello. On 10-05-2013 11:29, Jongsung Kim wrote: This patch cleans up the drivers code by: - using a consistent way to reference functions - removing unused macro-definitions - removing unnecessary new-lines - making ack_interrupt functions shorter. Signed-off-by: Jongsung Kim ---

[PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-10 Thread Jongsung Kim
This patch cleans up the drivers code by: - using a consistent way to reference functions - removing unused macro-definitions - removing unnecessary new-lines - making ack_interrupt functions shorter. Signed-off-by: Jongsung Kim --- drivers/net/phy/realtek.c | 29

[PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-10 Thread Jongsung Kim
This patch cleans up the drivers code by: - using a consistent way to reference functions - removing unused macro-definitions - removing unnecessary new-lines - making ack_interrupt functions shorter. Signed-off-by: Jongsung Kim neidhard@lge.com --- drivers/net/phy/realtek.c | 29

Re: [PATCH v2 4/4] net: phy: realtek: cleanup code

2013-05-10 Thread Sergei Shtylyov
Hello. On 10-05-2013 11:29, Jongsung Kim wrote: This patch cleans up the drivers code by: - using a consistent way to reference functions - removing unused macro-definitions - removing unnecessary new-lines - making ack_interrupt functions shorter. Signed-off-by: Jongsung Kim