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

QoL bugfixes #9

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

QoL bugfixes #9

wants to merge 3 commits into from

Conversation

oliv3r
Copy link

@oliv3r oliv3r commented Aug 28, 2024

A small number of inconsistency fixes

  • fix: npi_tl: Pointer-sign warning
  • fix: DisplayUart2: Use proper type for baudRate
  • boyscout: Drop log files from repository

Generated log files have no place in a repository.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
Within the whole system, uint32_t is used for the baudrate. We should do
the same here to avoid implicit type conversions.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
Resolves

/sdk/cc13xx_cc26xx_sdk/source/ti/zstack/npi/npi_tl.c:183:19: warning: passing 'Char[270]' (aka 'char[270]') to parameter of type 'uint8_t *' (aka 'unsigned char *') converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign]
    transportInit(npiRxBuf,npiTxBuf, NPITL_transmissionCallBack);
                  ^~~~~~~~
/sdk/cc13xx_cc26xx_sdk/source/ti/zstack/npi/npi_tl_uart.h:110:45: note: passing argument to parameter 'tRxBuf' here
void NPITLUART_initializeTransport(uint8_t *tRxBuf, uint8_t *tTxBuf, npiCB_t npiCBack);
                                            ^
/sdk/cc13xx_cc26xx_sdk/source/ti/zstack/npi/npi_tl.c:183:28: warning: passing 'Char[270]' (aka 'char[270]') to parameter of type 'uint8_t *' (aka 'unsigned char *') converts between pointers to integer types where one is of the unique plain 'char' type and the other is not [-Wpointer-sign]
    transportInit(npiRxBuf,npiTxBuf, NPITL_transmissionCallBack);
                           ^~~~~~~~
/sdk/cc13xx_cc26xx_sdk/source/ti/zstack/npi/npi_tl_uart.h:110:62: note: passing argument to parameter 'tTxBuf' here
void NPITLUART_initializeTransport(uint8_t *tRxBuf, uint8_t *tTxBuf, npiCB_t npiCBack);
                                                             ^
2 warnings generated.

Signed-off-by: Olliver Schinagl <oliver@schinagl.nl>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant