[OpenWrt-Devel] Remove ralink_default_fw_size_xxx variables

[Adrian Schmutzler]

Hi,

I've recently made an effort to check and fix IMAGE_SIZE variables in ramips target:

https://github.com/openwrt/openwrt/pull/2226

The target has four predefined variables ralink_default_fw_size_4M, ralink_default_fw_size_8M, ralink_default_fw_size_16M, ralink_default_fw_size_32M.

When checking/fixing IMAGE_SIZEs, I've encountered many cases where the IMAGE_SIZE was just set to a predefined $(ralink_default_fw_size_xxx) roughly matching the flash size, no matter what the actual firmware partition size was.
I've felt the same when looking at some recent PRs adding devices.

The more I think about this topic and the more I look at recent pull requests, the bigger my desire grows to remove ralink_default_fw_size_xxx variables completely. If they are not based on some inherent board property, but just on frequent choice of partitioning (and that's how it looks to me), it might be better to remove them.
This way the user is not tricked into assuming he can just put a variable without thinking.

Replacing variables by numbers is an easy sed job, so each device bears its IMAGE_SIZE in kiB (unless there is some common definition).

I've already done that in the second-to-last patch of my linked PR ("ramips: Remove ralink_default_fw_size_xxx variables").

Does anyone have an opposite opinion on this one?

Best

Adrian
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 834 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/openwrt-devel/attachments/20190722/b892f06f/attachment.sig>
-------------- next part --------------
_______________________________________________
openwrt-devel mailing list
openwrt-devel at lists.openwrt.org
https://lists.openwrt.org/mailman/listinfo/openwrt-devel