tp-freeforall/prod — Issue #16: gnode's cc1101 radio FSTEST is readonly register
Author: andresv
Created: 2012-03-06 08:58:41+00:00
State: open

I noticed that in gnode's cc1101 driver there is FSTEST register defined and actually value is written into that register, however actually it is read-only register.

Here it is defined: https://github.com/tp-freeforall/prod/blob/msp430-int/tos/platforms/gnode/chips/cc1101/ChipconRegisterValues.h

And here it is used: https://github.com/tp-freeforall/prod/blob/msp430-int/tos/platforms/gnode/chips/ccpacket/hal/HalChipconControlP.nc

```c
/**
* Load default configuration into registers.
*/
void configure() {
uint8_t i;
for (i = 0; i < NUM_REGISTERS * 2; i += 2) {
uint8_t reg = chipconRegisterValues[i];
uint8_t value = chipconRegisterValues[i+1];
call HplChipconSpi.writeRegister(reg, value);
}
}
```

It seems to produce no sideeffects, but it is not correct.

I am not willing to remove it yet, maybe I am missing something. It is better if mkonstapel would comment this issue first.