[coreboot-gerrit] Change in coreboot[master]: soc/cavium/cn81xx: Drop dead do_soft_reset() implementation

Patrick Georgi (Code Review) gerrit at coreboot.org
Wed Oct 17 14:02:38 CEST 2018


Patrick Georgi has submitted this change and it was merged. ( https://review.coreboot.org/29046 )

Change subject: soc/cavium/cn81xx: Drop dead do_soft_reset() implementation
......................................................................

soc/cavium/cn81xx: Drop dead do_soft_reset() implementation

Change-Id: I85f357739220f16497f65df1bb317d9d6eb54d9f
Signed-off-by: Nico Huber <nico.h at gmx.de>
Reviewed-on: https://review.coreboot.org/29046
Tested-by: build bot (Jenkins) <no-reply at coreboot.org>
Reviewed-by: Patrick Rudolph <siro at das-labor.org>
---
M src/soc/cavium/cn81xx/Makefile.inc
M src/soc/cavium/cn81xx/include/soc/addressmap.h
D src/soc/cavium/cn81xx/reset.c
3 files changed, 0 insertions(+), 28 deletions(-)

Approvals:
  build bot (Jenkins): Verified
  Patrick Rudolph: Looks good to me, approved



diff --git a/src/soc/cavium/cn81xx/Makefile.inc b/src/soc/cavium/cn81xx/Makefile.inc
index d212715..ede7d73 100644
--- a/src/soc/cavium/cn81xx/Makefile.inc
+++ b/src/soc/cavium/cn81xx/Makefile.inc
@@ -25,7 +25,6 @@
 bootblock-y += spi.c
 bootblock-y += uart.c
 bootblock-y += cpu.c
-bootblock-y += reset.c
 ifeq ($(CONFIG_BOOTBLOCK_CONSOLE),y)
 bootblock-$(CONFIG_DRIVERS_UART) += uart.c
 endif
@@ -40,7 +39,6 @@
 verstage-y += spi.c
 verstage-$(CONFIG_DRIVERS_UART) += uart.c
 verstage-y += cbmem.c
-verstage-y += reset.c
 
 ################################################################################
 # romstage
@@ -53,7 +51,6 @@
 romstage-y += uart.c
 romstage-$(CONFIG_DRIVERS_UART) += uart.c
 romstage-y += cbmem.c
-romstage-y += reset.c
 
 romstage-y += sdram.c
 romstage-y += mmu.c
@@ -74,7 +71,6 @@
 ramstage-y += cpu_secondary.S
 ramstage-y += ecam0.c
 ramstage-y += cbmem.c
-ramstage-y += reset.c
 
 ramstage-$(CONFIG_ARM64_USE_ARM_TRUSTED_FIRMWARE) += bl31_plat_params.c
 
diff --git a/src/soc/cavium/cn81xx/include/soc/addressmap.h b/src/soc/cavium/cn81xx/include/soc/addressmap.h
index f188961..f698306 100644
--- a/src/soc/cavium/cn81xx/include/soc/addressmap.h
+++ b/src/soc/cavium/cn81xx/include/soc/addressmap.h
@@ -62,7 +62,6 @@
 
 /* RST */
 #define RST_PF_BAR0		(0x87E006000000ULL + 0x1600)
-#define RST_SOFT_RESET	(RST_PF_BAR0 + 0x80ULL)
 #define RST_PP_AVAILABLE	(RST_PF_BAR0 + 0x138ULL)
 #define RST_PP_RESET		(RST_PF_BAR0 + 0x140ULL)
 #define RST_PP_PENDING		(RST_PF_BAR0 + 0x148ULL)
diff --git a/src/soc/cavium/cn81xx/reset.c b/src/soc/cavium/cn81xx/reset.c
deleted file mode 100644
index d3be7c9..0000000
--- a/src/soc/cavium/cn81xx/reset.c
+++ /dev/null
@@ -1,23 +0,0 @@
-/*
- * This file is part of the coreboot project.
- *
- * Copyright 2018-present Facebook, Inc.
- *
- * This program is free software; you can redistribute it and/or modify
- * it under the terms of the GNU General Public License as published by
- * the Free Software Foundation; version 2 of the License.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
- * GNU General Public License for more details.
- */
-
-#include <arch/io.h>
-#include <soc/addressmap.h>
-#include <reset.h>
-
-void do_soft_reset(void)
-{
-	write64((void *)RST_SOFT_RESET, 1);
-}

-- 
To view, visit https://review.coreboot.org/29046
To unsubscribe, or for help writing mail filters, visit https://review.coreboot.org/settings

Gerrit-Project: coreboot
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I85f357739220f16497f65df1bb317d9d6eb54d9f
Gerrit-Change-Number: 29046
Gerrit-PatchSet: 2
Gerrit-Owner: Nico Huber <nico.h at gmx.de>
Gerrit-Reviewer: Patrick Georgi <pgeorgi at google.com>
Gerrit-Reviewer: Patrick Rudolph <siro at das-labor.org>
Gerrit-Reviewer: build bot (Jenkins) <no-reply at coreboot.org>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.coreboot.org/pipermail/coreboot-gerrit/attachments/20181017/d9354f55/attachment.html>


More information about the coreboot-gerrit mailing list