[riot-notifications] [RIOT] fixed hwtimer on msp430 (#175)

Milan Babel notifications at github.com
Sat Aug 24 00:04:29 CEST 2013


> @@ -61,7 +61,7 @@ void TA0_unset(short timer)
>  
>  unsigned long hwtimer_arch_now()

Indeed this is a good idea, but I would prefer to merge this discussed request first.

This code is only related to the msp430 where no issues are expected, since all unsingned long have to be uint32_t etc.
Also no such variables are included in this request, only in the patched file.

> Am 23.08.2013 um 22:35 schrieb Christian Mehlis <notifications at github.com>:
> 
> In cpu/msp430-common/hwtimer_cpu.c:
> 
> > @@ -61,7 +61,7 @@ void TA0_unset(short timer)
> >  
> >  unsigned long hwtimer_arch_now()
> what about replacing all long / unsigned and so on types with the clear: uintXX_t ? I prefer this way to make sure that the code is portable an running on 16 and 32 bits.
> 
> This file is not that big....
> 
>> Reply to this email directly or view it on GitHub.

---
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/175/files#r5962412
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20130823/74315bf6/attachment.html>


More information about the notifications mailing list