@@ -117,7 +117,7 @@ static unsigned long initial_pmd_value __initdata = 0;
  */
 void __init init_default_cache_policy(unsigned long pmd)
 {
- int i;
+ unsigned i;

  initial_pmd_value = pmd;

@@ -140,20 +140,22 @@ void __init init_default_cache_policy(unsigned long pmd)
  */
 static int __init early_cachepolicy(char *p)
 {
- int i, selected = -1;
+ size_t i, selected = 0;

  for (i = 0; i < ARRAY_SIZE(cache_policies); i++) {
- int len = strlen(cache_policies[i].policy);
+ size_t len = strlen(cache_policies[i].policy);

  if (memcmp(p, cache_policies[i].policy, len) == 0) {
  selected = i;
- break;
+ goto no_err;
  }
  }

- if (selected == -1)
- pr_err("ERROR: unknown or unsupported cache policy\n");
+ pr_err("ERROR: unknown or unsupported cache policy\n");

+no_err:
+ if (selected != cachepolicy) {
+ unsigned long cr;
  /*
  * This restriction is partly to do with the way we boot; it is
  * unpredictable to have memory mapped using two different sets of

@@ -161,14 +163,12 @@ static int __init early_cachepolicy(char *p)
  * change these attributes once the initial assembly has setup the
  * page tables.
  */
- if (cpu_architecture() >= CPU_ARCH_ARMv6 && selected != cachepolicy) {
- pr_warn("Only cachepolicy=%s supported on ARMv6 and later\n",
- cache_policies[cachepolicy].policy);
- return 0;
- }
-
- if (selected != cachepolicy) {
- unsigned long cr = __clear_cr(cache_policies[selected].cr_mask);
+ if (cpu_architecture() >= CPU_ARCH_ARMv6) {
+ pr_warn("Only cachepolicy=%s supported on ARMv6 and later\n",
+ cache_policies[cachepolicy].policy);
+ return 0;
+ }
+ cr = __clear_cr(cache_policies[selected].cr_mask);
  cachepolicy = selected;
  flush_cache_all();
  set_cr(cr);

@@ -445,7 +445,7 @@ static void __init build_mem_type_table(void)
  unsigned int cr = get_cr();
  pteval_t user_pgprot, kern_pgprot, vecs_pgprot;
  int cpu_arch = cpu_architecture();
- int i;
+ size_t i;

  if (cpu_arch < CPU_ARCH_ARMv6) {
 #if defined(CONFIG_CPU_DCACHE_DISABLE)

@@ -723,9 +723,9 @@ static const char coverage_register_lookup[16] = {
 unsigned coverage_start_registers(const struct decode_header *h)
 {
  unsigned regs = 0;
- int i;
+ unsigned i;
  for (i = 0; i < 20; i += 4) {
- int r = (h->type_regs.bits >> (DECODE_TYPE_BITS + i)) & 0xf;
+ unsigned r = (h->type_regs.bits >> (DECODE_TYPE_BITS + i)) & 0xf;
  regs |= coverage_register_lookup[r] << i;
  }
  return regs;

@@ -774,12 +774,12 @@ static int coverage_start(const union decode_item *table)
 static void
 coverage_add_registers(struct coverage_entry *entry, kprobe_opcode_t insn)
 {
- int regs = entry->header->type_regs.bits >> DECODE_TYPE_BITS;
+ u32 regs = entry->header->type_regs.bits >> DECODE_TYPE_BITS;
  int i;
  for (i = 0; i < 20; i += 4) {
  enum decode_reg_type reg_type = (regs >> i) & 0xf;
  int reg = (insn >> i) & 0xf;
- int flag;
+ unsigned flag;

  if (!reg_type)
  continue;

@@ -227,7 +227,7 @@ static void vfp_raise_sigfpe(unsigned int sicode,

@@ -700,7 +700,7 @@ static struct undef_hook vfp_kmode_exception_hook[] = {{

 static int __init vfp_kmode_exception_hook_init(void)
 {
- int i;
+ size_t i;

  for (i = 0; i < ARRAY_SIZE(vfp_kmode_exception_hook); i++)
  register_undef_hook(&vfp_kmode_exception_hook[i]);

@@ -92,7 +92,7 @@ int set_foreign_p2m_mapping(struct

@@ -137,7 +137,7 @@ int clear_foreign_p2m_mapping(struct

@@ -809,7 +809,7 @@ static int alchemy_gpic_suspend(void)
 static void alchemy_gpic_resume(void)
 {
  void __iomem *base = (void __iomem *)KSEG1ADDR(AU1300_GPIC_PHYS_ADDR);
- int i;
+ u32 i;

  /* disable all first */
  __raw_writel(~0UL, base + AU1300_GPIC_IDIS + 0x0);

@@ -111,7 +111,7 @@ static inline unsigned char str2hexnum(unsigned char c)

 static inline void str2eaddr(unsigned char *ea, unsigned char *str)
 {
- int i;
+ unsigned i;

  for (i = 0; i < 6; i++) {
  unsigned char num;

@@ -28,7 +28,7 @@ static int bcsr_csc_base; /* linux-irq of first

@@ -122,10 +122,10 @@ int __init

@@ -144,7 +144,7 @@ static void __init add_adam2_var(char *name, char *value)

 static int __init parse_psp_env(void *psp_env_base)
 {
- int i, n;
+ unsigned i, n;
  char *name, *value;
  struct psp_env_chunk *chunks = (struct psp_env_chunk *)psp_env_data;

@@ -171,7 +171,7 @@ static int __init parse_psp_env(void *psp_env_base)

 static void __init ar7_init_env(struct env_var *env)
 {
- int i;
+ unsigned i;
  struct psbl_rec *psbl = (struct psbl_rec *)(KSEG1ADDR(0x14000300));
  void *psp_env = (void *)KSEG1ADDR(psbl->env_base);

@@ -25,13 +25,9 @@ void puts(const char *s)

 void puthex(unsigned long long val)
 {
-
- unsigned char buf[10];
- int i;
- for (i = 7; i >= 0; i--) {
- buf[i] = "0123456789ABCDEF"[val & 0x0F];
- val >>= 4;
- }
- buf[8] = '\0';
- puts(buf);
+    char buf[9];
+    size_t i = sizeof(buf) - 1;
+    for (buf[i] = '\0'; i; val >>= 4)
+        buf[--i] = "0123456789ABCDEF"[val & 0xF];
+    puts(buf);
 }

@@ -47,7 +47,7 @@ static char __init *move_firmware_args(int argc,

@@ -86,7 +86,7 @@ void __init prom_init_cmdline(int argc, LONG *argv)

  while (actr < argc) {
  for (i = 0; i < ARRAY_SIZE(ignored); i++) {
- int len = strlen(ignored[i]);
+ size_t len = strlen(ignored[i]);

  if (!strncmp(prom_argv(actr), ignored[i], len))
  goto pic_cont;

@@ -64,7 +64,7 @@ static int rs_write(struct tty_struct * tty,
 static void rs_poll(struct timer_list *unused)
 {
  struct tty_port *port = &serial_port;
- int i = 0;
+ size_t i = 0;
  int rd = 1;
  unsigned char c;

@@ -192,7 +192,7 @@ late_initcall(rs_init);
 static void iss_console_write(struct console *co, const char *s,