On 12/21/2013 07:52 PM, Rafał Miłecki wrote: > 2013/12/5 Cody P Schafer <de...@codyps.com>: >> F7D3301/F7D7301, F7D3302/F7D7302, and F7D4301 replace the "HDR0" with a >> custom byte sequence (which appears to be a date). Also, they all >> appear to support a single "QA" byte sequence. >> >> Just support the "QA" byte sequence, we don't need the others. > > Your patch seems to do the opposite by supporting all magics, not just > "QA". I'm little afraid of that. As you noted by yourself, these > magics are probably just dates. I'm afraid of having one magic per > device or even more. > > So why your patch actually adds support for detecting all these MAGICs > instead of just QA (like noted in the commit message)?
I think firmware upgrade over the Web administration interface only works with these dates as magic and not with this QA, that works only over CFE, the commit message should be fixed. I am a little frightened that we would detected the wrong bytes as a trx header and then apply a wrong partition design. As far as I know the trx version is always 1 or 2, so we could add an additional check for that. See this, it describes the trx header: https://github.com/RMerl/asuswrt-merlin/blob/master/release/src-rt-6.x.4708/include/trxhdr.h Hauke _______________________________________________ openwrt-devel mailing list openwrt-devel@lists.openwrt.org https://lists.openwrt.org/cgi-bin/mailman/listinfo/openwrt-devel
