From: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
To: barebox@lists.infradead.org
Subject: [PATCH 3/4] pci: improve debug output
Date: Sat, 14 Apr 2018 01:30:52 +0200 [thread overview]
Message-ID: <20180413233053.15570-3-u.kleine-koenig@pengutronix.de> (raw)
In-Reply-To: <20180413233053.15570-1-u.kleine-koenig@pengutronix.de>
Print a single line per bar with all relevant information and indent it
by a space to group them together per device.
Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
drivers/pci/pci.c | 15 ++++++++-------
1 file changed, 8 insertions(+), 7 deletions(-)
diff --git a/drivers/pci/pci.c b/drivers/pci/pci.c
index 87359b6de82c..837e17aa8397 100644
--- a/drivers/pci/pci.c
+++ b/drivers/pci/pci.c
@@ -181,7 +181,7 @@ static void setup_device(struct pci_dev *dev, int max_bar)
pci_write_config_dword(dev, PCI_BASE_ADDRESS_0 + bar * 4, orig);
if (barval == 0 || barval == 0xffffffff) {
- pr_debug("pbar%d set bad mask\n", bar);
+ pr_debug(" pbar%d: set bad mask (%08x)\n", bar, barval);
continue;
}
@@ -208,20 +208,21 @@ static void setup_device(struct pci_dev *dev, int max_bar)
size = pci_size(orig, barval, mask);
if (!size) {
- pr_debug("pbar%d bad %s mask\n", bar, type);
+ pr_debug(" pbar%d: bad %s size (orig=%08x, barval=%08x, type=%s)\n",
+ bar, type, orig, barval, type);
continue;
}
- pr_debug("pbar%d: barval=%08x type=%s size=0x%x\n",
- bar, barval, type, size);
-
if (ALIGN(*last, size) + size > dev->bus->resource[restype]->end) {
- pr_debug("BAR does not fit within bus %s res\n", type);
+ pr_debug(" pbar%d: !! type=%s allocbase=0x%08x + size=0x%08x > resource_end=%pa (barval=%08x)\n",
+ bar, type, ALIGN(*last, size), size, &dev->bus->resource[restype]->end, barval);
return;
}
*last = ALIGN(*last, size);
- pr_debug("pbar%d: allocated at 0x%08x\n", bar, *last);
+ pr_debug(" pbar%d: type=%s allocbase=0x%08x size=0x%08x (barval=%08x)\n",
+ bar, type, *last, size, barval);
+
pci_write_config_dword(dev, PCI_BASE_ADDRESS_0 + bar * 4, *last);
dev->resource[bar].flags = resflags;
dev->resource[bar].start = *last;
--
2.16.3
_______________________________________________
barebox mailing list
barebox@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/barebox
next prev parent reply other threads:[~2018-04-13 23:31 UTC|newest]
Thread overview: 8+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-04-13 23:30 [PATCH 1/4] pci: rename variable mask -> barval Uwe Kleine-König
2018-04-13 23:30 ` [PATCH 2/4] pci: refactor bar configuration Uwe Kleine-König
2018-04-13 23:30 ` Uwe Kleine-König [this message]
2018-04-13 23:58 ` [PATCH 3/4] pci: improve debug output Uwe Kleine-König
2018-04-13 23:30 ` [PATCH 4/4] pci: don't hide hard to understand sanity check in size calculation Uwe Kleine-König
2018-04-14 16:25 ` Andrey Smirnov
2018-04-15 18:27 ` Uwe Kleine-König
2018-04-16 13:42 ` Andrey Smirnov
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20180413233053.15570-3-u.kleine-koenig@pengutronix.de \
--to=u.kleine-koenig@pengutronix.de \
--cc=barebox@lists.infradead.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox