基于BasedHardware/Friend项目的按钮状态检测问题分析

问题背景

在基于BasedHardware/Friend项目的Omi Triangle v2硬件构建过程中，开发者遇到了按钮功能异常的问题。具体表现为按钮点击或按下释放时没有任何响应。经过排查发现，问题可能出在按钮状态检测的逻辑实现上。

技术分析

在项目中，按钮状态的检测是通过gpio_pin_get_raw函数实现的，该函数返回GPIO引脚的电平状态：

• 返回1表示引脚物理电平为高
• 返回0表示引脚物理电平为低

在原始代码中，按钮按下时的处理逻辑如下：

void button_pressed_callback(const struct device *dev, struct gpio_callback *cb,
            uint32_t pins)
{
    int temp = gpio_pin_get_raw(dev,d5_pin_input.pin); 
    printf("button_pressed_callback %d\n", temp);
    if (temp) 
    {
        was_pressed = false;
    }
    else 
    {
        was_pressed = true;
    }
}

问题根源

根据硬件电路设计原理，当按钮按下时，电路导通，D5引脚的物理电平应该为高电平（返回1）。然而原始代码中却将高电平状态对应设置为was_pressed = false，这与预期逻辑相反。

开发者通过测试验证了这一现象：

1. 使用原始代码时，按钮测试输出不符合预期，无法区分单击、双击和长按
2. 将逻辑反转后（高电平对应was_pressed = true），按钮功能恢复正常，测试输出符合预期

解决方案建议

针对这一问题，建议采取以下解决方案：

1. 修正状态检测逻辑：将高电平状态正确映射到按钮按下状态
2. 添加硬件说明：在文档中明确按钮电路设计原理，说明按下时GPIO的电平变化
3. 增加调试输出：在回调函数中加入更多调试信息，便于后续问题排查

修正后的代码示例如下：

void button_pressed_callback(const struct device *dev, struct gpio_callback *cb,
        uint32_t pins)
{
    int temp = gpio_pin_get_raw(dev,d5_pin_input.pin);
    printf("button_pressed_callback %d\n", temp);
    if (temp) 
    {
        was_pressed = true;  // 高电平表示按钮按下
    }
    else 
    {
        was_pressed = false; // 低电平表示按钮释放
    }
}

经验总结

在嵌入式开发中，GPIO状态检测是一个常见但容易出错的部分。开发者需要注意：

1. 理解硬件电路设计，明确按钮按下时的电平变化
2. 确保软件逻辑与硬件行为一致
3. 通过实际测试验证功能是否符合预期
4. 添加充分的调试信息，便于问题定位

这个问题也提醒我们，在开源项目协作中，清晰的硬件文档和代码注释对于其他开发者正确理解和使用项目至关重要。