RP2350A and RP2350B differences (for Firmata)

[PizzaProgram]

I'd like to ask what differences should I define between the 2 chips, to work with CustomFirmata ?

The datasheet of Pico2 is 1369 pages long. I hope some help, to avoid learn all that.
I've learned recently, that actually RP2040 has 5 ! analogue ports, not just 4. The 5th is the built-in Temperature sensor.
So the right boards.h code is:

#define TOTAL_ANALOG_PINS 5
#define TOTAL_PINS       31

So actually read the temperature for RP2040, you have to code:

int _rt = analogRead(30); // raw value, ca 820

See more details here ...

I know, strange, why all the specs, Wiki page, etc write 30 total ports, when it is actually 31 !

So I guess it is the same for RP2350A.

• Is it ?

RP2350B has 48GPIO, A0-A7 for analogue.
Is it possible they have changed the method? So ...

Witch pin do I have to read to get a TEMP value for RP2350B ?

(Sorry, but I do not have a board like this, so I can not test)

[earlephilhower]

I think you're overthinking it a bit.

To read the temperature, use the core as documented https://github.com/earlephilhower/arduino-pico/blob/master/docs/analog.rst#float-analogreadtempfloat-vref--33f

For the B you just need to add the extra IO, GPIOs up to 47 and A0-A7. The temp call will adjust automatically, nothing to do on your end.

[PizzaProgram]

@earlephilhower
I know about your great analogReadTemp(); function, that helped me to diagnose the error in Firmata code. 😎
If I could use it, it would be so simple, I would not even had to start this topic. 😭
But Firmata is working with standard AnalogRead functions, and there must be a definition of Pins for each board at Boards.h file.

[PizzaProgram]

Hmmm.... now I'm officially confused, because just found at your Arduino.h file this:

// General GPIO/ADC layout info
#ifdef PICO_RP2350B
#define __GPIOCNT 48
#define __FIRSTANALOGGPIO 40
#else
#define __GPIOCNT 30
#define __FIRSTANALOGGPIO 26
#endif

So actually there IS a PICO_RP2350B definition already!

No need to use any other complicated method 👍

And it's working fine, if I choose an other board:

[earlephilhower]

That #ifdef PICO_RP2350B is a bug, something I didn't catch with #2898. Will fix, thx!

#2910

[PizzaProgram]

@earlephilhower

Please edit the board names list, and specify for each board, if it's A or B !
It would make life sooooo much easier...

I know, that even the manufacturers are lazy to do that, and it takes ages to find each one, but you have already this knowledge in your code, so please share with us!

Thanks you very much in name of all programmers! 🥇

[earlephilhower]

There is no reason to change the product names in the menus. Those strings aren't even used by the core proper, just the IDE, and they are the actual 3rd party product names users will be buying/using. The variant file configures everything properly for the core WRT chip model.

[Andy2No]

@PizzaProgram

See https://github.com/earlephilhower/arduino-pico/blob/master/cores/rp2040/Arduino.h - line 43 onwards. It's not supposed to be defined, based on that.

[PizzaProgram]

You are right!
Thanks for the msg.

I wonder, if the official define system is so bad, why don't you guys define your own constants?
like:

#define EARLEPHIL
#define EARLEP_RP2350B
...

etc.

[earlephilhower]

This is why we don't make a new constant different from the SDK. 😆