ErichStyger / mcuoneclipse

McuOnEclipse Processor Expert components and example projects
Other
731 stars 1.29k forks source link

KDS USB Host MSC ISO99 cc isssue #19

Closed neilh10 closed 9 years ago

neilh10 commented 9 years ago

Not sure if this is worth mentioning, but using Comonent USB:FSL_USB_stack MSD Host I'm having a couple of compile errors. Either I'm doing something wrong (Learner plates still on advice appreciated) or upgrade to KDS issue.

I'm using (http://sourceforge.net/projects/mcuoneclipse/ Components 2014-10-17.zip) and set CPU:MK20DX128VLH5 (from my FRDM-K20D50 RevB brd) set the USB1:FSL_USB_Stack to Init_USB_OTG_VAR0, MSD Host, MSD This causes errors with asm() with reason and fix https://community.freescale.com/thread/326345

Then setting the compiler to GNU ISO C90 results in another error GeneratedCode/usbmsgq.c: 'index' redeclared as different kind of symbol usbmsgq.c /Mk20Dx50Logger/Generated_Code line 38 C/C++ Problem

This can be fixed by changing index --> index2

Of course whenever the PE regenerates the files, it goes back to previous problem. Suggested solution: change asm to __asm() per https://community.freescale.com/thread/326345

ErichStyger commented 9 years ago

Thanks for reporting that, I have the host stack not yet running on KDS, but I confirm the problem with asm. You can fix this on your side, the file is located in C:\ProgramData\Processor Expert\PEXDRV_PE5_3\Drivers\FSL_USB_Stack\Host\source\driver\usbprv.h ==> change there the asm() to asm(). I have a look at the index problem...

ErichStyger commented 9 years ago

I'm puzzled about that error message about index: it seems to me that the compiler sees another symbol named 'index', but not sure where?

ErichStyger commented 9 years ago

anyway, I have changed the name to elemIndex. Do you need new components?

neilh10 commented 9 years ago
Neil Hancock

Hi Eric
Thanks- based on your latest blog thought the right place to input
was github - and wow your fast.
I was puzzled by the 'index' as well - but solved it by changing the
name - so I can only assume there is another 'index' somwhere in the
includes - but doesn't make a lot of sense - so maybe a compiler bug
- you never know.
I've got a working environment right now, so if it gets fixed in the
next components release thats great.

many thanks
Neil 
On 11/1/2014 1:19 PM, Erich Styger
  wrote:

  anyway, I have changed the name to elemIndex. Do you need new
    components?
  —
    Reply to this email directly or view
      it on GitHub.
  {"@context":<a class="moz-txt-link-rfc2396E" href="http://schema.org">"http://schema.org"</a>,"@type":"EmailMessage","description":"View this Issue on GitHub","action":{"@type":"ViewAction","url":<a class="moz-txt-link-rfc2396E" href="https://github.com/ErichStyger/mcuoneclipse/issues/19#issuecomment-61381771">"https://github.com/ErichStyger/mcuoneclipse/issues/19#issuecomment-61381771"</a>,"name":"View Issue"}}
ErichStyger commented 9 years ago

yes, it will be fixed in the next version of the components on SourceForge.