[U-Boot] [PATCH v5 03/12] samsung: common: Add misc file and common function misc_init_r().
Przemyslaw Marczak
p.marczak at samsung.com
Tue Jan 14 14:55:55 CET 2014
Hello,
In case of discussion with Piotr Wilczek maybe it is better to make some
changes in this patch.
On 01/10/2014 03:31 PM, Przemyslaw Marczak wrote:
> Config: CONFIG_MISC_INIT_R enables implementation of misc_init_r()
> in common file::
> - board/samsung/common/misc.c
>
> Signed-off-by: Przemyslaw Marczak <p.marczak at samsung.com>
> Acked-by: Jaehoon Chung <jh80.chung at samsung.com>
> ---
> Changes v2:
> - change CONFIG_SAMSUNG to CONFIG_MISC_INIT_R
>
> Changes v3:
> - fix merge conflict in board/samsung/common/Makefile
>
> Changes v4:
> - none
>
> Changes v5:
> - add acked-by
>
> board/samsung/common/Makefile | 1 +
> board/samsung/common/misc.c | 14 ++++++++++++++
> 2 files changed, 15 insertions(+)
> create mode 100644 board/samsung/common/misc.c
>
> diff --git a/board/samsung/common/Makefile b/board/samsung/common/Makefile
> index 22bd6b1..79547a3 100644
> --- a/board/samsung/common/Makefile
> +++ b/board/samsung/common/Makefile
> @@ -8,6 +8,7 @@
> obj-$(CONFIG_SOFT_I2C_MULTI_BUS) += multi_i2c.o
> obj-$(CONFIG_THOR_FUNCTION) += thor.o
> obj-$(CONFIG_CMD_USB_MASS_STORAGE) += ums.o
> +obj-$(CONFIG_MISC_INIT_R) += misc.o
here change to:
obj-y += misc.o
>
> ifndef CONFIG_SPL_BUILD
> obj-$(CONFIG_BOARD_COMMON) += board.o
> diff --git a/board/samsung/common/misc.c b/board/samsung/common/misc.c
> new file mode 100644
> index 0000000..3764d12
> --- /dev/null
> +++ b/board/samsung/common/misc.c
> @@ -0,0 +1,14 @@
> +/*
> + * Copyright (C) 2013 Samsung Electronics
> + * Przemyslaw Marczak <p.marczak at samsung.com>
> + *
> + * SPDX-License-Identifier: GPL-2.0+
> + */
> +
> +#include <common.h>
> +
and here:
#ifdef CONFIG_MISC_INIT_R
> +/* Common for Samsung boards */
> +int misc_init_r(void)
> +{
> + return 0;
> +}
>
#endif
In this way we can add other functions in the future even without
CONFIG_MISC_INIT_R.
Is it better solution?
Thank you,
--
Przemyslaw Marczak
Samsung R&D Institute Poland
Samsung Electronics
p.marczak at samsung.com
More information about the U-Boot
mailing list