Code Review Requests

FreeEMS topics that aren't specific to hardware development or firmware development.
Post Reply
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Code Review Requests

Post by Fred »

Post here if you want others to check your work. Post here if you want to check the work of someone else on one of the projects, whether they ask for it or not.

I currently have Jeff and Sean up for review on the firmware. I'd like more variety in case they are busy, though. I run most of my commits directly to issues, so, if you'd like to review my firmware work please sign up on the tracker and let me know here and I'll send something your way to check.

Thanks in advance!

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
Peter
LQFP144 - On Top Of The Game
Posts: 268
Joined: Tue Dec 27, 2011 5:37 am

Re: Code Review Requests

Post by Peter »

https://github.com/pjohns30/freeems-van ... 16890fd0df

After sending one message it repeats that message over and over forever. I'm stuck to say the least.

Edit: This problem seems to be a feature of the CAN protocol. So no need for a code review, but here's an updated link anyway.
https://github.com/pjohns30/freeems-van ... 14a3d1cbb8
Last edited by Peter on Mon Dec 30, 2013 1:35 pm, edited 1 time in total.
:-p
User avatar
EssEss
LQFP112 - Up with the play
Posts: 244
Joined: Thu Sep 10, 2009 9:23 am
Location: Dayton, OH

Re: Code Review Requests

Post by EssEss »

w/CAN, you need someone else on the bus to assert the line and indicate that a frame was actually put out there error free. Without that the 'emitter' will begin sending error frames .. with enough of those, you begin to get into things like bus_off conditions where it takes itself off of the bus. I don't want to clog this thread up, so email me if you need some help.

Get a cheapo chinese CAN device - this works very well for me. Best 30$ I spent in a long time (it does some other things too).
User avatar
Fred
Moderator
Posts: 15431
Joined: Tue Jan 15, 2008 2:31 pm
Location: Home sweet home!
Contact:

Re: Code Review Requests

Post by Fred »

EssEss wrote:I don't want to clog this thread up
LOL! You're the last person whose posts would be unwelcome in this thread, or any thread, for that matter. If anything, best to split it off into another thread and link to that from here. Keeping it visible helps others learn through the same process.

Thanks for your input! Good info for CAN n00bs (such as me).

Fred.
DIYEFI.org - where Open Source means Open Source, and Free means Freedom
FreeEMS.org - the open source engine management system
FreeEMS dev diary and its comments thread and my turbo truck!
n00bs, do NOT PM or email tech questions! Use the forum!
The ever growing list of FreeEMS success stories!
Post Reply