[U-Boot] [PATCH 3/3] ARM: am57xx_evm: enable DFU support

[Sekhar Nori]

On Tuesday 30 August 2016 04:50 PM, Tom Rini wrote:
> 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.

Alright, I will start with dfu.h, and I guess over time we can move over
rest of the environment bits?

Thanks,
Sekhar