From: Andy Shevchenko <andy.shevchenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
To: Aaron Brice <aaron.brice-vSOi0k5x9wFWk0Htik3J/w@public.gmane.org>,
linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org,
Mark Brown <broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org>
Cc: Andy Shevchenko
<andy.shevchenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
Subject: [PATCH v1 0/2] spi: fsl-dspi: better approximation for baudrate
Date: Thu, 2 Apr 2015 12:11:45 +0300 [thread overview]
Message-ID: <1427965907-28125-1-git-send-email-andy.shevchenko@gmail.com> (raw)
This series makes better approximation when baud rate divisor parameters are
calculated.
The algorithm is represent as Python script here [1]. First parameter is
algorithm (0 - original, 1 - after patch 1/2, 2 - after patch 2/2, 3 - memory
vs. performance: use precalculated scales).
I played with let's say standard baud rates (which I used for Quark) and
separately run algorithms 0 and 1 for range 100-5000 baud. Input frequency is
64MHz. It seems my algo shows better results in all cases. Here is a diff for
'standard' baud rates
(speed_hz, DBR, i, BR, j, PBR, real baud rate, difference):
--- result0 2015-04-02 00:38:48.845761004 +0300
+++ result1 2015-04-02 00:38:48.913760154 +0300
@@ -5,7 +5,7 @@
25000000 0 0 2 0 2 16000000 9000000
20000000 0 0 2 0 2 16000000 4000000
16667000 0 0 2 0 2 16000000 667000
-13333000 0 0 2 0 2 16000000 2667000
+13333000 0 0 2 1 3 10666666 2666334
12500000 0 0 2 1 3 10666666 1833334
10000000 0 0 2 1 3 10666666 666666
8000000 0 1 4 0 2 8000000 0
@@ -17,36 +17,36 @@
3140500 0 1 4 2 5 3200000 59500
3125000 0 1 4 2 5 3200000 75000
3109500 0 1 4 2 5 3200000 90500
-2500000 0 1 4 3 7 2285714 214286
+2500000 0 3 8 1 3 2666666 166666
2000000 0 4 16 0 2 2000000 0
1563000 0 3 8 2 5 1600000 37000
-1250000 0 3 8 3 7 1142857 107143
+1250000 0 4 16 1 3 1333333 83333
1006000 0 5 32 0 2 1000000 6000
1000000 0 5 32 0 2 1000000 0
994000 0 5 32 0 2 1000000 6000
800000 0 4 16 2 5 800000 0
-781250 0 5 32 1 3 666666 114584
-625000 0 4 16 3 7 571428 53572
+781250 0 4 16 2 5 800000 18750
+625000 0 5 32 1 3 666666 41666
500000 0 6 64 0 2 500000 0
400000 0 5 32 2 5 400000 0
-390625 0 6 64 1 3 333333 57292
+390625 0 5 32 2 5 400000 9375
250000 0 7 128 0 2 250000 0
200000 0 6 64 2 5 200000 0
205313 0 6 64 2 5 200000 5313
-195313 0 7 128 1 3 166666 28647
-185313 0 7 128 1 3 166666 18647
+195313 0 6 64 2 5 200000 4687
+185313 0 6 64 2 5 200000 14687
125000 0 8 256 0 2 125000 0
100700 0 7 128 2 5 100000 700
100000 0 7 128 2 5 100000 0
-99300 0 8 256 1 3 83333 15967
+99300 0 7 128 2 5 100000 700
50000 0 8 256 2 5 50000 0
25000 0 9 512 2 5 25000 0
-10066 0 10 1024 3 7 8928 1138
-10016 0 10 1024 3 7 8928 1088
-9966 0 10 1024 3 7 8928 1038
-5065 0 11 2048 3 7 4464 601
-5040 0 11 2048 3 7 4464 576
-5015 0 11 2048 3 7 4464 551
+10066 0 11 2048 1 3 10416 350
+10016 0 11 2048 1 3 10416 400
+9966 0 11 2048 1 3 10416 450
+5065 0 12 4096 1 3 5208 143
+5040 0 12 4096 1 3 5208 168
+5015 0 12 4096 1 3 5208 193
1007 0 15 32768 0 2 976 31
1002 0 15 32768 0 2 976 26
997 0 15 32768 0 2 976 21
Patch 2 is RFC, since I don't know if all DSPI modules has that bit (at least
what I found on public for MC55xx) and how duty cycle is significant.
Not compiled. Not tested.
[1] http://pastebin.com/uy0CKc6b
Andy Shevchenko (2):
spi: fsl-dspi: increase precision of baud rate approximation
spi: fsl-dspi: use double baud rate in approximation
drivers/spi/spi-fsl-dspi.c | 50 +++++++++++++++++++++-------------------------
1 file changed, 23 insertions(+), 27 deletions(-)
--
1.9.3
--
To unsubscribe from this list: send the line "unsubscribe linux-spi" in
the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
next reply other threads:[~2015-04-02 9:11 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-04-02 9:11 Andy Shevchenko [this message]
[not found] ` <1427965907-28125-1-git-send-email-andy.shevchenko-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org>
2015-04-02 9:11 ` [PATCH v1 1/2] spi: fsl-dspi: increase precision of baud rate approximation Andy Shevchenko
2015-04-02 9:11 ` [PATCH v1 2/2][RFC] spi: fsl-dspi: use double baud rate in approximation Andy Shevchenko
2015-04-03 17:50 ` [PATCH v1 0/2] spi: fsl-dspi: better approximation for baudrate Aaron Brice
[not found] ` <551ED2F6.8020701-vSOi0k5x9wFWk0Htik3J/w@public.gmane.org>
2015-04-03 20:59 ` Andy Shevchenko
[not found] ` <CAHp75Vfa9b0s6rXMbUERbbGRowGp4bidKH1UZ4a8HAx23oSrLw-JsoAwUIsXosN+BqQ9rBEUg@public.gmane.org>
2015-04-03 22:44 ` Aaron Brice
[not found] ` <551F17B1.3010109-vSOi0k5x9wFWk0Htik3J/w@public.gmane.org>
2015-04-06 16:49 ` Mark Brown
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=1427965907-28125-1-git-send-email-andy.shevchenko@gmail.com \
--to=andy.shevchenko-re5jqeeqqe8avxtiumwx3w@public.gmane.org \
--cc=aaron.brice-vSOi0k5x9wFWk0Htik3J/w@public.gmane.org \
--cc=broonie-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org \
--cc=linux-spi-u79uwXL29TY76Z2rM5mHXA@public.gmane.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).