Re: [PATCH 3/4] mmc: Logging neatening

[NamJae Jeon]

2011/11/16 Joe Perches <joe@xxxxxxxxxxx>:
> Coalesce formats.
> Align arguments.
> Add pr_fmt, remove prefixes from format strings.
> Neaten DBG uses, rearrange argument order.
>
> Signed-off-by: Joe Perches <joe@xxxxxxxxxxx>
> ---
> Âdrivers/mmc/card/mmc_test.c  Â|  49 +++++-------
> Âdrivers/mmc/core/core.c    Â|  52 ++++++------
> Âdrivers/mmc/core/host.c    Â|  Â3 +-
> Âdrivers/mmc/core/mmc.c     |  22 ++---
> Âdrivers/mmc/core/mmc_ops.c   |  Â5 +-
> Âdrivers/mmc/core/sd.c     Â|  25 +++---
> Âdrivers/mmc/core/sdio_bus.c  Â|  Â2 +
> Âdrivers/mmc/core/sdio_io.c   |  14 ++--
> Âdrivers/mmc/core/sdio_irq.c  Â|  10 ++-
> Âdrivers/mmc/host/at91_mci.c  Â|  Â3 +-
> Âdrivers/mmc/host/au1xmmc.c   |  35 ++++----
> Âdrivers/mmc/host/bfin_sdh.c  Â|  Â3 +-
> Âdrivers/mmc/host/dw_mmc.c   Â|  16 ++--
> Âdrivers/mmc/host/mmc_spi.c   |  Â9 +-
> Âdrivers/mmc/host/mmci.c    Â|  Â3 +-
> Âdrivers/mmc/host/mvsdio.c   Â|  33 ++++----
> Âdrivers/mmc/host/omap_hsmmc.c Â| Â 11 +--
> Âdrivers/mmc/host/sdhci-pci.c  |  17 ++---
> Âdrivers/mmc/host/sdhci-pxav3.c | Â Â7 +-
> Âdrivers/mmc/host/sdhci.c    | Â175 ++++++++++++++++++---------------------
> Âdrivers/mmc/host/sdricoh_cs.c Â| Â 12 ++--
> Âdrivers/mmc/host/tifm_sd.c   |  Â9 +-
> Âdrivers/mmc/host/via-sdmmc.c  |  22 +++---
> Âdrivers/mmc/host/vub300.c   Â|  54 ++++++-------
> Âdrivers/mmc/host/wbsd.c    Â|  Â5 +-
> Â25 files changed, 277 insertions(+), 319 deletions(-)
>
> diff --git a/drivers/mmc/card/mmc_test.c b/drivers/mmc/card/mmc_test.c
> index b038c4a..f079555 100644
> --- a/drivers/mmc/card/mmc_test.c
> +++ b/drivers/mmc/card/mmc_test.c
> @@ -9,6 +9,8 @@
> Â* your option) any later version.
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/mmc/core.h>
> Â#include <linux/mmc/card.h>
> Â#include <linux/mmc/host.h>
> @@ -251,8 +253,7 @@ static int mmc_test_wait_busy(struct mmc_test_card *test)
> Â Â Â Â Â Â Â Âif (!busy && mmc_test_busy(&cmd)) {
> Â Â Â Â Â Â Â Â Â Â Â Âbusy = 1;
> Â Â Â Â Â Â Â Â Â Â Â Âif (test->card->host->caps & MMC_CAP_WAIT_WHILE_BUSY)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Warning: Host did not "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "wait for busy state to end.\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Warning: Host did not wait for busy state to end\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> Â Â Â Â Â Â Â Â}
> Â Â Â Â} while (mmc_test_busy(&cmd));
> @@ -553,12 +554,11 @@ static void mmc_test_print_rate(struct mmc_test_card *test, uint64_t bytes,
> Â Â Â Ârate = mmc_test_rate(bytes, &ts);
> Â Â Â Âiops = mmc_test_rate(100, &ts); /* I/O ops per sec x 100 */
>
> - Â Â Â pr_info("%s: Transfer of %u sectors (%u%s KiB) took %lu.%09lu "
> - Â Â Â Â Â Â Â Â Â Â Â Â"seconds (%u kB/s, %u KiB/s, %u.%02u IOPS)\n",
> - Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host), sectors, sectors >> 1,
> - Â Â Â Â Â Â Â Â Â Â Â Â(sectors & 1 ? ".5" : ""), (unsigned long)ts.tv_sec,
> - Â Â Â Â Â Â Â Â Â Â Â Â(unsigned long)ts.tv_nsec, rate / 1000, rate / 1024,
> - Â Â Â Â Â Â Â Â Â Â Â Âiops / 100, iops % 100);
> + Â Â Â pr_info("%s: Transfer of %u sectors (%u%s KiB) took %lu.%09lu seconds (%u kB/s, %u KiB/s, %u.%02u IOPS)\n",
> + Â Â Â Â Â Â Â mmc_hostname(test->card->host), sectors, sectors >> 1,
> + Â Â Â Â Â Â Â (sectors & 1 ? ".5" : ""), (unsigned long)ts.tv_sec,
> + Â Â Â Â Â Â Â (unsigned long)ts.tv_nsec, rate / 1000, rate / 1024,
> + Â Â Â Â Â Â Â iops / 100, iops % 100);
>
> Â Â Â Âmmc_test_save_transfer_result(test, 1, sectors, ts, rate, iops);
> Â}
> @@ -579,14 +579,12 @@ static void mmc_test_print_avg_rate(struct mmc_test_card *test, uint64_t bytes,
> Â Â Â Ârate = mmc_test_rate(tot, &ts);
> Â Â Â Âiops = mmc_test_rate(count * 100, &ts); /* I/O ops per sec x 100 */
>
> - Â Â Â pr_info("%s: Transfer of %u x %u sectors (%u x %u%s KiB) took "
> - Â Â Â Â Â Â Â Â Â Â Â Â"%lu.%09lu seconds (%u kB/s, %u KiB/s, "
> - Â Â Â Â Â Â Â Â Â Â Â Â"%u.%02u IOPS, sg_len %d)\n",
> - Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host), count, sectors, count,
> - Â Â Â Â Â Â Â Â Â Â Â Âsectors >> 1, (sectors & 1 ? ".5" : ""),
> - Â Â Â Â Â Â Â Â Â Â Â Â(unsigned long)ts.tv_sec, (unsigned long)ts.tv_nsec,
> - Â Â Â Â Â Â Â Â Â Â Â Ârate / 1000, rate / 1024, iops / 100, iops % 100,
> - Â Â Â Â Â Â Â Â Â Â Â Âtest->area.sg_len);
> + Â Â Â pr_info("%s: Transfer of %u x %u sectors (%u x %u%s KiB) took %lu.%09lu seconds (%u kB/s, %u KiB/s, %u.%02u IOPS, sg_len %d)\n",
> + Â Â Â Â Â Â Â mmc_hostname(test->card->host), count, sectors, count,
> + Â Â Â Â Â Â Â sectors >> 1, (sectors & 1 ? ".5" : ""),
> + Â Â Â Â Â Â Â (unsigned long)ts.tv_sec, (unsigned long)ts.tv_nsec,
> + Â Â Â Â Â Â Â rate / 1000, rate / 1024, iops / 100, iops % 100,
> + Â Â Â Â Â Â Â test->area.sg_len);
>
> Â Â Â Âmmc_test_save_transfer_result(test, count, sectors, ts, rate, iops);
> Â}
> @@ -1410,7 +1408,7 @@ static int mmc_test_multi_read_high(struct mmc_test_card *test)
> Âstatic int mmc_test_no_highmem(struct mmc_test_card *test)
> Â{
> Â Â Â Âpr_info("%s: Highmem not configured - test skipped\n",
> - Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> + Â Â Â Â Â Â Â mmc_hostname(test->card->host));
> Â Â Â Âreturn 0;
> Â}
>
> @@ -1437,7 +1435,7 @@ static int mmc_test_area_map(struct mmc_test_card *test, unsigned long sz,
> Â Â Â Â}
> Â Â Â Âif (err)
> Â Â Â Â Â Â Â Âpr_info("%s: Failed to map sg list\n",
> - Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> + Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(test->card->host));
> Â Â Â Âreturn err;
> Â}
>
> @@ -2709,8 +2707,7 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase)
> Â Â Â Â Â Â Â Âif (mmc_test_cases[i].prepare) {
> Â Â Â Â Â Â Â Â Â Â Â Âret = mmc_test_cases[i].prepare(test);
> Â Â Â Â Â Â Â Â Â Â Â Âif (ret) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: Prepare "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "stage failed! (%d)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: Prepare stage failed! (%d)\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âret);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcontinue;
> @@ -2747,13 +2744,11 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Âcase RESULT_UNSUP_HOST:
> - Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: UNSUPPORTED "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "(by host)\n",
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: UNSUPPORTED (by host)\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Âcase RESULT_UNSUP_CARD:
> - Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: UNSUPPORTED "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "(by card)\n",
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Result: UNSUPPORTED (by card)\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host));
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Âdefault:
> @@ -2768,8 +2763,7 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase)
> Â Â Â Â Â Â Â Âif (mmc_test_cases[i].cleanup) {
> Â Â Â Â Â Â Â Â Â Â Â Âret = mmc_test_cases[i].cleanup(test);
> Â Â Â Â Â Â Â Â Â Â Â Âif (ret) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Warning: Cleanup "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "stage failed! (%d)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: Warning: Cleanup stage failed! (%d)\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(test->card->host),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âret);
> Â Â Â Â Â Â Â Â Â Â Â Â}
> @@ -2778,8 +2772,7 @@ static void mmc_test_run(struct mmc_test_card *test, int testcase)
>
> Â Â Â Âmmc_release_host(test->card->host);
>
> - Â Â Â pr_info("%s: Tests completed.\n",
> - Â Â Â Â Â Â Â mmc_hostname(test->card->host));
> + Â Â Â pr_info("%s: Tests completed\n", mmc_hostname(test->card->host));
> Â}
>
> Âstatic void mmc_test_free_result(struct mmc_card *card)
> diff --git a/drivers/mmc/core/core.c b/drivers/mmc/core/core.c
> index 54d6a13..39e1311 100644
> --- a/drivers/mmc/core/core.c
> +++ b/drivers/mmc/core/core.c
> @@ -10,6 +10,9 @@
> Â* it under the terms of the GNU General Public License version 2 as
> Â* published by the Free Software Foundation.
> Â*/
> +
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/module.h>
> Â#include <linux/init.h>
> Â#include <linux/interrupt.h>
> @@ -153,22 +156,22 @@ void mmc_request_done(struct mmc_host *host, struct mmc_request *mrq)
> Â Â Â Â Â Â Â Âled_trigger_event(host->led, LED_OFF);
>
> Â Â Â Â Â Â Â Âpr_debug("%s: req done (CMD%u): %d: %08x %08x %08x %08x\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host), cmd->opcode, err,
> - Â Â Â Â Â Â Â Â Â Â Â cmd->resp[0], cmd->resp[1],
> - Â Â Â Â Â Â Â Â Â Â Â cmd->resp[2], cmd->resp[3]);
> + Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host), cmd->opcode, err,
> + Â Â Â Â Â Â Â Â Â Â Â Âcmd->resp[0], cmd->resp[1],
> + Â Â Â Â Â Â Â Â Â Â Â Âcmd->resp[2], cmd->resp[3]);
>
> Â Â Â Â Â Â Â Âif (mrq->data) {
> Â Â Â Â Â Â Â Â Â Â Â Âpr_debug("%s: Â Â %d bytes transferred: %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mrq->data->bytes_xfered, mrq->data->error);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host),
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmrq->data->bytes_xfered, mrq->data->error);
> Â Â Â Â Â Â Â Â}
>
> Â Â Â Â Â Â Â Âif (mrq->stop) {
> Â Â Â Â Â Â Â Â Â Â Â Âpr_debug("%s: Â Â (CMD%u): %d: %08x %08x %08x %08x\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host), mrq->stop->opcode,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mrq->stop->error,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mrq->stop->resp[0], mrq->stop->resp[1],
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mrq->stop->resp[2], mrq->stop->resp[3]);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host), mrq->stop->opcode,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmrq->stop->error,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmrq->stop->resp[0], mrq->stop->resp[1],
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmrq->stop->resp[2], mrq->stop->resp[3]);
> Â Â Â Â Â Â Â Â}
>
> Â Â Â Â Â Â Â Âif (mrq->done)
> @@ -193,12 +196,11 @@ mmc_start_request(struct mmc_host *host, struct mmc_request *mrq)
> Â Â Â Â Â Â Â Â mrq->cmd->arg, mrq->cmd->flags);
>
> Â Â Â Âif (mrq->data) {
> - Â Â Â Â Â Â Â pr_debug("%s: Â Â blksz %d blocks %d flags %08x "
> - Â Â Â Â Â Â Â Â Â Â Â "tsac %d ms nsac %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host), mrq->data->blksz,
> - Â Â Â Â Â Â Â Â Â Â Â mrq->data->blocks, mrq->data->flags,
> - Â Â Â Â Â Â Â Â Â Â Â mrq->data->timeout_ns / 1000000,
> - Â Â Â Â Â Â Â Â Â Â Â mrq->data->timeout_clks);
> + Â Â Â Â Â Â Â pr_debug("%s: Â Â blksz %d blocks %d flags %08x tsac %d ms nsac %d\n",
> + Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host), mrq->data->blksz,
> + Â Â Â Â Â Â Â Â Â Â Â Âmrq->data->blocks, mrq->data->flags,
> + Â Â Â Â Â Â Â Â Â Â Â Âmrq->data->timeout_ns / 1000000,
> + Â Â Â Â Â Â Â Â Â Â Â Âmrq->data->timeout_clks);
> Â Â Â Â}
>
> Â Â Â Âif (mrq->stop) {
> @@ -842,8 +844,7 @@ static inline void mmc_set_ios(struct mmc_host *host)
> Â{
> Â Â Â Âstruct mmc_ios *ios = &host->ios;
>
> - Â Â Â pr_debug("%s: clock %uHz busmode %u powermode %u cs %u Vdd %u "
> - Â Â Â Â Â Â Â "width %u timing %u\n",
> + Â Â Â pr_debug("%s: clock %uHz busmode %u powermode %u cs %u Vdd %u width %u timing %u\n",
> Â Â Â Â Â Â Â Â mmc_hostname(host), ios->clock, ios->bus_mode,
> Â Â Â Â Â Â Â Â ios->power_mode, ios->chip_select, ios->vdd,
> Â Â Â Â Â Â Â Â ios->bus_width, ios->timing);
> @@ -1310,8 +1311,7 @@ void mmc_power_off(struct mmc_host *host)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â notify_type, timeout);
>
> Â Â Â Â Â Â Â Âif (err && err != -EBADMSG)
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("Device failed to respond within %d poweroff "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"time. Forcefully powering down the device\n",
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("Device failed to respond within %d poweroff time. Forcefully powering down the device.\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â timeout);
>
> Â Â Â Â Â Â Â Â/* Set the card state to no notification after the poweroff */
> @@ -1642,8 +1642,8 @@ static int mmc_do_erase(struct mmc_card *card, unsigned int from,
> Â Â Â Âcmd.flags = MMC_RSP_SPI_R1 | MMC_RSP_R1 | MMC_CMD_AC;
> Â Â Â Âerr = mmc_wait_for_cmd(card->host, &cmd, 0);
> Â Â Â Âif (err) {
> - Â Â Â Â Â Â Â pr_err("mmc_erase: group start error %d, "
> - Â Â Â Â Â Â Â Â Â Â Â"status %#x\n", err, cmd.resp[0]);
> + Â Â Â Â Â Â Â pr_err("mmc_erase: group start error %d, status %#x\n",
> + Â Â Â Â Â Â Â Â Â Â Âerr, cmd.resp[0]);
> Â Â Â Â Â Â Â Âerr = -EIO;
> Â Â Â Â Â Â Â Âgoto out;
> Â Â Â Â}
> @@ -1688,7 +1688,7 @@ static int mmc_do_erase(struct mmc_card *card, unsigned int from,
> Â Â Â Â Â Â Â Âerr = mmc_wait_for_cmd(card->host, &cmd, 0);
> Â Â Â Â Â Â Â Âif (err || (cmd.resp[0] & 0xFDF92000)) {
> Â Â Â Â Â Â Â Â Â Â Â Âpr_err("error %d requesting status %#x\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â err, cmd.resp[0]);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âerr, cmd.resp[0]);
> Â Â Â Â Â Â Â Â Â Â Â Âerr = -EIO;
> Â Â Â Â Â Â Â Â Â Â Â Âgoto out;
> Â Â Â Â Â Â Â Â}
> @@ -2245,7 +2245,7 @@ int mmc_flush_cache(struct mmc_card *card)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂEXT_CSD_FLUSH_CACHE, 1, 0);
> Â Â Â Â Â Â Â Âif (err)
> Â Â Â Â Â Â Â Â Â Â Â Âpr_err("%s: cache flush error %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host), err);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host), err);
> Â Â Â Â}
>
> Â Â Â Âreturn err;
> @@ -2275,9 +2275,9 @@ int mmc_cache_ctrl(struct mmc_host *host, u8 enable)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂEXT_CSD_CACHE_CTRL, enable, 0);
> Â Â Â Â Â Â Â Âif (err)
> Â Â Â Â Â Â Â Â Â Â Â Âpr_err("%s: cache %s error %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â enable ? "on" : "off",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â err);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host),
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âenable ? "on" : "off",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âerr);
> Â Â Â Â Â Â Â Âelse
> Â Â Â Â Â Â Â Â Â Â Â Âcard->ext_csd.cache_ctrl = enable;
> Â Â Â Â}
> diff --git a/drivers/mmc/core/host.c b/drivers/mmc/core/host.c
> index 835e86a..7f963c0 100644
> --- a/drivers/mmc/core/host.c
> +++ b/drivers/mmc/core/host.c
> @@ -94,8 +94,7 @@ static void mmc_host_clk_gate_delayed(struct mmc_host *host)
> Â Â Â Âunsigned long flags;
>
> Â Â Â Âif (!freq) {
> - Â Â Â Â Â Â Â pr_debug("%s: frequency set to 0 in disable function, "
> - Â Â Â Â Â Â Â Â Â Â Â Â"this means the clock is already disabled.\n",
> + Â Â Â Â Â Â Â pr_debug("%s: frequency set to 0 in disable function, this means the clock is already disabled\n",
> Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host));
> Â Â Â Â Â Â Â Âreturn;
> Â Â Â Â}
> diff --git a/drivers/mmc/core/mmc.c b/drivers/mmc/core/mmc.c
> index bbd7adb..f6db788 100644
> --- a/drivers/mmc/core/mmc.c
> +++ b/drivers/mmc/core/mmc.c
> @@ -196,8 +196,8 @@ static int mmc_get_ext_csd(struct mmc_card *card, u8 **new_ext_csd)
> Â Â Â Â */
> Â Â Â Âext_csd = kmalloc(512, GFP_KERNEL);
> Â Â Â Âif (!ext_csd) {
> - Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer to "
> - Â Â Â Â Â Â Â Â Â Â Â "receive the ext_csd.\n", mmc_hostname(card->host));
> + Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer to receive the ext_csd\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âreturn -ENOMEM;
> Â Â Â Â}
>
> @@ -218,10 +218,8 @@ static int mmc_get_ext_csd(struct mmc_card *card, u8 **new_ext_csd)
> Â Â Â Â Â Â Â Â * stored in their CSD.
> Â Â Â Â Â Â Â Â */
> Â Â Â Â Â Â Â Âif (card->csd.capacity == (4096 * 512)) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: unable to read EXT_CSD "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "on a possible high capacity card. "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "Card will be ignored.\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: unable to read EXT_CSD on a possible high capacity card. Card will be ignored.\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Âpr_warn("%s: unable to read EXT_CSD, performance might suffer\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> @@ -251,9 +249,9 @@ static int mmc_read_ext_csd(struct mmc_card *card, u8 *ext_csd)
> Â Â Â Âcard->ext_csd.raw_ext_csd_structure = ext_csd[EXT_CSD_STRUCTURE];
> Â Â Â Âif (card->csd.structure == 3) {
> Â Â Â Â Â Â Â Âif (card->ext_csd.raw_ext_csd_structure > 2) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: unrecognised EXT_CSD structure "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "version %d\n", mmc_hostname(card->host),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â card->ext_csd.raw_ext_csd_structure);
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: unrecognised EXT_CSD structure version %d\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host),
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcard->ext_csd.raw_ext_csd_structure);
> Â Â Â Â Â Â Â Â Â Â Â Âerr = -EINVAL;
> Â Â Â Â Â Â Â Â Â Â Â Âgoto out;
> Â Â Â Â Â Â Â Â}
> @@ -984,8 +982,7 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
> Â Â Â Â Â Â Â Â Â Â Â Âerr = mmc_select_powerclass(card, ext_csd_bits[idx][0],
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âext_csd);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: power class selection to "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"bus width %d failed\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: power class selection to bus width %d failed\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â 1 << bus_width);
>
> @@ -1015,8 +1012,7 @@ static int mmc_init_card(struct mmc_host *host, u32 ocr,
> Â Â Â Â Â Â Â Â Â Â Â Âerr = mmc_select_powerclass(card, ext_csd_bits[idx][1],
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âext_csd);
> Â Â Â Â Â Â Â Â Â Â Â Âif (err)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: power class selection to "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"bus width %d ddr %d failed\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: power class selection to bus width %d ddr %d failed\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host),
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â 1 << bus_width, ddr);
>
> diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c
> index 32deb3e..7c7388c 100644
> --- a/drivers/mmc/core/mmc_ops.c
> +++ b/drivers/mmc/core/mmc_ops.c
> @@ -569,9 +569,8 @@ int mmc_send_hpi_cmd(struct mmc_card *card, u32 *status)
>
> Â Â Â Âerr = mmc_wait_for_cmd(card->host, &cmd, 0);
> Â Â Â Âif (err) {
> - Â Â Â Â Â Â Â pr_warn("%s: error %d interrupting operation. "
> - Â Â Â Â Â Â Â Â Â Â Â "HPI command response %#x\n", mmc_hostname(card->host),
> - Â Â Â Â Â Â Â Â Â Â Â err, cmd.resp[0]);
> + Â Â Â Â Â Â Â pr_warn("%s: error %d interrupting operation. HPI command response %#x\n",
> + Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host), err, cmd.resp[0]);
> Â Â Â Â Â Â Â Âreturn err;
> Â Â Â Â}
> Â Â Â Âif (status)
> diff --git a/drivers/mmc/core/sd.c b/drivers/mmc/core/sd.c
> index 52a1563..1492b7b 100644
> --- a/drivers/mmc/core/sd.c
> +++ b/drivers/mmc/core/sd.c
> @@ -10,6 +10,8 @@
> Â* published by the Free Software Foundation.
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/err.h>
> Â#include <linux/slab.h>
> Â#include <linux/stat.h>
> @@ -283,9 +285,8 @@ static int mmc_read_switch(struct mmc_card *card)
>
> Â Â Â Âstatus = kmalloc(64, GFP_KERNEL);
> Â Â Â Âif (!status) {
> - Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for "
> - Â Â Â Â Â Â Â Â Â Â Â "switch capabilities.\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(card->host));
> + Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for switch capabilities\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âreturn -ENOMEM;
> Â Â Â Â}
>
> @@ -299,7 +300,7 @@ static int mmc_read_switch(struct mmc_card *card)
> Â Â Â Â Â Â Â Âif (err != -EINVAL && err != -ENOSYS && err != -EFAULT)
> Â Â Â Â Â Â Â Â Â Â Â Âgoto out;
>
> - Â Â Â Â Â Â Â pr_warn("%s: problem reading Bus Speed modes.\n",
> + Â Â Â Â Â Â Â pr_warn("%s: problem reading Bus Speed modes\n",
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âerr = 0;
>
> @@ -381,8 +382,8 @@ int mmc_sd_switch_hs(struct mmc_card *card)
>
> Â Â Â Âstatus = kmalloc(64, GFP_KERNEL);
> Â Â Â Âif (!status) {
> - Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for "
> - Â Â Â Â Â Â Â Â Â Â Â "switch capabilities.\n", mmc_hostname(card->host));
> + Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for switch capabilities\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âreturn -ENOMEM;
> Â Â Â Â}
>
> @@ -391,7 +392,7 @@ int mmc_sd_switch_hs(struct mmc_card *card)
> Â Â Â Â Â Â Â Âgoto out;
>
> Â Â Â Âif ((status[16] & 0xF) != 1) {
> - Â Â Â Â Â Â Â pr_warn("%s: Problem switching card into high-speed mode!\n",
> + Â Â Â Â Â Â Â pr_warn("%s: problem switching card into high-speed mode!\n",
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âerr = 0;
> Â Â Â Â} else {
> @@ -456,7 +457,7 @@ static int sd_select_driver_type(struct mmc_card *card, u8 *status)
> Â Â Â Â Â Â Â Âreturn err;
>
> Â Â Â Âif ((status[15] & 0xF) != drive_strength) {
> - Â Â Â Â Â Â Â pr_warn("%s: Problem setting drive strength!\n",
> + Â Â Â Â Â Â Â pr_warn("%s: problem setting drive strength!\n",
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âreturn 0;
> Â Â Â Â}
> @@ -535,7 +536,7 @@ static int sd_set_bus_speed_mode(struct mmc_card *card, u8 *status)
> Â Â Â Â Â Â Â Âreturn err;
>
> Â Â Â Âif ((status[16] & 0xF) != card->sd_bus_speed)
> - Â Â Â Â Â Â Â pr_warn("%s: Problem setting bus speed mode!\n",
> + Â Â Â Â Â Â Â pr_warn("%s: problem setting bus speed mode!\n",
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Âelse {
> Â Â Â Â Â Â Â Âmmc_set_timing(card->host, timing);
> @@ -597,7 +598,7 @@ static int sd_set_current_limit(struct mmc_card *card, u8 *status)
> Â Â Â Â Â Â Â Âreturn err;
>
> Â Â Â Âif (((status[15] >> 4) & 0x0F) != current_limit)
> - Â Â Â Â Â Â Â pr_warn("%s: Problem setting current limit!\n",
> + Â Â Â Â Â Â Â pr_warn("%s: problem setting current limit!\n",
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
>
> Â Â Â Âreturn 0;
> @@ -619,8 +620,8 @@ static int mmc_sd_init_uhs_card(struct mmc_card *card)
>
> Â Â Â Âstatus = kmalloc(64, GFP_KERNEL);
> Â Â Â Âif (!status) {
> - Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for "
> - Â Â Â Â Â Â Â Â Â Â Â "switch capabilities.\n", mmc_hostname(card->host));
> + Â Â Â Â Â Â Â pr_err("%s: could not allocate a buffer for switch capabilities\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(card->host));
> Â Â Â Â Â Â Â Âreturn -ENOMEM;
> Â Â Â Â}
>
> diff --git a/drivers/mmc/core/sdio_bus.c b/drivers/mmc/core/sdio_bus.c
> index 3d82f12..57ce052 100644
> --- a/drivers/mmc/core/sdio_bus.c
> +++ b/drivers/mmc/core/sdio_bus.c
> @@ -11,6 +11,8 @@
> Â* SDIO function driver model
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/device.h>
> Â#include <linux/err.h>
> Â#include <linux/export.h>
> diff --git a/drivers/mmc/core/sdio_io.c b/drivers/mmc/core/sdio_io.c
> index 8f6f5ac..7c87732 100644
> --- a/drivers/mmc/core/sdio_io.c
> +++ b/drivers/mmc/core/sdio_io.c
> @@ -9,6 +9,8 @@
> Â* your option) any later version.
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/export.h>
> Â#include <linux/mmc/host.h>
> Â#include <linux/mmc/card.h>
> @@ -65,7 +67,7 @@ int sdio_enable_func(struct sdio_func *func)
> Â Â Â ÂBUG_ON(!func);
> Â Â Â ÂBUG_ON(!func->card);
>
> - Â Â Â pr_debug("SDIO: Enabling device %s...\n", sdio_func_id(func));
> + Â Â Â pr_debug("Enabling device %s...\n", sdio_func_id(func));
>
> Â Â Â Âret = mmc_io_rw_direct(func->card, 0, 0, SDIO_CCCR_IOEx, 0, &reg);
> Â Â Â Âif (ret)
> @@ -90,12 +92,12 @@ int sdio_enable_func(struct sdio_func *func)
> Â Â Â Â Â Â Â Â Â Â Â Âgoto err;
> Â Â Â Â}
>
> - Â Â Â pr_debug("SDIO: Enabled device %s\n", sdio_func_id(func));
> + Â Â Â pr_debug("Enabled device %s\n", sdio_func_id(func));
>
> Â Â Â Âreturn 0;
>
> Âerr:
> - Â Â Â pr_debug("SDIO: Failed to enable device %s\n", sdio_func_id(func));
> + Â Â Â pr_debug("Failed to enable device %s\n", sdio_func_id(func));
> Â Â Â Âreturn ret;
> Â}
> ÂEXPORT_SYMBOL_GPL(sdio_enable_func);
> @@ -115,7 +117,7 @@ int sdio_disable_func(struct sdio_func *func)
> Â Â Â ÂBUG_ON(!func);
> Â Â Â ÂBUG_ON(!func->card);
>
> - Â Â Â pr_debug("SDIO: Disabling device %s...\n", sdio_func_id(func));
> + Â Â Â pr_debug("Disabling device %s...\n", sdio_func_id(func));
>
> Â Â Â Âret = mmc_io_rw_direct(func->card, 0, 0, SDIO_CCCR_IOEx, 0, &reg);
> Â Â Â Âif (ret)
> @@ -127,12 +129,12 @@ int sdio_disable_func(struct sdio_func *func)
> Â Â Â Âif (ret)
> Â Â Â Â Â Â Â Âgoto err;
>
> - Â Â Â pr_debug("SDIO: Disabled device %s\n", sdio_func_id(func));
> + Â Â Â pr_debug("Disabled device %s\n", sdio_func_id(func));
>
> Â Â Â Âreturn 0;
>
> Âerr:
> - Â Â Â pr_debug("SDIO: Failed to disable device %s\n", sdio_func_id(func));
> + Â Â Â pr_debug("Failed to disable device %s\n", sdio_func_id(func));
> Â Â Â Âreturn -EIO;
> Â}
> ÂEXPORT_SYMBOL_GPL(sdio_disable_func);
> diff --git a/drivers/mmc/core/sdio_irq.c b/drivers/mmc/core/sdio_irq.c
> index bc9e67a..533d0e5 100644
> --- a/drivers/mmc/core/sdio_irq.c
> +++ b/drivers/mmc/core/sdio_irq.c
> @@ -13,6 +13,8 @@
> Â* your option) any later version.
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/kernel.h>
> Â#include <linux/sched.h>
> Â#include <linux/kthread.h>
> @@ -47,7 +49,7 @@ static int process_sdio_pending_irqs(struct mmc_card *card)
> Â Â Â Âret = mmc_io_rw_direct(card, 0, 0, SDIO_CCCR_INTx, 0, &pending);
> Â Â Â Âif (ret) {
> Â Â Â Â Â Â Â Âpr_debug("%s: error %d reading SDIO_CCCR_INTx\n",
> - Â Â Â Â Â Â Â Â Â Â Âmmc_card_id(card), ret);
> + Â Â Â Â Â Â Â Â Â Â Â Âmmc_card_id(card), ret);
> Â Â Â Â Â Â Â Âreturn ret;
> Â Â Â Â}
>
> @@ -233,10 +235,10 @@ int sdio_claim_irq(struct sdio_func *func, sdio_irq_handler_t *handler)
> Â Â Â ÂBUG_ON(!func);
> Â Â Â ÂBUG_ON(!func->card);
>
> - Â Â Â pr_debug("SDIO: Enabling IRQ for %s...\n", sdio_func_id(func));
> + Â Â Â pr_debug("Enabling IRQ for %s...\n", sdio_func_id(func));
>
> Â Â Â Âif (func->irq_handler) {
> - Â Â Â Â Â Â Â pr_debug("SDIO: IRQ for %s already in use.\n", sdio_func_id(func));
> + Â Â Â Â Â Â Â pr_debug("IRQ for %s already in use\n", sdio_func_id(func));
> Â Â Â Â Â Â Â Âreturn -EBUSY;
> Â Â Â Â}
>
> @@ -276,7 +278,7 @@ int sdio_release_irq(struct sdio_func *func)
> Â Â Â ÂBUG_ON(!func);
> Â Â Â ÂBUG_ON(!func->card);
>
> - Â Â Â pr_debug("SDIO: Disabling IRQ for %s...\n", sdio_func_id(func));
> + Â Â Â pr_debug("Disabling IRQ for %s...\n", sdio_func_id(func));
>
> Â Â Â Âif (func->irq_handler) {
> Â Â Â Â Â Â Â Âfunc->irq_handler = NULL;
> diff --git a/drivers/mmc/host/at91_mci.c b/drivers/mmc/host/at91_mci.c
> index f437c3e..d63f453 100644
> --- a/drivers/mmc/host/at91_mci.c
> +++ b/drivers/mmc/host/at91_mci.c
> @@ -968,8 +968,7 @@ static int __init at91_mci_probe(struct platform_device *pdev)
> Â Â Â Â Â Â Â Âif (at91mci_is_mci1rev2xx())
> Â Â Â Â Â Â Â Â Â Â Â Âmmc->caps |= MMC_CAP_4_BIT_DATA;
> Â Â Â Â Â Â Â Âelse
> - Â Â Â Â Â Â Â Â Â Â Â dev_warn(&pdev->dev, "4 wire bus mode not supported"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " - using 1 wire\n");
> + Â Â Â Â Â Â Â Â Â Â Â dev_warn(&pdev->dev, "4 wire bus mode not supported - using 1 wire\n");
> Â Â Â Â}
>
> Â Â Â Âhost->buffer = dma_alloc_coherent(&pdev->dev, MCI_BUFSIZE,
> diff --git a/drivers/mmc/host/au1xmmc.c b/drivers/mmc/host/au1xmmc.c
> index dbd0c8a..cfd325d 100644
> --- a/drivers/mmc/host/au1xmmc.c
> +++ b/drivers/mmc/host/au1xmmc.c
> @@ -32,6 +32,8 @@
> Â* (the low to high transition will not occur).
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/module.h>
> Â#include <linux/init.h>
> Â#include <linux/platform_device.h>
> @@ -54,10 +56,11 @@
> Â/* #define DEBUG */
>
> Â#ifdef DEBUG
> -#define DBG(fmt, idx, args...) \
> - Â Â Â pr_debug("au1xmmc(%d): DEBUG: " fmt, idx, ##args)
> +#define DBG(idx, fmt, ...) Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â \
> + Â Â Â pr_debug("(%d): DEBUG: " fmt, idx, ##__VA_ARGS__)
> Â#else
> -#define DBG(fmt, idx, args...) do {} while (0)
> +#define DBG(idx, fmt, ...) Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â \
> + Â Â Â no_printk(fmt, ##__VA_ARGS__)
> Â#endif
>
> Â/* Hardware definitions */
> @@ -277,8 +280,7 @@ static int au1xmmc_send_command(struct au1xmmc_host *host, int wait,
> Â Â Â Â Â Â Â Âmmccmd |= SD_CMD_RT_3;
> Â Â Â Â Â Â Â Âbreak;
> Â Â Â Âdefault:
> - Â Â Â Â Â Â Â pr_info("au1xmmc: unhandled response type %02x\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_resp_type(cmd));
> + Â Â Â Â Â Â Â pr_info("unhandled response type %02x\n", mmc_resp_type(cmd));
> Â Â Â Â Â Â Â Âreturn -EINVAL;
> Â Â Â Â}
>
> @@ -478,19 +480,19 @@ static void au1xmmc_receive_pio(struct au1xmmc_host *host)
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
>
> Â Â Â Â Â Â Â Âif (status & SD_STATUS_RC) {
> - Â Â Â Â Â Â Â Â Â Â Â DBG("RX CRC Error [%d + %d].\n", host->pdev->id,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, count);
> + Â Â Â Â Â Â Â Â Â Â Â DBG(host->pdev->id, "RX CRC Error [%d + %d]\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, count);
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â}
>
> Â Â Â Â Â Â Â Âif (status & SD_STATUS_RO) {
> - Â Â Â Â Â Â Â Â Â Â Â DBG("RX Overrun [%d + %d]\n", host->pdev->id,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, count);
> + Â Â Â Â Â Â Â Â Â Â Â DBG(host->pdev->id, "RX Overrun [%d + %d]\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, count);
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Âelse if (status & SD_STATUS_RU) {
> - Â Â Â Â Â Â Â Â Â Â Â DBG("RX Underrun [%d + %d]\n", host->pdev->id,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, Âcount);
> + Â Â Â Â Â Â Â Â Â Â Â DBG(host->pdev->id, "RX Underrun [%d + %d]\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio.len, count);
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â}
>
> @@ -835,8 +837,7 @@ static irqreturn_t au1xmmc_irq(int irq, void *dev_id)
> Â Â Â Â Â Â Â Â Â Â Â Âau1xmmc_receive_pio(host);
>
> Â Â Â Â} else if (status & 0x203F3C70) {
> - Â Â Â Â Â Â Â Â Â Â Â DBG("Unhandled status %8.8x\n", host->pdev->id,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â status);
> + Â Â Â Â Â Â Â Â Â Â Â DBG(host->pdev->id, "Unhandled status %8.8x\n", status);
> Â Â Â Â}
>
> Â Â Â Âau_writel(status, HOST_STATUS(host));
> @@ -1050,7 +1051,7 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
> Â Â Â Âif (has_dbdma()) {
> Â Â Â Â Â Â Â Âret = au1xmmc_dbdma_init(host);
> Â Â Â Â Â Â Â Âif (ret)
> - Â Â Â Â Â Â Â Â Â Â Â pr_info(DRIVER_NAME ": DBDMA init failed; using PIO\n");
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("DBDMA init failed; using PIO\n");
> Â Â Â Â}
>
> Â#ifdef CONFIG_LEDS_CLASS
> @@ -1075,8 +1076,8 @@ static int __devinit au1xmmc_probe(struct platform_device *pdev)
>
> Â Â Â Âplatform_set_drvdata(pdev, host);
>
> - Â Â Â pr_info(DRIVER_NAME ": MMC Controller %d set up at %8.8X"
> - Â Â Â Â Â Â Â " (mode=%s)\n", pdev->id, host->iobase,
> + Â Â Â pr_info("MMC Controller %d set up at %8.8X (mode=%s)\n",
> + Â Â Â Â Â Â Â pdev->id, host->iobase,
> Â Â Â Â Â Â Â Âhost->flags & HOST_F_DMA ? "dma" : "pio");
>
> Â Â Â Âreturn 0; Â Â Â /* all ok */
> @@ -1206,7 +1207,7 @@ static int __init au1xmmc_init(void)
> Â Â Â Â Â Â Â Â*/
> Â Â Â Â Â Â Â Âmemid = au1xxx_ddma_add_device(&au1xmmc_mem_dbdev);
> Â Â Â Â Â Â Â Âif (!memid)
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("au1xmmc: cannot add memory dbdma\n");
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("cannot add memory dbdma\n");
> Â Â Â Â}
> Â Â Â Âreturn platform_driver_register(&au1xmmc_driver);
> Â}
> diff --git a/drivers/mmc/host/bfin_sdh.c b/drivers/mmc/host/bfin_sdh.c
> index 0371bf5..780a1eb 100644
> --- a/drivers/mmc/host/bfin_sdh.c
> +++ b/drivers/mmc/host/bfin_sdh.c
> @@ -162,8 +162,7 @@ static int sdh_setup_data(struct sdh_host *host, struct mmc_data *data)
> Â Â Â Â Â Â Â Â Â Â Â Âhost->sg_cpu[i].cfg = dma_cfg;
> Â Â Â Â Â Â Â Â Â Â Â Âhost->sg_cpu[i].x_count = sg_dma_len(sg) / 4;
> Â Â Â Â Â Â Â Â Â Â Â Âhost->sg_cpu[i].x_modify = 4;
> - Â Â Â Â Â Â Â Â Â Â Â dev_dbg(mmc_dev(host->mmc), "%d: start_addr:0x%lx, "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "cfg:0x%x, x_count:0x%x, x_modify:0x%x\n",
> + Â Â Â Â Â Â Â Â Â Â Â dev_dbg(mmc_dev(host->mmc), "%d: start_addr:0x%lx, cfg:0x%x, x_count:0x%x, x_modify:0x%x\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âi, host->sg_cpu[i].start_addr,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->sg_cpu[i].cfg, host->sg_cpu[i].x_count,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->sg_cpu[i].x_modify);
> diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c
> index 3aaeb08..5d34c01 100644
> --- a/drivers/mmc/host/dw_mmc.c
> +++ b/drivers/mmc/host/dw_mmc.c
> @@ -554,8 +554,8 @@ static void dw_mci_setup_bus(struct dw_mci_slot *slot)
> Â Â Â Â Â Â Â Â Â Â Â Âdiv = (host->bus_hz Â/ slot->clock) >> 1;
>
> Â Â Â Â Â Â Â Âdev_info(&slot->mmc->class_dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â"Bus speed (slot %d) = %dHz (slot req %dHz, actual %dHZ"
> - Â Â Â Â Â Â Â Â Â Â Â Â" div = %d)\n", slot->id, host->bus_hz, slot->clock,
> + Â Â Â Â Â Â Â Â Â Â Â Â"Bus speed (slot %d) = %dHz (slot req %dHz, actual %dHZ div = %d)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Âslot->id, host->bus_hz, slot->clock,
> Â Â Â Â Â Â Â Â Â Â Â Â div ? ((host->bus_hz / div) >> 1) : host->bus_hz, div);
>
> Â Â Â Â Â Â Â Â/* disable clock */
> @@ -942,8 +942,7 @@ static void dw_mci_tasklet_func(unsigned long priv)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdata->error = -ETIMEDOUT;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_err(&host->pdev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "data FIFO error "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "(status=%08x)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "data FIFO error (status=%08x)\n",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âstatus);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdata->error = -EIO;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â}
> @@ -1776,8 +1775,8 @@ static void dw_mci_init_dma(struct dw_mci *host)
>
> Â Â Â Âif (host->dma_ops->init) {
> Â Â Â Â Â Â Â Âif (host->dma_ops->init(host)) {
> - Â Â Â Â Â Â Â Â Â Â Â dev_err(&host->pdev->dev, "%s: Unable to initialize "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "DMA Controller.\n", __func__);
> + Â Â Â Â Â Â Â Â Â Â Â dev_err(&host->pdev->dev, "%s: Unable to initialize DMA Controller\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â __func__);
> Â Â Â Â Â Â Â Â Â Â Â Âgoto no_dma;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â} else {
> @@ -1985,9 +1984,8 @@ static int dw_mci_probe(struct platform_device *pdev)
> Â Â Â Â Â Â Â Â Â DW_MCI_ERROR_FLAGS | SDMMC_INT_CD);
> Â Â Â Âmci_writel(host, CTRL, SDMMC_CTRL_INT_ENABLE); /* Enable mci interrupt */
>
> - Â Â Â dev_info(&pdev->dev, "DW MMC controller at irq %d, "
> - Â Â Â Â Â Â Â Â"%d bit host data width, "
> - Â Â Â Â Â Â Â Â"%u deep fifo\n",
> + Â Â Â dev_info(&pdev->dev,
> + Â Â Â "DW MMC controller at irq %d, %d bit host data width, %u deep fifo\n",
> Â Â Â Â Â Â Â Â irq, width, fifo_size);
I think that tab is needed this "DW MMC controller at irq ...." ?
> Â Â Â Âif (host->quirks & DW_MCI_QUIRK_IDMAC_DTO)
> Â Â Â Â Â Â Â Âdev_info(&pdev->dev, "Internal DMAC interrupt fix enabled.\n");
> diff --git a/drivers/mmc/host/mmc_spi.c b/drivers/mmc/host/mmc_spi.c
> index 92946b8..f77899f 100644
> --- a/drivers/mmc/host/mmc_spi.c
> +++ b/drivers/mmc/host/mmc_spi.c
> @@ -860,9 +860,9 @@ mmc_spi_readblock(struct mmc_spi_host *host, struct spi_transfer *t,
>
> Â Â Â Â Â Â Â Âbe16_to_cpus(&scratch->crc_val);
> Â Â Â Â Â Â Â Âif (scratch->crc_val != crc) {
> - Â Â Â Â Â Â Â Â Â Â Â dev_dbg(&spi->dev, "read - crc error: crc_val=0x%04x, "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "computed=0x%04x len=%d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â scratch->crc_val, crc, t->len);
> + Â Â Â Â Â Â Â Â Â Â Â dev_dbg(&spi->dev,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "read - crc error: crc_val=0x%04x, computed=0x%04x len=%d\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â scratch->crc_val, crc, t->len);
> Â Â Â Â Â Â Â Â Â Â Â Âreturn -EILSEQ;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> @@ -1253,8 +1253,7 @@ static void mmc_spi_set_ios(struct mmc_host *mmc, struct mmc_ios *ios)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmres = spi_setup(host->spi);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (mres < 0)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_dbg(&host->spi->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "switch back to SPI mode 3"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " failed\n");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "switch back to SPI mode 3 failed\n");
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â}
>
> diff --git a/drivers/mmc/host/mmci.c b/drivers/mmc/host/mmci.c
> index 0ea9935..33b3cd1 100644
> --- a/drivers/mmc/host/mmci.c
> +++ b/drivers/mmc/host/mmci.c
> @@ -1229,8 +1229,7 @@ static int __devinit mmci_probe(struct amba_device *dev,
> Â Â Â Â Â Â Â Â Â Â Â Âhost->mmc->ocr_avail = (u32) mask;
> Â Â Â Â Â Â Â Â Â Â Â Âif (plat->ocr_mask)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_warn(&dev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"Provided ocr_mask/setpower will not be used "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"(using regulator instead)\n");
> +"Provided ocr_mask/setpower will not be used (using regulator instead)\n");
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> Â#endif
> diff --git a/drivers/mmc/host/mvsdio.c b/drivers/mmc/host/mvsdio.c
> index 211a495..93012a0 100644
> --- a/drivers/mmc/host/mvsdio.c
> +++ b/drivers/mmc/host/mvsdio.c
> @@ -84,9 +84,9 @@ static int mvsd_setup_data(struct mvsd_host *host, struct mmc_data *data)
> Â Â Â Â Â Â Â Â Â Â Â Âhw_state = mvsd_read(MVSD_HW_STATE);
> Â Â Â Â Â Â Â Â Â Â Â Âcount++;
> Â Â Â Â Â Â Â Â} while (!(hw_state & (1 << 13)));
> - Â Â Â Â Â Â Â dev_dbg(host->dev, "*** wait for FIFO_EMPTY bit "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"(hw=0x%04x, count=%d, jiffies=%ld)\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhw_state, count, jiffies - (t - HZ));
> + Â Â Â Â Â Â Â dev_dbg(host->dev,
> +"*** wait for FIFO_EMPTY bit (hw=0x%04x, count=%d, jiffies=%ld)\n",
> + Â Â Â Â Â Â Â Â Â Â Â hw_state, count, jiffies - (t - HZ));
> Â Â Â Â}
>
> Â Â Â Â/* If timeout=0 then maximum timeout index is used. */
> @@ -117,10 +117,9 @@ static int mvsd_setup_data(struct mvsd_host *host, struct mmc_data *data)
> Â Â Â Â Â Â Â Âhost->pio_size = data->blocks * data->blksz;
> Â Â Â Â Â Â Â Âhost->pio_ptr = sg_virt(data->sg);
> Â Â Â Â Â Â Â Âif (!nodma)
> - Â Â Â Â Â Â Â Â Â Â Â pr_debug("%s: fallback to PIO for data "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "at 0x%p size %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â host->pio_ptr, host->pio_size);
> + Â Â Â Â Â Â Â Â Â Â Â pr_debug("%s: fallback to PIO for data at 0x%p size %d\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc),
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->pio_ptr, host->pio_size);
> Â Â Â Â Â Â Â Âreturn 1;
> Â Â Â Â} else {
> Â Â Â Â Â Â Â Âdma_addr_t phys_addr;
> @@ -489,9 +488,9 @@ static irqreturn_t mvsd_irq(int irq, void *dev)
> Â Â Â Âif (irq_handled)
> Â Â Â Â Â Â Â Âreturn IRQ_HANDLED;
>
> - Â Â Â pr_err("%s: unhandled interrupt status=0x%04x en=0x%04x "
> - Â Â Â Â Â Â Â Â Â Â Â "pio=%d\n", mmc_hostname(host->mmc), intr_status,
> - Â Â Â Â Â Â Â Â Â Â Â host->intr_en, host->pio_size);
> + Â Â Â pr_err("%s: unhandled interrupt status=0x%04x en=0x%04x pio=%d\n",
> + Â Â Â Â Â Â Âmmc_hostname(host->mmc), intr_status,
> + Â Â Â Â Â Â Âhost->intr_en, host->pio_size);
> Â Â Â Âreturn IRQ_NONE;
> Â}
>
> @@ -505,13 +504,13 @@ static void mvsd_timeout_timer(unsigned long data)
> Â Â Â Âspin_lock_irqsave(&host->lock, flags);
> Â Â Â Âmrq = host->mrq;
> Â Â Â Âif (mrq) {
> - Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware interrupt.\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc));
> - Â Â Â Â Â Â Â pr_err("%s: hw_state=0x%04x, intr_status=0x%04x "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "intr_en=0x%04x\n", mmc_hostname(host->mmc),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mvsd_read(MVSD_HW_STATE),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mvsd_read(MVSD_NOR_INTR_STATUS),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mvsd_read(MVSD_NOR_INTR_EN));
> + Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware interrupt\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â pr_err("%s: hw_state=0x%04x, intr_status=0x%04x intr_en=0x%04x\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc),
> + Â Â Â Â Â Â Â Â Â Â Âmvsd_read(MVSD_HW_STATE),
> + Â Â Â Â Â Â Â Â Â Â Âmvsd_read(MVSD_NOR_INTR_STATUS),
> + Â Â Â Â Â Â Â Â Â Â Âmvsd_read(MVSD_NOR_INTR_EN));
>
> Â Â Â Â Â Â Â Âhost->mrq = NULL;
>
> diff --git a/drivers/mmc/host/omap_hsmmc.c b/drivers/mmc/host/omap_hsmmc.c
> index 1455bd6..dd5159b 100644
> --- a/drivers/mmc/host/omap_hsmmc.c
> +++ b/drivers/mmc/host/omap_hsmmc.c
> @@ -1264,9 +1264,8 @@ static void omap_hsmmc_protect_card(struct omap_hsmmc_host *host)
> Â Â Â Âhost->reqs_blocked = 0;
> Â Â Â Âif (mmc_slot(host).get_cover_state(host->dev, host->slot_id)) {
> Â Â Â Â Â Â Â Âif (host->protect_card) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: cover is closed, "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"card is now accessible\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("%s: cover is closed, card is now accessible\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc));
> Â Â Â Â Â Â Â Â Â Â Â Âhost->protect_card = 0;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â} else {
> @@ -1967,8 +1966,7 @@ static int __init omap_hsmmc_probe(struct platform_device *pdev)
>
> Â Â Â Â Â Â Â Âif (host->got_dbclk)
> Â Â Â Â Â Â Â Â Â Â Â Âif (clk_enable(host->dbclk) != 0)
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â dev_dbg(mmc_dev(host->mmc), "Enabling debounce"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " clk failed\n");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â dev_dbg(mmc_dev(host->mmc), "Enabling debounce clk failed\n");
> Â Â Â Â}
>
> Â Â Â Â/* Since we do only SG emulation, we can have as many segs
> @@ -2169,8 +2167,7 @@ static int omap_hsmmc_suspend(struct device *dev)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->slot_id);
> Â Â Â Â Â Â Â Â Â Â Â Âif (ret) {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_dbg(mmc_dev(host->mmc),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "Unable to handle MMC board"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " level suspend\n");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "Unable to handle MMC board level suspend\n");
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->suspended = 0;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âreturn ret;
> Â Â Â Â Â Â Â Â Â Â Â Â}
> diff --git a/drivers/mmc/host/sdhci-pci.c b/drivers/mmc/host/sdhci-pci.c
> index 6878a94..67c5751 100644
> --- a/drivers/mmc/host/sdhci-pci.c
> +++ b/drivers/mmc/host/sdhci-pci.c
> @@ -460,8 +460,7 @@ static int jmicron_probe(struct sdhci_pci_chip *chip)
>
> Â Â Â Â Â Â Â Âif (sd_dev) {
> Â Â Â Â Â Â Â Â Â Â Â Âpci_dev_put(sd_dev);
> - Â Â Â Â Â Â Â Â Â Â Â dev_info(&chip->pdev->dev, "Refusing to bind to "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "secondary interface.\n");
> + Â Â Â Â Â Â Â Â Â Â Â dev_info(&chip->pdev->dev, "Refusing to bind to secondary interface\n");
> Â Â Â Â Â Â Â Â Â Â Â Âreturn -ENODEV;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> @@ -628,10 +627,10 @@ static int syskt_probe_slot(struct sdhci_pci_slot *slot)
>
> Â Â Â Âu8 board_rev = readb(slot->host->ioaddr + SYSKT_BOARD_REV);
> Â Â Â Âu8 Âchip_rev = readb(slot->host->ioaddr + SYSKT_CHIP_REV);
> - Â Â Â dev_info(&slot->chip->pdev->dev, "SysKonnect CardBus2SDIO, "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"board rev %d.%d, chip rev %d.%d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âboard_rev >> 4, board_rev & 0xf,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âchip_rev >> 4, Âchip_rev & 0xf);
> + Â Â Â dev_info(&slot->chip->pdev->dev,
> + Â Â Â Â Â Â Â Â"SysKonnect CardBus2SDIO, board rev %d.%d, chip rev %d.%d\n",
> + Â Â Â Â Â Â Â Âboard_rev >> 4, board_rev & 0xf,
> + Â Â Â Â Â Â Â Âchip_rev >> 4, Âchip_rev & 0xf);
> Â Â Â Âif (chip_rev >= 0x20)
> Â Â Â Â Â Â Â Âslot->host->quirks |= SDHCI_QUIRK_FORCE_DMA;
>
> @@ -925,8 +924,7 @@ static int sdhci_pci_enable_dma(struct sdhci_host *host)
> Â Â Â Âif (((pdev->class & 0xFFFF00) == (PCI_CLASS_SYSTEM_SDHCI << 8)) &&
> Â Â Â Â Â Â Â Â((pdev->class & 0x0000FF) != PCI_SDHCI_IFDMA) &&
> Â Â Â Â Â Â Â Â(host->flags & SDHCI_USE_SDMA)) {
> - Â Â Â Â Â Â Â dev_warn(&pdev->dev, "Will use DMA mode even though HW "
> - Â Â Â Â Â Â Â Â Â Â Â "doesn't fully claim to support it.\n");
> + Â Â Â Â Â Â Â dev_warn(&pdev->dev, "Will use DMA mode even though HW doesn't fully claim to support it\n");
> Â Â Â Â}
>
> Â Â Â Âret = pci_set_dma_mask(pdev, DMA_BIT_MASK(32));
> @@ -1202,8 +1200,7 @@ static struct sdhci_pci_slot * __devinit sdhci_pci_probe_slot(
> Â Â Â Â}
>
> Â Â Â Âif (pci_resource_len(pdev, bar) != 0x100) {
> - Â Â Â Â Â Â Â dev_err(&pdev->dev, "Invalid iomem size. You may "
> - Â Â Â Â Â Â Â Â Â Â Â "experience problems.\n");
> + Â Â Â Â Â Â Â dev_err(&pdev->dev, "Invalid iomem size. You may experience problems.\n");
> Â Â Â Â}
>
> Â Â Â Âif ((pdev->class & 0x0000FF) == PCI_SDHCI_IFVENDOR) {
> diff --git a/drivers/mmc/host/sdhci-pxav3.c b/drivers/mmc/host/sdhci-pxav3.c
> index 15673a7..0d8f061 100644
> --- a/drivers/mmc/host/sdhci-pxav3.c
> +++ b/drivers/mmc/host/sdhci-pxav3.c
> @@ -82,11 +82,8 @@ static void pxav3_gen_init_74_clocks(struct sdhci_host *host, u8 power_mode)
> Â Â Â Â Â Â Â Â Â Â Â Â&& power_mode == MMC_POWER_ON) {
>
> Â Â Â Â Â Â Â Âdev_dbg(mmc_dev(host->mmc),
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "%s: slot->power_mode = %d,"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "ios->power_mode = %d\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â __func__,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â pxa->power_mode,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â power_mode);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "%s: slot->power_mode = %d, ios->power_mode = %d\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â __func__, pxa->power_mode, power_mode);
>
> Â Â Â Â Â Â Â Â/* set we want notice of when 74 clocks are sent */
> Â Â Â Â Â Â Â Âtmp = readw(host->ioaddr + SD_CE_ATA_2);
> diff --git a/drivers/mmc/host/sdhci.c b/drivers/mmc/host/sdhci.c
> index d146bff..d31c31d 100644
> --- a/drivers/mmc/host/sdhci.c
> +++ b/drivers/mmc/host/sdhci.c
> @@ -13,6 +13,8 @@
> Â* Â Â - JMicron (hardware and technical support)
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/delay.h>
> Â#include <linux/highmem.h>
> Â#include <linux/io.h>
> @@ -32,8 +34,8 @@
>
> Â#define DRIVER_NAME "sdhci"
>
> -#define DBG(f, x...) \
> - Â Â Â pr_debug(DRIVER_NAME " [%s()]: " f, __func__,## x)
> +#define DBG(fmt, ...) Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â\
> + Â Â Â pr_debug("[%s()]: " fmt, __func__, ##__VA_ARGS__)
>
> Â#if defined(CONFIG_LEDS_CLASS) || (defined(CONFIG_LEDS_CLASS_MODULE) && \
> Â Â Â Âdefined(CONFIG_MMC_SDHCI_MODULE))
> @@ -68,51 +70,51 @@ static inline int sdhci_runtime_pm_put(struct sdhci_host *host)
>
> Âstatic void sdhci_dumpregs(struct sdhci_host *host)
> Â{
> - Â Â Â pr_debug(DRIVER_NAME ": =========== REGISTER DUMP (%s)===========\n",
> - Â Â Â Â Â Â Â mmc_hostname(host->mmc));
> -
> - Â Â Â pr_debug(DRIVER_NAME ": Sys addr: 0x%08x | Version: Â0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_DMA_ADDRESS),
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_HOST_VERSION));
> - Â Â Â pr_debug(DRIVER_NAME ": Blk size: 0x%08x | Blk cnt: Â0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_BLOCK_SIZE),
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_BLOCK_COUNT));
> - Â Â Â pr_debug(DRIVER_NAME ": Argument: 0x%08x | Trn mode: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_ARGUMENT),
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_TRANSFER_MODE));
> - Â Â Â pr_debug(DRIVER_NAME ": Present: Â0x%08x | Host ctl: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_PRESENT_STATE),
> - Â Â Â Â Â Â Â sdhci_readb(host, SDHCI_HOST_CONTROL));
> - Â Â Â pr_debug(DRIVER_NAME ": Power: Â Â0x%08x | Blk gap: Â0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readb(host, SDHCI_POWER_CONTROL),
> - Â Â Â Â Â Â Â sdhci_readb(host, SDHCI_BLOCK_GAP_CONTROL));
> - Â Â Â pr_debug(DRIVER_NAME ": Wake-up: Â0x%08x | Clock: Â Â0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readb(host, SDHCI_WAKE_UP_CONTROL),
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_CLOCK_CONTROL));
> - Â Â Â pr_debug(DRIVER_NAME ": Timeout: Â0x%08x | Int stat: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readb(host, SDHCI_TIMEOUT_CONTROL),
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_INT_STATUS));
> - Â Â Â pr_debug(DRIVER_NAME ": Int enab: 0x%08x | Sig enab: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_INT_ENABLE),
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_SIGNAL_ENABLE));
> - Â Â Â pr_debug(DRIVER_NAME ": AC12 err: 0x%08x | Slot int: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_ACMD12_ERR),
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_SLOT_INT_STATUS));
> - Â Â Â pr_debug(DRIVER_NAME ": Caps: Â Â 0x%08x | Caps_1: Â 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_CAPABILITIES),
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_CAPABILITIES_1));
> - Â Â Â pr_debug(DRIVER_NAME ": Cmd: Â Â Â0x%08x | Max curr: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_COMMAND),
> - Â Â Â Â Â Â Â sdhci_readl(host, SDHCI_MAX_CURRENT));
> - Â Â Â pr_debug(DRIVER_NAME ": Host ctl2: 0x%08x\n",
> - Â Â Â Â Â Â Â sdhci_readw(host, SDHCI_HOST_CONTROL2));
> + Â Â Â pr_debug("=========== REGISTER DUMP (%s)===========\n",
> + Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> +
> + Â Â Â pr_debug("Sys addr: 0x%08x | Version: Â0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_DMA_ADDRESS),
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_HOST_VERSION));
> + Â Â Â pr_debug("Blk size: 0x%08x | Blk cnt: Â0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_BLOCK_SIZE),
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_BLOCK_COUNT));
> + Â Â Â pr_debug("Argument: 0x%08x | Trn mode: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_ARGUMENT),
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_TRANSFER_MODE));
> + Â Â Â pr_debug("Present: Â0x%08x | Host ctl: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_PRESENT_STATE),
> + Â Â Â Â Â Â Â Âsdhci_readb(host, SDHCI_HOST_CONTROL));
> + Â Â Â pr_debug("Power: Â Â0x%08x | Blk gap: Â0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readb(host, SDHCI_POWER_CONTROL),
> + Â Â Â Â Â Â Â Âsdhci_readb(host, SDHCI_BLOCK_GAP_CONTROL));
> + Â Â Â pr_debug("Wake-up: Â0x%08x | Clock: Â Â0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readb(host, SDHCI_WAKE_UP_CONTROL),
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_CLOCK_CONTROL));
> + Â Â Â pr_debug("Timeout: Â0x%08x | Int stat: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readb(host, SDHCI_TIMEOUT_CONTROL),
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_INT_STATUS));
> + Â Â Â pr_debug("Int enab: 0x%08x | Sig enab: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_INT_ENABLE),
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_SIGNAL_ENABLE));
> + Â Â Â pr_debug("AC12 err: 0x%08x | Slot int: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_ACMD12_ERR),
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_SLOT_INT_STATUS));
> + Â Â Â pr_debug("Caps: Â Â 0x%08x | Caps_1: Â 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_CAPABILITIES),
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_CAPABILITIES_1));
> + Â Â Â pr_debug("Cmd: Â Â Â0x%08x | Max curr: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_COMMAND),
> + Â Â Â Â Â Â Â Âsdhci_readl(host, SDHCI_MAX_CURRENT));
> + Â Â Â pr_debug("Host ctl2: 0x%08x\n",
> + Â Â Â Â Â Â Â Âsdhci_readw(host, SDHCI_HOST_CONTROL2));
>
> Â Â Â Âif (host->flags & SDHCI_USE_ADMA)
> - Â Â Â Â Â Â Â pr_debug(DRIVER_NAME ": ADMA Err: 0x%08x | ADMA Ptr: 0x%08x\n",
> - Â Â Â Â Â Â Â Â Â Â Âreadl(host->ioaddr + SDHCI_ADMA_ERROR),
> - Â Â Â Â Â Â Â Â Â Â Âreadl(host->ioaddr + SDHCI_ADMA_ADDRESS));
> + Â Â Â Â Â Â Â pr_debug("ADMA Err: 0x%08x | ADMA Ptr: 0x%08x\n",
> + Â Â Â Â Â Â Â Â Â Â Â Âreadl(host->ioaddr + SDHCI_ADMA_ERROR),
> + Â Â Â Â Â Â Â Â Â Â Â Âreadl(host->ioaddr + SDHCI_ADMA_ADDRESS));
>
> - Â Â Â pr_debug(DRIVER_NAME ": ===========================================\n");
> + Â Â Â pr_debug("===========================================\n");
> Â}
>
> Â/*****************************************************************************\
> @@ -417,7 +419,7 @@ static void sdhci_transfer_pio(struct sdhci_host *host)
> Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â}
>
> - Â Â Â DBG("PIO transfer complete.\n");
> + Â Â Â DBG("PIO transfer complete\n");
> Â}
>
> Âstatic char *sdhci_kmap_atomic(struct scatterlist *sg, unsigned long *flags)
> @@ -745,9 +747,8 @@ static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command *cmd)
> Â Â Â Â Â Â Â Âif (unlikely(broken)) {
> Â Â Â Â Â Â Â Â Â Â Â Âfor_each_sg(data->sg, sg, data->sg_len, i) {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (sg->length & 0x3) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DBG("Reverting to PIO because of "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "transfer size (%d)\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sg->length);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DBG("Reverting to PIO because of transfer size (%d)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â sg->length);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->flags &= ~SDHCI_REQ_USE_DMA;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â}
> @@ -780,8 +781,7 @@ static void sdhci_prepare_data(struct sdhci_host *host, struct mmc_command *cmd)
> Â Â Â Â Â Â Â Âif (unlikely(broken)) {
> Â Â Â Â Â Â Â Â Â Â Â Âfor_each_sg(data->sg, sg, data->sg_len, i) {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âif (sg->offset & 0x3) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DBG("Reverting to PIO because of "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "bad alignment\n");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â DBG("Reverting to PIO because of bad alignment\n");
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âhost->flags &= ~SDHCI_REQ_USE_DMA;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â}
> @@ -972,8 +972,8 @@ static void sdhci_send_command(struct sdhci_host *host, struct mmc_command *cmd)
>
> Â Â Â Âwhile (sdhci_readl(host, SDHCI_PRESENT_STATE) & mask) {
> Â Â Â Â Â Â Â Âif (timeout == 0) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Controller never released "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "inhibit bit(s).\n", mmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Controller never released inhibit bit(s)\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> Â Â Â Â Â Â Â Â Â Â Â Âsdhci_dumpregs(host);
> Â Â Â Â Â Â Â Â Â Â Â Âcmd->error = -EIO;
> Â Â Â Â Â Â Â Â Â Â Â Âtasklet_schedule(&host->finish_tasklet);
> @@ -1154,8 +1154,8 @@ static void sdhci_set_clock(struct sdhci_host *host, unsigned int clock)
> Â Â Â Âwhile (!((clk = sdhci_readw(host, SDHCI_CLOCK_CONTROL))
> Â Â Â Â Â Â Â Â& SDHCI_CLOCK_INT_STABLE)) {
> Â Â Â Â Â Â Â Âif (timeout == 0) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Internal clock never "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "stabilised.\n", mmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Internal clock never stabilised\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> Â Â Â Â Â Â Â Â Â Â Â Âsdhci_dumpregs(host);
> Â Â Â Â Â Â Â Â Â Â Â Âreturn;
> Â Â Â Â Â Â Â Â}
> @@ -1592,8 +1592,7 @@ static int sdhci_do_start_signal_voltage_switch(struct sdhci_host *host,
> Â Â Â Â Â Â Â Âif (!(ctrl & SDHCI_CTRL_VDD_180))
> Â Â Â Â Â Â Â Â Â Â Â Âreturn 0;
> Â Â Â Â Â Â Â Âelse {
> - Â Â Â Â Â Â Â Â Â Â Â pr_info(DRIVER_NAME ": Switching to 3.3V "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "signalling voltage failed\n");
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("Switching to 3.3V signalling voltage failed\n");
> Â Â Â Â Â Â Â Â Â Â Â Âreturn -EIO;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â} else if (!(ctrl & SDHCI_CTRL_VDD_180) &&
> @@ -1651,8 +1650,7 @@ static int sdhci_do_start_signal_voltage_switch(struct sdhci_host *host,
> Â Â Â Â Â Â Â Âpwr |= SDHCI_POWER_ON;
> Â Â Â Â Â Â Â Âsdhci_writeb(host, pwr, SDHCI_POWER_CONTROL);
>
> - Â Â Â Â Â Â Â pr_info(DRIVER_NAME ": Switching to 1.8V signalling "
> - Â Â Â Â Â Â Â Â Â Â Â "voltage failed, retrying with S18R set to 0\n");
> + Â Â Â Â Â Â Â pr_info("Switching to 1.8V signalling voltage failed, retrying with S18R set to 0\n");
> Â Â Â Â Â Â Â Âreturn -EAGAIN;
> Â Â Â Â} else
> Â Â Â Â Â Â Â Â/* No signal voltage switch required */
> @@ -1774,10 +1772,7 @@ static int sdhci_execute_tuning(struct mmc_host *mmc)
> Â Â Â Â Â Â Â Âspin_lock(&host->lock);
>
> Â Â Â Â Â Â Â Âif (!host->tuning_done) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_info(DRIVER_NAME ": Timeout waiting for "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "Buffer Read Ready interrupt during tuning "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "procedure, falling back to fixed sampling "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "clock\n");
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("Timeout waiting for Buffer Read Ready interrupt during tuning procedure, falling back to fixed sampling clock\n");
> Â Â Â Â Â Â Â Â Â Â Â Âctrl = sdhci_readw(host, SDHCI_HOST_CONTROL2);
> Â Â Â Â Â Â Â Â Â Â Â Âctrl &= ~SDHCI_CTRL_TUNED_CLK;
> Â Â Â Â Â Â Â Â Â Â Â Âctrl &= ~SDHCI_CTRL_EXEC_TUNING;
> @@ -1804,9 +1799,7 @@ static int sdhci_execute_tuning(struct mmc_host *mmc)
> Â Â Â Â Â Â Â Âsdhci_writew(host, ctrl, SDHCI_HOST_CONTROL2);
> Â Â Â Â} else {
> Â Â Â Â Â Â Â Âif (!(ctrl & SDHCI_CTRL_TUNED_CLK)) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_info(DRIVER_NAME ": Tuning procedure"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " failed, falling back to fixed sampling"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " clock\n");
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("Tuning procedure failed, falling back to fixed sampling clock\n");
> Â Â Â Â Â Â Â Â Â Â Â Âerr = -EIO;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> @@ -2010,8 +2003,8 @@ static void sdhci_timeout_timer(unsigned long data)
> Â Â Â Âspin_lock_irqsave(&host->lock, flags);
>
> Â Â Â Âif (host->mrq) {
> - Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware "
> - Â Â Â Â Â Â Â Â Â Â Â "interrupt.\n", mmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware interrupt\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
> Â Â Â Â Â Â Â Âsdhci_dumpregs(host);
>
> Â Â Â Â Â Â Â Âif (host->data) {
> @@ -2056,9 +2049,8 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask)
> Â Â Â ÂBUG_ON(intmask == 0);
>
> Â Â Â Âif (!host->cmd) {
> - Â Â Â Â Â Â Â pr_err("%s: Got command interrupt 0x%08x even "
> - Â Â Â Â Â Â Â Â Â Â Â "though no command operation was in progress.\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), (unsigned)intmask);
> + Â Â Â Â Â Â Â pr_err("%s: Got command interrupt 0x%08x even though no command operation was in progress\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc), (unsigned)intmask);
> Â Â Â Â Â Â Â Âsdhci_dumpregs(host);
> Â Â Â Â Â Â Â Âreturn;
> Â Â Â Â}
> @@ -2087,8 +2079,7 @@ static void sdhci_cmd_irq(struct sdhci_host *host, u32 intmask)
> Â Â Â Â */
> Â Â Â Âif (host->cmd->flags & MMC_RSP_BUSY) {
> Â Â Â Â Â Â Â Âif (host->cmd->data)
> - Â Â Â Â Â Â Â Â Â Â Â DBG("Cannot wait for busy signal when also "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "doing a data transfer");
> + Â Â Â Â Â Â Â Â Â Â Â DBG("Cannot wait for busy signal when also doing a data transfer\n");
> Â Â Â Â Â Â Â Âelse if (!(host->quirks & SDHCI_QUIRK_NO_BUSY_IRQ))
> Â Â Â Â Â Â Â Â Â Â Â Âreturn;
>
> @@ -2156,9 +2147,8 @@ static void sdhci_data_irq(struct sdhci_host *host, u32 intmask)
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â}
>
> - Â Â Â Â Â Â Â pr_err("%s: Got data interrupt 0x%08x even "
> - Â Â Â Â Â Â Â Â Â Â Â "though no data operation was in progress.\n",
> - Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), (unsigned)intmask);
> + Â Â Â Â Â Â Â pr_err("%s: Got data interrupt 0x%08x even though no data operation was in progress\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc), (unsigned)intmask);
> Â Â Â Â Â Â Â Âsdhci_dumpregs(host);
>
> Â Â Â Â Â Â Â Âreturn;
> @@ -2204,10 +2194,9 @@ static void sdhci_data_irq(struct sdhci_host *host, u32 intmask)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â~(SDHCI_DEFAULT_BOUNDARY_SIZE - 1)) +
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂSDHCI_DEFAULT_BOUNDARY_SIZE;
> Â Â Â Â Â Â Â Â Â Â Â Âhost->data->bytes_xfered = dmanow - dmastart;
> - Â Â Â Â Â Â Â Â Â Â Â DBG("%s: DMA base 0x%08x, transferred 0x%06x bytes,"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " next 0x%08x\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), dmastart,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â host->data->bytes_xfered, dmanow);
> + Â Â Â Â Â Â Â Â Â Â Â DBG("%s: DMA base 0x%08x, transferred 0x%06x bytes, next 0x%08x\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), dmastart,
> + Â Â Â Â Â Â Â Â Â Â Â Â Â host->data->bytes_xfered, dmanow);
> Â Â Â Â Â Â Â Â Â Â Â Âsdhci_writel(host, dmanow, SDHCI_DMA_ADDRESS);
> Â Â Â Â Â Â Â Â}
>
> @@ -2250,7 +2239,7 @@ static irqreturn_t sdhci_irq(int irq, void *dev_id)
> Â Â Â Â}
>
> Â Â Â ÂDBG("*** %s got interrupt: 0x%08x\n",
> - Â Â Â Â Â Â Â mmc_hostname(host->mmc), intmask);
> + Â Â Â Â Â mmc_hostname(host->mmc), intmask);
>
> Â Â Â Âif (intmask & (SDHCI_INT_CARD_INSERT | SDHCI_INT_CARD_REMOVE)) {
> Â Â Â Â Â Â Â Âu32 present = sdhci_readl(host, SDHCI_PRESENT_STATE) &
> @@ -2548,9 +2537,8 @@ int sdhci_add_host(struct sdhci_host *host)
> Â Â Â Âhost->version = (host->version & SDHCI_SPEC_VER_MASK)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â>> SDHCI_SPEC_VER_SHIFT;
> Â Â Â Âif (host->version > SDHCI_SPEC_300) {
> - Â Â Â Â Â Â Â pr_err("%s: Unknown controller version (%d). "
> - Â Â Â Â Â Â Â Â Â Â Â "You may experience problems.\n", mmc_hostname(mmc),
> - Â Â Â Â Â Â Â Â Â Â Â host->version);
> + Â Â Â Â Â Â Â pr_err("%s: Unknown controller version (%d). You may experience problems.\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(mmc), host->version);
> Â Â Â Â}
>
> Â Â Â Âcaps[0] = (host->quirks & SDHCI_QUIRK_MISSING_CAPS) ? host->caps :
> @@ -2631,8 +2619,8 @@ int sdhci_add_host(struct sdhci_host *host)
> Â Â Â Âif (host->max_clk == 0 || host->quirks &
> Â Â Â Â Â Â Â Â Â Â Â ÂSDHCI_QUIRK_CAP_CLOCK_BASE_BROKEN) {
> Â Â Â Â Â Â Â Âif (!host->ops->get_max_clock) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't specify base clock "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"frequency.\n", mmc_hostname(mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't specify base clock frequency\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(mmc));
> Â Â Â Â Â Â Â Â Â Â Â Âreturn -ENODEV;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Âhost->max_clk = host->ops->get_max_clock(host);
> @@ -2677,8 +2665,8 @@ int sdhci_add_host(struct sdhci_host *host)
> Â Â Â Â Â Â Â Â Â Â Â Âhost->timeout_clk = host->ops->get_timeout_clock(host);
> Â Â Â Â Â Â Â Â} else if (!(host->quirks &
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂSDHCI_QUIRK_DATA_TIMEOUT_USES_SDCLK)) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't specify timeout clock "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"frequency.\n", mmc_hostname(mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't specify timeout clock frequency\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(mmc));
> Â Â Â Â Â Â Â Â Â Â Â Âreturn -ENODEV;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> @@ -2845,8 +2833,8 @@ int sdhci_add_host(struct sdhci_host *host)
> Â Â Â Â Â Â Â Âmmc->ocr_avail_mmc &= host->ocr_avail_mmc;
>
> Â Â Â Âif (mmc->ocr_avail == 0) {
> - Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't report any "
> - Â Â Â Â Â Â Â Â Â Â Â "support voltages.\n", mmc_hostname(mmc));
> + Â Â Â Â Â Â Â pr_err("%s: Hardware doesn't report any support voltages\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(mmc));
> Â Â Â Â Â Â Â Âreturn -ENODEV;
> Â Â Â Â}
>
> @@ -2994,8 +2982,8 @@ void sdhci_remove_host(struct sdhci_host *host, int dead)
> Â Â Â Â Â Â Â Âhost->flags |= SDHCI_DEVICE_DEAD;
>
> Â Â Â Â Â Â Â Âif (host->mrq) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Controller removed during "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " transfer!\n", mmc_hostname(host->mmc));
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Controller removed during transfer!\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc));
>
> Â Â Â Â Â Â Â Â Â Â Â Âhost->mrq->cmd->error = -ENOMEDIUM;
> Â Â Â Â Â Â Â Â Â Â Â Âtasklet_schedule(&host->finish_tasklet);
> @@ -3053,9 +3041,8 @@ EXPORT_SYMBOL_GPL(sdhci_free_host);
>
> Âstatic int __init sdhci_drv_init(void)
> Â{
> - Â Â Â pr_info(DRIVER_NAME
> - Â Â Â Â Â Â Â ": Secure Digital Host Controller Interface driver\n");
> - Â Â Â pr_info(DRIVER_NAME ": Copyright(c) Pierre Ossman\n");
> + Â Â Â pr_info("Secure Digital Host Controller Interface driver\n");
> + Â Â Â pr_info("Copyright(c) Pierre Ossman\n");
>
> Â Â Â Âreturn 0;
> Â}
> diff --git a/drivers/mmc/host/sdricoh_cs.c b/drivers/mmc/host/sdricoh_cs.c
> index 7009f17..1e30245 100644
> --- a/drivers/mmc/host/sdricoh_cs.c
> +++ b/drivers/mmc/host/sdricoh_cs.c
> @@ -310,8 +310,8 @@ static void sdricoh_request(struct mmc_host *mmc, struct mmc_request *mrq)
>
> Â Â Â Â/* transfer data */
> Â Â Â Âif (data && cmd->error == 0) {
> - Â Â Â Â Â Â Â dev_dbg(dev, "transfer: blksz %i blocks %i sg_len %i "
> - Â Â Â Â Â Â Â Â Â Â Â "sg length %i\n", data->blksz, data->blocks,
> + Â Â Â Â Â Â Â dev_dbg(dev, "transfer: blksz %i blocks %i sg_len %i sg length %i\n",
> + Â Â Â Â Â Â Â Â Â Â Â data->blksz, data->blocks,
> Â Â Â Â Â Â Â Â Â Â Â Âdata->sg_len, data->sg->length);
>
> Â Â Â Â Â Â Â Â/* enter data reading mode */
> @@ -330,8 +330,8 @@ static void sdricoh_request(struct mmc_host *mmc, struct mmc_request *mrq)
> Â Â Â Â Â Â Â Â Â Â Â Âkunmap(page);
> Â Â Â Â Â Â Â Â Â Â Â Âflush_dcache_page(page);
> Â Â Â Â Â Â Â Â Â Â Â Âif (result) {
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â dev_err(dev, "sdricoh_request: cmd %i "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "block transfer failed\n", cmd->opcode);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â dev_err(dev, "sdricoh_request: cmd %i block transfer failed\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â cmd->opcode);
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âcmd->error = result;
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âbreak;
> Â Â Â Â Â Â Â Â Â Â Â Â} else
> @@ -476,8 +476,8 @@ static int sdricoh_pcmcia_probe(struct pcmcia_device *pcmcia_dev)
> Â{
> Â Â Â Âstruct pci_dev *pci_dev = NULL;
>
> - Â Â Â dev_info(&pcmcia_dev->dev, "Searching MMC controller for pcmcia device"
> - Â Â Â Â Â Â Â " %s %s ...\n", pcmcia_dev->prod_id[0], pcmcia_dev->prod_id[1]);
> + Â Â Â dev_info(&pcmcia_dev->dev, "Searching MMC controller for pcmcia device %s %s ...\n",
> + Â Â Â Â Â Â Â Âpcmcia_dev->prod_id[0], pcmcia_dev->prod_id[1]);
>
> Â Â Â Â/* search pci cardbus bridge that contains the mmc controller */
> Â Â Â Â/* the io region is already claimed by yenta_socket... */
> diff --git a/drivers/mmc/host/tifm_sd.c b/drivers/mmc/host/tifm_sd.c
> index 55d6c60..d432597 100644
> --- a/drivers/mmc/host/tifm_sd.c
> +++ b/drivers/mmc/host/tifm_sd.c
> @@ -787,9 +787,9 @@ static void tifm_sd_abort(unsigned long data)
> Â{
> Â Â Â Âstruct tifm_sd *host = (struct tifm_sd*)data;
>
> - Â Â Â pr_err("%s : card failed to respond for a long period of time "
> - Â Â Â Â Â Â Â"(%x, %x)\n",
> - Â Â Â Â Â Â Âdev_name(&host->dev->dev), host->req->cmd->opcode, host->cmd_flags);
> + Â Â Â pr_err("%s : card failed to respond for a long period of time (%x, %x)\n",
> + Â Â Â Â Â Â Âdev_name(&host->dev->dev),
> + Â Â Â Â Â Â Âhost->req->cmd->opcode, host->cmd_flags);
>
> Â Â Â Âtifm_eject(host->dev);
> Â}
> @@ -803,8 +803,7 @@ static void tifm_sd_ios(struct mmc_host *mmc, struct mmc_ios *ios)
>
> Â Â Â Âspin_lock_irqsave(&sock->lock, flags);
>
> - Â Â Â dev_dbg(&sock->dev, "ios: clock = %u, vdd = %x, bus_mode = %x, "
> - Â Â Â Â Â Â Â "chip_select = %x, power_mode = %x, bus_width = %x\n",
> + Â Â Â dev_dbg(&sock->dev, "ios: clock = %u, vdd = %x, bus_mode = %x, chip_select = %x, power_mode = %x, bus_width = %x\n",
> Â Â Â Â Â Â Â Âios->clock, ios->vdd, ios->bus_mode, ios->chip_select,
> Â Â Â Â Â Â Â Âios->power_mode, ios->bus_width);
>
> diff --git a/drivers/mmc/host/via-sdmmc.c b/drivers/mmc/host/via-sdmmc.c
> index 4b83c43..dc479b0 100644
> --- a/drivers/mmc/host/via-sdmmc.c
> +++ b/drivers/mmc/host/via-sdmmc.c
> @@ -8,6 +8,8 @@
> Â* your option) any later version.
> Â*/
>
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/pci.h>
> Â#include <linux/module.h>
> Â#include <linux/dma-mapping.h>
> @@ -838,8 +840,7 @@ static void via_sdc_cmd_isr(struct via_crdr_mmc_host *host, u16 intmask)
> Â Â Â ÂBUG_ON(intmask == 0);
>
> Â Â Â Âif (!host->cmd) {
> - Â Â Â Â Â Â Â pr_err("%s: Got command interrupt 0x%x even "
> - Â Â Â Â Â Â Â Â Â Â Â"though no command operation was in progress.\n",
> + Â Â Â Â Â Â Â pr_err("%s: Got command interrupt 0x%x even though no command operation was in progress\n",
> Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), intmask);
> Â Â Â Â Â Â Â Âreturn;
> Â Â Â Â}
> @@ -941,8 +942,8 @@ static void via_sdc_timeout(unsigned long ulongdata)
> Â Â Â Âspin_lock_irqsave(&sdhost->lock, flags);
>
> Â Â Â Âif (sdhost->mrq) {
> - Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware interrupt."
> - Â Â Â Â Â Â Â Â Â Â Â"cmd:0x%x\n", mmc_hostname(sdhost->mmc),
> + Â Â Â Â Â Â Â pr_err("%s: Timeout waiting for hardware interrupt.cmd:0x%x\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(sdhost->mmc),
> Â Â Â Â Â Â Â Â Â Â Â sdhost->mrq->cmd->opcode);
>
> Â Â Â Â Â Â Â Âif (sdhost->data) {
> @@ -1091,8 +1092,7 @@ static int __devinit via_sd_probe(struct pci_dev *pcidev,
> Â Â Â Âu8 Âgatt;
> Â Â Â Âint ret;
>
> - Â Â Â pr_info(DRV_NAME
> - Â Â Â Â Â Â Â ": VIA SDMMC controller found at %s [%04x:%04x] (rev %x)\n",
> + Â Â Â pr_info("VIA SDMMC controller found at %s [%04x:%04x] (rev %x)\n",
> Â Â Â Â Â Â Â Âpci_name(pcidev), (int)pcidev->vendor, (int)pcidev->device,
> Â Â Â Â Â Â Â Â(int)pcidev->revision);
>
> @@ -1192,8 +1192,8 @@ static void __devexit via_sd_remove(struct pci_dev *pcidev)
> Â Â Â Âmmiowb();
>
> Â Â Â Âif (sdhost->mrq) {
> - Â Â Â Â Â Â Â pr_err("%s: Controller removed during "
> - Â Â Â Â Â Â Â Â Â Â Â "transfer\n", mmc_hostname(sdhost->mmc));
> + Â Â Â Â Â Â Â pr_err("%s: Controller removed during transfer\n",
> + Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(sdhost->mmc));
>
> Â Â Â Â Â Â Â Â/* make sure all DMA is stopped */
> Â Â Â Â Â Â Â Âwritel(VIA_CRDR_DMACTRL_SFTRST,
> @@ -1225,8 +1225,7 @@ static void __devexit via_sd_remove(struct pci_dev *pcidev)
> Â Â Â Âpci_release_regions(pcidev);
> Â Â Â Âpci_disable_device(pcidev);
>
> - Â Â Â pr_info(DRV_NAME
> - Â Â Â Â Â Â Â ": VIA SDMMC controller at %s [%04x:%04x] has been removed\n",
> + Â Â Â pr_info("VIA SDMMC controller at %s [%04x:%04x] has been removed\n",
> Â Â Â Â Â Â Â Âpci_name(pcidev), (int)pcidev->vendor, (int)pcidev->device);
> Â}
>
> @@ -1339,8 +1338,7 @@ static struct pci_driver via_sd_driver = {
>
> Âstatic int __init via_sd_drv_init(void)
> Â{
> - Â Â Â pr_info(DRV_NAME ": VIA SD/MMC Card Reader driver "
> - Â Â Â Â Â Â Â "(C) 2008 VIA Technologies, Inc.\n");
> + Â Â Â pr_info("VIA SD/MMC Card Reader driver (C) 2008 VIA Technologies, Inc.\n");
>
> Â Â Â Âreturn pci_register_driver(&via_sd_driver);
> Â}
> diff --git a/drivers/mmc/host/vub300.c b/drivers/mmc/host/vub300.c
> index e8f6e65..1908a25 100644
> --- a/drivers/mmc/host/vub300.c
> +++ b/drivers/mmc/host/vub300.c
> @@ -35,6 +35,9 @@
> Â* Â Â Â Â Â Â bits and driver data fields reflect that limit by using
> Â* Â Â Â Â Â Â u8, u16, u32
> Â*/
> +
> +#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> +
> Â#include <linux/kernel.h>
> Â#include <linux/errno.h>
> Â#include <linux/init.h>
> @@ -1257,9 +1260,8 @@ static void __download_offload_pseudocode(struct vub300_mmc_host *vub300,
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Âdev_err(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "not enough memory for xfer buffer to send"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " INTERRUPT_PSEUDOCODE for %s %s\n", fw->data,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â vub300->vub_name);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "not enough memory for xfer buffer to send INTERRUPT_PSEUDOCODE for %s %s\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â fw->data, vub300->vub_name);
> Â Â Â Â Â Â Â Â Â Â Â Âstrncpy(vub300->vub_name,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"SDIO interrupt pseudocode download failed",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsizeof(vub300->vub_name));
> @@ -1304,9 +1306,8 @@ static void __download_offload_pseudocode(struct vub300_mmc_host *vub300,
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Âdev_err(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "not enough memory for xfer buffer to send"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " TRANSFER_PSEUDOCODE for %s %s\n", fw->data,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â vub300->vub_name);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "not enough memory for xfer buffer to send TRANSFER_PSEUDOCODE for %s %s\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â fw->data, vub300->vub_name);
> Â Â Â Â Â Â Â Â Â Â Â Âstrncpy(vub300->vub_name,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"SDIO transfer pseudocode download failed",
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âsizeof(vub300->vub_name));
> @@ -2002,11 +2003,11 @@ static void __set_clock_speed(struct vub300_mmc_host *vub300, u8 buf[8],
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ÂUSB_DIR_OUT | USB_TYPE_VENDOR | USB_RECIP_DEVICE,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â0x00, 0x00, buf, buf_array_size, HZ);
> Â Â Â Âif (retval != 8) {
> - Â Â Â Â Â Â Â dev_err(&vub300->udev->dev, "SET_CLOCK_SPEED"
> - Â Â Â Â Â Â Â Â Â Â Â " %dkHz failed with retval=%d\n", kHzClock, retval);
> + Â Â Â Â Â Â Â dev_err(&vub300->udev->dev, "SET_CLOCK_SPEED %dkHz failed with retval=%d\n",
> + Â Â Â Â Â Â Â Â Â Â Â kHzClock, retval);
> Â Â Â Â} else {
> - Â Â Â Â Â Â Â dev_dbg(&vub300->udev->dev, "SET_CLOCK_SPEED"
> - Â Â Â Â Â Â Â Â Â Â Â " %dkHz\n", kHzClock);
> + Â Â Â Â Â Â Â dev_dbg(&vub300->udev->dev, "SET_CLOCK_SPEED %dkHz\n",
> + Â Â Â Â Â Â Â Â Â Â Â kHzClock);
> Â Â Â Â}
> Â}
>
> @@ -2251,8 +2252,7 @@ static int vub300_probe(struct usb_interface *interface,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âendpoint->bEndpointAddress;
> Â Â Â Â Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_warn(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"ignoring"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â" unexpected bulk_in endpoint");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"ignoring unexpected bulk_in endpoint\n");
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â} else if (usb_endpoint_is_bulk_out(endpoint)) {
> Â Â Â Â Â Â Â Â Â Â Â Âif (!vub300->cmnd_out_ep) {
> @@ -2263,20 +2263,18 @@ static int vub300_probe(struct usb_interface *interface,
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âendpoint->bEndpointAddress;
> Â Â Â Â Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdev_warn(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"ignoring"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â" unexpected bulk_out endpoint");
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"ignoring unexpected bulk_out endpoint\n");
> Â Â Â Â Â Â Â Â Â Â Â Â}
> Â Â Â Â Â Â Â Â} else {
> Â Â Â Â Â Â Â Â Â Â Â Âdev_warn(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"vub300 ignoring EndPoint(%d) %02X", i,
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âendpoint->bEndpointAddress);
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â"vub300 ignoring EndPoint(%d) %02X\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âi, endpoint->bEndpointAddress);
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> Â Â Â Âif (vub300->cmnd_res_ep && vub300->cmnd_out_ep &&
> Â Â Â Â Â Âvub300->data_inp_ep && vub300->data_out_ep) {
> Â Â Â Â Â Â Â Âdev_info(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â"vub300 %s packets"
> - Â Â Â Â Â Â Â Â Â Â Â Â" using EndPoints %02X %02X %02X %02X\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â"vub300 %s packets using EndPoints %02X %02X %02X %02X\n",
> Â Â Â Â Â Â Â Â Â Â Â Â vub300->large_usb_packets ? "LARGE" : "SMALL",
> Â Â Â Â Â Â Â Â Â Â Â Â vub300->cmnd_out_ep, vub300->cmnd_res_ep,
> Â Â Â Â Â Â Â Â Â Â Â Â vub300->data_out_ep, vub300->data_inp_ep);
> @@ -2341,13 +2339,11 @@ static int vub300_probe(struct usb_interface *interface,
> Â Â Â Âadd_timer(&vub300->inactivity_timer);
> Â Â Â Âif (vub300->card_present)
> Â Â Â Â Â Â Â Âdev_info(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â"USB vub300 remote SDIO host controller[%d]"
> - Â Â Â Â Â Â Â Â Â Â Â Â"connected with SD/SDIO card inserted\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â"USB vub300 remote SDIO host controller[%d]connected with SD/SDIO card inserted\n",
> Â Â Â Â Â Â Â Â Â Â Â Â interface_to_InterfaceNumber(interface));
> Â Â Â Âelse
> Â Â Â Â Â Â Â Âdev_info(&vub300->udev->dev,
> - Â Â Â Â Â Â Â Â Â Â Â Â"USB vub300 remote SDIO host controller[%d]"
> - Â Â Â Â Â Â Â Â Â Â Â Â"connected with no SD/SDIO card inserted\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â"USB vub300 remote SDIO host controller[%d]connected with no SD/SDIO card inserted\n",
> Â Â Â Â Â Â Â Â Â Â Â Â interface_to_InterfaceNumber(interface));
> Â Â Â Âmmc_add_host(mmc);
> Â Â Â Âreturn 0;
> @@ -2381,8 +2377,8 @@ static void vub300_disconnect(struct usb_interface *interface)
> Â Â Â Â Â Â Â Â Â Â Â Âvub300->interface = NULL;
> Â Â Â Â Â Â Â Â Â Â Â Âkref_put(&vub300->kref, vub300_delete);
> Â Â Â Â Â Â Â Â Â Â Â Âmmc_remove_host(mmc);
> - Â Â Â Â Â Â Â Â Â Â Â pr_info("USB vub300 remote SDIO host controller[%d]"
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â " now disconnected", ifnum);
> + Â Â Â Â Â Â Â Â Â Â Â pr_info("USB remote SDIO host controller[%d] now disconnected\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â ifnum);
> Â Â Â Â Â Â Â Â Â Â Â Âreturn;
> Â Â Â Â Â Â Â Â}
> Â Â Â Â}
> @@ -2448,29 +2444,29 @@ static int __init vub300_init(void)
> Â{ Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â/* NOT irq */
> Â Â Â Âint result;
>
> - Â Â Â pr_info("VUB300 Driver rom wait states = %02X irqpoll timeout = %04X",
> + Â Â Â pr_info("Driver rom wait states = %02X irqpoll timeout = %04X\n",
> Â Â Â Â Â Â Â Âfirmware_rom_wait_states, 0x0FFFF & firmware_irqpoll_timeout);
> Â Â Â Âcmndworkqueue = create_singlethread_workqueue("kvub300c");
> Â Â Â Âif (!cmndworkqueue) {
> - Â Â Â Â Â Â Â pr_err("not enough memory for the REQUEST workqueue");
> + Â Â Â Â Â Â Â pr_err("not enough memory for the REQUEST workqueue\n");
> Â Â Â Â Â Â Â Âresult = -ENOMEM;
> Â Â Â Â Â Â Â Âgoto out1;
> Â Â Â Â}
> Â Â Â Âpollworkqueue = create_singlethread_workqueue("kvub300p");
> Â Â Â Âif (!pollworkqueue) {
> - Â Â Â Â Â Â Â pr_err("not enough memory for the IRQPOLL workqueue");
> + Â Â Â Â Â Â Â pr_err("not enough memory for the IRQPOLL workqueue\n");
> Â Â Â Â Â Â Â Âresult = -ENOMEM;
> Â Â Â Â Â Â Â Âgoto out2;
> Â Â Â Â}
> Â Â Â Âdeadworkqueue = create_singlethread_workqueue("kvub300d");
> Â Â Â Âif (!deadworkqueue) {
> - Â Â Â Â Â Â Â pr_err("not enough memory for the EXPIRED workqueue");
> + Â Â Â Â Â Â Â pr_err("not enough memory for the EXPIRED workqueue\n");
> Â Â Â Â Â Â Â Âresult = -ENOMEM;
> Â Â Â Â Â Â Â Âgoto out3;
> Â Â Â Â}
> Â Â Â Âresult = usb_register(&vub300_driver);
> Â Â Â Âif (result) {
> - Â Â Â Â Â Â Â pr_err("usb_register failed. Error number %d", result);
> + Â Â Â Â Â Â Â pr_err("usb_register failed. Error number %d\n", result);
> Â Â Â Â Â Â Â Âgoto out4;
> Â Â Â Â}
> Â Â Â Âreturn 0;
> diff --git a/drivers/mmc/host/wbsd.c b/drivers/mmc/host/wbsd.c
> index deaed8c..7536b50 100644
> --- a/drivers/mmc/host/wbsd.c
> +++ b/drivers/mmc/host/wbsd.c
> @@ -721,9 +721,8 @@ static void wbsd_finish_data(struct wbsd_host *host, struct mmc_data *data)
> Â Â Â Â Â Â Â Â * Any leftover data?
> Â Â Â Â Â Â Â Â */
> Â Â Â Â Â Â Â Âif (count) {
> - Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Incomplete DMA transfer. "
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â "%d bytes left.\n",
> - Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â mmc_hostname(host->mmc), count);
> + Â Â Â Â Â Â Â Â Â Â Â pr_err("%s: Incomplete DMA transfer. %d bytes left.\n",
> + Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âmmc_hostname(host->mmc), count);
>
> Â Â Â Â Â Â Â Â Â Â Â Âif (!data->error)
> Â Â Â Â Â Â Â Â Â Â Â Â Â Â Â Âdata->error = -EIO;
> --
> 1.7.6.405.gc1be0
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
> the body of a message to majordomo@xxxxxxxxxxxxxxx
> More majordomo info at Âhttp://vger.kernel.org/majordomo-info.html
> Please read the FAQ at Âhttp://www.tux.org/lkml/
>
èº{.nÇ+‰·Ÿ®‰­†+%ŠËlzwm…ébëæìr¸›zX§»®w¥Š{ayºÊÚë,j­¢f£¢·hš‹àz¹®w¥¢¸¢·¦j:+v‰¨ŠwèjØm¶Ÿÿ¾«‘êçzZ+ƒùšŽŠÝj"ú!¶iO•æ¬z·švØ^¶m§ÿðÃnÆàþY&—