| Message ID | c1d0b86774e43880aa0ae8ab8118082ad8d6cc0e.1488285953.git.joe@perches.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
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 --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;
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(-)