Skip to content

v3.3.2 - fix RPi-Pico Analogue port number#183

Closed
PizzaProgram wants to merge 1 commit intofirmata:masterfrom
PizzaProgram:patch-5
Closed

v3.3.2 - fix RPi-Pico Analogue port number#183
PizzaProgram wants to merge 1 commit intofirmata:masterfrom
PizzaProgram:patch-5

Conversation

@PizzaProgram
Copy link
Copy Markdown
Contributor

@PizzaProgram PizzaProgram commented Apr 7, 2025

More info here...

@PizzaProgram
Copy link
Copy Markdown
Contributor Author

More info here ...

@PizzaProgram
Copy link
Copy Markdown
Contributor Author

PizzaProgram commented Apr 8, 2025

@pgrawehr Can you please merge this change too?

... and create a new RELEASE from the current state of code? v3.3.2

@pgrawehr
Copy link
Copy Markdown
Contributor

pgrawehr commented Apr 8, 2025

I'll look after it, but I'm rather busy at the moment, so might take a few days still.

@PizzaProgram
Copy link
Copy Markdown
Contributor Author

WAIT.

No need to accept this yet. Changing:

TOTAL_ANALOG_PINS       5

is not enough. Also:

#define TOTAL_PINS              31

must be increased.

Yet, these changes are still not enough to read the temperature value :-(

This code works fine at setup() :

#include <stdio.h>
  float f = analogReadTemp();
  char buf[50], buf2[10];
  snprintf(buf, 16, "{\"temp\": %f, ", f);
  strcat(buf, ", \"totalApins\": ");
  snprintf(buf2, 10, "%i }", TOTAL_ANALOG_PINS );
  strcat(buf, buf2);
Firmata.sendString(F( buf ));

@PizzaProgram
Copy link
Copy Markdown
Contributor Author

PizzaProgram commented Apr 13, 2025

Now Confirmed:

Need to change TOTAL for Pico board, from 30 -> to 31 too !

#define TOTAL_ANALOG_PINS     5
#define TOTAL_PINS              31

See here ...

@PizzaProgram
Copy link
Copy Markdown
Contributor Author

If we are about to change boards.h,
Would be great to add Pico2 board too.

@PizzaProgram PizzaProgram deleted the patch-5 branch April 21, 2025 12:41
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