Paul & Sabrina's cheap DIY 144v motor controller

[charlie_fd]

As I recall, we have had this debate a while back in this very thread.

It has been agreed, as far as I understand, to use the pre-defined names and macros from the gcc-avr include files allowing both human readability and portability between various controllers.

The refactored code (already in the svn) is now using these extensively and custom #define for most of the application specific constants. Why are we having this conversation again?

[JayC]

I'm not sure either. The refactored code does not have any of the items people were posting about? Are you guys looking at the latest code? It's using the _BV(bit) macro to create bit masks. It's a lot more readable.

To set a bit you use:
PORTB |= _BV(PB1); // PORTB = PORTB | (00000010)

To clear a bit you use:
PORTB &= ~_BV(PB1); // PORTB = PORTB & (11111101)

Finally, you can set multiple bits at once:
PORTB |= | _BV(PB7) | _BV(PB1) | _BV(PB0); // PORTB = PORTB | (10000000) | 0000010) | (00000001)

FWIW,
Jay

[dcb]

See, you find a bit of code out in the wild, it is well suited to its task, and efficient, and you can even recognize its lineage in some of its quirks. Choose:

A. appreciate it's character? Noting the interesting change in it's plumage that you don't normally see at the zoo, because the zookeeper painted all the zoo animals pink, because that made the zookeeper feel important.

B. get your paintbrush because you don't like what you see, even though it isn't doing anyone a damn bit of harm.

[JayC]

Um, are you just pontificating in general or are you directing those comments in any particular direction

[wjdennis]

Hey, guys. I think I must have missed the internal memo. Where do I go to see the latest code and control board layout?

Thanks.

Bill

[dcb]

Jay, it is In general, all the flibbertigibbeting drives me a little crazy

Oh, and here is the datasheet for the processor if any would-be development types would like to stop being spoon fed

[dcb]

wj, try http://ecomoder.com/wiki for a starting point, or that link in Pauls signature. Don't know how "latest" it is though.

[JayC]

Originally Posted by wjdennis

Hey, guys. I think I must have missed the internal memo. Where do I go to see the latest code and control board layout?

Thanks.

Bill

In the Wiki. No worries, I had to go find it a few times before I realized it's at the top of the page (wiki that is) and that there are a handful of wiki projects.

Direct link: software SVN. Going by age HighVoltageControllerRefactoredISRReorganized.c

Jay

[esoneson]

I am over joyed at all those who have been making great suggestions concerning the perfection of the controller code. Thanks to all who have contributed!!

I am one of those old f*arts who has lived through decades of debates concerning coding styles and best practices.

I suggest one Rule for the micro code. Let Paul assimilate all the proper FUNCTIONAL changes to the point where he and the testers are satisfied that the controller meets the functional needs of Beta Release 1.0 (or you pick a name).
Once that is done, anyone is free to take the code and run it through whatever "pretty" filter they want to make it look like they want. But for now, leave coding style out of the discussion to make it easier for Paul.

Paul is doing a great job with this project at the same time is taking care of his family in this tough economic time. He deserves all the help we can give, but lets focus on function for now and get this thing running.

Ok, I'll get off my soap-box now and think about more pleasant memories of coding and product release successes. OOOooooooohhhhhhhhhmmmmmm.
OOOoooooooohhhhhhhhhhhmmmmmmmmm. mantra....mantra.....mantra

Eric

[wolfraven]

W000t! Go Paul GO! You can do it!

Also, I've seen crazier looking code in a great deal of very popular computer games, with absolutely TERRIBLE commenting...Paul's work is the only bit that's managed to make any sense without looking every little bit up and swapping between all 56 files of includes/c/etc that the thing has just to do a teeny-weenie mod or find out how to change a couple of settings...Pfff...the big-shots make it look crazier than a lot of code I've looked at...keep up the good work!