[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 */
+    (uint32_t)0x000188A3,
+    /* 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:
https://github.com/RIOT-OS/RIOT/pull/13295#discussion_r423119875
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20200511/92814d04/attachment.htm>


More information about the notifications mailing list