static void ipoctal_irq_rx(struct ipoctal_channel *channel, u8 sr)
 {
        struct tty_port *port = &channel->tty_port;
-       unsigned char value;
-       unsigned char flag;
-       u8 isr;
+       u8 isr, value, flag;
 
        do {
                value = ioread8(&channel->regs->r.rhr);
 
 static void ipoctal_irq_tx(struct ipoctal_channel *channel)
 {
-       unsigned char value;
        unsigned int *pointer_write = &channel->pointer_write;
+       u8 value;
 
        if (channel->nb_bytes == 0)
                return;
        return res;
 }
 
-static inline int ipoctal_copy_write_buffer(struct ipoctal_channel *channel,
-                                           const u8 *buf, int count)
+static inline size_t ipoctal_copy_write_buffer(struct ipoctal_channel *channel,
+                                              const u8 *buf, size_t count)
 {
        unsigned long flags;
-       int i;
+       size_t i;
        unsigned int *pointer_read = &channel->pointer_read;
 
        /* Copy the bytes from the user buffer to the internal one */
                                 size_t count)
 {
        struct ipoctal_channel *channel = tty->driver_data;
-       unsigned int char_copied;
+       size_t char_copied;
 
        char_copied = ipoctal_copy_write_buffer(channel, buf, count);