Re: [PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-13 Thread Lee Jones
> >I'm not asking for in-depth analysis, just an overview. > > > >What's wrong with the default config? > >Why is the signal quality bad and what makes it bad? > >What did the old magic numbers do? > >How will the configuration differ if I applied your patch? I'm not sure I'm getting the answers I

Re: [PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-10 Thread wwang
于 2013年09月10日 19:08, Lee Jones 写道: I'm not asking for in-depth analysis, just an overview. What's wrong with the default config? Why is the signal quality bad and what makes it bad? What did the old magic numbers do? How will the configuration differ if I applied your patch? It is a little dif

Re: [PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-10 Thread Lee Jones
On Tue, 10 Sep 2013, wwang wrote: > 于 2013年09月10日 17:28, Lee Jones 写道: > >I would like some more information in the commit log though. You're > >making a lot of configuration changes here and due to the > >incomprehensible 'magic numbers' used previously, it's impossible to > >know what you're cha

Re: [PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-10 Thread Lee Jones
> From: Wei WANG > > In some platforms, specially Thinkpad series, rts5249 won't be > initialized properly. So we need adjust some phy parameters to > improve the compatibility issue. The code looks so much more readable now, thanks for that. I would like some more information in the commit log

Re: [PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-10 Thread wwang
于 2013年09月10日 17:28, Lee Jones 写道: I would like some more information in the commit log though. You're making a lot of configuration changes here and due to the incomprehensible 'magic numbers' used previously, it's impossible to know what you're changing by just reading the code. Why won't the

[PATCH v3] mfd: rtsx: Modify rts5249_optimize_phy

2013-09-10 Thread wei_wang
From: Wei WANG In some platforms, specially Thinkpad series, rts5249 won't be initialized properly. So we need adjust some phy parameters to improve the compatibility issue. Signed-off-by: Wei WANG --- drivers/mfd/rts5249.c| 48 -- include/linux/mf