forked frommicropython/micropython-lib
- Notifications
You must be signed in to change notification settings - Fork0
Commit60d1370
lora-sx126x: Change to class-level memoryview for _cmd buf.
Currently, the LoRa SX126x driver dynamically creates at least one,sometimes two, memoryview objects with each call to `_cmd`. This commitsimply provides the class with a long-lived memoryview object for `_cmd` toeasily slice as necessary.Unlike the SX127x chips, Semtech unfortunately designed the SX126x modemsto be more command-centric (as opposed to directly setting registers).Given the amount `_cmd` is called during normal device operation, even aminor improvement here should have a decent impact.Basic TX and RX tests pass on hardware.Signed-off-by: Max Holliday <maholli@stanford.edu>1 parentfbf7e12 commit60d1370
2 files changed
+7
-7
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
152 | 152 | | |
153 | 153 | | |
154 | 154 | | |
155 | | - | |
| 155 | + | |
156 | 156 | | |
157 | 157 | | |
158 | 158 | | |
| |||
704 | 704 | | |
705 | 705 | | |
706 | 706 | | |
707 | | - | |
| 707 | + | |
708 | 708 | | |
709 | | - | |
710 | | - | |
711 | | - | |
| 709 | + | |
| 710 | + | |
| 711 | + | |
712 | 712 | | |
713 | 713 | | |
714 | 714 | | |
| |||
723 | 723 | | |
724 | 724 | | |
725 | 725 | | |
726 | | - | |
| 726 | + | |
727 | 727 | | |
728 | 728 | | |
729 | 729 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
1 | | - | |
| 1 | + | |
2 | 2 | | |
3 | 3 | | |
0 commit comments
Comments
(0)