[#4231] get_vco defined in two places...

Document created by Aaronwu Employee on Aug 29, 2013
Version 1Show Document
  • View in full screen mode

[#4231] get_vco defined in two places...

Submitted By: Robin Getz

Open Date

2008-07-10 10:06:41     Close Date

2008-07-10 10:57:12

Priority:

High     Assignee:

Michael Hennerich

Status:

Closed     Fixed In Release:

N/A

Found In Release:

N/A     Release:

2008R1 branch

Category:

N/A     Board:

N/A

Processor:

N/A     Silicon Revision:

Is this bug repeatable?:

Yes     Resolution:

Fixed

Uboot version or rev.:

    Toolchain version or rev.:

all

App binary format:

N/A     

Summary: get_vco defined in two places...

Details:

 

./drivers/char/bfin_dpmc.c:

 

/* Returns VCO in Hz */

int get_vco(void)

{

        return ((CONFIG_CLKIN_HZ) * ((*(volatile unsigned short *)PLL_CTL >> 9)& 0x3F));

}

 

./arch/blackfin/kernel/setup.c:

static u_long get_vco(void)

{

        u_long msel;

        u_long vco;

 

        msel = (bfin_read_PLL_CTL() >> 9) & 0x3F;

        if (0 == msel)

                msel = 64;

 

        vco = CONFIG_CLKIN_HZ;

        vco >>= (1 & bfin_read_PLL_CTL());      /* DF bit */

        vco = msel * vco;

        return vco;

}

 

Follow-ups

 

--- Michael Hennerich                                        2008-07-10 10:23:26

Fixed - update your svn and

try again :-)

 

-Michael

 

--- Robin Getz                                               2008-07-10 10:57:12

Thanks

 

 

--- Robin Getz                                               2008-07-11 11:06:39

Fixed with this checkin:

 

  blackfin.uclinux.org/gf/project/linux-kernel/scmsvn/?action=browse&path=%2F&view=rev&revision=4951

 

 

 

    Files

    Changes

    Commits

    Dependencies

    Duplicates

    Associations

    Tags

 

File Name     File Type     File Size     Posted By

No Files Were Found

Attachments

    Outcomes