Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Grant Grundler
Hi Chris, On Wed, Mar 27, 2013 at 10:58 AM, Chris Ball wrote: > Hi Grant, ... > Please use the following (standard) syntax in the commit message: > > Commit e419990b5e8 ("mmc: dw_mmc: correct the calculation for CLKDIV") > fixed a bug in CLKDIV computation. [..] Ok - I didn't know that was "sta

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Grant Grundler
On Wed, Mar 27, 2013 at 8:07 AM, Doug Anderson wrote: > Grant, > > Thanks for posting! See below... thanks for reviewing/feedback! :) > On Tue, Mar 26, 2013 at 3:50 PM, Grant Grundler wrote: >> Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. >> But when debugging a related

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Chris Ball
Hi Grant, On Wed, Mar 27 2013, Grant Grundler wrote: > On Wed, Mar 27, 2013 at 5:25 AM, Seungwon Jeon wrote: >> On Wednesday, March 27, 2013, Grant Grundler wrote: >>> Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. >> For easily identifying, it would be good to point the com

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Grant Grundler
On Wed, Mar 27, 2013 at 5:25 AM, Seungwon Jeon wrote: > On Wednesday, March 27, 2013, Grant Grundler wrote: >> Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. > For easily identifying, it would be good to point the commit id and subject. commit id will be different for differ

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Grant Grundler
Sorry - please ignore the previous version. Did not include a copyright (implied to be mine...but it's not) nor license. Same thing but with proper attribution. cheers, grant On Tue, Mar 26, 2013 at 3:53 PM, Grant Grundler wrote: > I've attached the test program I wrote to compare the different

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Doug Anderson
Grant, Thanks for posting! See below... On Tue, Mar 26, 2013 at 3:50 PM, Grant Grundler wrote: > Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. > But when debugging a related issue (http://crbug.com/221828) I found > the code unreadable. This rewrite simplifies the comput

RE: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-27 Thread Seungwon Jeon
On Wednesday, March 27, 2013, Grant Grundler wrote: > Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. For easily identifying, it would be good to point the commit id and subject. > But when debugging a related issue (http://crbug.com/221828) I found It is not easy to catch up i

Re: [PATCH] mmc: dw_mmc: rewrite CLKDIV computation

2013-03-26 Thread Grant Grundler
I've attached the test program I wrote to compare the different flavors of CLKDIV computation: old (3.4 kernel), current upstream, and my rewrite. thanks grant On Tue, Mar 26, 2013 at 3:50 PM, Grant Grundler wrote: > Last year Seungwon Jeon (Samsung) fixed a bug in CLKDIV computation. > But when