USB CDC ACM example and ‘TX_PIN_NUMBER’ error

[Ubbi]

Hello all

I was just following the thread launched by Alex concerning nrf 52 dongle driven from serial port, which is also very important to me, and now that thread is already closed.

I was just trying to compile the usbd_cdc_acm example... I updated it for donge hardware (changed project properties) but when compilling the sample I see these errors:

‘TX_PIN_NUMBER’ undeclared (first use in this function); did you mean ‘P0_PIN_NUM’?
‘RX_PIN_NUMBER’ undeclared (first use in this function); did you mean ‘P0_PIN_NUM’?

Please, could you guide me to solve these compiler errors.

I am compilling it in ubuntu machine, using SES and SDK 15

Thank you

Ub

[awneil]

Ubbi said:

the thread launched by Alex concerning nrf 52 dongle driven from serial port

You mean this:

https://devzone.nordicsemi.com/f/nordic-q-a/50515/any-sample-to-drive-nrf-52-dongle-like-a-serial-port

Ubbi said:

I updated it for donge hardware (changed project properties)

Did you follow the link which Sigurd posted for the full details of the modifications required:

https://devzone.nordicsemi.com/f/nordic-q-a/50515/any-sample-to-drive-nrf-52-dongle-like-a-serial-port/201858#201858

Ubbi said:

Please, could you guide me to solve these compiler errors

The error tells you that those names are un-declared.

So the solution is to declare them!

I would suggest that you start by building the unmodified example, and then look at how & where they are declared in that.

[Ubbi]

The original project compilation goes well.

When I apply

"Update the Preprocessor Definitions by removing BOARD_PCA10056 and inserting BOARD_PCA10059"

and then compile it again the errors

‘TX_PIN_NUMBER’ undeclared (first use in this function); did you mean ‘P0_PIN_NUM’?
‘RX_PIN_NUMBER’ undeclared (first use in this function); did you mean ‘P0_PIN_NUM’?

appear

[awneil]

Ubbi said:

The original project compilation goes well

Good, so ...

awneil said:

then look at how & where they are declared in that

[Ubbi]

They are solely used in this function:

static void init_cli(void)
{
ret_code_t ret;
ret = bsp_cli_init(bsp_event_callback);
APP_ERROR_CHECK(ret);
nrf_drv_uart_config_t uart_config = NRF_DRV_UART_DEFAULT_CONFIG;
uart_config.pseltxd = TX_PIN_NUMBER;
uart_config.pselrxd = RX_PIN_NUMBER;
uart_config.hwfc = NRF_UART_HWFC_DISABLED;
ret = nrf_cli_init(&m_cli_uart, &uart_config, true, true, NRF_LOG_SEVERITY_INFO);
APP_ERROR_CHECK(ret);
ret = nrf_cli_start(&m_cli_uart);
APP_ERROR_CHECK(ret);
}

But not declared...

[awneil]

Ubbi said:

But not declared

That's why you get the error - you need to declare them before you can use them!

That's basic 'C' programming!

So, again, look at how & where they are declared in the unmodified example.

Then do the same in your modified example.

[Ubbi]

I was confused about that: 

is just need to change 

"Update the Preprocessor Definitions by removing BOARD_PCA10056 and inserting BOARD_PCA10059"

But it is not!!  :-)

I have checked the dongle pin numbers and realize that 

RX pin is 8
TX pin is 6

After declaring them :-)  it works

Thank you

[awneil]

Ubbi said:

is just need to change

No, that is not what Sigurd said!

Sigurd said:

in short, you need to ...

So that was just a quick summary - not the full details!

That's why I said earlier that you also need to follow his link for the full details!

Ubbi said:

After declaring them :-)  it works

Of course!

Now you can verify the answer