[U-Boot] [PATCH v4 2/7] part:efi: Move part_efi.h file to ./include

Lukasz Majewski l.majewski at samsung.com
Tue Nov 20 18:46:08 CET 2012


Hi Tom,

> On Mon, Nov 19, 2012 at 12:30:02PM -0700, Stephen Warren wrote:
> > On 11/09/2012 02:22 AM, Piotr Wilczek wrote:
> > > From: Lukasz Majewski <l.majewski at samsung.com>
> > > 
> > > This move is necessary to export gpt header and GPT partition
> > > entries to be used with other commands or subsystems (like DFU in
> > > the future) Additionally the part_efi.h file has been cleaned-up
> > > to supress checkpatch's warnings.
> > 
> > >  {disk => include}/part_efi.h |    0
> > 
> > I can understand the "gpt" command perhaps needing access to the
> > GPT-specific types, but I would hope that anything DFU-related
> > would be using types from include/part.h; I don't imagine that DFU
> > would need to know anything about GPT at all?
> 
> Indeed.  DFU must not get coupled to GPT (or for that matter,
> partitions) in the general code.  There should be "DFU write to a
> GPT-using backing store" code, if that's needed.  Not all eMMC would
> be GPT-using, for example.
> 

Yes, the commit description is now a bit misleading. 
The idea here is to have access to part_efi.h from gpt command.

DFU will use higher level of abstraction (as Stephen once suggested).

-- 
Best regards,

Lukasz Majewski

Samsung Poland R&D Center | Linux Platform Group


More information about the U-Boot mailing list