Embassy-rs项目中STM32 ADC通道选择问题的技术分析

问题背景

在嵌入式开发中，模数转换器(ADC)是常用的外设之一，用于将模拟信号转换为数字信号。在embassy-rs项目中，针对STM32系列微控制器的ADC驱动实现存在一个潜在的问题，特别是在通道选择方面。

问题描述

在STM32 L0系列微控制器的ADC实现中，发现通道选择逻辑可能存在错误。具体表现为：

1. 当使用PA4引脚作为ADC输入时，代码将通道号4直接写入寄存器
2. 但根据STM32参考手册，ADC通道选择寄存器(CHSELR)需要的是位掩码形式，即1<<n，而不是直接的通道号

技术细节分析

ADC通道选择寄存器结构

STM32的ADC通道选择寄存器(CHSELR)是一个位掩码寄存器，每个位对应一个通道：

• 位0：通道0选择
• 位1：通道1选择
• ...
• 位18：通道18选择

要选择某个通道，需要将对应位置1，而不是写入通道号本身。

当前实现的问题

当前代码中，impl_adc_pin!宏定义如下：

macro_rules! impl_adc_pin {
    ($inst:ident, $pin:ident, $ch:expr) => {
        impl crate::adc::AdcChannel<peripherals::$inst> for crate::peripherals::$pin {}
        impl crate::adc::SealedAdcChannel<peripherals::$inst> for crate::peripherals::$pin {
            #[cfg(any(adc_v1, adc_l0, adc_v2, adc_g4, adc_v4))]
            fn setup(&mut self) {
                <Self as crate::gpio::SealedPin>::set_as_analog(self);
            }

            fn channel(&self) -> u8 {
                $ch
            }
        }
    };
}

问题出在channel()方法直接返回了通道号，而没有转换为位掩码形式。

正确的实现方式

根据STM32 HAL头文件定义，正确的做法应该是返回1<<n的位掩码：

#define ADC_CHSELR_CHSEL4_Pos      (4U)       
#define ADC_CHSELR_CHSEL4_Msk      (0x1UL << ADC_CHSELR_CHSEL4_Pos) /*!< 0x00000010 */
#define ADC_CHSELR_CHSEL4          ADC_CHSELR_CHSEL4_Msk           /*!< Channel 4 selection */

因此，Rust实现应该修改为：

fn channel(&self) -> u8 {
    1 << $ch
}

底层寄存器访问分析

在embassy-rs的寄存器访问层，set_chselx方法的实现是正确的：

pub fn set_chselx(&mut self, n: usize, val: bool) {
    assert!(n < 19usize);
    let offs = 0usize + n * 1usize;
    self.0 = (self.0 & !(0x01 << offs)) | (((val as u32) & 0x01) << offs);
}

这个方法已经正确处理了位操作，只需要传入正确的通道号即可。问题在于上层提供的通道号格式不正确。

影响范围

这个问题可能影响以下STM32系列：

• ADCv1版本
• ADCv2版本
• ADCv4版本
• STM32L0系列
• STM32G4系列

解决方案建议

1. 修改impl_adc_pin!宏，使其返回位掩码而非直接通道号
2. 或者修改read方法，在写入寄存器前将通道号转换为位掩码
3. 更新相关文档，明确通道选择的使用方式

总结

在嵌入式开发中，对硬件寄存器的精确操作至关重要。STM32的ADC通道选择需要特别注意寄存器格式要求。这个问题提醒我们在开发硬件抽象层时，必须仔细对照芯片参考手册，确保寄存器访问的正确性。同时，良好的文档和注释也能帮助开发者避免类似的错误。