[U-Boot] [PATCH 2/3] board: samsung: add missing line breaks for pr_err()
Minkyu Kang
promsoft at gmail.com
Tue Jun 5 05:06:10 UTC 2018
Hi,
2018년 6월 4일 (월) 16:03, Seung-Woo Kim <sw0312.kim at samsung.com>님이 작성:
> After the commit 9b643e312d52 ("treewide: replace with error() with
> pr_err()"), there are some pr_err() with no line break. Add missing
> line breaks.
>
> Signed-off-by: Seung-Woo Kim <sw0312.kim at samsung.com>
> ---
> board/samsung/common/exynos5-dt.c | 2 +-
> board/samsung/common/misc.c | 2 +-
> board/samsung/odroid/odroid.c | 12 ++++++------
> 3 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/board/samsung/common/exynos5-dt.c
> b/board/samsung/common/exynos5-dt.c
> index 9f6f654..8c3a9ea 100644
> --- a/board/samsung/common/exynos5-dt.c
> +++ b/board/samsung/common/exynos5-dt.c
> @@ -164,7 +164,7 @@ int board_usb_init(int index, enum usb_init_type init)
> samsung_get_base_usb3_phy();
>
> if (!phy) {
> - pr_err("usb3 phy not supported");
> + pr_err("usb3 phy not supported\n");
> return -ENODEV;
> }
>
> diff --git a/board/samsung/common/misc.c b/board/samsung/common/misc.c
> index c9df7e6..05243fc 100644
> --- a/board/samsung/common/misc.c
> +++ b/board/samsung/common/misc.c
> @@ -456,7 +456,7 @@ void draw_logo(void)
>
> addr = panel_info.logo_addr;
> if (!addr) {
> - pr_err("There is no logo data.");
> + pr_err("There is no logo data.\n");
> return;
> }
>
> diff --git a/board/samsung/odroid/odroid.c b/board/samsung/odroid/odroid.c
> index 1c2bd01..552333f 100644
> --- a/board/samsung/odroid/odroid.c
> +++ b/board/samsung/odroid/odroid.c
> @@ -428,7 +428,7 @@ int exynos_power_init(void)
> };
>
> if (regulator_list_autoset(mmc_regulators, NULL, true))
> - pr_err("Unable to init all mmc regulators");
> + pr_err("Unable to init all mmc regulators\n");
>
> return 0;
> }
> @@ -441,7 +441,7 @@ static int s5pc210_phy_control(int on)
>
> ret = regulator_get_by_platname("VDD_UOTG_3.0V", &dev);
> if (ret) {
> - pr_err("Regulator get error: %d", ret);
> + pr_err("Regulator get error: %d\n", ret);
> return ret;
> }
>
> @@ -486,25 +486,25 @@ int board_usb_init(int index, enum usb_init_type
> init)
>
> ret = regulator_get_by_platname("VCC_P3V3_2.85V", &dev);
> if (ret) {
> - pr_err("Regulator get error: %d", ret);
> + pr_err("Regulator get error: %d\n", ret);
> return ret;
> }
>
> ret = regulator_set_enable(dev, true);
> if (ret) {
> - pr_err("Regulator %s enable setting error: %d", dev->name,
> ret);
> + pr_err("Regulator %s enable setting error: %d\n",
> dev->name, ret);
> return ret;
> }
>
> ret = regulator_set_value(dev, 750000);
> if (ret) {
> - pr_err("Regulator %s value setting error: %d", dev->name,
> ret);
> + pr_err("Regulator %s value setting error: %d\n",
> dev->name, ret);
> return ret;
> }
>
> ret = regulator_set_value(dev, 3300000);
> if (ret) {
> - pr_err("Regulator %s value setting error: %d", dev->name,
> ret);
> + pr_err("Regulator %s value setting error: %d\n",
> dev->name, ret);
> return ret;
> }
> #endif
> --
> 1.7.9.5
>
> _______________________________________________
> U-Boot mailing list
> U-Boot at lists.denx.de
> https://lists.denx.de/listinfo/u-boot
Reviewed-by: Minkyu Kang <mk7.kang at samsung.com>
Thanks,
Minkyu Kang.
More information about the U-Boot
mailing list