Looking To Advertise but Keep DFU Service Disabled

RE: Looking To Advertise but Keep DFU Service Disabled

Einar Thorsrud — Fri, 27 Mar 2020 10:18:01 GMT

Hi,

[quote user="aculverD5"]From what I've read that event gets generated when a mobile client writes 0x01 to the DFU service, correct?[/quote]

Yes.

[quote user="aculverD5"]I'm also curious if it would be better to send an RSP code back to the connected client when we ignore the control point write? We were considering sending back DFU_RSP_OPERATION_FAILED. Our accompanying mobile app has also made use of the DFU library. Does it matter if we send back an RSP code or not?[/quote]

It would probably be cleaner to make a new RSP code for this, something like DFU_RSP_OPERATION_NOT_ALLOWED, and add support for that in the DFU library on the mobile phone side as well. In any case, it is not implemented by us, so the details are up to you an how you prefer to do it.

RE: Looking To Advertise but Keep DFU Service Disabled

Anders — Thu, 26 Mar 2020 17:39:10 GMT

Hi Einar,

Thank you for the response, that makes sense. I'd been looking around ble_dfu_bonded.c already and saw that event. From what I've read that event gets generated when a mobile client writes 0x01 to the DFU service, correct?

I'm also curious if it would be better to send an RSP code back to the connected client when we ignore the control point write? We were considering sending back DFU_RSP_OPERATION_FAILED. Our accompanying mobile app has also made use of the DFU library. Does it matter if we send back an RSP code or not?

RE: Looking To Advertise but Keep DFU Service Disabled

Einar Thorsrud — Thu, 26 Mar 2020 15:40:11 GMT

Hi,

There is no support for this in the buttonless DFU service, but it should not be difficult to add. I suggest you check your flag in the handling of the DFU_OP_ENTER_BOOTLOADER event in ble_dfu_buttonless_on_ctrl_pt_write() in \ble_dfu_bonded.c or ble_dfu_unbonded (depending on which you use). There you could ignore commands to enter DFU mode if the peer is not authenticated.