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

Tom Rini trini at ti.com
Mon Nov 19 21:41:41 CET 2012


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.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20121119/585a5e55/attachment.pgp>


More information about the U-Boot mailing list