Dear Takahiro,
In message 20190719083548.GU21948@linaro.org you wrote:
+#if defined(CONFIG_ENV_FAT_INTERFACE) && \
- defined(CONFIG_ENV_FAT_DEVICE_AND_PART) && defined(CONFIG_ENV_FAT_FILE)
- [ENVCTX_UBOOT] = {
CONFIG_ENV_FAT_INTERFACE,CONFIG_ENV_FAT_DEVICE_AND_PART,CONFIG_ENV_FAT_FILE,- },
+#endif +};
Does it make sense to define this in a FAT specific way? I guess we could come up with a common structure that covers all supported storage devices and use this here.
But a different driver has different sets of configurations required. How can we *generalize* them?
Does it? At least all block devices probably share the properties of interface, device, partition and file name in one way or another. Or offset instead of file name when accessing the raw device.
I think it should be possible to at least support all file systems with exactly the sme code - ther eis nothing special about FAT here, or am I missing something?
Also, the "#if defined" looks really dangerous to me, as missing #defines will go unnoticed, and in the end you are accessing uninitialized data...
Yes, I have made this mistake before. But I think that all the drivers must be verified in any case before any changes are applied to the upstream. No?
Indeed, testing will be needed.
Best regards,
Wolfgang Denk