Changeset 21b6307 in mainline for kernel/genarch/src/drivers/ns16550/ns16550.c

Timestamp:

2012-10-31T18:26:45Z (12 years ago)

Author:

Martin Sucha <sucha14@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

3da166f0

Parents:

6bbe470

Message:

Merge ns16550_output into ns16550_init.

File:

• kernel/genarch/src/drivers/ns16550/ns16550.c (modified) (3 diffs)

kernel/genarch/src/drivers/ns16550/ns16550.c

@@ -102 +102 @@
  *
  * @param dev      Addrress of the beginning of the device in I/O space.
- * @param devno    Device number.
  * @param inr      Interrupt number.
  * @param cir      Clear interrupt function.
  * @param cir_arg  First argument to cir.
+ * @param output   Where to store pointer to the output device
+ *                 or NULL if the caller is not interested in
+ *                 writing to the serial port.
  *
  * @return Keyboard instance or NULL on failure.
  *
  */
-ns16550_instance_t *ns16550_init(ns16550_t *dev, inr_t inr, cir_t cir, void *cir_arg)
+ns16550_instance_t *ns16550_init(ns16550_t *dev, inr_t inr, cir_t cir,
+    void *cir_arg, outdev_t **output)
 {
         ns16550_instance_t *instance
@@ -118 +121 @@
                 instance->input = NULL;
                 instance->output = NULL;
+                
+                if (output) {
+                        instance->output = malloc(sizeof(outdev_t),
+                            FRAME_ATOMIC);
+                        if (!instance->output) {
+                                free(instance);
+                                return NULL;
+                        }
+                        
+                        outdev_initialize("ns16550", instance->output,
+                            &ns16550_ops);
+                        instance->output->data = instance;
+                        *output = instance->output;
+                }
                 
                 irq_initialize(&instance->irq);
@@ -153 +170 @@
 }
 
-outdev_t *ns16550_output(ns16550_instance_t *instance)
-{
-        ASSERT(instance);
-        
-        if (instance->output == NULL) {
-                instance->output = malloc(sizeof(outdev_t),
-                    FRAME_ATOMIC);
-                if (instance->output) {
-                        outdev_initialize("ns16550", instance->output,
-                            &ns16550_ops);
-                        instance->output->data = instance;
-                }
-        }
-        
-        return instance->output;
-}
-
 /** @}
  */