[OpenWrt-Devel] [PATCH] build: add mkrasimage

David Bauer mail at david-bauer.net
Sat Aug 18 21:04:24 EDT 2018


Hi Christian,

> I wrote the bash version because of issues in the past with C implementations
> of firmware packers for the Meraki routers (mkmerakifw.c and mkmerakifw-old.c).
> The issue there was that the main dev was coming from a sysadmin background
> and as with many OpenWrt users he had problems with C (endiannes!).
> 
> As for what implementation is better. Why not let Karl Palsson (or any
> other commentor that wants to join ;) ) decide?

I didn't want to start a race here, I was just more comfortable with 
finishing my C implementation as i have a problem with understanding 
such shell-scripts.

 From my understanding your linked script fixed the issues? I'm not sure 
why the script in OpenWRT is that old. IIRC i took it directly from your 
tree without further modifications.

> Note: The NBG6617 isn't the only device from ZyXEL that utilize the ras image.
> In fact, the "original" version of the notes above was written by Benjamin Berg
> and can still be found as a rather long comment in the NBG6616 image generation
> code under /target/linux/ar71xx/image/generic.mk [0]. (And there is at least
> one more device that can make use of ras: the NBG6817).
> 
> So chances are, there that whatever "wins" will need to be updated to support
> those routers as well.

Good point (didn't thought about that).  At first glance the NBG6817 
looks identical while the NBG6616 is missing the kernel-header part. I 
will see how i can support those two devices. There is also the NBG6716 
around, but his one is using legacy build-code. So for this patch i see 
it as out-of-scope (although the header looks identical to the NBG6616).

Best wishes
David

_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel



More information about the openwrt-devel mailing list