Skip to contentSkip to conversation

compile fixes for kernel driver for VICE coprocessor

PATCH New Andrew Randrianasulu 1 participant 1 message Nov 17, 2008

Subsystems (2)
lore
Share & tools
lore
#0 lore raw

PATCH 1/1 compile fixes for kernel driver for VICE coprocessor

Some function renaming and headers moving. Now it compiles (64 bit kernel, 2.6.28-rc5), but my changes very well may be done in wrong way.

Because i'm total newbie in kernel development - can I ask few question?
First - i've changed KSEG0ADDR -> CKSEG0ADDR unconditionally. probably bad idea, but before i add some if's around it - may be someone point me to better way for doing this? Second - i'm totally unsure about vice_vma_nopage function. probably i changed it in wrong way?
Third - eliminating <asm/pci.h> was only way for me to compile it. Vice.h includes <linux/pci.h>, hopefully it enough? If not - correct me, please.

I don't have hardware, so compile-tested only (crosscompiler - gcc-3.4.6. it gives me few warnings, may be under more recent compiler these warnings become errors?).

drivers/media/Kconfig +2 -0
index 93ea201..30978d1 100644
@@ -136,4 +136,6 @@ config USB_DABUSB
 	  module will be called dabusb.
 endif # DAB
 
+source "drivers/media/vice/Kconfig"
+
 endmenu
drivers/media/Makefile +1 -1
index 09a829d..5f8ad2e 100644
@@ -2,7 +2,7 @@
 # Makefile for the kernel multimedia device drivers.
 #
 
-obj-y += common/ video/
+obj-y += common/ video/ vice/
 
 obj-$(CONFIG_VIDEO_DEV) += radio/
 obj-$(CONFIG_DVB_CORE)  += dvb/
drivers/media/vice/Kconfig new file +35 -0
new file mode 100644
index 0000000..cad3376
@@ -0,0 +1,35 @@
+#
+# O2 VICE Engine configuration
+#
+
+config O2_VICE
+    tristate "O2 VICE Engine Support"
+    depends on SGI_IP32
+    ---help---
+      This option enables O2 VICE Engine support.
+      VICE stands for Video Image Compression Engine. This is very powerfull
+      piece of silicon, that can greatly speed up lots of graphics, vide, or
+      sound related tasks. To be able to use it, you will also need special
+      library, that can be found at <insert URL here>
+
+config O2_VICE_DBGG
+    bool "VICE Debugger Support. READ HELP!"
+    depends on O2_VICE
+    ---help---
+      This option enables features of VICE driver needed to debug VICE library.
+      This is probably serious security risk. You don't need it. If you think
+      you do, you are wrong. Say NO.
+
+config O2_VICE_DBG
+    bool "You seem to insist... did you read help? Yes? No? READ HELP!"
+    depends on O2_VICE_DBGG
+    ---help---
+      You are still here? Didn't I just tell you that it is not needed?
+      Or do you want to say you *legally* obtained information needed for
+      programming VICE? That you got all the tools needed?
+      You can get bitutils patch for dealing with BSP code at
+      http://www.total-knowledge.com/progs/mips/vice/binutils-bsp.diff
+      (but it doesn't work). You will also need a patch for compiling
+      MSP code.
+
+      Sigh... You've been warned...
drivers/media/vice/Makefile new file +8 -0
new file mode 100644
index 0000000..354b44b
@@ -0,0 +1,8 @@
+#
+# drivers/char/o2vice/Makefile
+#
+# Makefile for the O2 VICE Engine driver.
+#
+
+obj-$(CONFIG_O2_VICE)	+= main.o msp.o bsp.o dma.o
+#obj-$(CONFIG_O2_VICE_DBG) += vicedebug.o
drivers/media/vice/bsp.c new file +181 -0

181 lines not on this page · raw message

drivers/media/vice/dma.c new file +179 -0

179 lines not on this page · raw message

drivers/media/vice/main.c new file +419 -0

419 lines not on this page · raw message

drivers/media/vice/msp.c new file +160 -0

160 lines not on this page · raw message

include/linux/vice.h new file +451 -0

451 lines not on this page · raw message