On Tue, Aug 30, 2016 at 12:03:32PM +0530, Sekhar Nori wrote:
Hi Tom,
On Saturday 27 August 2016 02:31 AM, Tom Rini wrote:
This is another example of needing to find a place and a way to put vendor specific stuff somewhere. Along with needing to figure out how to move environment stuff out of include/configs/ if we can. The half idea I'm toying with right now is include/environment/.... and making various fragments be included in env_default.h.
I suppose to start with, we could move the TI-as-vendor-suggested DFU args there, once we have board/ti/Kconfig for the other TI-as-vendor-suggested stuff :)
To be clear, you are asking for a new file "include/environment/ti/omap.h" to be created and DFU args for TI's OMAP-style platforms consolidated there in v2 of this patch series. Right?
Something inside of include/environment/ should be created, yes. For the exact name(s), I think a re-read of all of the environment bits in include/configs/ti_* needs to be taken into consideration too, and the possibly used combinations there-of. My first thought is that you'd want a dfu.h for the DFU related parts. And something-else.h for other things and so forth.