gputils / Bugs / #292 usb registers missing for 18f4550 and variants in 1.5.0

Molnár Károly - 2016-08-23

assigned_to: Molnár Károly
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-08-23

--extended

Only incidentally I ask, are you sure that this is necessary for the compilation?

I've tried to search for the offending header(s) and/or linker files but haven't been able to find anything yet.

The missing symbols must be in the application code. They can not be found in the standard gputils header.

Károly

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Fred - 2016-08-23

Yes,

I searched as you did for the registers mentioned in the error messages. They "ARE" needed for compilation. The registers are USB enpoint registers within the 4550 and it's variants. What the usbmmmap.c code does is map structures to the internal 18f4550 registers. My question is why does it work in the previous version with a correctly working *.hex file but not in this version?

Cheers,

Fred

usbmmap.c

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-08-24

This is one file not enough to cause the error. I need as many source code which comes out of the problem. There are still some questions:
1.) The sdcc version.
2.) The sdcc installed from package from repository, or compiled on the own machine?

Károly

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-09-11

I can not reproduce this error.

Károly

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Fred - 2016-09-11

Károly,

My apologies for not following up on this sooner. I had let things slip due to personal and work issues. I have gathered up the offending code and have attached it here. Sorry for my "spaghetti" code. I am an engineer (not software engineer in any respects) and not a software person.

To recap on this, the code works with 1.4.x but not 1.5.0

Thank you again for looking at this.

Regards,

Fred

MCHPUSB-Bootloader_NewSDCC.tar.bz2

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-09-12

I found the cause of error. In the usbmmap.h header there is references to non-existent names. The new gplink manage this is strictly.

I propose a solution based on the usbmmap.c file:

#if(0 <= MAX_EP_NUMBER) volatile __far BDT ep0Bo; //Endpoint #0 BD Out volatile __far BDT ep0Bi; //Endpoint #0 BD In #endif . . . etc

The above way, must use conditional compilation in the usbmmap.h:

#if(0 <= MAX_EP_NUMBER) extern volatile __far BDT ep0Bo; //Endpoint #0 BD Out extern volatile __far BDT ep0Bi; //Endpoint #0 BD In #endif . . . etc

Thus eliminating the problem. I appended the modified usbmmap.h file.

Károly

usbmmap.h
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Fred - 2016-09-12

Károly,

Thank you very much for your assistance and patience in resolving this issue. I'm still looking at the changes that you made. I want to understand them, at least in part, for myself. I tried to build the project and it completed without errors. I am also in the process of reviewing the manual for gputils and espsecially gplink.

I think that this issue can be closed.

Cheers,

Fred

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-09-12

I created a correction, that do adjustable the behavior of gplink. [r1285]

-S [0|1|2], --strict [0|1|2] Set the strict level of the missing symbol. 0: This is the default. No message. 1: Show warning message if there is missing symbol. 2: Show error message if there is missing symbol.

gputils-1.5.0-1.tar.bz2
gputils-1.5.0-1.tar.gz
gputils-1.5.0-1.exe

The gplink will only warns, if the command line includes these switches:

"-S1" or "--strict 1" "-S2" or "--strict 2"

If these connections are not included on the command line, then the gplink operates the old way. Please try out with the original project.

Károly

Related

Commit: [r1285]
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Fred - 2016-09-12

It works with the original project. At least without the new switches. I am not sure how to pass those to sdcc.

Thanks Again,

Fred

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Comment has been marked as spam.
Undo

View and moderate all "bugs Discussion" comments posted by this user

Mark all as spam, and block user from posting to "Bugs"

Anonymous - 2016-09-13

Exactly the point is that there is no need for the new switches. These will be required only if you need the strict behavior.

Károly

Exactly the point is that there is no need for the new switches. These will be required only if you need the strict behavior. Károly

Add attachments
Cancel
You seem to have CSS turned off. Please don't fill out this field.

You seem to have CSS turned off. Please don't fill out this field.

New Attachment:

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

Molnár Károly - 2016-09-22

status: open --> closed-fixed
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Anonymous
  
  Add attachments
  Cancel
  You seem to have CSS turned off. Please don't fill out this field.
  
  You seem to have CSS turned off. Please don't fill out this field.

usb registers missing for 18f4550 and variants in 1.5.0

Group

Searches

Help

#292 usb registers missing for 18f4550 and variants in 1.5.0

Discussion

Related