Page 4 of 6 FirstFirst ... 23456 LastLast
Results 31 to 40 of 51

Thread: Auto upgrade from build 72 to build 81 kills serial wideband.

  1. #31
    Lifetime Member swingtan's Avatar
    Join Date
    Jul 2007
    Posts
    1,589

    Default

    Sorry, about the MIA Blacky, I just had a guy come round to pick up all the old Grey motor gear. Good work though, as usual

    Simon

  2. #32
    Joe (Moderator) joecar's Avatar
    Join Date
    Apr 2003
    Posts
    28,403

    Default Anyone care to post what the code should have been changed to?

    I'll try...

    Quote Originally Posted by Blacky View Post
    ...
    New code: ExternalPID_Index*sizeof(WORD)
    ...
    Should that have been:

    ExternalPID_Index*sizeof(Int16)

    Or something like:

    ExternalPID_Index*sizeof(ExternalPID_Array[1])

  3. #33
    EFILive Developer Site Admin Blacky's Avatar
    Join Date
    Mar 2003
    Posts
    9,503

    Default

    Quote Originally Posted by joecar View Post
    I'll try...

    Should that have been:

    ExternalPID_Index*sizeof(Int16)

    Or something like:

    ExternalPID_Index*sizeof(ExternalPID_Array[1])
    It should have been (sizeof(WORD)*8), because the value is supposed to be the bit size (not the byte size). I changed it to sizeof(WORD) which is only 2 (bytes) and so each value that was being extracted was only 2 bits apart instead of 16 bits apart.

    That's assuming sizeof() *always* returns the byte count of a field where a byte is *always* 8 bits - which I am sure is the case on all Intel based Windows systems.

    Regards
    Paul
    Before asking for help, please read this.

  4. #34
    Joe (Moderator) joecar's Avatar
    Join Date
    Apr 2003
    Posts
    28,403

    Default sizeof(WORD)*8

    Ah, yes of course, now I see clearly... I don't know what previously happened to me... ...Ross's programming joke was still with me...

    Yes I agree, sizeof(WORD) and sizeof(INT16) are 2 and not 16...

    lol, we both fell into the same hole...

    Does that mean if bit widths are being used a lot, then a macro like this may come in handy:

    #define bit_sizeof(x) (8*sizeof(x))

  5. #35
    Senior Member
    Join Date
    Jun 2004
    Posts
    128

    Default

    Quote Originally Posted by Blacky View Post
    Ok, I think I have found the problem. The diff command was my friend I compared the source code from build 81 to build 82...
    FWIW build 81 was broke for me using the serial wideband, 80 worked fine.

    Unless the build 81 I had was really build 82

  6. #36
    EFILive Developer Site Admin Blacky's Avatar
    Join Date
    Mar 2003
    Posts
    9,503

    Default

    Quote Originally Posted by kp View Post
    FWIW build 81 was broke for me using the serial wideband, 80 worked fine.

    Unless the build 81 I had was really build 82
    There was a mixup for about 30 minutes when build 82 was posted and it was labeled build 81. The best thing that can happen is for build 81 and 82 to die quietly and I'll get build 83 out tonight that will fix the problem.

    Regards
    Paul
    Before asking for help, please read this.

  7. #37
    Senior Member
    Join Date
    Jun 2004
    Posts
    128

    Default

    Quote Originally Posted by Blacky View Post
    There was a mixup for about 30 minutes when build 82 was posted and it was labeled build 81. The best thing that can happen is for build 81 and 82 to die quietly and I'll get build 83 out tonight that will fix the problem.

    Regards
    Paul
    Thanks for getting it fixed, of course I ended up in that 30 minute window lol.

  8. #38
    Lifetime Member Chevy366's Avatar
    Join Date
    Oct 2006
    Posts
    1,603

    Default

    Was going to do some AutoVE logging last night saw the thread and canceled the session , maybe tomorrow night .
    Great work on a fix .
    Old Sig :Always be wary of the Software Engineer who carries a screwdriver.
    Last edited by Chevy366; March 7th, 2009 at 02:12 PM.
    2005 1500 HD , Custom OS3 SD tune .
    2006 Trailblazer
    Dinosaurs and Plants gave their lives so that we may drive , long live fossil fuel .

  9. #39
    EFILive Developer Site Admin Blacky's Avatar
    Join Date
    Mar 2003
    Posts
    9,503

    Default

    Quote Originally Posted by tunedbyGM View Post
    Was going to do some AutoVE logging last night saw the thread and canceled the session , maybe tomorrow night .
    Great work on a fix .
    Your signature couldn't be more accurate except to say, beware of the programmer using a power saw - I managed to cut through the power cord last time I used a power saw.

    In other news a year or so ago, I managed to end up in A&E requiring stitches after watching the (professional) guys remove tiles with a jack hammer from our bathroom which was being renovated. It looked like so much fun, jack hammer verses porcelain tiles, what could be more fun that that? My wife told me not to touch the equipment, she knows me and power tools don't co-exists happily, but while they were having lunch I snuck in and had a go. Wife refused to take me to A&E, I had to convince the builder to drive me there. Got back, hand all stitched up and they were taking to a marble bath with a sledge hammer. Man I had to have a go of that!!! Well, one lump of marble in the face and one black eye later I called it quits for the day. As I've always said - I'm just a programmer.

    Regards
    Paul
    Before asking for help, please read this.

  10. #40
    Lifetime Member Chevy366's Avatar
    Join Date
    Oct 2006
    Posts
    1,603

    Default

    Quote Originally Posted by Blacky View Post
    Your signature couldn't be more accurate except to say, beware of the programmer using a power saw - I managed to cut through the power cord last time I used a power saw.

    In other news a year or so ago, I managed to end up in A&E requiring stitches after watching the (professional) guys remove tiles with a jack hammer from our bathroom which was being renovated. It looked like so much fun, jack hammer verses porcelain tiles, what could be more fun that that? My wife told me not to touch the equipment, she knows me and power tools don't co-exists happily, but while they were having lunch I snuck in and had a go. Wife refused to take me to A&E, I had to convince the builder to drive me there. Got back, hand all stitched up and they were taking to a marble bath with a sledge hammer. Man I had to have a go of that!!! Well, one lump of marble in the face and one black eye later I called it quits for the day. As I've always said - I'm just a programmer.

    Regards
    Paul
    Yeah , worked in the construction field for a while , and would get calls from ladies asking if I could fix or redo what their husbands had started or finished , I have to say Software Engineers were the worst . JK
    I had a wealthy client , say to me one time , "I wish I had your talent with tools" , I looked at him and said funny I wish I had your talent for making millions .

    Gil
    2005 1500 HD , Custom OS3 SD tune .
    2006 Trailblazer
    Dinosaurs and Plants gave their lives so that we may drive , long live fossil fuel .

Page 4 of 6 FirstFirst ... 23456 LastLast

Similar Threads

  1. Build 88
    By Gordy M in forum General (Petrol, Gas, Ethanol)
    Replies: 4
    Last Post: December 1st, 2009, 09:07 AM
  2. Build 82, WTF?
    By mr.prick in forum FlashScan V2 BB Logging
    Replies: 30
    Last Post: March 9th, 2009, 11:20 AM
  3. build 80 and 78 ?
    By JST in forum FlashScan V2
    Replies: 10
    Last Post: February 11th, 2009, 12:59 PM
  4. 7.5.4 build 66
    By eboggs_jkvl in forum FlashScan V2
    Replies: 12
    Last Post: October 22nd, 2008, 11:22 AM
  5. Replies: 10
    Last Post: August 12th, 2006, 03:25 AM

Posting Permissions

  • You may not post new threads
  • You may not post replies
  • You may not post attachments
  • You may not edit your posts
  •