Changes in uspace/drv/ns8250/ns8250.c [33dbbd2:49698fa] in mainline

File:

• uspace/drv/ns8250/ns8250.c (modified) (15 diffs)

uspace/drv/ns8250/ns8250.c

@@ -123 +123 @@
 static void delete_ns8250_dev_data(ns8250_dev_data_t *data)
 {
-        if (data != NULL)
+        if (NULL != data)
                 free(data);
 }
@@ -251 +251 @@
 static void ns8250_dev_cleanup(device_t *dev)
 {
-        if (dev->driver_data != NULL) {
+        if (NULL != dev->driver_data) {
                 delete_ns8250_dev_data((ns8250_dev_data_t*) dev->driver_data);
                 dev->driver_data = NULL;
@@ -332 +332 @@
         /* Allocate driver data for the device. */
         ns8250_dev_data_t *data = create_ns8250_dev_data();
-        if (data == NULL)
+        if (NULL == data)
                 return ENOMEM;
         dev->driver_data = data;
@@ -436 +436 @@
         /* Enable interrupt globally. */
         res = interrupt_enable(data->irq);
-        if (res != EOK)
+        if (EOK != res)
                 return res;
         
@@ -480 +480 @@
         uint8_t div_low, div_high;
         
-        if (baud_rate < 50 || MAX_BAUD_RATE % baud_rate != 0) {
+        if (50 > baud_rate || 0 != MAX_BAUD_RATE % baud_rate) {
                 printf(NAME ": error - somebody tried to set invalid baud rate "
                     "%d\n", baud_rate);
@@ -534 +534 @@
  * @param stop_bits     The number of stop bits used (one or two).
  */
-static void ns8250_port_get_com_props(ioport8_t *port, unsigned int *parity,
+static void
+ns8250_port_get_com_props(ioport8_t *port, unsigned int *parity,
     unsigned int *word_length, unsigned int *stop_bits)
 {
@@ -571 +572 @@
  *                      is invalid.
  */
-static int ns8250_port_set_com_props(ioport8_t *port, unsigned int parity,
+static int
+ns8250_port_set_com_props(ioport8_t *port, unsigned int parity,
     unsigned int word_length, unsigned int stop_bits)
 {
@@ -689 +691 @@
  * @param dev           The serial port device.
  */
-static inline void ns8250_interrupt_handler(device_t *dev, ipc_callid_t iid,
-    ipc_call_t *icall)
+static inline void
+ns8250_interrupt_handler(device_t *dev, ipc_callid_t iid, ipc_call_t *icall)
 {
         ns8250_read_from_device(dev);
@@ -724 +726 @@
  * @param dev           The serial port device.
  */
-static int ns8250_add_device(device_t *dev)
+static int ns8250_add_device(device_t *dev) 
 {
         printf(NAME ": ns8250_add_device %s (handle = %d)\n",
@@ -730 +732 @@
         
         int res = ns8250_dev_initialize(dev);
-        if (res != EOK)
+        if (EOK != res)
                 return res;
         
@@ -748 +750 @@
         
         /* Register interrupt handler. */
-        if (ns8250_register_interrupt_handler(dev) != EOK) {
+        if (EOK != ns8250_register_interrupt_handler(dev)) {
                 printf(NAME ": failed to register interrupt handler.\n");
                 ns8250_dev_cleanup(dev);
@@ -756 +758 @@
         /* Enable interrupt. */
         res = ns8250_interrupt_enable(dev);
-        if (res != EOK) {
+        if (EOK != res) {
                 printf(NAME ": failed to enable the interrupt. Error code = "
                     "%d.\n", res);
@@ -857 +859 @@
  * @param stop_bits     The number of stop bits to be used.
  */
-static int ns8250_set_props(device_t *dev, unsigned int baud_rate,
-    unsigned int parity, unsigned int word_length, unsigned int stop_bits)
+static int
+ns8250_set_props(device_t *dev, unsigned int baud_rate, unsigned int parity,
+    unsigned int word_length, unsigned int stop_bits)
 {
         printf(NAME ": ns8250_set_props: baud rate %d, parity 0x%x, word "
@@ -871 +874 @@
         ns8250_port_interrupts_disable(port);
         ret = ns8250_port_set_baud_rate(port, baud_rate);
-        if (ret == EOK)
+        if (EOK == ret)
                 ret = ns8250_port_set_com_props(port, parity, word_length, stop_bits);
         ns8250_port_interrupts_enable(port);
@@ -884 +887 @@
  * Configure the parameters of the serial communication.
  */
-static void ns8250_default_handler(device_t *dev, ipc_callid_t callid,
-    ipc_call_t *call)
+static void
+ns8250_default_handler(device_t *dev, ipc_callid_t callid, ipc_call_t *call)
 {
         ipcarg_t method = IPC_GET_METHOD(*call);