[Skiboot] [PATCH 07/11] phb4: Remove some unnecessary debug output

Benjamin Herrenschmidt benh at kernel.crashing.org
Wed Mar 15 20:58:56 AEDT 2017


.. and update a comment while at it.

Signed-off-by: Benjamin Herrenschmidt <benh at kernel.crashing.org>
---
 hw/phb4.c | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/hw/phb4.c b/hw/phb4.c
index 7117668..314e0f7 100644
--- a/hw/phb4.c
+++ b/hw/phb4.c
@@ -3034,8 +3034,6 @@ static int64_t phb4_dd1_lsi_set_xive(struct irq_source *is, uint32_t isn,
 	struct phb4 *p = is->data;
 	uint32_t idx = isn - p->base_lsi;
 
-	PHBERR(p, "DD1 LSI set_xive idx %d prio=%d\n", idx, priority);
-
 	if (idx > 8)
 		return OPAL_PARAMETER;
 
@@ -3045,16 +3043,17 @@ static int64_t phb4_dd1_lsi_set_xive(struct irq_source *is, uint32_t isn,
 
 	/* Mask using P=0,Q=1, unmask using P=1,Q=0 followed by EOI */
 	/* XXX FIXME: A quick mask/umask can make us shoot an interrupt
-	 * more than once to a queue. We need to keep track better
+	 * more than once to a queue. We need to keep track better.
+	 *
+	 * Thankfully, this is only on DD1 and for LSIs, so will go away
+	 * soon enough.
 	 */
-	PHBERR(p, " LIST before: %016llx\n", in_be64(p->regs + PHB_IODA_DATA0));
 	if (priority == 0xff)
 		out_be64(p->regs + PHB_IODA_DATA0, IODA3_LIST_Q);
 	else {
 		out_be64(p->regs + PHB_IODA_DATA0, IODA3_LIST_P);
 		__irq_source_eoi(is, isn);
 	}
-	PHBERR(p, " LIST after: %016llx\n", in_be64(p->regs + PHB_IODA_DATA0));
 
 	phb_unlock(&p->phb);
 
-- 
2.9.3



More information about the Skiboot mailing list