[U-Boot] [RFC PATCH v2 3/8] disk: part: add support of GPT partitioning over MTD

Simon Glass sjg at chromium.org
Thu Dec 1 03:21:13 CET 2016


Hi Patrick,

On 30 November 2016 at 04:01, Patrick Delaunay
<patrick.delaunay73 at gmail.com> wrote:
> From: Patrick Delaunay <patrick.delaunay at st.com>
>
> Signed-off-by: Patrick Delaunay <patrick.delaunay at st.com>
> Signed-off-by: Patrick Delaunay <patrick.delaunay73 at gmail.com>
> ---
>
> Changes in v2: None
>
>  Kconfig            |  12 ++
>  disk/part_efi.c    | 445 +++++++++++++++++++++++++++++++++++++++++++++++++++++
>  doc/README.gpt.mtd | 189 +++++++++++++++++++++++
>  include/part.h     |  13 +-
>  4 files changed, 657 insertions(+), 2 deletions(-)
>  create mode 100644 doc/README.gpt.mtd

This seems OK to me, but here are some general comments:

- Return -EINVAL or something like that instead of -1
- malloc()/calloc() can fail, so check for NULL and return -ENOMEM
- how about putting your changes todisk/part_efi.c in their own file?

Regards,
Simon


More information about the U-Boot mailing list