[ell-i-developers] Re: PWM working for TIM2

  • From: Pekka Nikander <pekka.nikander@xxxxxx>
  • To: ell-i-developers@xxxxxxxxxxxxx
  • Date: Mon, 27 Jan 2014 21:21:04 +0200

> Finally the PWM ouptput for the TIM2 is working. The main problem was that 
> TIM2 is a timer with some 32bit registers.


> Generates this particular assembly:
>  80002a2:    8022          strh    r2, [r4, #0]
> That slightly changes the assembly, the most important part is this:
> 80002a2:    6022          str    r2, [r4, #0]

Good finding!

> I am not sure if the c code is the correct one, as I know that typecast is 
> not advisable.

No, it is not, let's work together a better solution.  And let's consider the 
other things in your mail only later.

> BWT, now I understand a little why the STM library takes so much space, 
> seeing the assembly opened my eyes :)


> What would be the next step??

Clean up your change, commit it, and push it to github.  You may want to push 
it to some temp branch, but YMMV.  You can also create a pull request already, 
if you want to.  If you want to commit it to a temp branch, the easiest way is 
something like
  git push origin master:temp-branch-name  


  git push origin master:tentative-fix-16bit-init

Anyway, please point me to the right file in github, either via a pull request 
or a with direct link to the right *line*.  (Note that you can copy a link to a 
particular line by clicking the line number and then copying the link from the 
URL box.)

I really need to see the whole code in order to think about how to get it 
right, just a few lines of code out of the context is not enough.  More 
generally, I think it is better to commit a temp branch to github and point to 
it, rather than copy & paste code to e-mail.  You can then leave the temp 
branches there in github, if you wish, they don't harm anybody but document the 


Other related posts: