commit 7d714a24d7258db1188abe3c7c9c46a1cc7ab8be finalized adding
support for multiple env locations in U-Boot and I'm really thankful
for this work!
I would like to extend this such that each env can have a unique
size/offset/redund configuration. This would affect all the env
drivers so before I code it up I wanted to see if there was agreement
on how this should be done.
We could define a new struct in environment.h:
struct env_details
{
enum env_location loc;
unsigned long size;
unsigned long offset;
unsigned long offset_redund;
};
This can be added as a parameter to env_get_location(..) and would get
initialized such that size=CONFIG_ENV_SIZE, offset=CONFIG_ENV_OFFSET,
offset_redund=CONFIG_ENV_OFFSET_REDUND to that current users could do
nothing for current before but boards that are interested could differ
the details of the storage based on the type.
Then these details would need to be passed to each env drivers
init/load/save functions to be used instead of the currently hard
coded configs (CONFIG_ENV_SIZE, CONFIG_ENV_OFFSET, and
CONFIG_ENV_OFFSET_REDUND). There are likely some more I'll run into
along the way such as hardware partition etc.
Any opposing thoughts or suggestions before I code this up and submit a patch?
Tim
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
