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. -- Tom
signature.asc
Description: Digital signature
_______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot