/*
         * Map greybus values to power_supply values.  Hopefully these are
-        * "identical" which should allow gcc to optomize the code away to
+        * "identical" which should allow gcc to optimize the code away to
         * nothing.
         */
        technology = le32_to_cpu(tech_response.technology);
 
        /*
         * Map greybus values to power_supply values.  Hopefully these are
-        * "identical" which should allow gcc to optomize the code away to
+        * "identical" which should allow gcc to optimize the code away to
         * nothing.
         */
        battery_status = le16_to_cpu(status_response.battery_status);
 
 
        u16                             op_cycle;
        struct list_head                operations;
-       struct list_head                pending;        /* awaiting reponse */
+       struct list_head                pending;        /* awaiting response */
 
        void                            *private;
 };
 
        id = gb_module_match_id(gmod, driver->id_table);
        if (id)
                return 1;
-       /* FIXME - Dyanmic ids? */
+       /* FIXME - Dynamic ids? */
        return 0;
 }
 
 
         *
         * This protocol is "needed" due to some hardware restrictions on the
         * current generation of Unipro controllers.  Think about it for a
-        * minute, this is a USB driver, talking to a Unipro bridge, impediance
+        * minute, this is a USB driver, talking to a Unipro bridge, impedance
         * mismatch is huge, yet the Unipro controller are even more
         * underpowered than this little USB controller.  We rely on the round
         * trip to keep stalls in the Unipro controllers from happening so that
 
 
 /*
  * Validate the given descriptor.  Its reported size must fit within
- * the number of bytes reamining, and it must have a recognized
+ * the number of bytes remaining, and it must have a recognized
  * type.  Check that the reported size is at least as big as what
  * we expect to see.  (It could be bigger, perhaps for a new version
  * of the format.)
 
 };
 
 /*
- * A Greybus module represents a user-replacable component on an Ara
+ * A Greybus module represents a user-replicable component on an Ara
  * phone.
  *
  * Create a gb_module structure to represent a discovered module.
 
 }
 
 /*
- * An operations's response message has arrived.  If no callback was
+ * An operation's response message has arrived.  If no callback was
  * supplied it was submitted for asynchronous completion, so we notify
  * any waiters.  Otherwise we assume calling the completion is enough
  * and nobody else will be waiting.
        if (ret < 0)
                gb_message_cancel(operation->request);
        return ret;
-
 }
 
 static void gb_operation_request_handle(struct gb_operation *operation)
 }
 
 /*
- * Map an enum gb_operation_status value (which is represted in a
- * message as a single back a single byte) to an appropriate Linux
- * negative errno.
+ * Map an enum gb_operation_status value (which is represented in a
+ * message as a single byte) to an appropriate Linux negative errno.
  */
 int gb_operation_status_map(u8 status)
 {
 
 /*
  * Create a Greybus operation to be sent over the given connection.
- * The request buffer will big enough for a payload of the given
+ * The request buffer will be big enough for a payload of the given
  * size.  Outgoing requests must specify the size of the response
  * buffer size, which must be sufficient to hold all expected
  * response data.
 /*
  * We've received data that appears to be an operation response
  * message.  Look up the operation, and record that we've received
- * its repsonse.
+ * its response.
  *
  * This is called in interrupt context, so just copy the incoming
  * data into the response buffer and handle the rest via workqueue.
 
        return NULL;
 }
 
-/* Returns true if protocol was succesfully registered, false otherwise */
+/* Returns true if protocol was successfully registered, false otherwise */
 bool gb_protocol_register(struct gb_protocol *protocol)
 {
        struct gb_protocol *existing;
  * XXX Currently this fails (and reports an error to the caller) if
  * XXX the protocol is currently in use.  We may want to forcefully
  * XXX kill off a protocol and all its active users at some point.
- * XXX But I think that's better handled by quescing modules that
+ * XXX But I think that's better handled by quiescing modules that
  * XXX have users and having those users drop their reference.
  *
  * Returns true if successful, false otherwise.
 
 
        tty_unregister_device(gb_tty_driver, gb_tty->minor);
 
-       /* FIXME - free transmit / recieve buffers */
+       /* FIXME - free transmit / receive buffers */
 
        tty_port_put(&gb_tty->port);