我正在使用以下宏功能
#define setAsOutput(bit, i) { *bit ## _DDR[i] |= (1 << bit[i]); }
简化定义和设置一些寄存器值
// registers
volatile uint8_t *FOO_DDR[] = {&DDRA, &DDRA};
uint8_t FOO[] = {PA1, PA2};
setAsOutput(FOO, 0);
// these are defined somewhere else
#define PA1 1
#define PA2 2
#define DDRA _SFR_IO8(0X01)
这给出了相当于
的代码DDRA |= (1 << PA1);
然而,volatile uint8_t *FOO_DDR[] = {&DDRA, &DDRA};
行实际上是多余的,因为A
中的DDRA
始终在FOO
值中重复,即PA1
和{{1} }。
理想情况下,它可以完全删除,宏变为类似
PA2
但似乎无法获取#define setAsOutput(bit, i) { DDR ## <second char of bit[i]> |= (1 << bit[i]); }
的名称的第二个字符。
有没有办法重写宏函数,因此bit[i]
不需要明确定义,而是可以隐含FOO_DDR
?
答案 0 :(得分:1)
如果您能提供MCVE通常会有帮助,以便其他人可以轻松编译您的代码,查看其工作原理,并尝试调整它。
您不需要在代码中定义DDRA
和PA1
之类的内容。只需将适当的选项传递给编译器,以指定您正在使用的AVR(例如-mmcu=atmega1284p
),然后在程序顶部添加#include <avr/io.h>
以获取这些定义。将这些定义从io.h
复制到StackOverflow上的问题通常没什么意义,因为它们非常标准。这些定义来自avr-libc,所以如果你真的想提供这些细节,你可以说你正在使用什么版本的avr-libc。
您问题的一个主要前提是您使用数组和宏发布的代码等同于DDRA |= (1 << PA1);
。不幸的是,这个前提是不正确的。当GCC看到DDRA |= (1 << PA1);
时,它实际上可以将其编译为单个原子AVR指令,该指令设置DDRA寄存器的第1位。当GCC看到你的代码时,它会做一些更复杂的事情,最终会读取,写入和修改寄存器。因此,如果中断可能会修改DDRA寄存器,则阵列代码会浪费CPU周期并且不安全使用。
如果你不相信我,你可以看看这个godbolt.org链接,它比较两种方法的汇编:
看起来您只需向数组中添加const
限定符即可解决此问题。然后编译器将知道您的数组在编译时保存的值,并且可以生成良好的代码。
volatile uint8_t * const FOO_DDR[] = {&DDRA, &DDRA};
uint8_t const FOO[] = {PA1, PA2};
现在回答您的主要问题,即如何摆脱冗余阵列。我不认为有一种简单的方法可以做到这一点,并且在程序中有两个const数组并不是什么大问题,它们可能会在编译时被优化掉。你可以做的是扩展这些数组,使它们包含芯片上每个引脚的条目。然后,当您想要写入引脚时,您只需使用一个引脚号,它是数组的索引(而不是需要定义新数组)。绝大多数代码将处理这些引脚号,而不用担心数组。所以这就是我写它的方式:
#include <avr/io.h>
// Here is a general GPIO library for your chip.
// TODO: expand these arrays to cover every pin on the chip
#define setAsOutput(i) { *pin_dir[i] |= (1 << pin_bit[i]); }
#define setHigh(i) { *pin_value[i] |= (1 << pin_bit[i]); }
static volatile uint8_t * const pin_dir[] = {
&DDRA, // Pin 0
&DDRA, // Pin 1
};
static volatile uint8_t * const pin_value[] = {
&PORTA, // Pin 0
&PORTA, // Pin 1
};
static const uint8_t pin_bit[] = {
PA1, // Pin 0
PA2, // Pin 1
};
// Pin definitions for your particular project.
// (e.g. pin 0 is connected to a green LED)
#define GREEN_LED_PIN 0
void nice()
{
setAsOutput(GREEN_LED_PIN);
setHigh(GREEN_LED_PIN);
}
上面的每个GPIO函数调用最终都会编译成一个汇编指令。
如果你在Arduino核心代码中挖掘,你会发现这样的数组。 (但Arduino人错误地在pinMode
和digitalWrite
函数中以浪费的方式访问这些数组。)
请注意,使用上面提供的代码,您很可能会意外地传递一个不是编译时常量的引脚号,因此编译器将无法优化它,并产生浪费/不安全的代码。这就是使用内联汇编和FastGPIO library之类的C ++模板更好的原因之一。
答案 1 :(得分:1)
如果希望代码等效于DDRA |= (1 << PA1);
- 即在编译时没有读/写数组和指向IO寄存器的指针的最简单指令。你可以这样做。
1)我们假设我们已经在某处定义(例如通过<avr/io.h>
)
#define PA1 1
#define PA2 2
...
#define DDRA _SFR_IO8(0X01)
#define PB1 1
#define PB2 2
...
#define DDRB _SFR_IO8(0X01)
2)你想要这样的声明:
#define BIG_RED_LED PA1
#define SMALL_GREEN_LED PB2
然后只是像
一样使用它们setAsOutput(BIG_RED_LED);
setAsOutput(SMALL_GREEN_LED);
setLow(BIG_RED_LED);
setHigh(SMALL_GREEN_LED);
等,其中每一行都是对相应IO寄存器中的BIT的简单写入。
要实现这一目标,您可以定义大量的
#define DDR_PA0 DDRA
#define PORT_PA0 PORTA
#define PIN_PA0 PINA
#define DDR_PA1 DDRA
#define PORT_PA1 PORTA
#define PIN_PA1 PINA
...
#define DDR_PB0 DDRB
#define PORT_PB0 PORTB
#define PIN_PB0 PINB
...
然后
#define setAsOutput(px) { DDR_ ## px |= (1 << px); }
#define setHigh(px) { PORT_ ## px |= (1 << px); }
#define setLow(px) { PORT_ ## px &= ~(1 << px); }
etc.
然后,每次在您的代码中发生类似setAsOutput(PA1)
的内容时,它将被编译为完全与DDRA | =(1&lt;&lt;&lt;&lt;&lt;&lt; PA1);
但是
如果你想将它们存储在数组中并通过数组索引进行访问,就像在你的例子中一样,那么除了定义两个数组或结构数组之外你没有别的办法,其中两个元素都包含位数或位掩码和指向IO /寄存器的指针。
因为虽然名称PA1
PA2
等中有A
个字母,但在运行时它会被编译成它的值。即'PA1'将为1,但PB1
也将为1。因此,只考虑该数组中的索引,编译器无法知道访问哪个寄存器。
但在这里,我可以给你一些小生命:
1)由于寄存器PINx,DDRx,PORTx是almsot总是按顺序连续运行(参见数据手册中的寄存器组汇总),你不需要全部存储,只需存储对PINx寄存器的引用,并且计算DDRx和PORTx的位置只需将1或2添加到地址,因为AVR具有通过位移来纠正存储器访问的指令,代码将足够有效。
2)这些寄存器位于较低的存储器地址中,因此您可以将它们转换为byte
而不是存储2/4字节的指针,并在访问时将它们转换回指针。它不仅可以节省空间,还可以加快速度。此外,将这种表存储在闪存中也是一种好习惯,而不是浪费RAM。
3)AVR架构只有一个位移位指令,因此(1 uint8_t FOO[] = {(1 << PA1), (1 << PA2)};
,而不是存储预先计算的掩码值。
答案 2 :(得分:0)
最后,我使用了_MMIO_BYTE
中的avr/sfr_defs.h
宏函数,将新的位操作函数基于:
#define SET_OUTPUT(pin) (_MMIO_BYTE(OFFSET_ADDR((pin)[0] + 0x1)) |= _BV((pin)[1]))
#define SET_INPUT(pin) (_MMIO_BYTE(OFFSET_ADDR((pin)[0] + 0x1)) &= ~_BV((pin)[1]))
// etc
这提供了简单的引脚定义,如引脚阵列或单个引脚:
#define NUM_LEDS 3
const uint16_t LEDS[NUM_LEDS][2] = {
{PB, 4},
{PB, 5},
{PB, 6}
};
const uint16_t BUTTON[2] = {PB, 7};
然后可以像这样操纵引脚:
SET_INPUT(BUTTON);
ENABLE_PULLUP(BUTTON);
for (int i = 0; i < NUM_LEDS; ++i) {
SET_OUTPUT(LEDS[i]);
SET_HIGH(LEDS[i]);
}
源代码在这里:https://github.com/morefigs/avr-bit-funcs。
这仅是为Mega 2560编写的,但应易于适应其他主板。