[U-Boot] [PATCH v4 2/2] part: always enable part_get_info_ptr() for driver

Kever Yang kever.yang at rock-chips.com
Thu Nov 28 03:09:36 UTC 2019


Hi Tom, Simon,

  Is there anything I need to update for this patch?

Thanks,
- Kever

Kever Yang <kever.yang at rock-chips.com> 于2019年8月15日周四 下午4:32写道:

> The partition driver has its Kconfig option, and the part_get_info_prt()
> interface are mendatory interface for partition drivers, always enable the
> macro to make partition driver works correctly.
>
> Signed-off-by: Kever Yang <kever.yang at rock-chips.com>
> ---
>
> Changes in v4:
> - formate commit message to ~75 columns
>
> Changes in v3: None
> Changes in v2:
> - add patch to use SPL_PARTITIONS so that we don't add disk driver for
>   boards who don't need it.
>
>  include/part.h | 11 ++---------
>  1 file changed, 2 insertions(+), 9 deletions(-)
>
> diff --git a/include/part.h b/include/part.h
> index ebca546db5..7d00fae56f 100644
> --- a/include/part.h
> +++ b/include/part.h
> @@ -241,22 +241,15 @@ static inline int blk_get_device_part_str(const char
> *ifname,
>  #endif
>
>  /*
> - * We don't support printing partition information in SPL and only support
> - * getting partition information in a few cases.
> + * We don't support printing partition information in SPL
>   */
>  #ifdef CONFIG_SPL_BUILD
>  # define part_print_ptr(x)     NULL
> -# if defined(CONFIG_SPL_FS_EXT4) || defined(CONFIG_SPL_FS_FAT) || \
> -       defined(CONFIG_SYS_MMCSD_RAW_MODE_U_BOOT_PARTITION)
> -#  define part_get_info_ptr(x) x
> -# else
> -#  define part_get_info_ptr(x) NULL
> -# endif
>  #else
>  #define part_print_ptr(x)      x
> -#define part_get_info_ptr(x)   x
>  #endif
>
> +#define part_get_info_ptr(x)   x
>
>  struct part_driver {
>         const char *name;
> --
> 2.17.1
>
>


More information about the U-Boot mailing list