[#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