diff mbox

[1/3] gpu: drm: amd/radeon: Convert printk(KERN_<LEVEL> to pr_<level>

Message ID c1d0b86774e43880aa0ae8ab8118082ad8d6cc0e.1488285953.git.joe@perches.com (mailing list archive)
State New, archived
Headers show

Commit Message

Joe Perches Feb. 28, 2017, 12:55 p.m. UTC
Use a more common logging style.

Miscellanea:

o Coalesce formats and realign arguments
o Neaten a few macros now using pr_<level>

Signed-off-by: Joe Perches <joe@perches.com>
---
 drivers/gpu/drm/amd/amdgpu/amdgpu.h                |  3 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c           |  4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c   |  4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_device.c         |  4 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c             |  3 +-
 drivers/gpu/drm/amd/amdgpu/amdgpu_test.c           |  2 +-
 drivers/gpu/drm/amd/amdgpu/atom.c                  | 44 ++++++++---------
 drivers/gpu/drm/amd/amdgpu/ci_dpm.c                |  4 +-
 drivers/gpu/drm/amd/amdgpu/cik_sdma.c              |  4 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c              |  4 +-
 drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c              |  4 +-
 drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c              |  8 ++--
 drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c              |  8 ++--
 drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c             |  4 +-
 drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c             |  4 +-
 drivers/gpu/drm/amd/include/amd_pcie_helpers.h     |  4 +-
 drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c   |  2 +-
 drivers/gpu/drm/amd/powerplay/inc/pp_debug.h       |  2 +-
 drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c    |  4 +-
 drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 14 +++---
 .../gpu/drm/amd/powerplay/smumgr/polaris10_smc.c   |  4 +-
 drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c   |  4 +-
 drivers/gpu/drm/radeon/atom.c                      | 46 ++++++++----------
 drivers/gpu/drm/radeon/cik.c                       | 56 ++++++++--------------
 drivers/gpu/drm/radeon/evergreen.c                 |  2 +-
 drivers/gpu/drm/radeon/evergreen_cs.c              |  7 ++-
 drivers/gpu/drm/radeon/ni.c                        | 22 +++------
 drivers/gpu/drm/radeon/r100.c                      | 18 +++----
 drivers/gpu/drm/radeon/r200.c                      |  3 +-
 drivers/gpu/drm/radeon/r300.c                      | 13 ++---
 drivers/gpu/drm/radeon/r420.c                      |  9 ++--
 drivers/gpu/drm/radeon/r520.c                      |  3 +-
 drivers/gpu/drm/radeon/r600.c                      | 21 +++-----
 drivers/gpu/drm/radeon/r600_cs.c                   |  7 ++-
 drivers/gpu/drm/radeon/radeon.h                    |  3 +-
 drivers/gpu/drm/radeon/radeon_atpx_handler.c       |  4 +-
 drivers/gpu/drm/radeon/radeon_audio.c              |  4 +-
 drivers/gpu/drm/radeon/radeon_clocks.c             |  2 +-
 drivers/gpu/drm/radeon/radeon_device.c             |  8 ++--
 drivers/gpu/drm/radeon/radeon_fb.c                 |  3 +-
 drivers/gpu/drm/radeon/radeon_gem.c                |  4 +-
 drivers/gpu/drm/radeon/radeon_test.c               |  6 +--
 drivers/gpu/drm/radeon/rs400.c                     |  4 +-
 drivers/gpu/drm/radeon/rs690.c                     |  3 +-
 drivers/gpu/drm/radeon/rv515.c                     |  9 ++--
 drivers/gpu/drm/radeon/si.c                        | 45 ++++++-----------
 46 files changed, 172 insertions(+), 268 deletions(-)

Comments

Alex Deucher March 1, 2017, 11:19 p.m. UTC | #1
On Tue, Feb 28, 2017 at 7:55 AM, Joe Perches <joe@perches.com> wrote:
> Use a more common logging style.
>
> Miscellanea:
>
> o Coalesce formats and realign arguments
> o Neaten a few macros now using pr_<level>
>
> Signed-off-by: Joe Perches <joe@perches.com>

Applied.  Thanks!

Alex


> ---
>  drivers/gpu/drm/amd/amdgpu/amdgpu.h                |  3 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c           |  4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c   |  4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_device.c         |  4 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c             |  3 +-
>  drivers/gpu/drm/amd/amdgpu/amdgpu_test.c           |  2 +-
>  drivers/gpu/drm/amd/amdgpu/atom.c                  | 44 ++++++++---------
>  drivers/gpu/drm/amd/amdgpu/ci_dpm.c                |  4 +-
>  drivers/gpu/drm/amd/amdgpu/cik_sdma.c              |  4 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c              |  4 +-
>  drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c              |  4 +-
>  drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c              |  8 ++--
>  drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c              |  8 ++--
>  drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c             |  4 +-
>  drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c             |  4 +-
>  drivers/gpu/drm/amd/include/amd_pcie_helpers.h     |  4 +-
>  drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c   |  2 +-
>  drivers/gpu/drm/amd/powerplay/inc/pp_debug.h       |  2 +-
>  drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c    |  4 +-
>  drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c | 14 +++---
>  .../gpu/drm/amd/powerplay/smumgr/polaris10_smc.c   |  4 +-
>  drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c   |  4 +-
>  drivers/gpu/drm/radeon/atom.c                      | 46 ++++++++----------
>  drivers/gpu/drm/radeon/cik.c                       | 56 ++++++++--------------
>  drivers/gpu/drm/radeon/evergreen.c                 |  2 +-
>  drivers/gpu/drm/radeon/evergreen_cs.c              |  7 ++-
>  drivers/gpu/drm/radeon/ni.c                        | 22 +++------
>  drivers/gpu/drm/radeon/r100.c                      | 18 +++----
>  drivers/gpu/drm/radeon/r200.c                      |  3 +-
>  drivers/gpu/drm/radeon/r300.c                      | 13 ++---
>  drivers/gpu/drm/radeon/r420.c                      |  9 ++--
>  drivers/gpu/drm/radeon/r520.c                      |  3 +-
>  drivers/gpu/drm/radeon/r600.c                      | 21 +++-----
>  drivers/gpu/drm/radeon/r600_cs.c                   |  7 ++-
>  drivers/gpu/drm/radeon/radeon.h                    |  3 +-
>  drivers/gpu/drm/radeon/radeon_atpx_handler.c       |  4 +-
>  drivers/gpu/drm/radeon/radeon_audio.c              |  4 +-
>  drivers/gpu/drm/radeon/radeon_clocks.c             |  2 +-
>  drivers/gpu/drm/radeon/radeon_device.c             |  8 ++--
>  drivers/gpu/drm/radeon/radeon_fb.c                 |  3 +-
>  drivers/gpu/drm/radeon/radeon_gem.c                |  4 +-
>  drivers/gpu/drm/radeon/radeon_test.c               |  6 +--
>  drivers/gpu/drm/radeon/rs400.c                     |  4 +-
>  drivers/gpu/drm/radeon/rs690.c                     |  3 +-
>  drivers/gpu/drm/radeon/rv515.c                     |  9 ++--
>  drivers/gpu/drm/radeon/si.c                        | 45 ++++++-----------
>  46 files changed, 172 insertions(+), 268 deletions(-)
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> index c1b913541739..3f636632c289 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
> @@ -1515,7 +1515,8 @@ void amdgpu_mm_wdoorbell(struct amdgpu_device *adev, u32 index, u32 v);
>   */
>  #define RREG32(reg) amdgpu_mm_rreg(adev, (reg), false)
>  #define RREG32_IDX(reg) amdgpu_mm_rreg(adev, (reg), true)
> -#define DREG32(reg) printk(KERN_INFO "REGISTER: " #reg " : 0x%08X\n", amdgpu_mm_rreg(adev, (reg), false))
> +#define DREG32(reg) pr_info("REGISTER: " #reg " : 0x%08X\n",           \
> +                           amdgpu_mm_rreg(adev, (reg), false))
>  #define WREG32(reg, v) amdgpu_mm_wreg(adev, (reg), (v), false)
>  #define WREG32_IDX(reg, v) amdgpu_mm_wreg(adev, (reg), (v), true)
>  #define REG_SET(FIELD, v) (((v) << FIELD##_SHIFT) & FIELD##_MASK)
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
> index 857ba0897159..3889486f71fe 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
> @@ -74,9 +74,9 @@ static void amdgpu_afmt_calc_cts(uint32_t clock, int *CTS, int *N, int freq)
>
>         /* Check that we are in spec (not always possible) */
>         if (n < (128*freq/1500))
> -               printk(KERN_WARNING "Calculated ACR N value is too small. You may experience audio problems.\n");
> +               pr_warn("Calculated ACR N value is too small. You may experience audio problems.\n");
>         if (n > (128*freq/300))
> -               printk(KERN_WARNING "Calculated ACR N value is too large. You may experience audio problems.\n");
> +               pr_warn("Calculated ACR N value is too large. You may experience audio problems.\n");
>
>         *N = n;
>         *CTS = cts;
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
> index 6c343a933182..c13c51af0b68 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
> @@ -583,8 +583,8 @@ static bool amdgpu_atpx_detect(void)
>
>         if (has_atpx && vga_count == 2) {
>                 acpi_get_name(amdgpu_atpx_priv.atpx.handle, ACPI_FULL_PATHNAME, &buffer);
> -               printk(KERN_INFO "vga_switcheroo: detected switching method %s handle\n",
> -                      acpi_method_name);
> +               pr_info("vga_switcheroo: detected switching method %s handle\n",
> +                       acpi_method_name);
>                 amdgpu_atpx_priv.atpx_detected = true;
>                 amdgpu_atpx_priv.bridge_pm_usable = d3_supported;
>                 amdgpu_atpx_init();
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> index 6abb238b25c9..85eeae05ac27 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
> @@ -1059,7 +1059,7 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
>         if (state == VGA_SWITCHEROO_ON) {
>                 unsigned d3_delay = dev->pdev->d3_delay;
>
> -               printk(KERN_INFO "amdgpu: switched on\n");
> +               pr_info("amdgpu: switched on\n");
>                 /* don't suspend or resume card normally */
>                 dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
>
> @@ -1070,7 +1070,7 @@ static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
>                 dev->switch_power_state = DRM_SWITCH_POWER_ON;
>                 drm_kms_helper_poll_enable(dev);
>         } else {
> -               printk(KERN_INFO "amdgpu: switched off\n");
> +               pr_info("amdgpu: switched off\n");
>                 drm_kms_helper_poll_disable(dev);
>                 dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
>                 amdgpu_device_suspend(dev, true, true);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
> index 36ce3cac81ba..35a287280363 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
> @@ -150,8 +150,7 @@ static int amdgpufb_create_pinned_object(struct amdgpu_fbdev *rfbdev,
>                                        AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS,
>                                        true, &gobj);
>         if (ret) {
> -               printk(KERN_ERR "failed to allocate framebuffer (%d)\n",
> -                      aligned_size);
> +               pr_err("failed to allocate framebuffer (%d)\n", aligned_size);
>                 return -ENOMEM;
>         }
>         abo = gem_to_amdgpu_bo(gobj);
> diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
> index e05a24325eeb..b0483e6e536f 100644
> --- a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
> +++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
> @@ -228,7 +228,7 @@ static void amdgpu_do_test_moves(struct amdgpu_device *adev)
>  out_cleanup:
>         kfree(gtt_obj);
>         if (r) {
> -               printk(KERN_WARNING "Error while testing BO move.\n");
> +               pr_warn("Error while testing BO move\n");
>         }
>  }
>
> diff --git a/drivers/gpu/drm/amd/amdgpu/atom.c b/drivers/gpu/drm/amd/amdgpu/atom.c
> index 1b50e6c13fb3..d1444aaca038 100644
> --- a/drivers/gpu/drm/amd/amdgpu/atom.c
> +++ b/drivers/gpu/drm/amd/amdgpu/atom.c
> @@ -166,7 +166,7 @@ static uint32_t atom_iio_execute(struct atom_context *ctx, int base,
>                 case ATOM_IIO_END:
>                         return temp;
>                 default:
> -                       printk(KERN_INFO "Unknown IIO opcode.\n");
> +                       pr_info("Unknown IIO opcode\n");
>                         return 0;
>                 }
>  }
> @@ -190,22 +190,19 @@ static uint32_t atom_get_src_int(atom_exec_context *ctx, uint8_t attr,
>                         val = gctx->card->reg_read(gctx->card, idx);
>                         break;
>                 case ATOM_IO_PCI:
> -                       printk(KERN_INFO
> -                              "PCI registers are not implemented.\n");
> +                       pr_info("PCI registers are not implemented\n");
>                         return 0;
>                 case ATOM_IO_SYSIO:
> -                       printk(KERN_INFO
> -                              "SYSIO registers are not implemented.\n");
> +                       pr_info("SYSIO registers are not implemented\n");
>                         return 0;
>                 default:
>                         if (!(gctx->io_mode & 0x80)) {
> -                               printk(KERN_INFO "Bad IO mode.\n");
> +                               pr_info("Bad IO mode\n");
>                                 return 0;
>                         }
>                         if (!gctx->iio[gctx->io_mode & 0x7F]) {
> -                               printk(KERN_INFO
> -                                      "Undefined indirect IO read method %d.\n",
> -                                      gctx->io_mode & 0x7F);
> +                               pr_info("Undefined indirect IO read method %d\n",
> +                                       gctx->io_mode & 0x7F);
>                                 return 0;
>                         }
>                         val =
> @@ -469,22 +466,19 @@ static void atom_put_dst(atom_exec_context *ctx, int arg, uint8_t attr,
>                                 gctx->card->reg_write(gctx->card, idx, val);
>                         break;
>                 case ATOM_IO_PCI:
> -                       printk(KERN_INFO
> -                              "PCI registers are not implemented.\n");
> +                       pr_info("PCI registers are not implemented\n");
>                         return;
>                 case ATOM_IO_SYSIO:
> -                       printk(KERN_INFO
> -                              "SYSIO registers are not implemented.\n");
> +                       pr_info("SYSIO registers are not implemented\n");
>                         return;
>                 default:
>                         if (!(gctx->io_mode & 0x80)) {
> -                               printk(KERN_INFO "Bad IO mode.\n");
> +                               pr_info("Bad IO mode\n");
>                                 return;
>                         }
>                         if (!gctx->iio[gctx->io_mode & 0xFF]) {
> -                               printk(KERN_INFO
> -                                      "Undefined indirect IO write method %d.\n",
> -                                      gctx->io_mode & 0x7F);
> +                               pr_info("Undefined indirect IO write method %d\n",
> +                                       gctx->io_mode & 0x7F);
>                                 return;
>                         }
>                         atom_iio_execute(gctx, gctx->iio[gctx->io_mode & 0xFF],
> @@ -850,17 +844,17 @@ static void atom_op_postcard(atom_exec_context *ctx, int *ptr, int arg)
>
>  static void atom_op_repeat(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_restorereg(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_savereg(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_setdatablock(atom_exec_context *ctx, int *ptr, int arg)
> @@ -1023,7 +1017,7 @@ static void atom_op_switch(atom_exec_context *ctx, int *ptr, int arg)
>                         }
>                         (*ptr) += 2;
>                 } else {
> -                       printk(KERN_INFO "Bad case.\n");
> +                       pr_info("Bad case\n");
>                         return;
>                 }
>         (*ptr) += 2;
> @@ -1316,14 +1310,14 @@ struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
>         ctx->bios = bios;
>
>         if (CU16(0) != ATOM_BIOS_MAGIC) {
> -               printk(KERN_INFO "Invalid BIOS magic.\n");
> +               pr_info("Invalid BIOS magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
>         if (strncmp
>             (CSTR(ATOM_ATI_MAGIC_PTR), ATOM_ATI_MAGIC,
>              strlen(ATOM_ATI_MAGIC))) {
> -               printk(KERN_INFO "Invalid ATI magic.\n");
> +               pr_info("Invalid ATI magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
> @@ -1332,7 +1326,7 @@ struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
>         if (strncmp
>             (CSTR(base + ATOM_ROM_MAGIC_PTR), ATOM_ROM_MAGIC,
>              strlen(ATOM_ROM_MAGIC))) {
> -               printk(KERN_INFO "Invalid ATOM magic.\n");
> +               pr_info("Invalid ATOM magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
> @@ -1356,7 +1350,7 @@ struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
>                         break;
>                 }
>         }
> -       printk(KERN_INFO "ATOM BIOS: %s\n", name);
> +       pr_info("ATOM BIOS: %s\n", name);
>
>         return ctx;
>  }
> diff --git a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
> index f97ecb49972e..b59e7e5d7279 100644
> --- a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
> +++ b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
> @@ -5804,9 +5804,7 @@ static int ci_dpm_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "cik_smc: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("cik_smc: Failed to load firmware \"%s\"\n", fw_name);
>                 release_firmware(adev->pm.fw);
>                 adev->pm.fw = NULL;
>         }
> diff --git a/drivers/gpu/drm/amd/amdgpu/cik_sdma.c b/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
> index 810bba533975..c33bc1bb4655 100644
> --- a/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
> +++ b/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
> @@ -142,9 +142,7 @@ static int cik_sdma_init_microcode(struct amdgpu_device *adev)
>         }
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "cik_sdma: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("cik_sdma: Failed to load firmware \"%s\"\n", fw_name);
>                 for (i = 0; i < adev->sdma.num_instances; i++) {
>                         release_firmware(adev->sdma.instance[i].fw);
>                         adev->sdma.instance[i].fw = NULL;
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> index 2086e7e68de4..c8f9c1414940 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
> @@ -378,9 +378,7 @@ static int gfx_v6_0_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "gfx6: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("gfx6: Failed to load firmware \"%s\"\n", fw_name);
>                 release_firmware(adev->gfx.pfp_fw);
>                 adev->gfx.pfp_fw = NULL;
>                 release_firmware(adev->gfx.me_fw);
> diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> index 1f9354541f29..82c574cf3658 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
> @@ -972,9 +972,7 @@ static int gfx_v7_0_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "gfx7: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("gfx7: Failed to load firmware \"%s\"\n", fw_name);
>                 release_firmware(adev->gfx.pfp_fw);
>                 adev->gfx.pfp_fw = NULL;
>                 release_firmware(adev->gfx.me_fw);
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> index 8d05e0c4e3d7..52594b9fd182 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
> @@ -161,9 +161,7 @@ static int gmc_v7_0_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "cik_mc: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("cik_mc: Failed to load firmware \"%s\"\n", fw_name);
>                 release_firmware(adev->mc.fw);
>                 adev->mc.fw = NULL;
>         }
> @@ -938,12 +936,12 @@ static int gmc_v7_0_sw_init(void *handle)
>         if (r) {
>                 adev->need_dma32 = true;
>                 dma_bits = 32;
> -               printk(KERN_WARNING "amdgpu: No suitable DMA available.\n");
> +               pr_warn("amdgpu: No suitable DMA available\n");
>         }
>         r = pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(dma_bits));
>         if (r) {
>                 pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(32));
> -               printk(KERN_WARNING "amdgpu: No coherent DMA available.\n");
> +               pr_warn("amdgpu: No coherent DMA available\n");
>         }
>
>         r = gmc_v7_0_init_microcode(adev);
> diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> index 7669b3259f35..421507aea36a 100644
> --- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
> @@ -245,9 +245,7 @@ static int gmc_v8_0_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "mc: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("mc: Failed to load firmware \"%s\"\n", fw_name);
>                 release_firmware(adev->mc.fw);
>                 adev->mc.fw = NULL;
>         }
> @@ -950,12 +948,12 @@ static int gmc_v8_0_sw_init(void *handle)
>         if (r) {
>                 adev->need_dma32 = true;
>                 dma_bits = 32;
> -               printk(KERN_WARNING "amdgpu: No suitable DMA available.\n");
> +               pr_warn("amdgpu: No suitable DMA available\n");
>         }
>         r = pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(dma_bits));
>         if (r) {
>                 pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(32));
> -               printk(KERN_WARNING "amdgpu: No coherent DMA available.\n");
> +               pr_warn("amdgpu: No coherent DMA available\n");
>         }
>
>         r = gmc_v8_0_init_microcode(adev);
> diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c b/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
> index 896be64b7013..a881cf475a19 100644
> --- a/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
> +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
> @@ -170,9 +170,7 @@ static int sdma_v2_4_init_microcode(struct amdgpu_device *adev)
>
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "sdma_v2_4: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("sdma_v2_4: Failed to load firmware \"%s\"\n", fw_name);
>                 for (i = 0; i < adev->sdma.num_instances; i++) {
>                         release_firmware(adev->sdma.instance[i].fw);
>                         adev->sdma.instance[i].fw = NULL;
> diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
> index 011800f621c6..255fab5fceee 100644
> --- a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
> +++ b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
> @@ -321,9 +321,7 @@ static int sdma_v3_0_init_microcode(struct amdgpu_device *adev)
>         }
>  out:
>         if (err) {
> -               printk(KERN_ERR
> -                      "sdma_v3_0: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("sdma_v3_0: Failed to load firmware \"%s\"\n", fw_name);
>                 for (i = 0; i < adev->sdma.num_instances; i++) {
>                         release_firmware(adev->sdma.instance[i].fw);
>                         adev->sdma.instance[i].fw = NULL;
> diff --git a/drivers/gpu/drm/amd/include/amd_pcie_helpers.h b/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
> index 5725bf85eacc..7e5a965450c7 100644
> --- a/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
> +++ b/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
> @@ -82,7 +82,7 @@ static inline uint16_t get_pcie_lane_support(uint32_t pcie_lane_width_cap,
>
>         switch (pcie_lane_width_cap) {
>         case 0:
> -               printk(KERN_ERR "No valid PCIE lane width reported");
> +               pr_err("No valid PCIE lane width reported\n");
>                 break;
>         case CAIL_PCIE_LINK_WIDTH_SUPPORT_X1:
>                 new_pcie_lanes = 1;
> @@ -126,7 +126,7 @@ static inline uint16_t get_pcie_lane_support(uint32_t pcie_lane_width_cap,
>                                                         }
>                                                 }
>                                                 if (j > 7)
> -                                                       printk(KERN_ERR "Cannot find a valid PCIE lane width!");
> +                                                       pr_err("Cannot find a valid PCIE lane width!\n");
>                                         }
>                                 }
>                                 break;
> diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> index f75ee33ec5bb..418f87e805b2 100644
> --- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> +++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
> @@ -1544,7 +1544,7 @@ static int smu7_get_evv_voltages(struct pp_hwmgr *hwmgr)
>                                         if (vddc >= 2000 || vddc == 0)
>                                                 return -EINVAL;
>                                 } else {
> -                                       pr_warning("failed to retrieving EVV voltage!\n");
> +                                       pr_warn("failed to retrieving EVV voltage!\n");
>                                         continue;
>                                 }
>
> diff --git a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
> index 072880130cfb..f3f9ebb631a5 100644
> --- a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
> +++ b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
> @@ -37,7 +37,7 @@
>  #define PP_ASSERT_WITH_CODE(cond, msg, code)   \
>         do {                                    \
>                 if (!(cond)) {                  \
> -                       pr_warning("%s\n", msg);        \
> +                       pr_warn("%s\n", msg);   \
>                         code;                   \
>                 }                               \
>         } while (0)
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
> index 0f7a77b7312e..5450f5ef8e89 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
> @@ -2131,7 +2131,7 @@ uint32_t fiji_get_offsetof(uint32_t type, uint32_t member)
>                         return offsetof(SMU73_Discrete_DpmTable, LowSclkInterruptThreshold);
>                 }
>         }
> -       pr_warning("can't get the offset of type %x member %x\n", type, member);
> +       pr_warn("can't get the offset of type %x member %x\n", type, member);
>         return 0;
>  }
>
> @@ -2156,7 +2156,7 @@ uint32_t fiji_get_mac_definition(uint32_t value)
>                 return SMU73_MAX_LEVELS_MVDD;
>         }
>
> -       pr_warning("can't get the mac of %x\n", value);
> +       pr_warn("can't get the mac of %x\n", value);
>         return 0;
>  }
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
> index ad82161df831..b609ab7e9b97 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
> @@ -122,7 +122,7 @@ static void iceland_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr)
>                 break;
>         default:
>                 smu_data->power_tune_defaults = &defaults_iceland;
> -               pr_warning("Unknown V.I. Device ID.\n");
> +               pr_warn("Unknown V.I. Device ID\n");
>                 break;
>         }
>         return;
> @@ -378,7 +378,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
>                         return -EINVAL);
>
>         if (NULL == hwmgr->dyn_state.cac_leakage_table) {
> -               pr_warning("CAC Leakage Table does not exist, using vddc.\n");
> +               pr_warn("CAC Leakage Table does not exist, using vddc\n");
>                 return 0;
>         }
>
> @@ -394,7 +394,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
>                                 *lo = hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * VOLTAGE_SCALE;
>                                 *hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage * VOLTAGE_SCALE);
>                         } else {
> -                               pr_warning("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table.\n");
> +                               pr_warn("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table\n");
>                                 *lo = hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Vddc * VOLTAGE_SCALE;
>                                 *hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Leakage * VOLTAGE_SCALE);
>                         }
> @@ -414,7 +414,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
>                                         *lo = hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * VOLTAGE_SCALE;
>                                         *hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage) * VOLTAGE_SCALE;
>                                 } else {
> -                                       pr_warning("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index in second look up, using maximum index from CAC table.");
> +                                       pr_warn("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index in second look up, using maximum index from CAC table\n");
>                                         *lo = hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Vddc * VOLTAGE_SCALE;
>                                         *hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Leakage * VOLTAGE_SCALE);
>                                 }
> @@ -423,7 +423,7 @@ static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
>                 }
>
>                 if (!vol_found)
> -                       pr_warning("Unable to get std_vddc from SCLK/VDDC Dependency Table, using vddc.\n");
> +                       pr_warn("Unable to get std_vddc from SCLK/VDDC Dependency Table, using vddc\n");
>         }
>
>         return 0;
> @@ -2146,7 +2146,7 @@ uint32_t iceland_get_offsetof(uint32_t type, uint32_t member)
>                         return offsetof(SMU71_Discrete_DpmTable, LowSclkInterruptThreshold);
>                 }
>         }
> -       pr_warning("can't get the offset of type %x member %x\n", type, member);
> +       pr_warn("can't get the offset of type %x member %x\n", type, member);
>         return 0;
>  }
>
> @@ -2169,7 +2169,7 @@ uint32_t iceland_get_mac_definition(uint32_t value)
>                 return SMU71_MAX_LEVELS_MVDD;
>         }
>
> -       pr_warning("can't get the mac of %x\n", value);
> +       pr_warn("can't get the mac of %x\n", value);
>         return 0;
>  }
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
> index 80e2329a1b9e..c9ca7a1ac171 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
> @@ -2184,7 +2184,7 @@ uint32_t polaris10_get_offsetof(uint32_t type, uint32_t member)
>                         return offsetof(SMU74_Discrete_DpmTable, LowSclkInterruptThreshold);
>                 }
>         }
> -       pr_warning("can't get the offset of type %x member %x\n", type, member);
> +       pr_warn("can't get the offset of type %x member %x\n", type, member);
>         return 0;
>  }
>
> @@ -2211,7 +2211,7 @@ uint32_t polaris10_get_mac_definition(uint32_t value)
>                 return SMU7_UVD_MCLK_HANDSHAKE_DISABLE;
>         }
>
> -       pr_warning("can't get the mac of %x\n", value);
> +       pr_warn("can't get the mac of %x\n", value);
>         return 0;
>  }
>
> diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
> index 331b0aba4a13..4d1439a498eb 100644
> --- a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
> +++ b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
> @@ -2657,7 +2657,7 @@ uint32_t tonga_get_offsetof(uint32_t type, uint32_t member)
>                         return offsetof(SMU72_Discrete_DpmTable, LowSclkInterruptThreshold);
>                 }
>         }
> -       pr_warning("can't get the offset of type %x member %x\n", type, member);
> +       pr_warn("can't get the offset of type %x member %x\n", type, member);
>         return 0;
>  }
>
> @@ -2681,7 +2681,7 @@ uint32_t tonga_get_mac_definition(uint32_t value)
>         case SMU_MAX_LEVELS_MVDD:
>                 return SMU72_MAX_LEVELS_MVDD;
>         }
> -       pr_warning("can't get the mac value %x\n", value);
> +       pr_warn("can't get the mac value %x\n", value);
>
>         return 0;
>  }
> diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon/atom.c
> index f66c33dd21a3..6a2e091aa7b6 100644
> --- a/drivers/gpu/drm/radeon/atom.c
> +++ b/drivers/gpu/drm/radeon/atom.c
> @@ -170,7 +170,7 @@ static uint32_t atom_iio_execute(struct atom_context *ctx, int base,
>                 case ATOM_IIO_END:
>                         return temp;
>                 default:
> -                       printk(KERN_INFO "Unknown IIO opcode.\n");
> +                       pr_info("Unknown IIO opcode\n");
>                         return 0;
>                 }
>  }
> @@ -194,22 +194,19 @@ static uint32_t atom_get_src_int(atom_exec_context *ctx, uint8_t attr,
>                         val = gctx->card->reg_read(gctx->card, idx);
>                         break;
>                 case ATOM_IO_PCI:
> -                       printk(KERN_INFO
> -                              "PCI registers are not implemented.\n");
> +                       pr_info("PCI registers are not implemented\n");
>                         return 0;
>                 case ATOM_IO_SYSIO:
> -                       printk(KERN_INFO
> -                              "SYSIO registers are not implemented.\n");
> +                       pr_info("SYSIO registers are not implemented\n");
>                         return 0;
>                 default:
>                         if (!(gctx->io_mode & 0x80)) {
> -                               printk(KERN_INFO "Bad IO mode.\n");
> +                               pr_info("Bad IO mode\n");
>                                 return 0;
>                         }
>                         if (!gctx->iio[gctx->io_mode & 0x7F]) {
> -                               printk(KERN_INFO
> -                                      "Undefined indirect IO read method %d.\n",
> -                                      gctx->io_mode & 0x7F);
> +                               pr_info("Undefined indirect IO read method %d\n",
> +                                       gctx->io_mode & 0x7F);
>                                 return 0;
>                         }
>                         val =
> @@ -473,22 +470,19 @@ static void atom_put_dst(atom_exec_context *ctx, int arg, uint8_t attr,
>                                 gctx->card->reg_write(gctx->card, idx, val);
>                         break;
>                 case ATOM_IO_PCI:
> -                       printk(KERN_INFO
> -                              "PCI registers are not implemented.\n");
> +                       pr_info("PCI registers are not implemented\n");
>                         return;
>                 case ATOM_IO_SYSIO:
> -                       printk(KERN_INFO
> -                              "SYSIO registers are not implemented.\n");
> +                       pr_info("SYSIO registers are not implemented\n");
>                         return;
>                 default:
>                         if (!(gctx->io_mode & 0x80)) {
> -                               printk(KERN_INFO "Bad IO mode.\n");
> +                               pr_info("Bad IO mode\n");
>                                 return;
>                         }
>                         if (!gctx->iio[gctx->io_mode & 0xFF]) {
> -                               printk(KERN_INFO
> -                                      "Undefined indirect IO write method %d.\n",
> -                                      gctx->io_mode & 0x7F);
> +                               pr_info("Undefined indirect IO write method %d\n",
> +                                       gctx->io_mode & 0x7F);
>                                 return;
>                         }
>                         atom_iio_execute(gctx, gctx->iio[gctx->io_mode & 0xFF],
> @@ -819,17 +813,17 @@ static void atom_op_postcard(atom_exec_context *ctx, int *ptr, int arg)
>
>  static void atom_op_repeat(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_restorereg(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_savereg(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static void atom_op_setdatablock(atom_exec_context *ctx, int *ptr, int arg)
> @@ -992,7 +986,7 @@ static void atom_op_switch(atom_exec_context *ctx, int *ptr, int arg)
>                         }
>                         (*ptr) += 2;
>                 } else {
> -                       printk(KERN_INFO "Bad case.\n");
> +                       pr_info("Bad case\n");
>                         return;
>                 }
>         (*ptr) += 2;
> @@ -1026,7 +1020,7 @@ static void atom_op_xor(atom_exec_context *ctx, int *ptr, int arg)
>
>  static void atom_op_debug(atom_exec_context *ctx, int *ptr, int arg)
>  {
> -       printk(KERN_INFO "unimplemented!\n");
> +       pr_info("unimplemented!\n");
>  }
>
>  static struct {
> @@ -1280,14 +1274,14 @@ struct atom_context *atom_parse(struct card_info *card, void *bios)
>         ctx->bios = bios;
>
>         if (CU16(0) != ATOM_BIOS_MAGIC) {
> -               printk(KERN_INFO "Invalid BIOS magic.\n");
> +               pr_info("Invalid BIOS magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
>         if (strncmp
>             (CSTR(ATOM_ATI_MAGIC_PTR), ATOM_ATI_MAGIC,
>              strlen(ATOM_ATI_MAGIC))) {
> -               printk(KERN_INFO "Invalid ATI magic.\n");
> +               pr_info("Invalid ATI magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
> @@ -1296,7 +1290,7 @@ struct atom_context *atom_parse(struct card_info *card, void *bios)
>         if (strncmp
>             (CSTR(base + ATOM_ROM_MAGIC_PTR), ATOM_ROM_MAGIC,
>              strlen(ATOM_ROM_MAGIC))) {
> -               printk(KERN_INFO "Invalid ATOM magic.\n");
> +               pr_info("Invalid ATOM magic\n");
>                 kfree(ctx);
>                 return NULL;
>         }
> @@ -1320,7 +1314,7 @@ struct atom_context *atom_parse(struct card_info *card, void *bios)
>                         break;
>                 }
>         }
> -       printk(KERN_INFO "ATOM BIOS: %s\n", name);
> +       pr_info("ATOM BIOS: %s\n", name);
>
>         return ctx;
>  }
> diff --git a/drivers/gpu/drm/radeon/cik.c b/drivers/gpu/drm/radeon/cik.c
> index f6ff41a0eed6..b338d58e281f 100644
> --- a/drivers/gpu/drm/radeon/cik.c
> +++ b/drivers/gpu/drm/radeon/cik.c
> @@ -2069,8 +2069,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->pfp_fw->size != pfp_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->pfp_fw->size, fw_name);
>                         err = -EINVAL;
>                         goto out;
> @@ -2078,8 +2077,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>         } else {
>                 err = radeon_ucode_validate(rdev->pfp_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2095,16 +2093,14 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->me_fw->size != me_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->me_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->me_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2120,16 +2116,14 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->ce_fw->size != ce_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->ce_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->ce_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2145,16 +2139,14 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->mec_fw->size != mec_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->mec_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->mec_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2185,16 +2177,14 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->rlc_fw->size != rlc_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_rlc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_rlc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->rlc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->rlc_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2210,16 +2200,14 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->sdma_fw->size != sdma_req_size) {
> -                       printk(KERN_ERR
> -                              "cik_sdma: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("cik_sdma: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->sdma_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->sdma_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "cik_fw: validation failed for firmware \"%s\"\n",
> +                       pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -2242,8 +2230,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                         }
>                         if ((rdev->mc_fw->size != mc_req_size) &&
>                             (rdev->mc_fw->size != mc2_req_size)){
> -                               printk(KERN_ERR
> -                                      "cik_mc: Bogus length %zu in firmware \"%s\"\n",
> +                               pr_err("cik_mc: Bogus length %zu in firmware \"%s\"\n",
>                                        rdev->mc_fw->size, fw_name);
>                                 err = -EINVAL;
>                         }
> @@ -2251,8 +2238,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                 } else {
>                         err = radeon_ucode_validate(rdev->mc_fw);
>                         if (err) {
> -                               printk(KERN_ERR
> -                                      "cik_fw: validation failed for firmware \"%s\"\n",
> +                               pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                        fw_name);
>                                 goto out;
>                         } else {
> @@ -2269,23 +2255,20 @@ static int cik_init_microcode(struct radeon_device *rdev)
>                         snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
>                         err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
>                         if (err) {
> -                               printk(KERN_ERR
> -                                      "smc: error loading firmware \"%s\"\n",
> +                               pr_err("smc: error loading firmware \"%s\"\n",
>                                        fw_name);
>                                 release_firmware(rdev->smc_fw);
>                                 rdev->smc_fw = NULL;
>                                 err = 0;
>                         } else if (rdev->smc_fw->size != smc_req_size) {
> -                               printk(KERN_ERR
> -                                      "cik_smc: Bogus length %zu in firmware \"%s\"\n",
> +                               pr_err("cik_smc: Bogus length %zu in firmware \"%s\"\n",
>                                        rdev->smc_fw->size, fw_name);
>                                 err = -EINVAL;
>                         }
>                 } else {
>                         err = radeon_ucode_validate(rdev->smc_fw);
>                         if (err) {
> -                               printk(KERN_ERR
> -                                      "cik_fw: validation failed for firmware \"%s\"\n",
> +                               pr_err("cik_fw: validation failed for firmware \"%s\"\n",
>                                        fw_name);
>                                 goto out;
>                         } else {
> @@ -2297,7 +2280,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>         if (new_fw == 0) {
>                 rdev->new_fw = false;
>         } else if (new_fw < num_fw) {
> -               printk(KERN_ERR "ci_fw: mixing new and old firmware!\n");
> +               pr_err("ci_fw: mixing new and old firmware!\n");
>                 err = -EINVAL;
>         } else {
>                 rdev->new_fw = true;
> @@ -2306,8 +2289,7 @@ static int cik_init_microcode(struct radeon_device *rdev)
>  out:
>         if (err) {
>                 if (err != -EINVAL)
> -                       printk(KERN_ERR
> -                              "cik_cp: Failed to load firmware \"%s\"\n",
> +                       pr_err("cik_cp: Failed to load firmware \"%s\"\n",
>                                fw_name);
>                 release_firmware(rdev->pfp_fw);
>                 rdev->pfp_fw = NULL;
> diff --git a/drivers/gpu/drm/radeon/evergreen.c b/drivers/gpu/drm/radeon/evergreen.c
> index 0b6b5766216f..d1b1e0cc3c25 100644
> --- a/drivers/gpu/drm/radeon/evergreen.c
> +++ b/drivers/gpu/drm/radeon/evergreen.c
> @@ -2486,7 +2486,7 @@ void evergreen_pcie_gart_tlb_flush(struct radeon_device *rdev)
>                 tmp = RREG32(VM_CONTEXT0_REQUEST_RESPONSE);
>                 tmp = (tmp & RESPONSE_TYPE_MASK) >> RESPONSE_TYPE_SHIFT;
>                 if (tmp == 2) {
> -                       printk(KERN_WARNING "[drm] r600 flush TLB failed\n");
> +                       pr_warn("[drm] r600 flush TLB failed\n");
>                         return;
>                 }
>                 if (tmp) {
> diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
> index f8b05090232a..54324330b91f 100644
> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
> @@ -1061,8 +1061,7 @@ static int evergreen_packet0_check(struct radeon_cs_parser *p,
>                 }
>                 break;
>         default:
> -               printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
> -                      reg, idx);
> +               pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
>                 return -EINVAL;
>         }
>         return 0;
> @@ -2776,7 +2775,7 @@ int evergreen_cs_parse(struct radeon_cs_parser *p)
>         } while (p->idx < p->chunk_ib->length_dw);
>  #if 0
>         for (r = 0; r < p->ib.length_dw; r++) {
> -               printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
> +               pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
>                 mdelay(1);
>         }
>  #endif
> @@ -3215,7 +3214,7 @@ int evergreen_dma_cs_parse(struct radeon_cs_parser *p)
>         } while (p->idx < p->chunk_ib->length_dw);
>  #if 0
>         for (r = 0; r < p->ib->length_dw; r++) {
> -               printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
> +               pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
>                 mdelay(1);
>         }
>  #endif
> diff --git a/drivers/gpu/drm/radeon/ni.c b/drivers/gpu/drm/radeon/ni.c
> index a0d4a0522fdc..9eccd0c81d88 100644
> --- a/drivers/gpu/drm/radeon/ni.c
> +++ b/drivers/gpu/drm/radeon/ni.c
> @@ -774,8 +774,7 @@ int ni_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->pfp_fw->size != pfp_req_size) {
> -               printk(KERN_ERR
> -                      "ni_cp: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("ni_cp: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->pfp_fw->size, fw_name);
>                 err = -EINVAL;
>                 goto out;
> @@ -786,8 +785,7 @@ int ni_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->me_fw->size != me_req_size) {
> -               printk(KERN_ERR
> -                      "ni_cp: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("ni_cp: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->me_fw->size, fw_name);
>                 err = -EINVAL;
>         }
> @@ -797,8 +795,7 @@ int ni_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->rlc_fw->size != rlc_req_size) {
> -               printk(KERN_ERR
> -                      "ni_rlc: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("ni_rlc: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->rlc_fw->size, fw_name);
>                 err = -EINVAL;
>         }
> @@ -810,8 +807,7 @@ int ni_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->mc_fw->size != mc_req_size) {
> -                       printk(KERN_ERR
> -                              "ni_mc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("ni_mc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->mc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
> @@ -821,15 +817,12 @@ int ni_init_microcode(struct radeon_device *rdev)
>                 snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
>                 err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "smc: error loading firmware \"%s\"\n",
> -                              fw_name);
> +                       pr_err("smc: error loading firmware \"%s\"\n", fw_name);
>                         release_firmware(rdev->smc_fw);
>                         rdev->smc_fw = NULL;
>                         err = 0;
>                 } else if (rdev->smc_fw->size != smc_req_size) {
> -                       printk(KERN_ERR
> -                              "ni_mc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("ni_mc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->mc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
> @@ -838,8 +831,7 @@ int ni_init_microcode(struct radeon_device *rdev)
>  out:
>         if (err) {
>                 if (err != -EINVAL)
> -                       printk(KERN_ERR
> -                              "ni_cp: Failed to load firmware \"%s\"\n",
> +                       pr_err("ni_cp: Failed to load firmware \"%s\"\n",
>                                fw_name);
>                 release_firmware(rdev->pfp_fw);
>                 rdev->pfp_fw = NULL;
> diff --git a/drivers/gpu/drm/radeon/r100.c b/drivers/gpu/drm/radeon/r100.c
> index e3399310d41d..aefca0b03f38 100644
> --- a/drivers/gpu/drm/radeon/r100.c
> +++ b/drivers/gpu/drm/radeon/r100.c
> @@ -1042,11 +1042,9 @@ static int r100_cp_init_microcode(struct radeon_device *rdev)
>
>         err = request_firmware(&rdev->me_fw, fw_name, rdev->dev);
>         if (err) {
> -               printk(KERN_ERR "radeon_cp: Failed to load firmware \"%s\"\n",
> -                      fw_name);
> +               pr_err("radeon_cp: Failed to load firmware \"%s\"\n", fw_name);
>         } else if (rdev->me_fw->size % 8) {
> -               printk(KERN_ERR
> -                      "radeon_cp: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("radeon_cp: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->me_fw->size, fw_name);
>                 err = -EINVAL;
>                 release_firmware(rdev->me_fw);
> @@ -1087,8 +1085,7 @@ static void r100_cp_load_microcode(struct radeon_device *rdev)
>         int i, size;
>
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>
>         if (rdev->me_fw) {
> @@ -1246,8 +1243,7 @@ void r100_cp_disable(struct radeon_device *rdev)
>         WREG32(RADEON_CP_CSQ_CNTL, 0);
>         WREG32(R_000770_SCRATCH_UMSK, 0);
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>  }
>
> @@ -1881,8 +1877,7 @@ static int r100_packet0_check(struct radeon_cs_parser *p,
>                 track->tex_dirty = true;
>                 break;
>         default:
> -               printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
> -                      reg, idx);
> +               pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
>                 return -EINVAL;
>         }
>         return 0;
> @@ -2486,8 +2481,7 @@ int r100_gui_wait_for_idle(struct radeon_device *rdev)
>         uint32_t tmp;
>
>         if (r100_rbbm_fifo_wait_for_entry(rdev, 64)) {
> -               printk(KERN_WARNING "radeon: wait for empty RBBM fifo failed !"
> -                      " Bad things might happen.\n");
> +               pr_warn("radeon: wait for empty RBBM fifo failed! Bad things might happen.\n");
>         }
>         for (i = 0; i < rdev->usec_timeout; i++) {
>                 tmp = RREG32(RADEON_RBBM_STATUS);
> diff --git a/drivers/gpu/drm/radeon/r200.c b/drivers/gpu/drm/radeon/r200.c
> index c70e6d5bcd19..c22321cc5a41 100644
> --- a/drivers/gpu/drm/radeon/r200.c
> +++ b/drivers/gpu/drm/radeon/r200.c
> @@ -537,8 +537,7 @@ int r200_packet0_check(struct radeon_cs_parser *p,
>                 track->tex_dirty = true;
>                 break;
>         default:
> -               printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
> -                      reg, idx);
> +               pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
>                 return -EINVAL;
>         }
>         return 0;
> diff --git a/drivers/gpu/drm/radeon/r300.c b/drivers/gpu/drm/radeon/r300.c
> index 7e417d8dc733..21161aa8acbf 100644
> --- a/drivers/gpu/drm/radeon/r300.c
> +++ b/drivers/gpu/drm/radeon/r300.c
> @@ -387,8 +387,7 @@ static void r300_gpu_init(struct radeon_device *rdev)
>         WREG32(R300_GB_TILE_CONFIG, gb_tile_config);
>
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>
>         tmp = RREG32(R300_DST_PIPE_CONFIG);
> @@ -399,14 +398,12 @@ static void r300_gpu_init(struct radeon_device *rdev)
>                R300_DC_DC_DISABLE_IGNORE_PE);
>
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>         if (r300_mc_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait MC idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
>         }
> -       DRM_INFO("radeon: %d quad pipes, %d Z pipes initialized.\n",
> +       DRM_INFO("radeon: %d quad pipes, %d Z pipes initialized\n",
>                  rdev->num_gb_pipes, rdev->num_z_pipes);
>  }
>
> @@ -1165,7 +1162,7 @@ static int r300_packet0_check(struct radeon_cs_parser *p,
>         }
>         return 0;
>  fail:
> -       printk(KERN_ERR "Forbidden register 0x%04X in cs at %d (val=%08x)\n",
> +       pr_err("Forbidden register 0x%04X in cs at %d (val=%08x)\n",
>                reg, idx, idx_value);
>         return -EINVAL;
>  }
> diff --git a/drivers/gpu/drm/radeon/r420.c b/drivers/gpu/drm/radeon/r420.c
> index 2828605aef3f..3eb0c4f9f796 100644
> --- a/drivers/gpu/drm/radeon/r420.c
> +++ b/drivers/gpu/drm/radeon/r420.c
> @@ -92,8 +92,7 @@ void r420_pipes_init(struct radeon_device *rdev)
>                (1 << 2) | (1 << 3));
>         /* add idle wait as per freedesktop.org bug 24041 */
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>         /* get max number of pipes */
>         gb_pipe_select = RREG32(R400_GB_PIPE_SELECT);
> @@ -128,8 +127,7 @@ void r420_pipes_init(struct radeon_device *rdev)
>         tmp |= R300_TILE_SIZE_16 | R300_ENABLE_TILING;
>         WREG32(R300_GB_TILE_CONFIG, tmp);
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>
>         tmp = RREG32(R300_DST_PIPE_CONFIG);
> @@ -141,8 +139,7 @@ void r420_pipes_init(struct radeon_device *rdev)
>                R300_DC_DC_DISABLE_IGNORE_PE);
>
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
>         }
>
>         if (rdev->family == CHIP_RV530) {
> diff --git a/drivers/gpu/drm/radeon/r520.c b/drivers/gpu/drm/radeon/r520.c
> index 98d6053c36c6..074cf752faef 100644
> --- a/drivers/gpu/drm/radeon/r520.c
> +++ b/drivers/gpu/drm/radeon/r520.c
> @@ -86,8 +86,7 @@ static void r520_gpu_init(struct radeon_device *rdev)
>               (((gb_pipe_select >> 8) & 0xF) << 4);
>         WREG32_PLL(0x000D, tmp);
>         if (r520_mc_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait MC idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
>         }
>  }
>
> diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c
> index a951881c2a50..0a085176e79b 100644
> --- a/drivers/gpu/drm/radeon/r600.c
> +++ b/drivers/gpu/drm/radeon/r600.c
> @@ -1094,7 +1094,7 @@ void r600_pcie_gart_tlb_flush(struct radeon_device *rdev)
>                 tmp = RREG32(VM_CONTEXT0_REQUEST_RESPONSE);
>                 tmp = (tmp & RESPONSE_TYPE_MASK) >> RESPONSE_TYPE_SHIFT;
>                 if (tmp == 2) {
> -                       printk(KERN_WARNING "[drm] r600 flush TLB failed\n");
> +                       pr_warn("[drm] r600 flush TLB failed\n");
>                         return;
>                 }
>                 if (tmp) {
> @@ -2550,8 +2550,7 @@ int r600_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->pfp_fw->size != pfp_req_size) {
> -               printk(KERN_ERR
> -                      "r600_cp: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("r600_cp: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->pfp_fw->size, fw_name);
>                 err = -EINVAL;
>                 goto out;
> @@ -2562,8 +2561,7 @@ int r600_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->me_fw->size != me_req_size) {
> -               printk(KERN_ERR
> -                      "r600_cp: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("r600_cp: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->me_fw->size, fw_name);
>                 err = -EINVAL;
>         }
> @@ -2573,8 +2571,7 @@ int r600_init_microcode(struct radeon_device *rdev)
>         if (err)
>                 goto out;
>         if (rdev->rlc_fw->size != rlc_req_size) {
> -               printk(KERN_ERR
> -                      "r600_rlc: Bogus length %zu in firmware \"%s\"\n",
> +               pr_err("r600_rlc: Bogus length %zu in firmware \"%s\"\n",
>                        rdev->rlc_fw->size, fw_name);
>                 err = -EINVAL;
>         }
> @@ -2583,15 +2580,12 @@ int r600_init_microcode(struct radeon_device *rdev)
>                 snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", smc_chip_name);
>                 err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "smc: error loading firmware \"%s\"\n",
> -                              fw_name);
> +                       pr_err("smc: error loading firmware \"%s\"\n", fw_name);
>                         release_firmware(rdev->smc_fw);
>                         rdev->smc_fw = NULL;
>                         err = 0;
>                 } else if (rdev->smc_fw->size != smc_req_size) {
> -                       printk(KERN_ERR
> -                              "smc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("smc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->smc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
> @@ -2600,8 +2594,7 @@ int r600_init_microcode(struct radeon_device *rdev)
>  out:
>         if (err) {
>                 if (err != -EINVAL)
> -                       printk(KERN_ERR
> -                              "r600_cp: Failed to load firmware \"%s\"\n",
> +                       pr_err("r600_cp: Failed to load firmware \"%s\"\n",
>                                fw_name);
>                 release_firmware(rdev->pfp_fw);
>                 rdev->pfp_fw = NULL;
> diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
> index 595a19736458..97fd58e97043 100644
> --- a/drivers/gpu/drm/radeon/r600_cs.c
> +++ b/drivers/gpu/drm/radeon/r600_cs.c
> @@ -932,8 +932,7 @@ static int r600_packet0_check(struct radeon_cs_parser *p,
>                 }
>                 break;
>         default:
> -               printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
> -                      reg, idx);
> +               pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
>                 return -EINVAL;
>         }
>         return 0;
> @@ -2320,7 +2319,7 @@ int r600_cs_parse(struct radeon_cs_parser *p)
>         } while (p->idx < p->chunk_ib->length_dw);
>  #if 0
>         for (r = 0; r < p->ib.length_dw; r++) {
> -               printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
> +               pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
>                 mdelay(1);
>         }
>  #endif
> @@ -2528,7 +2527,7 @@ int r600_dma_cs_parse(struct radeon_cs_parser *p)
>         } while (p->idx < p->chunk_ib->length_dw);
>  #if 0
>         for (r = 0; r < p->ib->length_dw; r++) {
> -               printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
> +               pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
>                 mdelay(1);
>         }
>  #endif
> diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
> index 44e0c5ed6418..2e400dc414e3 100644
> --- a/drivers/gpu/drm/radeon/radeon.h
> +++ b/drivers/gpu/drm/radeon/radeon.h
> @@ -2535,7 +2535,8 @@ static inline struct radeon_fence *to_radeon_fence(struct dma_fence *f)
>  #define WREG16(reg, v) writew(v, (rdev->rmmio) + (reg))
>  #define RREG32(reg) r100_mm_rreg(rdev, (reg), false)
>  #define RREG32_IDX(reg) r100_mm_rreg(rdev, (reg), true)
> -#define DREG32(reg) printk(KERN_INFO "REGISTER: " #reg " : 0x%08X\n", r100_mm_rreg(rdev, (reg), false))
> +#define DREG32(reg) pr_info("REGISTER: " #reg " : 0x%08X\n",   \
> +                           r100_mm_rreg(rdev, (reg), false))
>  #define WREG32(reg, v) r100_mm_wreg(rdev, (reg), (v), false)
>  #define WREG32_IDX(reg, v) r100_mm_wreg(rdev, (reg), (v), true)
>  #define REG_SET(FIELD, v) (((v) << FIELD##_SHIFT) & FIELD##_MASK)
> diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> index 0ae13cd2adda..40be4068ca69 100644
> --- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> +++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
> @@ -577,8 +577,8 @@ static bool radeon_atpx_detect(void)
>
>         if (has_atpx && vga_count == 2) {
>                 acpi_get_name(radeon_atpx_priv.atpx.handle, ACPI_FULL_PATHNAME, &buffer);
> -               printk(KERN_INFO "vga_switcheroo: detected switching method %s handle\n",
> -                      acpi_method_name);
> +               pr_info("vga_switcheroo: detected switching method %s handle\n",
> +                       acpi_method_name);
>                 radeon_atpx_priv.atpx_detected = true;
>                 radeon_atpx_priv.bridge_pm_usable = d3_supported;
>                 radeon_atpx_init();
> diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
> index b214663b370d..aaacac190d26 100644
> --- a/drivers/gpu/drm/radeon/radeon_audio.c
> +++ b/drivers/gpu/drm/radeon/radeon_audio.c
> @@ -576,9 +576,9 @@ static void radeon_audio_calc_cts(unsigned int clock, int *CTS, int *N, int freq
>
>         /* Check that we are in spec (not always possible) */
>         if (n < (128*freq/1500))
> -               printk(KERN_WARNING "Calculated ACR N value is too small. You may experience audio problems.\n");
> +               pr_warn("Calculated ACR N value is too small. You may experience audio problems.\n");
>         if (n > (128*freq/300))
> -               printk(KERN_WARNING "Calculated ACR N value is too large. You may experience audio problems.\n");
> +               pr_warn("Calculated ACR N value is too large. You may experience audio problems.\n");
>
>         *N = n;
>         *CTS = cts;
> diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
> index c1135feb93c1..e55146cdf543 100644
> --- a/drivers/gpu/drm/radeon/radeon_clocks.c
> +++ b/drivers/gpu/drm/radeon/radeon_clocks.c
> @@ -111,7 +111,7 @@ static bool radeon_read_clocks_OF(struct drm_device *dev)
>                 return false;
>         val = of_get_property(dp, "ATY,RefCLK", NULL);
>         if (!val || !*val) {
> -               printk(KERN_WARNING "radeonfb: No ATY,RefCLK property !\n");
> +               pr_warn("radeonfb: No ATY,RefCLK property !\n");
>                 return false;
>         }
>         p1pll->reference_freq = p2pll->reference_freq = (*val) / 10;
> diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
> index 4b0c388be3f5..dde59c36ba92 100644
> --- a/drivers/gpu/drm/radeon/radeon_device.c
> +++ b/drivers/gpu/drm/radeon/radeon_device.c
> @@ -1249,7 +1249,7 @@ static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
>         if (state == VGA_SWITCHEROO_ON) {
>                 unsigned d3_delay = dev->pdev->d3_delay;
>
> -               printk(KERN_INFO "radeon: switched on\n");
> +               pr_info("radeon: switched on\n");
>                 /* don't suspend or resume card normally */
>                 dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
>
> @@ -1263,7 +1263,7 @@ static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
>                 dev->switch_power_state = DRM_SWITCH_POWER_ON;
>                 drm_kms_helper_poll_enable(dev);
>         } else {
> -               printk(KERN_INFO "radeon: switched off\n");
> +               pr_info("radeon: switched off\n");
>                 drm_kms_helper_poll_disable(dev);
>                 dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
>                 radeon_suspend_kms(dev, true, true, false);
> @@ -1410,12 +1410,12 @@ int radeon_device_init(struct radeon_device *rdev,
>         if (r) {
>                 rdev->need_dma32 = true;
>                 dma_bits = 32;
> -               printk(KERN_WARNING "radeon: No suitable DMA available.\n");
> +               pr_warn("radeon: No suitable DMA available\n");
>         }
>         r = pci_set_consistent_dma_mask(rdev->pdev, DMA_BIT_MASK(dma_bits));
>         if (r) {
>                 pci_set_consistent_dma_mask(rdev->pdev, DMA_BIT_MASK(32));
> -               printk(KERN_WARNING "radeon: No coherent DMA available.\n");
> +               pr_warn("radeon: No coherent DMA available\n");
>         }
>
>         /* Registers mapping */
> diff --git a/drivers/gpu/drm/radeon/radeon_fb.c b/drivers/gpu/drm/radeon/radeon_fb.c
> index 2be4fe9c7217..53aa94f705a7 100644
> --- a/drivers/gpu/drm/radeon/radeon_fb.c
> +++ b/drivers/gpu/drm/radeon/radeon_fb.c
> @@ -149,8 +149,7 @@ static int radeonfb_create_pinned_object(struct radeon_fbdev *rfbdev,
>                                        RADEON_GEM_DOMAIN_VRAM,
>                                        0, true, &gobj);
>         if (ret) {
> -               printk(KERN_ERR "failed to allocate framebuffer (%d)\n",
> -                      aligned_size);
> +               pr_err("failed to allocate framebuffer (%d)\n", aligned_size);
>                 return -ENOMEM;
>         }
>         rbo = gem_to_radeon_bo(gobj);
> diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
> index 96683f5b2b1b..9b0b123ce079 100644
> --- a/drivers/gpu/drm/radeon/radeon_gem.c
> +++ b/drivers/gpu/drm/radeon/radeon_gem.c
> @@ -106,7 +106,7 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj,
>         }
>         if (!domain) {
>                 /* Do nothings */
> -               printk(KERN_WARNING "Set domain without domain !\n");
> +               pr_warn("Set domain without domain !\n");
>                 return 0;
>         }
>         if (domain == RADEON_GEM_DOMAIN_CPU) {
> @@ -116,7 +116,7 @@ static int radeon_gem_set_domain(struct drm_gem_object *gobj,
>                         r = -EBUSY;
>
>                 if (r < 0 && r != -EINTR) {
> -                       printk(KERN_ERR "Failed to wait for object: %li\n", r);
> +                       pr_err("Failed to wait for object: %li\n", r);
>                         return r;
>                 }
>         }
> diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c
> index 791818165c76..4fdc7bda7a7d 100644
> --- a/drivers/gpu/drm/radeon/radeon_test.c
> +++ b/drivers/gpu/drm/radeon/radeon_test.c
> @@ -246,7 +246,7 @@ static void radeon_do_test_moves(struct radeon_device *rdev, int flag)
>  out_cleanup:
>         kfree(gtt_obj);
>         if (r) {
> -               printk(KERN_WARNING "Error while testing BO move.\n");
> +               pr_warn("Error while testing BO move\n");
>         }
>  }
>
> @@ -394,7 +394,7 @@ void radeon_test_ring_sync(struct radeon_device *rdev,
>                 radeon_fence_unref(&fence2);
>
>         if (r)
> -               printk(KERN_WARNING "Error while testing ring sync (%d).\n", r);
> +               pr_warn("Error while testing ring sync (%d)\n", r);
>  }
>
>  static void radeon_test_ring_sync2(struct radeon_device *rdev,
> @@ -504,7 +504,7 @@ static void radeon_test_ring_sync2(struct radeon_device *rdev,
>                 radeon_fence_unref(&fenceB);
>
>         if (r)
> -               printk(KERN_WARNING "Error while testing ring sync (%d).\n", r);
> +               pr_warn("Error while testing ring sync (%d)\n", r);
>  }
>
>  static bool radeon_test_sync_possible(struct radeon_ring *ringA,
> diff --git a/drivers/gpu/drm/radeon/rs400.c b/drivers/gpu/drm/radeon/rs400.c
> index 34e3235f41d2..4121209c183e 100644
> --- a/drivers/gpu/drm/radeon/rs400.c
> +++ b/drivers/gpu/drm/radeon/rs400.c
> @@ -255,8 +255,8 @@ static void rs400_gpu_init(struct radeon_device *rdev)
>         /* FIXME: is this correct ? */
>         r420_pipes_init(rdev);
>         if (rs400_mc_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "rs400: Failed to wait MC idle while "
> -                      "programming pipes. Bad things might happen. %08x\n", RREG32(RADEON_MC_STATUS));
> +               pr_warn("rs400: Failed to wait MC idle while programming pipes. Bad things might happen. %08x\n",
> +                       RREG32(RADEON_MC_STATUS));
>         }
>  }
>
> diff --git a/drivers/gpu/drm/radeon/rs690.c b/drivers/gpu/drm/radeon/rs690.c
> index 6bc44c24e837..1bae33e43f3c 100644
> --- a/drivers/gpu/drm/radeon/rs690.c
> +++ b/drivers/gpu/drm/radeon/rs690.c
> @@ -52,8 +52,7 @@ static void rs690_gpu_init(struct radeon_device *rdev)
>         /* FIXME: is this correct ? */
>         r420_pipes_init(rdev);
>         if (rs690_mc_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait MC idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
>         }
>  }
>
> diff --git a/drivers/gpu/drm/radeon/rv515.c b/drivers/gpu/drm/radeon/rv515.c
> index c55d653aaf5f..ffbd2c006f60 100644
> --- a/drivers/gpu/drm/radeon/rv515.c
> +++ b/drivers/gpu/drm/radeon/rv515.c
> @@ -154,8 +154,7 @@ static void rv515_gpu_init(struct radeon_device *rdev)
>         unsigned pipe_select_current, gb_pipe_select, tmp;
>
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "resetting GPU. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while resetting GPU. Bad things might happen.\n");
>         }
>         rv515_vga_render_disable(rdev);
>         r420_pipes_init(rdev);
> @@ -166,12 +165,10 @@ static void rv515_gpu_init(struct radeon_device *rdev)
>               (((gb_pipe_select >> 8) & 0xF) << 4);
>         WREG32_PLL(0x000D, tmp);
>         if (r100_gui_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait GUI idle while "
> -                      "resetting GPU. Bad things might happen.\n");
> +               pr_warn("Failed to wait GUI idle while resetting GPU. Bad things might happen.\n");
>         }
>         if (rv515_mc_wait_for_idle(rdev)) {
> -               printk(KERN_WARNING "Failed to wait MC idle while "
> -                      "programming pipes. Bad things might happen.\n");
> +               pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
>         }
>  }
>
> diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
> index 414776811e71..528e5a49a214 100644
> --- a/drivers/gpu/drm/radeon/si.c
> +++ b/drivers/gpu/drm/radeon/si.c
> @@ -1762,8 +1762,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->pfp_fw->size != pfp_req_size) {
> -                       printk(KERN_ERR
> -                              "si_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->pfp_fw->size, fw_name);
>                         err = -EINVAL;
>                         goto out;
> @@ -1771,8 +1770,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>         } else {
>                 err = radeon_ucode_validate(rdev->pfp_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1788,16 +1786,14 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->me_fw->size != me_req_size) {
> -                       printk(KERN_ERR
> -                              "si_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->me_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->me_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1813,16 +1809,14 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->ce_fw->size != ce_req_size) {
> -                       printk(KERN_ERR
> -                              "si_cp: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->ce_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->ce_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1838,16 +1832,14 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 if (err)
>                         goto out;
>                 if (rdev->rlc_fw->size != rlc_req_size) {
> -                       printk(KERN_ERR
> -                              "si_rlc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_rlc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->rlc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->rlc_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1871,8 +1863,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 }
>                 if ((rdev->mc_fw->size != mc_req_size) &&
>                     (rdev->mc_fw->size != mc2_req_size)) {
> -                       printk(KERN_ERR
> -                              "si_mc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_mc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->mc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
> @@ -1880,8 +1871,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>         } else {
>                 err = radeon_ucode_validate(rdev->mc_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1900,23 +1890,19 @@ static int si_init_microcode(struct radeon_device *rdev)
>                 snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
>                 err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "smc: error loading firmware \"%s\"\n",
> -                              fw_name);
> +                       pr_err("smc: error loading firmware \"%s\"\n", fw_name);
>                         release_firmware(rdev->smc_fw);
>                         rdev->smc_fw = NULL;
>                         err = 0;
>                 } else if (rdev->smc_fw->size != smc_req_size) {
> -                       printk(KERN_ERR
> -                              "si_smc: Bogus length %zu in firmware \"%s\"\n",
> +                       pr_err("si_smc: Bogus length %zu in firmware \"%s\"\n",
>                                rdev->smc_fw->size, fw_name);
>                         err = -EINVAL;
>                 }
>         } else {
>                 err = radeon_ucode_validate(rdev->smc_fw);
>                 if (err) {
> -                       printk(KERN_ERR
> -                              "si_cp: validation failed for firmware \"%s\"\n",
> +                       pr_err("si_cp: validation failed for firmware \"%s\"\n",
>                                fw_name);
>                         goto out;
>                 } else {
> @@ -1927,7 +1913,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>         if (new_fw == 0) {
>                 rdev->new_fw = false;
>         } else if (new_fw < 6) {
> -               printk(KERN_ERR "si_fw: mixing new and old firmware!\n");
> +               pr_err("si_fw: mixing new and old firmware!\n");
>                 err = -EINVAL;
>         } else {
>                 rdev->new_fw = true;
> @@ -1935,8 +1921,7 @@ static int si_init_microcode(struct radeon_device *rdev)
>  out:
>         if (err) {
>                 if (err != -EINVAL)
> -                       printk(KERN_ERR
> -                              "si_cp: Failed to load firmware \"%s\"\n",
> +                       pr_err("si_cp: Failed to load firmware \"%s\"\n",
>                                fw_name);
>                 release_firmware(rdev->pfp_fw);
>                 rdev->pfp_fw = NULL;
> --
> 2.10.0.rc2.1.g053435c
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/dri-devel
diff mbox

Patch

diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu.h b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
index c1b913541739..3f636632c289 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu.h
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu.h
@@ -1515,7 +1515,8 @@  void amdgpu_mm_wdoorbell(struct amdgpu_device *adev, u32 index, u32 v);
  */
 #define RREG32(reg) amdgpu_mm_rreg(adev, (reg), false)
 #define RREG32_IDX(reg) amdgpu_mm_rreg(adev, (reg), true)
-#define DREG32(reg) printk(KERN_INFO "REGISTER: " #reg " : 0x%08X\n", amdgpu_mm_rreg(adev, (reg), false))
+#define DREG32(reg) pr_info("REGISTER: " #reg " : 0x%08X\n",		\
+			    amdgpu_mm_rreg(adev, (reg), false))
 #define WREG32(reg, v) amdgpu_mm_wreg(adev, (reg), (v), false)
 #define WREG32_IDX(reg, v) amdgpu_mm_wreg(adev, (reg), (v), true)
 #define REG_SET(FIELD, v) (((v) << FIELD##_SHIFT) & FIELD##_MASK)
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
index 857ba0897159..3889486f71fe 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_afmt.c
@@ -74,9 +74,9 @@  static void amdgpu_afmt_calc_cts(uint32_t clock, int *CTS, int *N, int freq)
 
 	/* Check that we are in spec (not always possible) */
 	if (n < (128*freq/1500))
-		printk(KERN_WARNING "Calculated ACR N value is too small. You may experience audio problems.\n");
+		pr_warn("Calculated ACR N value is too small. You may experience audio problems.\n");
 	if (n > (128*freq/300))
-		printk(KERN_WARNING "Calculated ACR N value is too large. You may experience audio problems.\n");
+		pr_warn("Calculated ACR N value is too large. You may experience audio problems.\n");
 
 	*N = n;
 	*CTS = cts;
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
index 6c343a933182..c13c51af0b68 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_atpx_handler.c
@@ -583,8 +583,8 @@  static bool amdgpu_atpx_detect(void)
 
 	if (has_atpx && vga_count == 2) {
 		acpi_get_name(amdgpu_atpx_priv.atpx.handle, ACPI_FULL_PATHNAME, &buffer);
-		printk(KERN_INFO "vga_switcheroo: detected switching method %s handle\n",
-		       acpi_method_name);
+		pr_info("vga_switcheroo: detected switching method %s handle\n",
+			acpi_method_name);
 		amdgpu_atpx_priv.atpx_detected = true;
 		amdgpu_atpx_priv.bridge_pm_usable = d3_supported;
 		amdgpu_atpx_init();
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
index 6abb238b25c9..85eeae05ac27 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_device.c
@@ -1059,7 +1059,7 @@  static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
 	if (state == VGA_SWITCHEROO_ON) {
 		unsigned d3_delay = dev->pdev->d3_delay;
 
-		printk(KERN_INFO "amdgpu: switched on\n");
+		pr_info("amdgpu: switched on\n");
 		/* don't suspend or resume card normally */
 		dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
 
@@ -1070,7 +1070,7 @@  static void amdgpu_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
 		dev->switch_power_state = DRM_SWITCH_POWER_ON;
 		drm_kms_helper_poll_enable(dev);
 	} else {
-		printk(KERN_INFO "amdgpu: switched off\n");
+		pr_info("amdgpu: switched off\n");
 		drm_kms_helper_poll_disable(dev);
 		dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
 		amdgpu_device_suspend(dev, true, true);
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
index 36ce3cac81ba..35a287280363 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_fb.c
@@ -150,8 +150,7 @@  static int amdgpufb_create_pinned_object(struct amdgpu_fbdev *rfbdev,
 				       AMDGPU_GEM_CREATE_VRAM_CONTIGUOUS,
 				       true, &gobj);
 	if (ret) {
-		printk(KERN_ERR "failed to allocate framebuffer (%d)\n",
-		       aligned_size);
+		pr_err("failed to allocate framebuffer (%d)\n", aligned_size);
 		return -ENOMEM;
 	}
 	abo = gem_to_amdgpu_bo(gobj);
diff --git a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
index e05a24325eeb..b0483e6e536f 100644
--- a/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
+++ b/drivers/gpu/drm/amd/amdgpu/amdgpu_test.c
@@ -228,7 +228,7 @@  static void amdgpu_do_test_moves(struct amdgpu_device *adev)
 out_cleanup:
 	kfree(gtt_obj);
 	if (r) {
-		printk(KERN_WARNING "Error while testing BO move.\n");
+		pr_warn("Error while testing BO move\n");
 	}
 }
 
diff --git a/drivers/gpu/drm/amd/amdgpu/atom.c b/drivers/gpu/drm/amd/amdgpu/atom.c
index 1b50e6c13fb3..d1444aaca038 100644
--- a/drivers/gpu/drm/amd/amdgpu/atom.c
+++ b/drivers/gpu/drm/amd/amdgpu/atom.c
@@ -166,7 +166,7 @@  static uint32_t atom_iio_execute(struct atom_context *ctx, int base,
 		case ATOM_IIO_END:
 			return temp;
 		default:
-			printk(KERN_INFO "Unknown IIO opcode.\n");
+			pr_info("Unknown IIO opcode\n");
 			return 0;
 		}
 }
@@ -190,22 +190,19 @@  static uint32_t atom_get_src_int(atom_exec_context *ctx, uint8_t attr,
 			val = gctx->card->reg_read(gctx->card, idx);
 			break;
 		case ATOM_IO_PCI:
-			printk(KERN_INFO
-			       "PCI registers are not implemented.\n");
+			pr_info("PCI registers are not implemented\n");
 			return 0;
 		case ATOM_IO_SYSIO:
-			printk(KERN_INFO
-			       "SYSIO registers are not implemented.\n");
+			pr_info("SYSIO registers are not implemented\n");
 			return 0;
 		default:
 			if (!(gctx->io_mode & 0x80)) {
-				printk(KERN_INFO "Bad IO mode.\n");
+				pr_info("Bad IO mode\n");
 				return 0;
 			}
 			if (!gctx->iio[gctx->io_mode & 0x7F]) {
-				printk(KERN_INFO
-				       "Undefined indirect IO read method %d.\n",
-				       gctx->io_mode & 0x7F);
+				pr_info("Undefined indirect IO read method %d\n",
+					gctx->io_mode & 0x7F);
 				return 0;
 			}
 			val =
@@ -469,22 +466,19 @@  static void atom_put_dst(atom_exec_context *ctx, int arg, uint8_t attr,
 				gctx->card->reg_write(gctx->card, idx, val);
 			break;
 		case ATOM_IO_PCI:
-			printk(KERN_INFO
-			       "PCI registers are not implemented.\n");
+			pr_info("PCI registers are not implemented\n");
 			return;
 		case ATOM_IO_SYSIO:
-			printk(KERN_INFO
-			       "SYSIO registers are not implemented.\n");
+			pr_info("SYSIO registers are not implemented\n");
 			return;
 		default:
 			if (!(gctx->io_mode & 0x80)) {
-				printk(KERN_INFO "Bad IO mode.\n");
+				pr_info("Bad IO mode\n");
 				return;
 			}
 			if (!gctx->iio[gctx->io_mode & 0xFF]) {
-				printk(KERN_INFO
-				       "Undefined indirect IO write method %d.\n",
-				       gctx->io_mode & 0x7F);
+				pr_info("Undefined indirect IO write method %d\n",
+					gctx->io_mode & 0x7F);
 				return;
 			}
 			atom_iio_execute(gctx, gctx->iio[gctx->io_mode & 0xFF],
@@ -850,17 +844,17 @@  static void atom_op_postcard(atom_exec_context *ctx, int *ptr, int arg)
 
 static void atom_op_repeat(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_restorereg(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_savereg(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_setdatablock(atom_exec_context *ctx, int *ptr, int arg)
@@ -1023,7 +1017,7 @@  static void atom_op_switch(atom_exec_context *ctx, int *ptr, int arg)
 			}
 			(*ptr) += 2;
 		} else {
-			printk(KERN_INFO "Bad case.\n");
+			pr_info("Bad case\n");
 			return;
 		}
 	(*ptr) += 2;
@@ -1316,14 +1310,14 @@  struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
 	ctx->bios = bios;
 
 	if (CU16(0) != ATOM_BIOS_MAGIC) {
-		printk(KERN_INFO "Invalid BIOS magic.\n");
+		pr_info("Invalid BIOS magic\n");
 		kfree(ctx);
 		return NULL;
 	}
 	if (strncmp
 	    (CSTR(ATOM_ATI_MAGIC_PTR), ATOM_ATI_MAGIC,
 	     strlen(ATOM_ATI_MAGIC))) {
-		printk(KERN_INFO "Invalid ATI magic.\n");
+		pr_info("Invalid ATI magic\n");
 		kfree(ctx);
 		return NULL;
 	}
@@ -1332,7 +1326,7 @@  struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
 	if (strncmp
 	    (CSTR(base + ATOM_ROM_MAGIC_PTR), ATOM_ROM_MAGIC,
 	     strlen(ATOM_ROM_MAGIC))) {
-		printk(KERN_INFO "Invalid ATOM magic.\n");
+		pr_info("Invalid ATOM magic\n");
 		kfree(ctx);
 		return NULL;
 	}
@@ -1356,7 +1350,7 @@  struct atom_context *amdgpu_atom_parse(struct card_info *card, void *bios)
 			break;
 		}
 	}
-	printk(KERN_INFO "ATOM BIOS: %s\n", name);
+	pr_info("ATOM BIOS: %s\n", name);
 
 	return ctx;
 }
diff --git a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
index f97ecb49972e..b59e7e5d7279 100644
--- a/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
+++ b/drivers/gpu/drm/amd/amdgpu/ci_dpm.c
@@ -5804,9 +5804,7 @@  static int ci_dpm_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "cik_smc: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("cik_smc: Failed to load firmware \"%s\"\n", fw_name);
 		release_firmware(adev->pm.fw);
 		adev->pm.fw = NULL;
 	}
diff --git a/drivers/gpu/drm/amd/amdgpu/cik_sdma.c b/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
index 810bba533975..c33bc1bb4655 100644
--- a/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
+++ b/drivers/gpu/drm/amd/amdgpu/cik_sdma.c
@@ -142,9 +142,7 @@  static int cik_sdma_init_microcode(struct amdgpu_device *adev)
 	}
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "cik_sdma: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("cik_sdma: Failed to load firmware \"%s\"\n", fw_name);
 		for (i = 0; i < adev->sdma.num_instances; i++) {
 			release_firmware(adev->sdma.instance[i].fw);
 			adev->sdma.instance[i].fw = NULL;
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
index 2086e7e68de4..c8f9c1414940 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v6_0.c
@@ -378,9 +378,7 @@  static int gfx_v6_0_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "gfx6: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("gfx6: Failed to load firmware \"%s\"\n", fw_name);
 		release_firmware(adev->gfx.pfp_fw);
 		adev->gfx.pfp_fw = NULL;
 		release_firmware(adev->gfx.me_fw);
diff --git a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
index 1f9354541f29..82c574cf3658 100644
--- a/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gfx_v7_0.c
@@ -972,9 +972,7 @@  static int gfx_v7_0_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "gfx7: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("gfx7: Failed to load firmware \"%s\"\n", fw_name);
 		release_firmware(adev->gfx.pfp_fw);
 		adev->gfx.pfp_fw = NULL;
 		release_firmware(adev->gfx.me_fw);
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
index 8d05e0c4e3d7..52594b9fd182 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v7_0.c
@@ -161,9 +161,7 @@  static int gmc_v7_0_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "cik_mc: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("cik_mc: Failed to load firmware \"%s\"\n", fw_name);
 		release_firmware(adev->mc.fw);
 		adev->mc.fw = NULL;
 	}
@@ -938,12 +936,12 @@  static int gmc_v7_0_sw_init(void *handle)
 	if (r) {
 		adev->need_dma32 = true;
 		dma_bits = 32;
-		printk(KERN_WARNING "amdgpu: No suitable DMA available.\n");
+		pr_warn("amdgpu: No suitable DMA available\n");
 	}
 	r = pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(dma_bits));
 	if (r) {
 		pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(32));
-		printk(KERN_WARNING "amdgpu: No coherent DMA available.\n");
+		pr_warn("amdgpu: No coherent DMA available\n");
 	}
 
 	r = gmc_v7_0_init_microcode(adev);
diff --git a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
index 7669b3259f35..421507aea36a 100644
--- a/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/gmc_v8_0.c
@@ -245,9 +245,7 @@  static int gmc_v8_0_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "mc: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("mc: Failed to load firmware \"%s\"\n", fw_name);
 		release_firmware(adev->mc.fw);
 		adev->mc.fw = NULL;
 	}
@@ -950,12 +948,12 @@  static int gmc_v8_0_sw_init(void *handle)
 	if (r) {
 		adev->need_dma32 = true;
 		dma_bits = 32;
-		printk(KERN_WARNING "amdgpu: No suitable DMA available.\n");
+		pr_warn("amdgpu: No suitable DMA available\n");
 	}
 	r = pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(dma_bits));
 	if (r) {
 		pci_set_consistent_dma_mask(adev->pdev, DMA_BIT_MASK(32));
-		printk(KERN_WARNING "amdgpu: No coherent DMA available.\n");
+		pr_warn("amdgpu: No coherent DMA available\n");
 	}
 
 	r = gmc_v8_0_init_microcode(adev);
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c b/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
index 896be64b7013..a881cf475a19 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v2_4.c
@@ -170,9 +170,7 @@  static int sdma_v2_4_init_microcode(struct amdgpu_device *adev)
 
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "sdma_v2_4: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("sdma_v2_4: Failed to load firmware \"%s\"\n", fw_name);
 		for (i = 0; i < adev->sdma.num_instances; i++) {
 			release_firmware(adev->sdma.instance[i].fw);
 			adev->sdma.instance[i].fw = NULL;
diff --git a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
index 011800f621c6..255fab5fceee 100644
--- a/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
+++ b/drivers/gpu/drm/amd/amdgpu/sdma_v3_0.c
@@ -321,9 +321,7 @@  static int sdma_v3_0_init_microcode(struct amdgpu_device *adev)
 	}
 out:
 	if (err) {
-		printk(KERN_ERR
-		       "sdma_v3_0: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("sdma_v3_0: Failed to load firmware \"%s\"\n", fw_name);
 		for (i = 0; i < adev->sdma.num_instances; i++) {
 			release_firmware(adev->sdma.instance[i].fw);
 			adev->sdma.instance[i].fw = NULL;
diff --git a/drivers/gpu/drm/amd/include/amd_pcie_helpers.h b/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
index 5725bf85eacc..7e5a965450c7 100644
--- a/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
+++ b/drivers/gpu/drm/amd/include/amd_pcie_helpers.h
@@ -82,7 +82,7 @@  static inline uint16_t get_pcie_lane_support(uint32_t pcie_lane_width_cap,
 
 	switch (pcie_lane_width_cap) {
 	case 0:
-		printk(KERN_ERR "No valid PCIE lane width reported");
+		pr_err("No valid PCIE lane width reported\n");
 		break;
 	case CAIL_PCIE_LINK_WIDTH_SUPPORT_X1:
 		new_pcie_lanes = 1;
@@ -126,7 +126,7 @@  static inline uint16_t get_pcie_lane_support(uint32_t pcie_lane_width_cap,
 							}
 						}
 						if (j > 7)
-							printk(KERN_ERR "Cannot find a valid PCIE lane width!");
+							pr_err("Cannot find a valid PCIE lane width!\n");
 					}
 				}
 				break;
diff --git a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
index f75ee33ec5bb..418f87e805b2 100644
--- a/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
+++ b/drivers/gpu/drm/amd/powerplay/hwmgr/smu7_hwmgr.c
@@ -1544,7 +1544,7 @@  static int smu7_get_evv_voltages(struct pp_hwmgr *hwmgr)
 					if (vddc >= 2000 || vddc == 0)
 						return -EINVAL;
 				} else {
-					pr_warning("failed to retrieving EVV voltage!\n");
+					pr_warn("failed to retrieving EVV voltage!\n");
 					continue;
 				}
 
diff --git a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
index 072880130cfb..f3f9ebb631a5 100644
--- a/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
+++ b/drivers/gpu/drm/amd/powerplay/inc/pp_debug.h
@@ -37,7 +37,7 @@ 
 #define PP_ASSERT_WITH_CODE(cond, msg, code)	\
 	do {					\
 		if (!(cond)) {			\
-			pr_warning("%s\n", msg);	\
+			pr_warn("%s\n", msg);	\
 			code;			\
 		}				\
 	} while (0)
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
index 0f7a77b7312e..5450f5ef8e89 100644
--- a/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/fiji_smc.c
@@ -2131,7 +2131,7 @@  uint32_t fiji_get_offsetof(uint32_t type, uint32_t member)
 			return offsetof(SMU73_Discrete_DpmTable, LowSclkInterruptThreshold);
 		}
 	}
-	pr_warning("can't get the offset of type %x member %x\n", type, member);
+	pr_warn("can't get the offset of type %x member %x\n", type, member);
 	return 0;
 }
 
@@ -2156,7 +2156,7 @@  uint32_t fiji_get_mac_definition(uint32_t value)
 		return SMU73_MAX_LEVELS_MVDD;
 	}
 
-	pr_warning("can't get the mac of %x\n", value);
+	pr_warn("can't get the mac of %x\n", value);
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
index ad82161df831..b609ab7e9b97 100644
--- a/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/iceland_smc.c
@@ -122,7 +122,7 @@  static void iceland_initialize_power_tune_defaults(struct pp_hwmgr *hwmgr)
 		break;
 	default:
 		smu_data->power_tune_defaults = &defaults_iceland;
-		pr_warning("Unknown V.I. Device ID.\n");
+		pr_warn("Unknown V.I. Device ID\n");
 		break;
 	}
 	return;
@@ -378,7 +378,7 @@  static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
 			return -EINVAL);
 
 	if (NULL == hwmgr->dyn_state.cac_leakage_table) {
-		pr_warning("CAC Leakage Table does not exist, using vddc.\n");
+		pr_warn("CAC Leakage Table does not exist, using vddc\n");
 		return 0;
 	}
 
@@ -394,7 +394,7 @@  static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
 				*lo = hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * VOLTAGE_SCALE;
 				*hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage * VOLTAGE_SCALE);
 			} else {
-				pr_warning("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table.\n");
+				pr_warn("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index, using maximum index from CAC table\n");
 				*lo = hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Vddc * VOLTAGE_SCALE;
 				*hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Leakage * VOLTAGE_SCALE);
 			}
@@ -414,7 +414,7 @@  static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
 					*lo = hwmgr->dyn_state.cac_leakage_table->entries[v_index].Vddc * VOLTAGE_SCALE;
 					*hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[v_index].Leakage) * VOLTAGE_SCALE;
 				} else {
-					pr_warning("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index in second look up, using maximum index from CAC table.");
+					pr_warn("Index from SCLK/VDDC Dependency Table exceeds the CAC Leakage Table index in second look up, using maximum index from CAC table\n");
 					*lo = hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Vddc * VOLTAGE_SCALE;
 					*hi = (uint16_t)(hwmgr->dyn_state.cac_leakage_table->entries[hwmgr->dyn_state.cac_leakage_table->count - 1].Leakage * VOLTAGE_SCALE);
 				}
@@ -423,7 +423,7 @@  static int iceland_get_std_voltage_value_sidd(struct pp_hwmgr *hwmgr,
 		}
 
 		if (!vol_found)
-			pr_warning("Unable to get std_vddc from SCLK/VDDC Dependency Table, using vddc.\n");
+			pr_warn("Unable to get std_vddc from SCLK/VDDC Dependency Table, using vddc\n");
 	}
 
 	return 0;
@@ -2146,7 +2146,7 @@  uint32_t iceland_get_offsetof(uint32_t type, uint32_t member)
 			return offsetof(SMU71_Discrete_DpmTable, LowSclkInterruptThreshold);
 		}
 	}
-	pr_warning("can't get the offset of type %x member %x\n", type, member);
+	pr_warn("can't get the offset of type %x member %x\n", type, member);
 	return 0;
 }
 
@@ -2169,7 +2169,7 @@  uint32_t iceland_get_mac_definition(uint32_t value)
 		return SMU71_MAX_LEVELS_MVDD;
 	}
 
-	pr_warning("can't get the mac of %x\n", value);
+	pr_warn("can't get the mac of %x\n", value);
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
index 80e2329a1b9e..c9ca7a1ac171 100644
--- a/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/polaris10_smc.c
@@ -2184,7 +2184,7 @@  uint32_t polaris10_get_offsetof(uint32_t type, uint32_t member)
 			return offsetof(SMU74_Discrete_DpmTable, LowSclkInterruptThreshold);
 		}
 	}
-	pr_warning("can't get the offset of type %x member %x\n", type, member);
+	pr_warn("can't get the offset of type %x member %x\n", type, member);
 	return 0;
 }
 
@@ -2211,7 +2211,7 @@  uint32_t polaris10_get_mac_definition(uint32_t value)
 		return SMU7_UVD_MCLK_HANDSHAKE_DISABLE;
 	}
 
-	pr_warning("can't get the mac of %x\n", value);
+	pr_warn("can't get the mac of %x\n", value);
 	return 0;
 }
 
diff --git a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
index 331b0aba4a13..4d1439a498eb 100644
--- a/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
+++ b/drivers/gpu/drm/amd/powerplay/smumgr/tonga_smc.c
@@ -2657,7 +2657,7 @@  uint32_t tonga_get_offsetof(uint32_t type, uint32_t member)
 			return offsetof(SMU72_Discrete_DpmTable, LowSclkInterruptThreshold);
 		}
 	}
-	pr_warning("can't get the offset of type %x member %x\n", type, member);
+	pr_warn("can't get the offset of type %x member %x\n", type, member);
 	return 0;
 }
 
@@ -2681,7 +2681,7 @@  uint32_t tonga_get_mac_definition(uint32_t value)
 	case SMU_MAX_LEVELS_MVDD:
 		return SMU72_MAX_LEVELS_MVDD;
 	}
-	pr_warning("can't get the mac value %x\n", value);
+	pr_warn("can't get the mac value %x\n", value);
 
 	return 0;
 }
diff --git a/drivers/gpu/drm/radeon/atom.c b/drivers/gpu/drm/radeon/atom.c
index f66c33dd21a3..6a2e091aa7b6 100644
--- a/drivers/gpu/drm/radeon/atom.c
+++ b/drivers/gpu/drm/radeon/atom.c
@@ -170,7 +170,7 @@  static uint32_t atom_iio_execute(struct atom_context *ctx, int base,
 		case ATOM_IIO_END:
 			return temp;
 		default:
-			printk(KERN_INFO "Unknown IIO opcode.\n");
+			pr_info("Unknown IIO opcode\n");
 			return 0;
 		}
 }
@@ -194,22 +194,19 @@  static uint32_t atom_get_src_int(atom_exec_context *ctx, uint8_t attr,
 			val = gctx->card->reg_read(gctx->card, idx);
 			break;
 		case ATOM_IO_PCI:
-			printk(KERN_INFO
-			       "PCI registers are not implemented.\n");
+			pr_info("PCI registers are not implemented\n");
 			return 0;
 		case ATOM_IO_SYSIO:
-			printk(KERN_INFO
-			       "SYSIO registers are not implemented.\n");
+			pr_info("SYSIO registers are not implemented\n");
 			return 0;
 		default:
 			if (!(gctx->io_mode & 0x80)) {
-				printk(KERN_INFO "Bad IO mode.\n");
+				pr_info("Bad IO mode\n");
 				return 0;
 			}
 			if (!gctx->iio[gctx->io_mode & 0x7F]) {
-				printk(KERN_INFO
-				       "Undefined indirect IO read method %d.\n",
-				       gctx->io_mode & 0x7F);
+				pr_info("Undefined indirect IO read method %d\n",
+					gctx->io_mode & 0x7F);
 				return 0;
 			}
 			val =
@@ -473,22 +470,19 @@  static void atom_put_dst(atom_exec_context *ctx, int arg, uint8_t attr,
 				gctx->card->reg_write(gctx->card, idx, val);
 			break;
 		case ATOM_IO_PCI:
-			printk(KERN_INFO
-			       "PCI registers are not implemented.\n");
+			pr_info("PCI registers are not implemented\n");
 			return;
 		case ATOM_IO_SYSIO:
-			printk(KERN_INFO
-			       "SYSIO registers are not implemented.\n");
+			pr_info("SYSIO registers are not implemented\n");
 			return;
 		default:
 			if (!(gctx->io_mode & 0x80)) {
-				printk(KERN_INFO "Bad IO mode.\n");
+				pr_info("Bad IO mode\n");
 				return;
 			}
 			if (!gctx->iio[gctx->io_mode & 0xFF]) {
-				printk(KERN_INFO
-				       "Undefined indirect IO write method %d.\n",
-				       gctx->io_mode & 0x7F);
+				pr_info("Undefined indirect IO write method %d\n",
+					gctx->io_mode & 0x7F);
 				return;
 			}
 			atom_iio_execute(gctx, gctx->iio[gctx->io_mode & 0xFF],
@@ -819,17 +813,17 @@  static void atom_op_postcard(atom_exec_context *ctx, int *ptr, int arg)
 
 static void atom_op_repeat(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_restorereg(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_savereg(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static void atom_op_setdatablock(atom_exec_context *ctx, int *ptr, int arg)
@@ -992,7 +986,7 @@  static void atom_op_switch(atom_exec_context *ctx, int *ptr, int arg)
 			}
 			(*ptr) += 2;
 		} else {
-			printk(KERN_INFO "Bad case.\n");
+			pr_info("Bad case\n");
 			return;
 		}
 	(*ptr) += 2;
@@ -1026,7 +1020,7 @@  static void atom_op_xor(atom_exec_context *ctx, int *ptr, int arg)
 
 static void atom_op_debug(atom_exec_context *ctx, int *ptr, int arg)
 {
-	printk(KERN_INFO "unimplemented!\n");
+	pr_info("unimplemented!\n");
 }
 
 static struct {
@@ -1280,14 +1274,14 @@  struct atom_context *atom_parse(struct card_info *card, void *bios)
 	ctx->bios = bios;
 
 	if (CU16(0) != ATOM_BIOS_MAGIC) {
-		printk(KERN_INFO "Invalid BIOS magic.\n");
+		pr_info("Invalid BIOS magic\n");
 		kfree(ctx);
 		return NULL;
 	}
 	if (strncmp
 	    (CSTR(ATOM_ATI_MAGIC_PTR), ATOM_ATI_MAGIC,
 	     strlen(ATOM_ATI_MAGIC))) {
-		printk(KERN_INFO "Invalid ATI magic.\n");
+		pr_info("Invalid ATI magic\n");
 		kfree(ctx);
 		return NULL;
 	}
@@ -1296,7 +1290,7 @@  struct atom_context *atom_parse(struct card_info *card, void *bios)
 	if (strncmp
 	    (CSTR(base + ATOM_ROM_MAGIC_PTR), ATOM_ROM_MAGIC,
 	     strlen(ATOM_ROM_MAGIC))) {
-		printk(KERN_INFO "Invalid ATOM magic.\n");
+		pr_info("Invalid ATOM magic\n");
 		kfree(ctx);
 		return NULL;
 	}
@@ -1320,7 +1314,7 @@  struct atom_context *atom_parse(struct card_info *card, void *bios)
 			break;
 		}
 	}
-	printk(KERN_INFO "ATOM BIOS: %s\n", name);
+	pr_info("ATOM BIOS: %s\n", name);
 
 	return ctx;
 }
diff --git a/drivers/gpu/drm/radeon/cik.c b/drivers/gpu/drm/radeon/cik.c
index f6ff41a0eed6..b338d58e281f 100644
--- a/drivers/gpu/drm/radeon/cik.c
+++ b/drivers/gpu/drm/radeon/cik.c
@@ -2069,8 +2069,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->pfp_fw->size != pfp_req_size) {
-			printk(KERN_ERR
-			       "cik_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->pfp_fw->size, fw_name);
 			err = -EINVAL;
 			goto out;
@@ -2078,8 +2077,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 	} else {
 		err = radeon_ucode_validate(rdev->pfp_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2095,16 +2093,14 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->me_fw->size != me_req_size) {
-			printk(KERN_ERR
-			       "cik_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->me_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->me_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2120,16 +2116,14 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->ce_fw->size != ce_req_size) {
-			printk(KERN_ERR
-			       "cik_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->ce_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->ce_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2145,16 +2139,14 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->mec_fw->size != mec_req_size) {
-			printk(KERN_ERR
-			       "cik_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->mec_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->mec_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2185,16 +2177,14 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->rlc_fw->size != rlc_req_size) {
-			printk(KERN_ERR
-			       "cik_rlc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_rlc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->rlc_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->rlc_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2210,16 +2200,14 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->sdma_fw->size != sdma_req_size) {
-			printk(KERN_ERR
-			       "cik_sdma: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("cik_sdma: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->sdma_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->sdma_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "cik_fw: validation failed for firmware \"%s\"\n",
+			pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -2242,8 +2230,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 			}
 			if ((rdev->mc_fw->size != mc_req_size) &&
 			    (rdev->mc_fw->size != mc2_req_size)){
-				printk(KERN_ERR
-				       "cik_mc: Bogus length %zu in firmware \"%s\"\n",
+				pr_err("cik_mc: Bogus length %zu in firmware \"%s\"\n",
 				       rdev->mc_fw->size, fw_name);
 				err = -EINVAL;
 			}
@@ -2251,8 +2238,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 		} else {
 			err = radeon_ucode_validate(rdev->mc_fw);
 			if (err) {
-				printk(KERN_ERR
-				       "cik_fw: validation failed for firmware \"%s\"\n",
+				pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 				       fw_name);
 				goto out;
 			} else {
@@ -2269,23 +2255,20 @@  static int cik_init_microcode(struct radeon_device *rdev)
 			snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
 			err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
 			if (err) {
-				printk(KERN_ERR
-				       "smc: error loading firmware \"%s\"\n",
+				pr_err("smc: error loading firmware \"%s\"\n",
 				       fw_name);
 				release_firmware(rdev->smc_fw);
 				rdev->smc_fw = NULL;
 				err = 0;
 			} else if (rdev->smc_fw->size != smc_req_size) {
-				printk(KERN_ERR
-				       "cik_smc: Bogus length %zu in firmware \"%s\"\n",
+				pr_err("cik_smc: Bogus length %zu in firmware \"%s\"\n",
 				       rdev->smc_fw->size, fw_name);
 				err = -EINVAL;
 			}
 		} else {
 			err = radeon_ucode_validate(rdev->smc_fw);
 			if (err) {
-				printk(KERN_ERR
-				       "cik_fw: validation failed for firmware \"%s\"\n",
+				pr_err("cik_fw: validation failed for firmware \"%s\"\n",
 				       fw_name);
 				goto out;
 			} else {
@@ -2297,7 +2280,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 	if (new_fw == 0) {
 		rdev->new_fw = false;
 	} else if (new_fw < num_fw) {
-		printk(KERN_ERR "ci_fw: mixing new and old firmware!\n");
+		pr_err("ci_fw: mixing new and old firmware!\n");
 		err = -EINVAL;
 	} else {
 		rdev->new_fw = true;
@@ -2306,8 +2289,7 @@  static int cik_init_microcode(struct radeon_device *rdev)
 out:
 	if (err) {
 		if (err != -EINVAL)
-			printk(KERN_ERR
-			       "cik_cp: Failed to load firmware \"%s\"\n",
+			pr_err("cik_cp: Failed to load firmware \"%s\"\n",
 			       fw_name);
 		release_firmware(rdev->pfp_fw);
 		rdev->pfp_fw = NULL;
diff --git a/drivers/gpu/drm/radeon/evergreen.c b/drivers/gpu/drm/radeon/evergreen.c
index 0b6b5766216f..d1b1e0cc3c25 100644
--- a/drivers/gpu/drm/radeon/evergreen.c
+++ b/drivers/gpu/drm/radeon/evergreen.c
@@ -2486,7 +2486,7 @@  void evergreen_pcie_gart_tlb_flush(struct radeon_device *rdev)
 		tmp = RREG32(VM_CONTEXT0_REQUEST_RESPONSE);
 		tmp = (tmp & RESPONSE_TYPE_MASK) >> RESPONSE_TYPE_SHIFT;
 		if (tmp == 2) {
-			printk(KERN_WARNING "[drm] r600 flush TLB failed\n");
+			pr_warn("[drm] r600 flush TLB failed\n");
 			return;
 		}
 		if (tmp) {
diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
index f8b05090232a..54324330b91f 100644
--- a/drivers/gpu/drm/radeon/evergreen_cs.c
+++ b/drivers/gpu/drm/radeon/evergreen_cs.c
@@ -1061,8 +1061,7 @@  static int evergreen_packet0_check(struct radeon_cs_parser *p,
 		}
 		break;
 	default:
-		printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
-		       reg, idx);
+		pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
 		return -EINVAL;
 	}
 	return 0;
@@ -2776,7 +2775,7 @@  int evergreen_cs_parse(struct radeon_cs_parser *p)
 	} while (p->idx < p->chunk_ib->length_dw);
 #if 0
 	for (r = 0; r < p->ib.length_dw; r++) {
-		printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
+		pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
 		mdelay(1);
 	}
 #endif
@@ -3215,7 +3214,7 @@  int evergreen_dma_cs_parse(struct radeon_cs_parser *p)
 	} while (p->idx < p->chunk_ib->length_dw);
 #if 0
 	for (r = 0; r < p->ib->length_dw; r++) {
-		printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
+		pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
 		mdelay(1);
 	}
 #endif
diff --git a/drivers/gpu/drm/radeon/ni.c b/drivers/gpu/drm/radeon/ni.c
index a0d4a0522fdc..9eccd0c81d88 100644
--- a/drivers/gpu/drm/radeon/ni.c
+++ b/drivers/gpu/drm/radeon/ni.c
@@ -774,8 +774,7 @@  int ni_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->pfp_fw->size != pfp_req_size) {
-		printk(KERN_ERR
-		       "ni_cp: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("ni_cp: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->pfp_fw->size, fw_name);
 		err = -EINVAL;
 		goto out;
@@ -786,8 +785,7 @@  int ni_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->me_fw->size != me_req_size) {
-		printk(KERN_ERR
-		       "ni_cp: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("ni_cp: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->me_fw->size, fw_name);
 		err = -EINVAL;
 	}
@@ -797,8 +795,7 @@  int ni_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->rlc_fw->size != rlc_req_size) {
-		printk(KERN_ERR
-		       "ni_rlc: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("ni_rlc: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->rlc_fw->size, fw_name);
 		err = -EINVAL;
 	}
@@ -810,8 +807,7 @@  int ni_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->mc_fw->size != mc_req_size) {
-			printk(KERN_ERR
-			       "ni_mc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("ni_mc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->mc_fw->size, fw_name);
 			err = -EINVAL;
 		}
@@ -821,15 +817,12 @@  int ni_init_microcode(struct radeon_device *rdev)
 		snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
 		err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
 		if (err) {
-			printk(KERN_ERR
-			       "smc: error loading firmware \"%s\"\n",
-			       fw_name);
+			pr_err("smc: error loading firmware \"%s\"\n", fw_name);
 			release_firmware(rdev->smc_fw);
 			rdev->smc_fw = NULL;
 			err = 0;
 		} else if (rdev->smc_fw->size != smc_req_size) {
-			printk(KERN_ERR
-			       "ni_mc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("ni_mc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->mc_fw->size, fw_name);
 			err = -EINVAL;
 		}
@@ -838,8 +831,7 @@  int ni_init_microcode(struct radeon_device *rdev)
 out:
 	if (err) {
 		if (err != -EINVAL)
-			printk(KERN_ERR
-			       "ni_cp: Failed to load firmware \"%s\"\n",
+			pr_err("ni_cp: Failed to load firmware \"%s\"\n",
 			       fw_name);
 		release_firmware(rdev->pfp_fw);
 		rdev->pfp_fw = NULL;
diff --git a/drivers/gpu/drm/radeon/r100.c b/drivers/gpu/drm/radeon/r100.c
index e3399310d41d..aefca0b03f38 100644
--- a/drivers/gpu/drm/radeon/r100.c
+++ b/drivers/gpu/drm/radeon/r100.c
@@ -1042,11 +1042,9 @@  static int r100_cp_init_microcode(struct radeon_device *rdev)
 
 	err = request_firmware(&rdev->me_fw, fw_name, rdev->dev);
 	if (err) {
-		printk(KERN_ERR "radeon_cp: Failed to load firmware \"%s\"\n",
-		       fw_name);
+		pr_err("radeon_cp: Failed to load firmware \"%s\"\n", fw_name);
 	} else if (rdev->me_fw->size % 8) {
-		printk(KERN_ERR
-		       "radeon_cp: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("radeon_cp: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->me_fw->size, fw_name);
 		err = -EINVAL;
 		release_firmware(rdev->me_fw);
@@ -1087,8 +1085,7 @@  static void r100_cp_load_microcode(struct radeon_device *rdev)
 	int i, size;
 
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 
 	if (rdev->me_fw) {
@@ -1246,8 +1243,7 @@  void r100_cp_disable(struct radeon_device *rdev)
 	WREG32(RADEON_CP_CSQ_CNTL, 0);
 	WREG32(R_000770_SCRATCH_UMSK, 0);
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 }
 
@@ -1881,8 +1877,7 @@  static int r100_packet0_check(struct radeon_cs_parser *p,
 		track->tex_dirty = true;
 		break;
 	default:
-		printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
-		       reg, idx);
+		pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
 		return -EINVAL;
 	}
 	return 0;
@@ -2486,8 +2481,7 @@  int r100_gui_wait_for_idle(struct radeon_device *rdev)
 	uint32_t tmp;
 
 	if (r100_rbbm_fifo_wait_for_entry(rdev, 64)) {
-		printk(KERN_WARNING "radeon: wait for empty RBBM fifo failed !"
-		       " Bad things might happen.\n");
+		pr_warn("radeon: wait for empty RBBM fifo failed! Bad things might happen.\n");
 	}
 	for (i = 0; i < rdev->usec_timeout; i++) {
 		tmp = RREG32(RADEON_RBBM_STATUS);
diff --git a/drivers/gpu/drm/radeon/r200.c b/drivers/gpu/drm/radeon/r200.c
index c70e6d5bcd19..c22321cc5a41 100644
--- a/drivers/gpu/drm/radeon/r200.c
+++ b/drivers/gpu/drm/radeon/r200.c
@@ -537,8 +537,7 @@  int r200_packet0_check(struct radeon_cs_parser *p,
 		track->tex_dirty = true;
 		break;
 	default:
-		printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
-		       reg, idx);
+		pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
 		return -EINVAL;
 	}
 	return 0;
diff --git a/drivers/gpu/drm/radeon/r300.c b/drivers/gpu/drm/radeon/r300.c
index 7e417d8dc733..21161aa8acbf 100644
--- a/drivers/gpu/drm/radeon/r300.c
+++ b/drivers/gpu/drm/radeon/r300.c
@@ -387,8 +387,7 @@  static void r300_gpu_init(struct radeon_device *rdev)
 	WREG32(R300_GB_TILE_CONFIG, gb_tile_config);
 
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 
 	tmp = RREG32(R300_DST_PIPE_CONFIG);
@@ -399,14 +398,12 @@  static void r300_gpu_init(struct radeon_device *rdev)
 	       R300_DC_DC_DISABLE_IGNORE_PE);
 
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 	if (r300_mc_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait MC idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
 	}
-	DRM_INFO("radeon: %d quad pipes, %d Z pipes initialized.\n",
+	DRM_INFO("radeon: %d quad pipes, %d Z pipes initialized\n",
 		 rdev->num_gb_pipes, rdev->num_z_pipes);
 }
 
@@ -1165,7 +1162,7 @@  static int r300_packet0_check(struct radeon_cs_parser *p,
 	}
 	return 0;
 fail:
-	printk(KERN_ERR "Forbidden register 0x%04X in cs at %d (val=%08x)\n",
+	pr_err("Forbidden register 0x%04X in cs at %d (val=%08x)\n",
 	       reg, idx, idx_value);
 	return -EINVAL;
 }
diff --git a/drivers/gpu/drm/radeon/r420.c b/drivers/gpu/drm/radeon/r420.c
index 2828605aef3f..3eb0c4f9f796 100644
--- a/drivers/gpu/drm/radeon/r420.c
+++ b/drivers/gpu/drm/radeon/r420.c
@@ -92,8 +92,7 @@  void r420_pipes_init(struct radeon_device *rdev)
 	       (1 << 2) | (1 << 3));
 	/* add idle wait as per freedesktop.org bug 24041 */
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 	/* get max number of pipes */
 	gb_pipe_select = RREG32(R400_GB_PIPE_SELECT);
@@ -128,8 +127,7 @@  void r420_pipes_init(struct radeon_device *rdev)
 	tmp |= R300_TILE_SIZE_16 | R300_ENABLE_TILING;
 	WREG32(R300_GB_TILE_CONFIG, tmp);
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 
 	tmp = RREG32(R300_DST_PIPE_CONFIG);
@@ -141,8 +139,7 @@  void r420_pipes_init(struct radeon_device *rdev)
 	       R300_DC_DC_DISABLE_IGNORE_PE);
 
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while programming pipes. Bad things might happen.\n");
 	}
 
 	if (rdev->family == CHIP_RV530) {
diff --git a/drivers/gpu/drm/radeon/r520.c b/drivers/gpu/drm/radeon/r520.c
index 98d6053c36c6..074cf752faef 100644
--- a/drivers/gpu/drm/radeon/r520.c
+++ b/drivers/gpu/drm/radeon/r520.c
@@ -86,8 +86,7 @@  static void r520_gpu_init(struct radeon_device *rdev)
 	      (((gb_pipe_select >> 8) & 0xF) << 4);
 	WREG32_PLL(0x000D, tmp);
 	if (r520_mc_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait MC idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
 	}
 }
 
diff --git a/drivers/gpu/drm/radeon/r600.c b/drivers/gpu/drm/radeon/r600.c
index a951881c2a50..0a085176e79b 100644
--- a/drivers/gpu/drm/radeon/r600.c
+++ b/drivers/gpu/drm/radeon/r600.c
@@ -1094,7 +1094,7 @@  void r600_pcie_gart_tlb_flush(struct radeon_device *rdev)
 		tmp = RREG32(VM_CONTEXT0_REQUEST_RESPONSE);
 		tmp = (tmp & RESPONSE_TYPE_MASK) >> RESPONSE_TYPE_SHIFT;
 		if (tmp == 2) {
-			printk(KERN_WARNING "[drm] r600 flush TLB failed\n");
+			pr_warn("[drm] r600 flush TLB failed\n");
 			return;
 		}
 		if (tmp) {
@@ -2550,8 +2550,7 @@  int r600_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->pfp_fw->size != pfp_req_size) {
-		printk(KERN_ERR
-		       "r600_cp: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("r600_cp: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->pfp_fw->size, fw_name);
 		err = -EINVAL;
 		goto out;
@@ -2562,8 +2561,7 @@  int r600_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->me_fw->size != me_req_size) {
-		printk(KERN_ERR
-		       "r600_cp: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("r600_cp: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->me_fw->size, fw_name);
 		err = -EINVAL;
 	}
@@ -2573,8 +2571,7 @@  int r600_init_microcode(struct radeon_device *rdev)
 	if (err)
 		goto out;
 	if (rdev->rlc_fw->size != rlc_req_size) {
-		printk(KERN_ERR
-		       "r600_rlc: Bogus length %zu in firmware \"%s\"\n",
+		pr_err("r600_rlc: Bogus length %zu in firmware \"%s\"\n",
 		       rdev->rlc_fw->size, fw_name);
 		err = -EINVAL;
 	}
@@ -2583,15 +2580,12 @@  int r600_init_microcode(struct radeon_device *rdev)
 		snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", smc_chip_name);
 		err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
 		if (err) {
-			printk(KERN_ERR
-			       "smc: error loading firmware \"%s\"\n",
-			       fw_name);
+			pr_err("smc: error loading firmware \"%s\"\n", fw_name);
 			release_firmware(rdev->smc_fw);
 			rdev->smc_fw = NULL;
 			err = 0;
 		} else if (rdev->smc_fw->size != smc_req_size) {
-			printk(KERN_ERR
-			       "smc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("smc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->smc_fw->size, fw_name);
 			err = -EINVAL;
 		}
@@ -2600,8 +2594,7 @@  int r600_init_microcode(struct radeon_device *rdev)
 out:
 	if (err) {
 		if (err != -EINVAL)
-			printk(KERN_ERR
-			       "r600_cp: Failed to load firmware \"%s\"\n",
+			pr_err("r600_cp: Failed to load firmware \"%s\"\n",
 			       fw_name);
 		release_firmware(rdev->pfp_fw);
 		rdev->pfp_fw = NULL;
diff --git a/drivers/gpu/drm/radeon/r600_cs.c b/drivers/gpu/drm/radeon/r600_cs.c
index 595a19736458..97fd58e97043 100644
--- a/drivers/gpu/drm/radeon/r600_cs.c
+++ b/drivers/gpu/drm/radeon/r600_cs.c
@@ -932,8 +932,7 @@  static int r600_packet0_check(struct radeon_cs_parser *p,
 		}
 		break;
 	default:
-		printk(KERN_ERR "Forbidden register 0x%04X in cs at %d\n",
-		       reg, idx);
+		pr_err("Forbidden register 0x%04X in cs at %d\n", reg, idx);
 		return -EINVAL;
 	}
 	return 0;
@@ -2320,7 +2319,7 @@  int r600_cs_parse(struct radeon_cs_parser *p)
 	} while (p->idx < p->chunk_ib->length_dw);
 #if 0
 	for (r = 0; r < p->ib.length_dw; r++) {
-		printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
+		pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
 		mdelay(1);
 	}
 #endif
@@ -2528,7 +2527,7 @@  int r600_dma_cs_parse(struct radeon_cs_parser *p)
 	} while (p->idx < p->chunk_ib->length_dw);
 #if 0
 	for (r = 0; r < p->ib->length_dw; r++) {
-		printk(KERN_INFO "%05d  0x%08X\n", r, p->ib.ptr[r]);
+		pr_info("%05d  0x%08X\n", r, p->ib.ptr[r]);
 		mdelay(1);
 	}
 #endif
diff --git a/drivers/gpu/drm/radeon/radeon.h b/drivers/gpu/drm/radeon/radeon.h
index 44e0c5ed6418..2e400dc414e3 100644
--- a/drivers/gpu/drm/radeon/radeon.h
+++ b/drivers/gpu/drm/radeon/radeon.h
@@ -2535,7 +2535,8 @@  static inline struct radeon_fence *to_radeon_fence(struct dma_fence *f)
 #define WREG16(reg, v) writew(v, (rdev->rmmio) + (reg))
 #define RREG32(reg) r100_mm_rreg(rdev, (reg), false)
 #define RREG32_IDX(reg) r100_mm_rreg(rdev, (reg), true)
-#define DREG32(reg) printk(KERN_INFO "REGISTER: " #reg " : 0x%08X\n", r100_mm_rreg(rdev, (reg), false))
+#define DREG32(reg) pr_info("REGISTER: " #reg " : 0x%08X\n",	\
+			    r100_mm_rreg(rdev, (reg), false))
 #define WREG32(reg, v) r100_mm_wreg(rdev, (reg), (v), false)
 #define WREG32_IDX(reg, v) r100_mm_wreg(rdev, (reg), (v), true)
 #define REG_SET(FIELD, v) (((v) << FIELD##_SHIFT) & FIELD##_MASK)
diff --git a/drivers/gpu/drm/radeon/radeon_atpx_handler.c b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
index 0ae13cd2adda..40be4068ca69 100644
--- a/drivers/gpu/drm/radeon/radeon_atpx_handler.c
+++ b/drivers/gpu/drm/radeon/radeon_atpx_handler.c
@@ -577,8 +577,8 @@  static bool radeon_atpx_detect(void)
 
 	if (has_atpx && vga_count == 2) {
 		acpi_get_name(radeon_atpx_priv.atpx.handle, ACPI_FULL_PATHNAME, &buffer);
-		printk(KERN_INFO "vga_switcheroo: detected switching method %s handle\n",
-		       acpi_method_name);
+		pr_info("vga_switcheroo: detected switching method %s handle\n",
+			acpi_method_name);
 		radeon_atpx_priv.atpx_detected = true;
 		radeon_atpx_priv.bridge_pm_usable = d3_supported;
 		radeon_atpx_init();
diff --git a/drivers/gpu/drm/radeon/radeon_audio.c b/drivers/gpu/drm/radeon/radeon_audio.c
index b214663b370d..aaacac190d26 100644
--- a/drivers/gpu/drm/radeon/radeon_audio.c
+++ b/drivers/gpu/drm/radeon/radeon_audio.c
@@ -576,9 +576,9 @@  static void radeon_audio_calc_cts(unsigned int clock, int *CTS, int *N, int freq
 
 	/* Check that we are in spec (not always possible) */
 	if (n < (128*freq/1500))
-		printk(KERN_WARNING "Calculated ACR N value is too small. You may experience audio problems.\n");
+		pr_warn("Calculated ACR N value is too small. You may experience audio problems.\n");
 	if (n > (128*freq/300))
-		printk(KERN_WARNING "Calculated ACR N value is too large. You may experience audio problems.\n");
+		pr_warn("Calculated ACR N value is too large. You may experience audio problems.\n");
 
 	*N = n;
 	*CTS = cts;
diff --git a/drivers/gpu/drm/radeon/radeon_clocks.c b/drivers/gpu/drm/radeon/radeon_clocks.c
index c1135feb93c1..e55146cdf543 100644
--- a/drivers/gpu/drm/radeon/radeon_clocks.c
+++ b/drivers/gpu/drm/radeon/radeon_clocks.c
@@ -111,7 +111,7 @@  static bool radeon_read_clocks_OF(struct drm_device *dev)
 		return false;
 	val = of_get_property(dp, "ATY,RefCLK", NULL);
 	if (!val || !*val) {
-		printk(KERN_WARNING "radeonfb: No ATY,RefCLK property !\n");
+		pr_warn("radeonfb: No ATY,RefCLK property !\n");
 		return false;
 	}
 	p1pll->reference_freq = p2pll->reference_freq = (*val) / 10;
diff --git a/drivers/gpu/drm/radeon/radeon_device.c b/drivers/gpu/drm/radeon/radeon_device.c
index 4b0c388be3f5..dde59c36ba92 100644
--- a/drivers/gpu/drm/radeon/radeon_device.c
+++ b/drivers/gpu/drm/radeon/radeon_device.c
@@ -1249,7 +1249,7 @@  static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
 	if (state == VGA_SWITCHEROO_ON) {
 		unsigned d3_delay = dev->pdev->d3_delay;
 
-		printk(KERN_INFO "radeon: switched on\n");
+		pr_info("radeon: switched on\n");
 		/* don't suspend or resume card normally */
 		dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
 
@@ -1263,7 +1263,7 @@  static void radeon_switcheroo_set_state(struct pci_dev *pdev, enum vga_switchero
 		dev->switch_power_state = DRM_SWITCH_POWER_ON;
 		drm_kms_helper_poll_enable(dev);
 	} else {
-		printk(KERN_INFO "radeon: switched off\n");
+		pr_info("radeon: switched off\n");
 		drm_kms_helper_poll_disable(dev);
 		dev->switch_power_state = DRM_SWITCH_POWER_CHANGING;
 		radeon_suspend_kms(dev, true, true, false);
@@ -1410,12 +1410,12 @@  int radeon_device_init(struct radeon_device *rdev,
 	if (r) {
 		rdev->need_dma32 = true;
 		dma_bits = 32;
-		printk(KERN_WARNING "radeon: No suitable DMA available.\n");
+		pr_warn("radeon: No suitable DMA available\n");
 	}
 	r = pci_set_consistent_dma_mask(rdev->pdev, DMA_BIT_MASK(dma_bits));
 	if (r) {
 		pci_set_consistent_dma_mask(rdev->pdev, DMA_BIT_MASK(32));
-		printk(KERN_WARNING "radeon: No coherent DMA available.\n");
+		pr_warn("radeon: No coherent DMA available\n");
 	}
 
 	/* Registers mapping */
diff --git a/drivers/gpu/drm/radeon/radeon_fb.c b/drivers/gpu/drm/radeon/radeon_fb.c
index 2be4fe9c7217..53aa94f705a7 100644
--- a/drivers/gpu/drm/radeon/radeon_fb.c
+++ b/drivers/gpu/drm/radeon/radeon_fb.c
@@ -149,8 +149,7 @@  static int radeonfb_create_pinned_object(struct radeon_fbdev *rfbdev,
 				       RADEON_GEM_DOMAIN_VRAM,
 				       0, true, &gobj);
 	if (ret) {
-		printk(KERN_ERR "failed to allocate framebuffer (%d)\n",
-		       aligned_size);
+		pr_err("failed to allocate framebuffer (%d)\n", aligned_size);
 		return -ENOMEM;
 	}
 	rbo = gem_to_radeon_bo(gobj);
diff --git a/drivers/gpu/drm/radeon/radeon_gem.c b/drivers/gpu/drm/radeon/radeon_gem.c
index 96683f5b2b1b..9b0b123ce079 100644
--- a/drivers/gpu/drm/radeon/radeon_gem.c
+++ b/drivers/gpu/drm/radeon/radeon_gem.c
@@ -106,7 +106,7 @@  static int radeon_gem_set_domain(struct drm_gem_object *gobj,
 	}
 	if (!domain) {
 		/* Do nothings */
-		printk(KERN_WARNING "Set domain without domain !\n");
+		pr_warn("Set domain without domain !\n");
 		return 0;
 	}
 	if (domain == RADEON_GEM_DOMAIN_CPU) {
@@ -116,7 +116,7 @@  static int radeon_gem_set_domain(struct drm_gem_object *gobj,
 			r = -EBUSY;
 
 		if (r < 0 && r != -EINTR) {
-			printk(KERN_ERR "Failed to wait for object: %li\n", r);
+			pr_err("Failed to wait for object: %li\n", r);
 			return r;
 		}
 	}
diff --git a/drivers/gpu/drm/radeon/radeon_test.c b/drivers/gpu/drm/radeon/radeon_test.c
index 791818165c76..4fdc7bda7a7d 100644
--- a/drivers/gpu/drm/radeon/radeon_test.c
+++ b/drivers/gpu/drm/radeon/radeon_test.c
@@ -246,7 +246,7 @@  static void radeon_do_test_moves(struct radeon_device *rdev, int flag)
 out_cleanup:
 	kfree(gtt_obj);
 	if (r) {
-		printk(KERN_WARNING "Error while testing BO move.\n");
+		pr_warn("Error while testing BO move\n");
 	}
 }
 
@@ -394,7 +394,7 @@  void radeon_test_ring_sync(struct radeon_device *rdev,
 		radeon_fence_unref(&fence2);
 
 	if (r)
-		printk(KERN_WARNING "Error while testing ring sync (%d).\n", r);
+		pr_warn("Error while testing ring sync (%d)\n", r);
 }
 
 static void radeon_test_ring_sync2(struct radeon_device *rdev,
@@ -504,7 +504,7 @@  static void radeon_test_ring_sync2(struct radeon_device *rdev,
 		radeon_fence_unref(&fenceB);
 
 	if (r)
-		printk(KERN_WARNING "Error while testing ring sync (%d).\n", r);
+		pr_warn("Error while testing ring sync (%d)\n", r);
 }
 
 static bool radeon_test_sync_possible(struct radeon_ring *ringA,
diff --git a/drivers/gpu/drm/radeon/rs400.c b/drivers/gpu/drm/radeon/rs400.c
index 34e3235f41d2..4121209c183e 100644
--- a/drivers/gpu/drm/radeon/rs400.c
+++ b/drivers/gpu/drm/radeon/rs400.c
@@ -255,8 +255,8 @@  static void rs400_gpu_init(struct radeon_device *rdev)
 	/* FIXME: is this correct ? */
 	r420_pipes_init(rdev);
 	if (rs400_mc_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "rs400: Failed to wait MC idle while "
-		       "programming pipes. Bad things might happen. %08x\n", RREG32(RADEON_MC_STATUS));
+		pr_warn("rs400: Failed to wait MC idle while programming pipes. Bad things might happen. %08x\n",
+			RREG32(RADEON_MC_STATUS));
 	}
 }
 
diff --git a/drivers/gpu/drm/radeon/rs690.c b/drivers/gpu/drm/radeon/rs690.c
index 6bc44c24e837..1bae33e43f3c 100644
--- a/drivers/gpu/drm/radeon/rs690.c
+++ b/drivers/gpu/drm/radeon/rs690.c
@@ -52,8 +52,7 @@  static void rs690_gpu_init(struct radeon_device *rdev)
 	/* FIXME: is this correct ? */
 	r420_pipes_init(rdev);
 	if (rs690_mc_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait MC idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
 	}
 }
 
diff --git a/drivers/gpu/drm/radeon/rv515.c b/drivers/gpu/drm/radeon/rv515.c
index c55d653aaf5f..ffbd2c006f60 100644
--- a/drivers/gpu/drm/radeon/rv515.c
+++ b/drivers/gpu/drm/radeon/rv515.c
@@ -154,8 +154,7 @@  static void rv515_gpu_init(struct radeon_device *rdev)
 	unsigned pipe_select_current, gb_pipe_select, tmp;
 
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "resetting GPU. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while resetting GPU. Bad things might happen.\n");
 	}
 	rv515_vga_render_disable(rdev);
 	r420_pipes_init(rdev);
@@ -166,12 +165,10 @@  static void rv515_gpu_init(struct radeon_device *rdev)
 	      (((gb_pipe_select >> 8) & 0xF) << 4);
 	WREG32_PLL(0x000D, tmp);
 	if (r100_gui_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait GUI idle while "
-		       "resetting GPU. Bad things might happen.\n");
+		pr_warn("Failed to wait GUI idle while resetting GPU. Bad things might happen.\n");
 	}
 	if (rv515_mc_wait_for_idle(rdev)) {
-		printk(KERN_WARNING "Failed to wait MC idle while "
-		       "programming pipes. Bad things might happen.\n");
+		pr_warn("Failed to wait MC idle while programming pipes. Bad things might happen.\n");
 	}
 }
 
diff --git a/drivers/gpu/drm/radeon/si.c b/drivers/gpu/drm/radeon/si.c
index 414776811e71..528e5a49a214 100644
--- a/drivers/gpu/drm/radeon/si.c
+++ b/drivers/gpu/drm/radeon/si.c
@@ -1762,8 +1762,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->pfp_fw->size != pfp_req_size) {
-			printk(KERN_ERR
-			       "si_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->pfp_fw->size, fw_name);
 			err = -EINVAL;
 			goto out;
@@ -1771,8 +1770,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 	} else {
 		err = radeon_ucode_validate(rdev->pfp_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1788,16 +1786,14 @@  static int si_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->me_fw->size != me_req_size) {
-			printk(KERN_ERR
-			       "si_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->me_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->me_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1813,16 +1809,14 @@  static int si_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->ce_fw->size != ce_req_size) {
-			printk(KERN_ERR
-			       "si_cp: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_cp: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->ce_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->ce_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1838,16 +1832,14 @@  static int si_init_microcode(struct radeon_device *rdev)
 		if (err)
 			goto out;
 		if (rdev->rlc_fw->size != rlc_req_size) {
-			printk(KERN_ERR
-			       "si_rlc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_rlc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->rlc_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->rlc_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1871,8 +1863,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 		}
 		if ((rdev->mc_fw->size != mc_req_size) &&
 		    (rdev->mc_fw->size != mc2_req_size)) {
-			printk(KERN_ERR
-			       "si_mc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_mc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->mc_fw->size, fw_name);
 			err = -EINVAL;
 		}
@@ -1880,8 +1871,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 	} else {
 		err = radeon_ucode_validate(rdev->mc_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1900,23 +1890,19 @@  static int si_init_microcode(struct radeon_device *rdev)
 		snprintf(fw_name, sizeof(fw_name), "radeon/%s_smc.bin", chip_name);
 		err = request_firmware(&rdev->smc_fw, fw_name, rdev->dev);
 		if (err) {
-			printk(KERN_ERR
-			       "smc: error loading firmware \"%s\"\n",
-			       fw_name);
+			pr_err("smc: error loading firmware \"%s\"\n", fw_name);
 			release_firmware(rdev->smc_fw);
 			rdev->smc_fw = NULL;
 			err = 0;
 		} else if (rdev->smc_fw->size != smc_req_size) {
-			printk(KERN_ERR
-			       "si_smc: Bogus length %zu in firmware \"%s\"\n",
+			pr_err("si_smc: Bogus length %zu in firmware \"%s\"\n",
 			       rdev->smc_fw->size, fw_name);
 			err = -EINVAL;
 		}
 	} else {
 		err = radeon_ucode_validate(rdev->smc_fw);
 		if (err) {
-			printk(KERN_ERR
-			       "si_cp: validation failed for firmware \"%s\"\n",
+			pr_err("si_cp: validation failed for firmware \"%s\"\n",
 			       fw_name);
 			goto out;
 		} else {
@@ -1927,7 +1913,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 	if (new_fw == 0) {
 		rdev->new_fw = false;
 	} else if (new_fw < 6) {
-		printk(KERN_ERR "si_fw: mixing new and old firmware!\n");
+		pr_err("si_fw: mixing new and old firmware!\n");
 		err = -EINVAL;
 	} else {
 		rdev->new_fw = true;
@@ -1935,8 +1921,7 @@  static int si_init_microcode(struct radeon_device *rdev)
 out:
 	if (err) {
 		if (err != -EINVAL)
-			printk(KERN_ERR
-			       "si_cp: Failed to load firmware \"%s\"\n",
+			pr_err("si_cp: Failed to load firmware \"%s\"\n",
 			       fw_name);
 		release_firmware(rdev->pfp_fw);
 		rdev->pfp_fw = NULL;