Hello Tom,

On 11/23/2015 11:44 PM, Tom Rini wrote:
On Fri, Nov 20, 2015 at 08:06:16AM +0100, Lukasz Majewski wrote:

This commit provides definition and declaration of GPT verification
functions - namely gpt_verify_headers() and gpt_verify_partitions().
The former is used to only check CRC32 of GPT's header and PTEs.
The latter examines each partition entry and compare attributes such as:
name, start offset and size with ones provided at '$partitions' env
variable.

Signed-off-by: Lukasz Majewski <l.majew...@majess.pl>
Reviewed-by: Tom Rini <tr...@konsulko.com>
Reviewed-by: Przemyslaw Marczak <p.marc...@samsung.com>

Applied to u-boot/master, thanks!



_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Why merged so quickly?

I tested this patchset on my device and posted about the issues. [1]

This should be reworked, since the verify assumptions are too simple and doesn't fully match the GPT header creation. So this command will fail for some cases of write/verify sequence, depending on what the $partitions includes.

[1] https://www.mail-archive.com/u-boot@lists.denx.de/msg193216.html

Best regards,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marc...@samsung.com
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to