Attention is currently required from: dexter, fixeria.

laforge has posted comments on this change by dexter. ( 
https://gerrit.osmocom.org/c/pysim/+/37839?usp=email )

Change subject: pySim-shell: prevent opening/closing logical channel 0
......................................................................


Patch Set 7: Code-Review+1

(1 comment)

File pySim-shell.py:

https://gerrit.osmocom.org/c/pysim/+/37839/comment/4cd0fa48_d493a729?usp=email :
PS4, Line 968: choices=range(1,4),
> Yes, range(1,4) excludes the 4 (i find that a bit confusing, bit it seems to 
> be some half open inter […]
sysmocom cards only support basic logical channels, while pySim contains  
(untested) support for extended logical channels. The ATR tells us about the 
support:

```
ATR: 3B 9F 96 80 1F 87 80 31 E0 73 FE 21 1B 67 4A 35 75 30 35 02 51 CC
...
      Command chaining, length fields and logical channels: 1B
        - Logical channel number assignment: by the interface device and card
        - Maximum number of logical channels: 4
```



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

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I4eebd9f15fadd18e1caeb033fda36c59446fcab8
Gerrit-Change-Number: 37839
Gerrit-PatchSet: 7
Gerrit-Owner: dexter <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <[email protected]>
Gerrit-CC: fixeria <[email protected]>
Gerrit-Attention: fixeria <[email protected]>
Gerrit-Attention: dexter <[email protected]>
Gerrit-Comment-Date: Tue, 27 Aug 2024 14:15:11 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Comment-In-Reply-To: fixeria <[email protected]>
Comment-In-Reply-To: dexter <[email protected]>

Reply via email to