Problem in lpc17xx_i2c.c

cancel
Showing results for 
Show  only  | Search instead for 
Did you mean: 

Problem in lpc17xx_i2c.c

251 Views
lpcware
NXP Employee
NXP Employee
Content originally posted in LPCWare by mfc on Sun Dec 05 02:56:55 MST 2010
I discovered a problem in the subject source code today -- I cannot find where to report it, so I'm putting it here in the hope that it might save someone some time.   The problem it causes is that the i2c2 clock runs twice as fast as specified, which could make some devices unreliable.

Mike Cowlishaw

Details:

[FONT=Courier New]In lpc17xx_i2c.c the I2C_SetClock function has an error; in the case of I2Cx == LPC_I2C2, the line[/FONT]
[INDENT][FONT=Courier New]  temp = CLKPWR_GetPCLK (CLKPWR_PCLKSEL_I2C1) / target_clock;[/FONT]
[/INDENT][FONT=Courier New]should be:[/FONT]
[INDENT][FONT=Courier New]  temp = CLKPWR_GetPCLK (CLKPWR_PCLKSEL_I2C2) / target_clock;[/FONT]

[/INDENT]
0 Kudos
2 Replies

239 Views
lpcware
NXP Employee
NXP Employee
Content originally posted in LPCWare by mfc on Mon Dec 06 12:36:14 MST 2010

Quote: NXP_USA
Hi Mike,

That is a good catch. This change will be rolled into an upcoming LPC17xx example package release.

-NXP



It is an easy fix :-).   Sounds good.  Where is the right place to get the current updates, release, etc?  It's hard to find (I have not found it yet!).  And, ideally, a way to be notified of new updates.

Thanks -- Mike
0 Kudos

239 Views
lpcware
NXP Employee
NXP Employee
Content originally posted in LPCWare by NXP_USA on Mon Dec 06 11:28:46 MST 2010
Hi Mike,

That is a good catch. This change will be rolled into an upcoming LPC17xx example package release.

-NXP
0 Kudos