Philippe Teuwen

Results 226 comments of Philippe Teuwen
trafficstars

currently the timestamp in version.c is when version.c is created, not the zip folder timestamp.

Hmm no, it's just about supported features so far. What we need is kind of API version check. Ideally every time a command has been changed both on arm &...

@iceman1001 Any reason why mbedtls is in common/ ? I though common/ is for common stuff between bootrom & armsrc while mbedtls is used only in client.

Today: * common: code used only in armsrc & bootrom and not client (I think) * common/mbedtls: code used only in client For AES/DES in pm3, mbedtls would be very...

ok we'll see when the first one will make the effort of compiling mbedtls in armsrc... BTW this will very probably require two different common/mbedtls/config.h for client & pm3

Yep, that's the idea, and see how fast we can get

stupid idea, just implement real time sniffing over USB-CDC

"ARM" We only did *some* warnings on *client* so far. With `mymakeclang.sh`: * bootrom: 115 warinings * armsrc 2354 warnings * client 302 warnings

I added stats to the script: ``` client: 11 [-Wcast-align] only with clang, in src/cmdtrace.c, casts of trace 29 [-Wswitch-enum] 92 [-Wunused-parameter] 165 [-Wsign-compare] fpga_compress: mfkey: nonce2key: bootrom: 1 [-Wsign-compare]...

I splitted the script into host & arm and unified warnings gcc/clang, host : ``` [=] LD nonce2key g++ obj/crypto1.o obj/crapto1.o obj/bucketsort.o obj/nonce2key.o -o nonce2key CLANG ===== Total: 297 client:...