Attention is currently required from: osmith.

Hoernchen has posted comments on this change. ( 
https://gerrit.osmocom.org/c/osmo-trx/+/33737 )

Change subject: USRPDevice:updateAlignment: remove byteswap code
......................................................................


Patch Set 2: Code-Review+1

(1 comment)

Patchset:

PS2:
union {
        short data[4] = {0x00, 0x02, 0x00, 0x00};
        uint32_t d4[2];
    };
    uint32_t *wordPtr = &d4[0];
    *wordPtr = host_to_usrp_u32(*wordPtr);

    might work,  at least the compiler does not complain, but



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

Gerrit-Project: osmo-trx
Gerrit-Branch: master
Gerrit-Change-Id: I806d8c1432cb20efca1830a2752a4cbc70384b54
Gerrit-Change-Number: 33737
Gerrit-PatchSet: 2
Gerrit-Owner: osmith <[email protected]>
Gerrit-Reviewer: Hoernchen <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: pespin <[email protected]>
Gerrit-Attention: osmith <[email protected]>
Gerrit-Comment-Date: Thu, 13 Jul 2023 12:41:54 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment

Reply via email to