USB HID (physical) + BLE

RE: USB HID (physical) + BLE

mauro_nx — Thu, 23 Jan 2020 14:49:32 GMT

Maybe you can update on the next version of the SDK to have the definition of the 'ret' instance (on the app_usbd_hid_generic.c file) outside of the critical_region. That worked out for me... I guess it's because the Softdevice is enabled.

Again, thank you so much 🙂

RE: USB HID (physical) + BLE

Edvin — Thu, 23 Jan 2020 14:31:34 GMT

No worries 🙂 Glad it worked out!

Best regards,

Edvin

RE: USB HID (physical) + BLE

mauro_nx — Thu, 23 Jan 2020 13:18:26 GMT

Hi Edvin,

the project now successfully compiles. It was an error from my side. I didn't know about the 'Output' option to see the details of the compiling process. Now we know it.

I didn't realize that in one of our libraries there was a definition on a .h file... So compiler was saying that there was a redefinition of some variables. I placed the definition on the .c file and now it works.

Sorry for the mistake... and thank you so much for all your help.

RE: USB HID (physical) + BLE

Edvin — Thu, 23 Jan 2020 08:42:16 GMT

Before we do that, can you please try to look at the "output" from the build?

From your last screenshot, there is a dropdown menu saying "Task". Can you change it to "Output", and copy-paste all the text from there?

RE: USB HID (physical) + BLE

mauro_nx — Wed, 22 Jan 2020 16:47:18 GMT

Hi Edvin,

yes, this is the instance that the error points to. On the app_usbd_hid_generic.c :

If I try what you suggested, declaring "ret" outside from the critical region, this is what happens... Build fails and there is any error message:

I was using SES v4.18 (64 bits). I updated it to v4.40 (64 bits) as you recommended, and the behavior is the same, Build failed with no error message. If I try to double click the Build failed message... nothing happens.

Do you think that sending you the project... Would help you to review it?

RE: USB HID (physical) + BLE

Edvin — Wed, 22 Jan 2020 14:27:53 GMT

[quote user="mauroaristi"]I made the change because of this error... but not sure if that was ok. [/quote]

I see. I would try to move only the "rety_code_t ret" outside the critical region if this caused any issues during compilation:

NRF_DRV_USBD_TRANSFER_IN(transfer, p_buff, size);
ret_code_t ret;

CRITICAL_REGION_ENTER();
ret = app_usbd_ep_transfer(ep_addr, &transfer);
if (ret == NRF_SUCCESS)
{
    app_usbd_hid_state_flag_set(&p_generic_ctx->hid_ctx,

But are you sure this "ret" is the "ret" instance that it was actually pointing to?

what happens if you double click the "build failed"? Does anything else pop up?

It looks like you are using SES, is that correct? What version? Can you try to update it to a later version from here?

RE: USB HID (physical) + BLE

mauro_nx — Wed, 22 Jan 2020 13:53:35 GMT

Hello Edvin,

thank you so much for you fast reply. Please find attached the warnings that it shows... Those warnings are from main.c... and on the previous version of the project (where it didn't has USB functionality, the same warnings appeared, and it compiled successfully):

Current version - build failed:

But... i modified one file: app_usbd_hid_generic.c, as follows:

I made the change because of this error... but not sure if that was ok.

On the previous version of the project (where no USB functionality was added... The same warnings appeared, and it compiled successfully:

Probably my mistake was to change the app_usbd_hid_generic.c file. But... how can I avoid that error?

Thank you so much!

RE: USB HID (physical) + BLE

Edvin — Wed, 22 Jan 2020 12:42:13 GMT

Hello,

What are the warnings and errors you get when you try to build the project? If you double click the warnings, you should see what they are, and you probably have at least one error as well, since the build failed.

Best regards,

Edvin