sinara-hw / Thermostat

2-channel temperature controller
5 stars 3 forks source link

Review microprocessor connections #53

Closed hartytp closed 5 years ago

hartytp commented 5 years ago

Can someone sign off on these before the next round (eg @jordens @astro @sbourdeauducq )

gkasprow commented 5 years ago

the STM32CubeMX project is here

gkasprow commented 5 years ago

SPI assignments: SPI2: ADC SPI3: extension header SPI4: DAC0 SPI5: DAC1

Timers TIM1_CH1: PWM_maxIPos0 TIM1_CH2: PWM_maxIPos1 TIM1_CH3: PWM_maxINeg0 TIM1_CH4: PWM_maxINeg1 TIM3_CH1: PWM_maxV0 TIM3_CH2: PWM_maxV1 TIM4_CH1: ADC CLKIO TIM4_CH1: ADC SYNC

ADC assignments:

ADC1_IN0: TEC Vref0 ADC1_IN3: TEC Vref1 ADC1_IN4: DAC_FB0 ADC1_IN5: DAC_FB1 ADC1_IN6: ITEC0 ADC1_IN8: ITEC1 ADC1_IN9: extension header.3 ADC1_IN10: P5V0 sense ADC1_IN12: TEC_U_meas0 ADC1_IN13: TEC_U_meas1

UART1: header I2C1: EEPROM

astro commented 5 years ago

I cannot find RMII_TXD0. According to the Datasheet, the only possible pin seems to be PB12 which is used for ADC.

gkasprow commented 5 years ago

well, the STM32CubeMx says that PG13 is also possible obraz

astro commented 5 years ago

Oh, I didn't recognize the second rectangle as the other half of the MCU. Continuing review...

astro commented 5 years ago

Looks good to me.

hartytp commented 5 years ago

@astro if you're happy with this, please close the issue. Thanks!

hartytp commented 5 years ago

@gkasprow my understanding is that the microprocessor has 3 ADCs, with muxes to give 24 channels. We should make sure to connect Vref0 to the same ADC as DAC_FB0, same for channel 1, to maximise accuracy.

gkasprow commented 5 years ago

Vref0 obraz

DAC FB0 obraz

Vref1 obraz

DAC FB1 obraz

so we can assin all these inputs to ADC1 or ADC2.