Gitweb:     
http://git.kernel.org/git/?p=linux/kernel/git/torvalds/linux-2.6.git;a=commit;h=d4a96b53125c3d31266c05f2a8432d956dd26141
Commit:     d4a96b53125c3d31266c05f2a8432d956dd26141
Parent:     e5d809d774fc8aa76899bde3235afb046728feed
Author:     Krzysztof Helt <[EMAIL PROTECTED]>
AuthorDate: Tue May 8 00:39:33 2007 -0700
Committer:  Linus Torvalds <[EMAIL PROTECTED]>
CommitDate: Tue May 8 11:15:32 2007 -0700

    pm2fb: pixclock setting restriction
    
    This patch adds restrictions to calculations of m, n and p factors
    which sets the pixclock for the Permedia 2V. I found during tests
    that synchronization is unstable if m (divisor) is bigger than
    half of the n (numerator). The patch disallows such settings
    combination.
    
    Signed-off-by: Krzysztof Helt <[EMAIL PROTECTED]>
    Signed-off-by: Antonino Daplas <[EMAIL PROTECTED]>
    Signed-off-by: Andrew Morton <[EMAIL PROTECTED]>
    Signed-off-by: Linus Torvalds <[EMAIL PROTECTED]>
---
 drivers/video/pm2fb.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/video/pm2fb.c b/drivers/video/pm2fb.c
index 6f634e3..2a8ba60 100644
--- a/drivers/video/pm2fb.c
+++ b/drivers/video/pm2fb.c
@@ -302,10 +302,10 @@ static void pm2v_mnp(u32 clk, unsigned char* mm, unsigned 
char* nn,
        s32 delta = 1000;
 
        *mm = *nn = *pp = 0;
-       for (n = 1; n; n++) {
-               for ( m = 1; m; m++) {
+       for ( m = 1; m < 128; m++) {
+               for (n = 2 * m + 1; n; n++) {
                        for ( p = 0; p < 2; p++) {
-                               f = PM2_REFERENCE_CLOCK * n / (m * (1 << (p + 
1)));
+                               f = ( PM2_REFERENCE_CLOCK >> ( p + 1 )) * n / m;
                                if ( clk > f - delta && clk < f + delta ) {
                                        delta = ( clk > f ) ? clk - f : f - clk;
                                        *mm=m;
-
To unsubscribe from this list: send the line "unsubscribe git-commits-head" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to