Cosmetic changes
authorDaniel Röhrig <daniel.roehrig@mailbox.tu-berlin.de>
Fri, 22 Nov 2013 16:01:43 +0000 (17:01 +0100)
committerDaniel Röhrig <daniel.roehrig@mailbox.tu-berlin.de>
Fri, 22 Nov 2013 16:01:43 +0000 (17:01 +0100)
Just some refactoring.

framework/fw.c
framework/fw.h
framework/fw_comm.c
framework/modules/rr_module.c
framework/os/linux/Makefile
framework/os/os.h

index 03319e9..c3b418b 100644 (file)
@@ -1,5 +1,15 @@
-#include "fw.h"
+/*
+ * fw.c
+ *
+ * Framework functionality is mapped to os functions.
+ *
+ *      Author: Daniel Roehrig, Christian Rene Sechting
+ */
+#include <fw.h>
+#include <os/os.h>
+
 struct fw_task *fw_idle_task;
+int fw_module_count;
 
 
 void *fw_malloc(fw_size_t size)
@@ -34,19 +44,14 @@ fw_task_t *fw_current_task(int cpu)
        return NULL;
 }
 
-int fw_module_count;
 int fw_register_module(){
        return (fw_module_count+1) == FW_MAX_MODULES ? -1 : fw_module_count++;
 }
 
-int init_sched_pub_sub(void);
-int init_sched_rr_module(void);
 void fw_init()
 {
        fw_module_count = 0;
-
        os_init_os();
-
        fw_init_comm();
        fw_modules_init();
 }
index 2ee3375..7faa2f8 100644 (file)
@@ -1,7 +1,6 @@
 #ifndef FW_H_
 #define FW_H_
 
-#include <os/os.h>
 #include <fw_types.h>
 #include <fw_list.h>
 #include <fw_task.h>
@@ -13,8 +12,7 @@
  *
  * @param formatstring The format string to print on screen.
  */
-#define fw_printf(fmt, ...)  os_printf(fmt ,## __VA_ARGS__)
-
+void fw_printf(const char* formatstring, ...);
 /**
  * \brief Dynamic allocation of memory.
  *
index b628ed5..4b3bfd0 100644 (file)
@@ -1,5 +1,12 @@
-#include "fw_comm.h"
-
+/*
+ * fw_comm.c
+ *
+ * Simplistic Pub-Sub-System
+ *
+ *             Author: Anselm Busse, Daniel Roehrig, Christian Rene Sechting
+ */
+
+#include <fw_comm.h>
 #include <fw.h>
 
 fw_list_head_t topics[LAST-FIRST];
@@ -21,13 +28,6 @@ void fw_subscribe(topic_t topic, void *callback)
 void fw_notify(topic_t topic, void *data)
 {
        struct fw_topic_element *iter;
-
-       //TODO: Kill this also
-       /*if(data!=NULL){
-               struct fw_task *task = register_task(data);
-               data = task;
-       }*/
-
        fw_list_for_each_entry(iter, &topics[topic], list)
                ((void (*)(void *))iter->callback)(data);
 }
index fc29d45..6058e50 100644 (file)
@@ -8,7 +8,7 @@
 
 //#include "mod_sched_framework.h"
 #include <fw.h>
-#include "../fw_comm.h"
+#include <fw_comm.h>
 //void subscribe_to_topic(int i,void *dingens);
 
 struct rq{
index 04c7ec1..00d1b9a 100644 (file)
@@ -8,8 +8,8 @@ OBJ+=$(MOD_SCHED_DIR)/os/linux/fw_task_linux.o
 mod_sched_build: $(OBJ)
        @echo "--- Scheduler-Module Linking... ---"
        $(LD) -o kernel/sched.new/mod_sched.o $(OBJ) -r
-       # Use printk whenever os_printf is called
-       objcopy --redefine-sym os_printf=printk kernel/sched.new/mod_sched.o
+       # Use printk whenever fw_printf is called
+       objcopy --redefine-sym fw_printf=printk kernel/sched.new/mod_sched.o
        @echo "--- Scheduler-Module Linked! ---"                
        
 %.o: %.c
index 1cb6ab0..43426e2 100644 (file)
@@ -9,15 +9,8 @@
 #ifndef OS_H_
 #define OS_H_
 
-#include "../fw_types.h"
-#include "../fw_task.h"
-
-/**
- * \brief A printf function.
- *
- * @param formatstring The format string to print on screen.
- */
-void os_printf(const char* formatstring, ...);
+#include <fw_types.h>
+#include <fw_task.h>
 
 /**
  * \brief Dynamic allocation of memory.