Attention is currently required from: laforge.
Hoernchen has posted comments on this change by Hoernchen. ( https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437?usp=email )
Change subject: add fw update crcstub ......................................................................
Patch Set 12:
(2 comments)
File crc_code.c:
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/ca6428e5_5ae9e5d8... : PS10, Line 102: // uint32_t length = strtoul(argv[3], NULL, 16);
can this commented out code be removed? I guess every C developer knows that diferent base arguments […]
Done
File crc_params.c:
https://gerrit.osmocom.org/c/osmo-asf4-dfu/+/39437/comment/67c3abd4_39ba744e... : PS10, Line 5: // __attribute__((used, section(".crc_par"))) const uint32_t __CRC_End = 0xFFFFFFFF; /* Placeholder */
why do we have this commented-out code here? is this some earlier attempts? shouldn't it be removed?
those still exist in the ld file and might be needed in case the linker, gcc, make, objdump, objcopy, or any other part of this brittle house of cards breaks again and I need anther approach to patch the files.