Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2014-02-21 Thread Tom Rini
On Mon, Oct 28, 2013 at 11:19:53PM +, Måns Rullgård wrote: > Wolfgang Denk writes: > > > Dear Matt, > > > > I hope you are the right person to address this to - if not, please > > help to redirect to the current responsible developer. > > > > Function pll_sigma_delta_val() in arch/arm/cpu/ar

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2014-01-28 Thread Matt Porter
On Tue, Oct 29, 2013 at 08:23:07AM -0400, Tom Rini wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 10/29/2013 06:48 AM, Wolfgang Denk wrote: > > Dear Måns Rullgård, > > > > In message you wrote: > >> > Something like this should be equivalent. That said, it > looks su

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-29 Thread Måns Rullgård
Wolfgang Denk writes: > Dear Måns Rullgård, > > In message you wrote: >> >> >> Something like this should be equivalent. That said, it looks >> >> suspiciously like it's meant to simply do a division and round up. If >> >> that is the case, +225 should be +249. It probably makes no difference

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-29 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/29/2013 06:48 AM, Wolfgang Denk wrote: > Dear Måns Rullgård, > > In message you wrote: >> Something like this should be equivalent. That said, it looks suspiciously like it's meant to simply do a division and round up. If that

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-29 Thread Wolfgang Denk
Dear Måns Rullgård, In message you wrote: > > >> Something like this should be equivalent. That said, it looks > >> suspiciously like it's meant to simply do a division and round up. If > >> that is the case, +225 should be +249. It probably makes no difference > >> for the values actually enc

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-28 Thread Måns Rullgård
Wolfgang Denk writes: > Dear Måns Rullgård, > > In message you wrote: >> >> Something like this should be equivalent. That said, it looks >> suspiciously like it's meant to simply do a division and round up. If >> that is the case, +225 should be +249. It probably makes no difference >> for

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-28 Thread Wolfgang Denk
Dear Måns Rullgård, In message you wrote: > > Something like this should be equivalent. That said, it looks > suspiciously like it's meant to simply do a division and round up. If > that is the case, +225 should be +249. It probably makes no difference > for the values actually encountered.

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-28 Thread Måns Rullgård
Wolfgang Denk writes: > Dear Matt, > > I hope you are the right person to address this to - if not, please > help to redirect to the current responsible developer. > > Function pll_sigma_delta_val() in arch/arm/cpu/armv7/am33xx/clock_ti814x.c > incorrectly uses "float" data, which results in FP o

Re: [U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-28 Thread Tom Rini
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 10/27/2013 05:11 PM, Wolfgang Denk wrote: > Dear Matt, > > I hope you are the right person to address this to - if not, > please help to redirect to the current responsible developer. > > Function pll_sigma_delta_val() in > arch/arm/cpu/armv7/am33

[U-Boot] Illegal use of FP ops in clock_ti814x.c

2013-10-27 Thread Wolfgang Denk
Dear Matt, I hope you are the right person to address this to - if not, please help to redirect to the current responsible developer. Function pll_sigma_delta_val() in arch/arm/cpu/armv7/am33xx/clock_ti814x.c incorrectly uses "float" data, which results in FP operations which are not permitted in