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

drivers/ftdi: fix an incompatible pointer type error #9

Merged
merged 1 commit into from
Nov 27, 2023

Conversation

ivq
Copy link

@ivq ivq commented Nov 26, 2023

Fix the build error:

src/jtag/drivers/ftdi.c:1082:61: error: passing argument 4 of 'oscan1_mpsse_clock_data' from incompatible pointer type [-Werror=incompatible-pointer-types]
 1082 |                 oscan1_mpsse_clock_data(mpsse_ctx, NULL, 0, &back0, 0, 32, ftdi_jtag_mode);
      |                                                             ^~~~~~
      |                                                             |
      |                                                             uint32_t * {aka unsigned int *}
src/jtag/drivers/ftdi.c:812:110: note: expected 'uint8_t *' {aka 'unsigned char *'} but argument is of type 'uint32_t *' {aka 'unsigned int *'}
  812 | static void oscan1_mpsse_clock_data(struct mpsse_ctx *ctx, const uint8_t *out, unsigned out_offset, uint8_t *in,
      |                                                                                                     ~~~~~~~~~^~
cc1: all warnings being treated as errors

Tested on archlinux, GCC 13.2.1.

Change-Id: Ie86b7fae104aae9b4a74835a615eef302e3a6a03
Signed-off-by: Chien Wong <[email protected]>
Copy link
Member

@fanghuaqi fanghuaqi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fanghuaqi fanghuaqi merged commit 56292c0 into riscv-mcu:nuclei/2023.10 Nov 27, 2023
1 of 4 checks passed
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.

2 participants