Attention is currently required from: Timur Davydov, pespin.

Jenkins Builder has posted comments on this change by Timur Davydov. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/42411?usp=email )

Change subject: transceiver: add optional WebSDR device support
......................................................................


Patch Set 9:

(11 comments)

File Transceiver52M/Transceiver.h:

Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/4bc93112_23854306?usp=email
 :
PS9, Line 246:   int driveReceiveRadioShortVector(std::vector<short *> &bufs, 
int samples, int underrun,
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/a4d30d4a_47c3e2f4?usp=email
 :
PS9, Line 247:                                                                 
TIMESTAMP ts, unsigned samples_skipped);
code indent should use tabs where possible


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/e30d2bd9_1edce613?usp=email
 :
PS9, Line 249:   bool driveReceiveRadio(unsigned samples_skipped = 0);
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/ab25580d_7a7ac85f?usp=email
 :
PS9, Line 258:   bool driveReceiveFIFO(size_t chan, char *burstBuffer = 
nullptr, size_t burstBufferLen = 0);
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/44175132_47a56361?usp=email
 :
PS9, Line 264:     @brief modulation and sorting of GSM bursts from GSM core
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/74bcb7c6_5d6abc9a?usp=email
 :
PS9, Line 265:     @param chan channel number
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/35d71303_01e0d0da?usp=email
 :
PS9, Line 266:     @param burstBuffer optional buffer containing burst to 
transmit, if not provided, will read from data socket
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/4b8fe87c_49ebba63?usp=email
 :
PS9, Line 267:     @param burstBufferLen optional length of burst buffer, if 
not provided, will read from data socket
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/ade66149_a9e9eff5?usp=email
 :
PS9, Line 270:   bool driveTxPriorityQueue(size_t chan, char *burstBuffer = 
nullptr, size_t burstBufferLen = 0);
please, no spaces at the start of a line


File Transceiver52M/radioInterface.h:

Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/ce558366_c1e7fcc5?usp=email
 :
PS9, Line 118:   int driveReceiveRadio(unsigned samples_skipped = 0);
please, no spaces at the start of a line


Robot Comment from checkpatch (run ID ):
https://gerrit.osmocom.org/c/osmo-trx/+/42411/comment/75c851e4_33a74666?usp=email
 :
PS9, Line 137:   int fillPullBuffer(std::vector<short *> &bufs, size_t samples, 
int recvunderrun, TIMESTAMP ts);
please, no spaces at the start of a line



--
To view, visit https://gerrit.osmocom.org/c/osmo-trx/+/42411?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: comment
Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: Ia0d340c323c2eea28fbe82601ba0af7cfbd68f6d
Gerrit-Change-Number: 42411
Gerrit-PatchSet: 9
Gerrit-Owner: Timur Davydov <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Attention: Timur Davydov <[email protected]>
Gerrit-Comment-Date: Wed, 18 Mar 2026 13:11:42 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to