Remove the usage of Python to build the version (from GitHub Tag) and build DFU for STM32F405
=> https://github.com/hydrabus/hydrafw_hydranfc_shield_v2/tree/master/src/build-scripts

1. Remove hydrafw-revision.py / hydrafw-version.py

• To be done with GitHub Actions after each Commit ...
  • To check integration of https://github.com/marketplace/actions/gittools
• Other idea ?

2. Remove dfu-convert.py

• Replace it with a native C code (multi-platform) to be built with GCC/MinGW2 ...
  • To check integration of https://github.com/encedo/hex2dfu
• Other idea ?

3. Modification of Makefile (to remove anything related to Python script)

• https://github.com/hydrabus/hydrafw_hydranfc_shield_v2/blob/master/src/Makefile

I want to use a vicinity card in dfnc menu, so I want to configure it like in rfalNfcvPollerInitialize:

    EXIT_ON_ERR( ret, rfalSetMode( RFAL_MODE_POLL_NFCV, RFAL_BR_26p48, RFAL_BR_26p48 ) );

I go in dnfc mode to set NFCV

dnfc2> set-nfc-mode nfc-mode 7
nfc-mode = 7
nfc-mode-tx_br = 0
nfc-mode-rx_br = 0
rfalSetMode Error 7

But, the error 7 is raised. Why? We go in hydranfc_v2_dnfc_mode.c file, in exec function. First, the mode is copied

memcpy(&nfc_mode, p->buf + p->tokens[t], sizeof(int));

Then, it is set

		case T_SET_NFC_MODE:
		{
			ReturnCode err;
			cprintf(con, "nfc-mode = %d\r\n", nfc_mode);
			cprintf(con, "nfc-mode-tx_br = %d\r\n", nfc_mode_tx_bitrate);
			cprintf(con, "nfc-mode-rx_br = %d\r\n", nfc_mode_rx_bitrate);
			err = rfalSetMode(nfc_mode, nfc_mode_tx_bitrate, nfc_mode_rx_bitrate);

but the default value for nfc_mode_tx_bitrate/nfc_mode_rx_bitrate are not compatible, so an error is raised.

Ok, but if I want to update the rx mode for instance, the mode modification is not taken into account:

dnfc2> set-nfc-mode nfc-mode-tx_br 236
nfc-mode = 0
nfc-mode-tx_br = 236
nfc-mode-rx_br = 0
rfalSetMode Error 15

It comes from the previous modification was not recorded: default value are retrieved at the beginning of exec function:

	action = nfc_mode =  nfc_mode_tx_bitrate = nfc_mode_rx_bitrate = 0;

nfc_mode, nfc_mode_tx_bitrate, nfc_mode_rx_bitrate could be move out the function and be defined as global to resolve this problem.

See commit 7ddfa04

1. Modify BBIO_NFC_CMD_SEND_BITS to use a generic command to send bits as rfalNfcaPollerCheckPresence() is specific to NFC-A
2. Modify BBIO_NFC_CMD_SEND_BYTES to do not use hard coded value for fwt (Frame Waiting Time in 1/fc) parameter
3. Add BBIO READER mode for NFC-V (see code BBIO_NFC_SET_MODE_ISO_15693 TODO)
4. Add BBIO READER mode for NFC-B (BBIO_NFC_SET_MODE_ISO_14443B to be created)
5. Cleanup code remove comments ...

Improve the scan with option nfc-all as some specific tags are not read when nfc-all is used
To check: Different norm/standard on the subject to be more compliant to scan different technology with fastest possible speed.

Idea1:
Switch NFC field off/on between each technology drawback it will slow down the scan (can be done with a special option to be defined (Phone with NFC does not use this as NFC field is always ON during scan of different technology)

• Refactor code for nfc/dnfc/nfc bbio mode before to release the v0.3

  • nfc/dnfc/nfc bbio mode refactoring / cleanup
    • 77ea467
  • nfc/dnfc/nfc bbio modes refactoring cleanup (removed some headers)
    • b2bb3a3
    • 9db3eab

• RAM optimization

• caf1bd1

• f586147