Blackfin: cleanup printk() usage in module code
authorMike Frysinger <vapier@gentoo.org>
Fri, 26 Jun 2009 00:35:24 +0000 (00:35 +0000)
committerMike Frysinger <vapier@gentoo.org>
Thu, 17 Sep 2009 01:28:48 +0000 (21:28 -0400)
Convert all printk() statements to use the common pr_xxx() funcs and use
the new pr_fmt() function to standardize all of the output.

Signed-off-by: Mike Frysinger <vapier@gentoo.org>
arch/blackfin/kernel/module.c

index bb940784cf2e84ad94ee00835dcc7cd344e8f4a1..b8f4c67eb2ec4d90c1efd5bcf2a6bfaac9ab233b 100644 (file)
@@ -27,6 +27,7 @@
  * 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA
  */
 
+#define pr_fmt(fmt) "module %s: " fmt
 
 #include <linux/moduleloader.h>
 #include <linux/elf.h>
@@ -72,9 +73,8 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l1_inst_sram_alloc(s->sh_size);
                        mod->arch.text_l1 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                      "module %s: L1 instruction memory allocation failed\n",
-                                      mod->name);
+                               pr_err("L1 inst memory allocation failed\n",
+                                       mod->name);
                                return -1;
                        }
                        dma_memcpy(dest, (void *)s->sh_addr, s->sh_size);
@@ -87,8 +87,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l1_data_sram_alloc(s->sh_size);
                        mod->arch.data_a_l1 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L1 data memory allocation failed\n",
+                               pr_err("L1 data memory allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -102,8 +101,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l1_data_sram_alloc(s->sh_size);
                        mod->arch.bss_a_l1 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L1 data memory allocation failed\n",
+                               pr_err("L1 data memory allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -115,8 +113,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l1_data_B_sram_alloc(s->sh_size);
                        mod->arch.data_b_l1 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L1 data memory allocation failed\n",
+                               pr_err("L1 data memory allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -128,8 +125,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l1_data_B_sram_alloc(s->sh_size);
                        mod->arch.bss_b_l1 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L1 data memory allocation failed\n",
+                               pr_err("L1 data memory allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -143,9 +139,8 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l2_sram_alloc(s->sh_size);
                        mod->arch.text_l2 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                      "module %s: L2 SRAM allocation failed\n",
-                                      mod->name);
+                               pr_err("L2 SRAM allocation failed\n",
+                                       mod->name);
                                return -1;
                        }
                        memcpy(dest, (void *)s->sh_addr, s->sh_size);
@@ -158,8 +153,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l2_sram_alloc(s->sh_size);
                        mod->arch.data_l2 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L2 SRAM allocation failed\n",
+                               pr_err("L2 SRAM allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -173,8 +167,7 @@ module_frob_arch_sections(Elf_Ehdr * hdr, Elf_Shdr * sechdrs,
                        dest = l2_sram_alloc(s->sh_size);
                        mod->arch.bss_l2 = dest;
                        if (dest == NULL) {
-                               printk(KERN_ERR
-                                       "module %s: L2 SRAM allocation failed\n",
+                               pr_err("L2 SRAM allocation failed\n",
                                        mod->name);
                                return -1;
                        }
@@ -190,7 +183,7 @@ int
 apply_relocate(Elf_Shdr * sechdrs, const char *strtab,
               unsigned int symindex, unsigned int relsec, struct module *me)
 {
-       printk(KERN_ERR "module %s: .rel unsupported\n", me->name);
+       pr_err(".rel unsupported\n", me->name);
        return -ENOEXEC;
 }
 
@@ -217,8 +210,8 @@ apply_relocate_add(Elf_Shdr * sechdrs, const char *strtab,
        uint16_t *location16;
        uint32_t value;
 
-       pr_debug("Applying relocate section %u to %u\n", relsec,
-              sechdrs[relsec].sh_info);
+       pr_debug("applying relocate section %u to %u\n", mod->name,
+               relsec, sechdrs[relsec].sh_info);
        for (i = 0; i < sechdrs[relsec].sh_size / sizeof(*rel); i++) {
                /* This is where to make the change */
                location16 =
@@ -231,21 +224,22 @@ apply_relocate_add(Elf_Shdr * sechdrs, const char *strtab,
                    + ELF32_R_SYM(rel[i].r_info);
                value = sym->st_value;
                value += rel[i].r_addend;
-               pr_debug("location is %x, value is %x type is %d \n",
-                        (unsigned int) location32, value,
-                        ELF32_R_TYPE(rel[i].r_info));
+
 #ifdef CONFIG_SMP
                if ((unsigned long)location16 >= COREB_L1_DATA_A_START) {
-                       printk(KERN_ERR "module %s: cannot relocate in L1: %u (SMP kernel)",
-                                      mod->name, ELF32_R_TYPE(rel[i].r_info));
+                       pr_err("cannot relocate in L1: %u (SMP kernel)",
+                               mod->name, ELF32_R_TYPE(rel[i].r_info));
                        return -ENOEXEC;
                }
 #endif
+
+               pr_debug("location is %lx, value is %x type is %d\n",
+                       mod->name, (unsigned long)location32, value,
+                       ELF32_R_TYPE(rel[i].r_info));
+
                switch (ELF32_R_TYPE(rel[i].r_info)) {
 
                case R_BFIN_LUIMM16:
-                       pr_debug("before %x after %x\n", *location16,
-                                      (value & 0xffff));
                        tmp = (value & 0xffff);
                        if ((unsigned long)location16 >= L1_CODE_START) {
                                dma_memcpy(location16, &tmp, 2);
@@ -253,8 +247,6 @@ apply_relocate_add(Elf_Shdr * sechdrs, const char *strtab,
                                *location16 = tmp;
                        break;
                case R_BFIN_HUIMM16:
-                       pr_debug("before %x after %x\n", *location16,
-                                      ((value >> 16) & 0xffff));
                        tmp = ((value >> 16) & 0xffff);
                        if ((unsigned long)location16 >= L1_CODE_START) {
                                dma_memcpy(location16, &tmp, 2);
@@ -265,7 +257,6 @@ apply_relocate_add(Elf_Shdr * sechdrs, const char *strtab,
                        *location16 = (value & 0xffff);
                        break;
                case R_BFIN_BYTE4_DATA:
-                       pr_debug("before %x after %x\n", *location32, value);
                        *location32 = value;
                        break;
                case R_BFIN_PCREL24:
@@ -273,12 +264,12 @@ apply_relocate_add(Elf_Shdr * sechdrs, const char *strtab,
                case R_BFIN_PCREL12_JUMP:
                case R_BFIN_PCREL12_JUMP_S:
                case R_BFIN_PCREL10:
-                       printk(KERN_ERR "module %s: Unsupported relocation: %u (no -mlong-calls?)\n"
+                       pr_err("unsupported relocation: %u (no -mlong-calls?)\n",
                                mod->name, ELF32_R_TYPE(rel[i].r_info));
                        return -ENOEXEC;
                default:
-                       printk(KERN_ERR "module %s: Unknown relocation: %u\n",
-                              mod->name, ELF32_R_TYPE(rel[i].r_info));
+                       pr_err("unknown relocation: %u\n", mod->name,
+                               ELF32_R_TYPE(rel[i].r_info));
                        return -ENOEXEC;
                }
        }