Attention is currently required from: Hoernchen. lynxis lazus has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42157?usp=email )
Change subject: ccid: fix rdr_to_pc_parameters and rdr_to_pc_hardware_error ...................................................................... Patch Set 1: Code-Review-1 (1 comment) Commit Message: https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42157/comment/75f5fc67_bd7bd1bd?usp=email : PS1, Line 8: > The title literally says that "what was wrong" is "ccid:" and two named > messages, so two ccid messag […] You're fixing two messages which were wrong in different ways and had different problems. Also those changes are quiet important. Please split up the commit in two commits: One for the missing bProtocolNum / ccid_gen_parameters One for the wrong interrupt header in ccid_rdr_to_pc_hardware_error. -- To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/42157?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: comment Gerrit-Project: osmo-ccid-firmware Gerrit-Branch: master Gerrit-Change-Id: I67d5cb876d20b29f4759d5b194606f34b8b966c4 Gerrit-Change-Number: 42157 Gerrit-PatchSet: 1 Gerrit-Owner: Hoernchen <[email protected]> Gerrit-Reviewer: Jenkins Builder Gerrit-Reviewer: lynxis lazus <[email protected]> Gerrit-Reviewer: pespin <[email protected]> Gerrit-Attention: Hoernchen <[email protected]> Gerrit-Comment-Date: Sat, 21 Feb 2026 14:55:48 +0000 Gerrit-HasComments: Yes Gerrit-Has-Labels: Yes Comment-In-Reply-To: Hoernchen <[email protected]> Comment-In-Reply-To: lynxis lazus <[email protected]>
