Stm32h7 - Githubissues

sinara-hw / Thermostat_EEM

Thermostat with 4 TEC and 8 sensor channels in EEM form factor

CERN Open Hardware Licence Version 2 - Weakly Reciprocal

2 stars 1 forks source link

Stm32h7 #43

Closed jordens closed 3 years ago

jordens commented 3 years ago

Let's consider moving to the stm32h7 right away. Price is almost the same, more power, newer architecture, lots of experience with it on stabilizer now. I think there is a pin compatible stm32h743 to the f4 that we are using. It might just be a bom.change.

@gkasprow what do you think?

gkasprow commented 3 years ago

Let's do whatever is easier from SW's point of view.

jordens commented 3 years ago

Ok. Let's do stm32h743zit6 as on Stabilizer then. The H7 teething problems appear to be over and I can see the need for some additional computation power here. I think this is a drop-in replacement but it's probably worth verifying the complete pin config.

c.f. old discussion at https://github.com/sinara-hw/Thermostat/issues/12

gkasprow commented 3 years ago

done, please check the CubeMX project Thermostat__EEM_8CH.zip

nkrackow commented 3 years ago

Hi, I looked at the schematic and I couldn't find anything wrong with the pinout. But I also don't know the exact timer/ADC configuration etc. and generally this is the first time I properly looked at Thermostat in depth. So maybe someone else should check as well.

jordens commented 3 years ago

@gkasprow

~Is there a chance to group the timer channels a bit clearer? Specifically: Can we put PWM_maxINeg1 onto the same timer (TIM8) as the three other? I.e. swap FAN_PWM and PWM_maxINeg1.~ We can just use TIM3.

Looks good from our end.

One thing to note: please ensure that there are only the 16 reference resistors on the breakout board are mounted, not the ones on the main board.

gkasprow commented 3 years ago

they are not mounted by default obraz