mmc: sdio: Print contents of unknown CIS tuples

Print out the contents of the offending tuples when we do print them.
This can make it easier to debug, since these tuples are not exposed to
userspace anywhere else. We are limited to 64 bytes, so keep printing
out the full length in case the tuple is truncated.

Signed-off-by: Sean Anderson <sean.anderson@seco.com>
Link: https://lore.kernel.org/r/20210726163654.1110969-2-sean.anderson@seco.com
Signed-off-by: Ulf Hansson <ulf.hansson@linaro.org>

diff --git a/drivers/mmc/core/sdio_cis.c b/drivers/mmc/core/sdio_cis.c
index 7518b27..a705ba6 100644 (file)
--- a/drivers/mmc/core/sdio_cis.c
+++ b/drivers/mmc/core/sdio_cis.c
@@ -334,7 +334,7 @@ static int sdio_read_cis(struct mmc_card *card, struct sdio_func *func)
                                if (time_after(jiffies, timeout))
                                        break;
 
-#define FMT(type) "%s: queuing " type " CIS tuple 0x%02x (%u bytes)\n"
+#define FMT(type) "%s: queuing " type " CIS tuple 0x%02x [%*ph] (%u bytes)\n"
                                /*
                                 * Tuples in this range are reserved for
                                 * vendors, so don't warn about them
@@ -342,11 +342,13 @@ static int sdio_read_cis(struct mmc_card *card, struct sdio_func *func)
                                if (tpl_code >= 0x80 && tpl_code <= 0x8f)
                                        pr_debug_ratelimited(FMT("vendor"),
                                                mmc_hostname(card->host),
-                                               tpl_code, tpl_link);
+                                               tpl_code, tpl_link, this->data,
+                                               tpl_link);
                                else
                                        pr_warn_ratelimited(FMT("unknown"),
                                                mmc_hostname(card->host),
-                                               tpl_code, tpl_link);
+                                               tpl_code, tpl_link, this->data,
+                                               tpl_link);
                        }
 
                        /* keep on analyzing tuples */