[riot-notifications] [RIOT-OS/RIOT] CC1312R Sub-GHz netdev driver (Proof of Concept) (#13295)
Jean Pierre Dudey
notifications at github.com
Mon May 11 17:22:52 CEST 2020
@jeandudey commented on this pull request.
> + (uint32_t)0x00F788D3,
+ /* override_tc146.xml
+ * Tx: Configure PA ramp time, PACTL2.RC=0x3 (in ADI0, set PACTL2[4:3]=0x3)
+ RFC_ADI_2HALFREG_OVERRIDE(0, 16, 0x8, 0x8, 17, 0x1, 0x1),
+ /* Tx: Configure PA ramping, set wait time before turning off (0x1A ticks
+ of 16/24 us = 17.3 us).
+ RFC_HW_REG_OVERRIDE(0x6028, 0x001A),
+ /* Rx: Set AGC reference level to 0x16 (default: 0x2E) */
+ RFC_HW_REG_OVERRIDE(0x609C, 0x0016),
+ /* Rx: Set RSSI offset to adjust reported RSSI by -1 dB (default: -2),
+ * trimmed for external bias and differential configuration */
+ /* Rx: Set anti-aliasing filter bandwidth to 0x8 (in ADI0, set
+ * IFAMPCTL3[7:4]=0x8) */
+ RFC_ADI_HALFREG_OVERRIDE(0, 61, 0xF, 0x8),
+ /* TX power override
+ * Tx: Set PA trim to max to maximize its output power (in ADI0,
+ * set PACTL0=0xF8) */
+ RFC_ADI_REG_OVERRIDE(0, 12, 0xF8),
I haven't moved them to proper defines because it's automatically generated by a TI tool, I don't know if it makes sense to manually change them. Sorry for not clarifying it with a comment on the code, should I move the magic values to defines?
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the notifications