[coreboot-gerrit] New patch to review for coreboot: soc: Remove newline from `CHIP_NAME`

Paul Menzel (paulepanter@users.sourceforge.net) gerrit at coreboot.org
Sun Jul 3 10:08:46 CEST 2016


Paul Menzel (paulepanter at users.sourceforge.net) just uploaded a new patch set to gerrit, which you can find at https://review.coreboot.org/15540

-gerrit

commit d0eb46e9ed3138ba5763f2b2ba85f6927e33baf1
Author: Paul Menzel <paulepanter at users.sourceforge.net>
Date:   Sun Jul 3 09:45:26 2016 +0200

    soc: Remove newline from `CHIP_NAME`
    
    The name must not terminated with a newline character `\n` as it would
    make it hard to use it strings. So, remove the newline from the two SoCs
    with it.
    
    Change-Id: I7570442b38a455e7c497d7f461c208fb0a88296d
    Signed-off-by: Paul Menzel <paulepanter at users.sourceforge.net>
---
 src/soc/mediatek/mt8173/soc.c | 2 +-
 src/soc/rockchip/rk3399/soc.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/soc/mediatek/mt8173/soc.c b/src/soc/mediatek/mt8173/soc.c
index 5b560f5..686f973 100644
--- a/src/soc/mediatek/mt8173/soc.c
+++ b/src/soc/mediatek/mt8173/soc.c
@@ -39,6 +39,6 @@ static void enable_soc_dev(device_t dev)
 }
 
 struct chip_operations soc_mediatek_mt8173_ops = {
-	CHIP_NAME("SOC Mediatek MT8173\n")
+	CHIP_NAME("SOC Mediatek MT8173")
 	.enable_dev = enable_soc_dev,
 };
diff --git a/src/soc/rockchip/rk3399/soc.c b/src/soc/rockchip/rk3399/soc.c
index ef0c9c7..1adf3f6 100644
--- a/src/soc/rockchip/rk3399/soc.c
+++ b/src/soc/rockchip/rk3399/soc.c
@@ -55,6 +55,6 @@ static void enable_soc_dev(device_t dev)
 }
 
 struct chip_operations soc_rockchip_rk3399_ops = {
-	CHIP_NAME("SOC Rockchip RK3399\n")
+	CHIP_NAME("SOC Rockchip RK3399")
 	    .enable_dev = enable_soc_dev,
 };



More information about the coreboot-gerrit mailing list