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

[Tom Rini]

On Tue, Aug 30, 2016 at 04:55:35PM +0530, Sekhar Nori wrote:
> 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?

Yeah, we don't have to do all of the environment stuff at first, but it
does need to be moved as that's kind of the final blocker to 100%
Kconfig migration, long term :)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20160830/9566fc04/attachment.sig>