Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Most of the examples don't actually work because the mc3 file is out of sync with the generated code #2

Open
otonoton opened this issue Mar 5, 2024 · 2 comments

Comments

@otonoton
Copy link

otonoton commented Mar 5, 2024

Most of the MC3 files that use multiple endpoints (in+out) have the endpoint numbers BOTH equal to 1!

I spent an entire week chasing down this problem.

The generated files in this repo are correct but not the mc3 files. So if you ever press the generate button, everything breaks. Unbelievable.

@shijasm
Copy link
Collaborator

shijasm commented Apr 22, 2024

@otonoton , I couldn't reproduce the issue with the latest USB repositories. Please let me know the demo name and configuration name. Thanks!

@otonoton
Copy link
Author

otonoton commented Apr 24, 2024

@shijasm

$ cat apps/hid_basic/firmware/pic32mz_ef_curiosity_2_0.X/hid_basic_pic32mz_ef_curiosity_2_0.mc3 |grep -A2 _ENDPOINT_NUMBER                                                                                                                                            <key class="com.microchip.mcc.core.tokenManager.CustomKey" moduleName="usb_device_hid_0" name="CONFIG_USB_DEVICE_FUNCTION_INT_IN_ENDPOINT_NUMBER"/>
         <value>&lt;?xml version=&quot;1.0&quot; encoding=&quot;UTF-8&quot;?&gt;&lt;usb_device_hid_0&gt;
  &lt;usb_device_hid_0 dnOrder=&quot;0&quot; id=&quot;CONFIG_USB_DEVICE_FUNCTION_INT_IN_ENDPOINT_NUMBER&quot;&gt;
    &lt;Values dnOrder=&quot;0&quot;&gt;
      &lt;Dynamic dnOrder=&quot;0&quot; id=&quot;usb_device_hid_0&quot; value=&quot;1&quot;/&gt;
--
         <key class="com.microchip.mcc.core.tokenManager.CustomKey" moduleName="usb_device_hid_0" name="CONFIG_USB_DEVICE_FUNCTION_INT_OUT_ENDPOINT_NUMBER"/>
         <value>&lt;?xml version=&quot;1.0&quot; encoding=&quot;UTF-8&quot;?&gt;&lt;usb_device_hid_0&gt;
  &lt;usb_device_hid_0 dnOrder=&quot;0&quot; id=&quot;CONFIG_USB_DEVICE_FUNCTION_INT_OUT_ENDPOINT_NUMBER&quot;&gt;
    &lt;Values dnOrder=&quot;0&quot;&gt;
      &lt;Dynamic dnOrder=&quot;0&quot; id=&quot;usb_device_hid_0&quot; value=&quot;1&quot;/&gt;

Here you can see both in/out endpoint numbers are set to 1. Many other files in this repo do the same thing, but not every one. This is on a fresh clone of the repo just now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants