Change in osmo-ccid-firmware[master]: t0: fix wtime calculation, for real this time

This is merely a historical archive of years 2008-2021, before the migration to mailman3.

A maintained and still updated list archive can be found at https://lists.osmocom.org/hyperkitty/list/gerrit-log@lists.osmocom.org/.

Hoernchen gerrit-no-reply at lists.osmocom.org
Sat Oct 17 17:23:49 UTC 2020


Hello Jenkins Builder, 

I'd like you to reexamine a change. Please visit

    https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/20691

to look at the new patch set (#2).

Change subject: t0: fix wtime calculation, for real this time
......................................................................

t0: fix wtime calculation, for real this time

ETU calculation is 1 etu = fi/di * 1/f as per chapter 7.1.

wtime/WWT is usually mentioned as being 9600 etu in iso 7816-3 - but
it's actually specified for t=0 as WT=WI*960*Fi/f in chapter 10.2 - no
di involved here, so the rightmost fraction is not ETU!

-> as soon as di is not 1 after a pps exchange the timeout value was off
by a factor of di if it was just used as a multiplier for ETU.

Change-Id: Icd4fa83e1177873e90200529331d3a1fce8be1c6
---
M ccid_common/ccid_slot_fsm.c
1 file changed, 2 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-ccid-firmware refs/changes/91/20691/2
-- 
To view, visit https://gerrit.osmocom.org/c/osmo-ccid-firmware/+/20691
To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings

Gerrit-Project: osmo-ccid-firmware
Gerrit-Branch: master
Gerrit-Change-Id: Icd4fa83e1177873e90200529331d3a1fce8be1c6
Gerrit-Change-Number: 20691
Gerrit-PatchSet: 2
Gerrit-Owner: Hoernchen <ewild at sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-MessageType: newpatchset
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osmocom.org/pipermail/gerrit-log/attachments/20201017/cf69bfb1/attachment.htm>


More information about the gerrit-log mailing list