Attention is currently required from: pespin.

Timur Davydov 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 7:

(1 comment)

Patchset:

PS4:
> What I'm still missing here in order to properly review, is an explanation on 
> why do you need a new  […]
I've tried to reuse as much of the existing code as possible.

The WebSDR setup has different transport requirements compared to the existing 
implementations. In this case bursts are not exchanged through the usual 
socket/file-descriptor path. Instead they are prepared and passed via callbacks 
to the web frontend, which then communicates with the SDR device over WebUSB.

Originally I placed the WebSDR-related code in separate files in order to keep 
the existing implementation as untouched as possible. The intention was to 
reuse the original logic where possible and only provide alternative 
implementations for the parts where the transport mechanism differs.



--
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: 7
Gerrit-Owner: Timur Davydov <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: pespin <[email protected]>
Gerrit-Comment-Date: Wed, 18 Mar 2026 12:21:47 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: pespin <[email protected]>

Reply via email to