WIP: start adding interrupt handlers

This commit should not compile and was made just to save current state.
framepars and sensor_common were converted to platform dirver, some file
headers were updated and ad-hoc updates to doxygen comments were
introduced.

src/drivers/elphel/Makefile

@@ -8,7 +8,7 @@ obj-$(CONFIG_ELPHELDRVONMICROZED) += elphel393-mem.o
 obj-$(CONFIG_ELPHEL393_INIT)      += elphel393-init.o   
 
 obj-$(CONFIG_ELPHEL393)           += framepars.o
-obj-$(CONFIG_ELPHEL393)           += sensor_common.o
+obj-$(CONFIG_ELPHEL393)           += sensor_common.o x393.o
 obj-$(CONFIG_ELPHEL393)           += quantization_tables.o
 obj-$(CONFIG_ELPHEL393)           += circbuf.o jpeghead.o
 
@@ -18,4 +18,4 @@ obj-$(CONFIG_ELPHEL393)           += circbuf.o jpeghead.o
 #obj-$(CONFIG_ELPHEL393_EXTERNAL)  += fpars.o
 #obj-$(CONFIG_ELPHEL393_EXTERNAL)  += sc.o
 #obj-$(CONFIG_ELPHEL393_EXTERNAL)  += cbuf.o
-#obj-$(CONFIG_ELPHEL393_EXTERNAL)  += circbuf.o jpeghead.o quantization_tables.o 
+#obj-$(CONFIG_ELPHEL393_EXTERNAL)  += circbuf.o jpeghead.o quantization_tables.o 
\ No newline at end of file

src/drivers/elphel/framepars.c

+/** @file framepars.c */
 /*!********************************************************************************
 *! FILE NAME  : framepars.c
 *! DESCRIPTION: Handling of frame parameters, making use of FPGA i2c
@@ -207,6 +208,9 @@
 #include <linux/init.h>
 //#include <linux/autoconf.h>
 #include <linux/vmalloc.h>
+#include <linux/platform_device.h>
+#include <linux/of.h>
+#include <linux/of_device.h>
 
 //#include <asm/system.h>
 #include <asm/byteorder.h> // endians
@@ -230,7 +234,7 @@
 /// needed for lseek commands
 //#include "cxdma.h" // x313_dma_init
 //#include "cci2c.h" // to use void i2c_reset_wait(void), reset shadow static 'i2c_hardware_on'
-
+#include "x393_macro.h"
 
 /**
  * \def MDF1(x) optional debug output 
@@ -283,11 +287,11 @@
 //#define ELP_KERR(x) printk("%s:%d:%s: ERROR ",__FILE__,__LINE__,__FUNCTION__); x
 
 /**
- * \def X3X3_FRAMEPARS_DRIVER_NAME driver name to display
+ * \def FRAMEPARS_DRIVER_NAME driver name to display
  */
+#define  FRAMEPARS_DRIVER_NAME "Elphel (R) Model 393 Frame Parameters device driver"
 
-#define  X3X3_FRAMEPARS_DRIVER_NAME "Elphel (R) Model 353 Frame Parameters device driver"
-
+static const struct of_device_id elphel393_framepars_of_match[];
 static struct framepars_all_t sFrameParsAll __attribute__ ((aligned (PAGE_SIZE)));   ///< Sensor Parameters, currently 8 pages all and 2048 pages some, static struct
 unsigned long frameParsInitialized; /// set to 0 at startup, 1 after initialization that is triggered by setParsAtomic()
 #define  thisFrameNumber GLOBALPARS(G_THIS_FRAME) // Current frame number (may lag from the hardware)
@@ -332,7 +336,6 @@ int        framepars_release(struct inode *inode, struct file *filp);
 loff_t     framepars_lseek  (struct file * file, loff_t offset, int orig);
 ssize_t    framepars_write  (struct file * file, const char * buf, size_t count, loff_t *off);
 int        framepars_mmap   (struct file *file, struct vm_area_struct *vma);
-static int __init framepars_init(void);
 
 /**
  * @brief Reset hardware sequencers (i2c, command) and initialize framepars structure
@@ -1321,27 +1324,61 @@ int framepars_mmap (struct file *file, struct vm_area_struct *vma) {
 }
 
 /**
- * @brief framepars driver init
- * @return 0
+ * @brief framepars driver probing function
+ * @param[in]   pdev   pointer to \b platform_device structure
+ * @return      0 on success or negative error code otherwise
  */
-static int __init framepars_init(void) {
+static int framepars_init(struct platform_device *pdev)
+{
    int res;
+   struct device *dev = &pdev->dev;
+   const struct of_device_id *match;
+
+   /* sanity check */
+   match = of_match_device(elphel393_framepars_of_match, dev);
+   if (!match)
+	   return -EINVAL;
+
    init_framepars_ptr();
    initGlobalPars(); /// sets default debug if enabled - not anymore. Add here?
    initMultiPars(); /// just clear - needs to be called again when sensor is recognized
    frameParsInitialized=0;
+
    res = register_chrdev(FRAMEPARS_MAJOR, "framepars_operations", &framepars_fops);
    if(res < 0) {
      printk(KERN_ERR "\nframepars_init: couldn't get a major number %d.\n",FRAMEPARS_MAJOR);
      return res;
    }
    init_waitqueue_head(&framepars_wait_queue);
-   printk(X3X3_FRAMEPARS_DRIVER_NAME" - %d \n",FRAMEPARS_MAJOR);
+   dev_info(dev, "registered MAJOR: %d\n", FRAMEPARS_MAJOR);
+
    return 0;
 }
 
+static int framepars_remove(struct platform_device *pdev)
+{
+	unregister_chrdev(FRAMEPARS_MAJOR, "framepars_operations");
+
+	return 0;
+}
+
+static const struct of_device_id elphel393_framepars_of_match[] = {
+		{ .compatible = "elphel,elphel393-framepars-1.00" },
+		{ /* end of list */ }
+};
+MODULE_DEVICE_TABLE(of, elphel393_framepars_of_match);
+
+static struct platform_driver elphel393_framepars = {
+		.probe          = framepars_init,
+		.remove         = framepars_remove,
+		.driver = {
+				.name = FRAMEPARS_DRIVER_NAME,
+				.of_match_table = elphel393_framepars_of_match,
+		},
+};
+
+module_platform_driver(elphel393_framepars);
 
-module_init(framepars_init);
 MODULE_LICENSE("GPL");
 MODULE_AUTHOR("Andrey Filippov <andrey@elphel.com>.");
 MODULE_DESCRIPTION(X3X3_FRAMEPARS_DRIVER_NAME);

src/drivers/elphel/jpeghead.c

-/*!***************************************************************************
-*! FILE NAME  : jpeghead.c
-*! DESCRIPTION: handleng JPEG file headers
-*!
-*! Copyright (C) 2008 Elphel, Inc
-*! -----------------------------------------------------------------------------**
-*!
-*!  This program is free software: you can redistribute it and/or modify
-*!  it under the terms of the GNU General Public License as published by
-*!  the Free Software Foundation, either version 3 of the License, or
-*!  (at your option) any later version.
-*!
-*!  This program is distributed in the hope that it will be useful,
-*!  but WITHOUT ANY WARRANTY; without even the implied warranty of
-*!  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
-*!  GNU General Public License for more details.
-*!
-*!  You should have received a copy of the GNU General Public License
-*!  along with this program.  If not, see <http://www.gnu.org/licenses/>.
-*! -----------------------------------------------------------------------------**
-*!  $Log: jpeghead.c,v $
-*!  Revision 1.1.1.1  2008/11/27 20:04:01  elphel
-*!
-*!
-*!  Revision 1.10  2008/11/03 18:43:18  elphel
-*!  8.0.alpha12 with working apps/astreamer
-*!
-*!  Revision 1.9  2008/10/29 04:18:28  elphel
-*!  v.8.0.alpha10 made a separate structure for global parameters (not related to particular frames in a frame queue)
-*!
-*!  Revision 1.8  2008/10/12 16:46:22  elphel
-*!  snapshot
-*!
-*!  Revision 1.7  2008/10/06 08:31:08  elphel
-*!  snapshot, first images
-*!
-*!  Revision 1.6  2008/09/22 22:55:48  elphel
-*!  snapshot
-*!
-*!  Revision 1.5  2008/09/19 04:37:25  elphel
-*!  snapshot
-*!
-*!  Revision 1.4  2008/09/12 20:40:12  elphel
-*!  snapshot
-*!
-*!  Revision 1.3  2008/09/12 00:23:59  elphel
-*!  removed cc353.c, cc353.h
-*!
-*!  Revision 1.2  2008/09/11 01:05:32  elphel
-*!  snapshot
-*!
-*!  Revision 1.1  2008/09/07 19:48:09  elphel
-*!  snapshot
-*!
+/** @file jpeghead.c
+ *
+ * @brief This file contains methods for JPEG tables and headers generation and
+ * JPEG files composition from data compressed by FPGA.
+ *
+ * Copyright (C) 2016 Elphel, Inc
+ *
+ *  This program is free software: you can redistribute it and/or modify
+ *  it under the terms of the GNU General Public License as published by
+ *  the Free Software Foundation, either version 3 of the License, or
+ *  (at your option) any later version.
+ *
+ *  This program is distributed in the hope that it will be useful,
+ *  but WITHOUT ANY WARRANTY; without even the implied warranty of
+ *  MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+ *  GNU General Public License for more details.
+ *
+ *  You should have received a copy of the GNU General Public License
+ *  along with this program.  If not, see <http://www.gnu.org/licenses/>.
 */
 
-/****************** INCLUDE FILES SECTION ***********************************/
-
-
 #include <linux/module.h>
 #include <linux/mm.h>
 #include <linux/sched.h>
@@ -98,8 +60,8 @@
 //#include "cxdma.h"
 //#include "circbuf.h"
 //#include "sensor_common.h"
-#include "x313_macro.h"
 #include "exif.h"
+#include "x393_macro.h"
 
 #if ELPHEL_DEBUG
  #define   MDF(x) {printk("%s:%d:%s ",__FILE__,__LINE__,__FUNCTION__ );x ;}

src/drivers/elphel/x313_macro.h → src/drivers/elphel/x393_macro.h

-#ifndef _X313_MACRO
-#define _X313_MACRO
+/**
+ * @file x393_macro.h
+ * @brief This file contains various macros used in multiple files.
+ */
+#ifndef _X393_MACRO
+#define _X393_MACRO
+
+/** @brief Number of image channels*/
+#define IMAGE_CHN_NUM   4
 
 /* These macro were removed from sensor_common.h*/
 #define X313_LENGTH_MASK      0xff000000
@@ -7,4 +14,4 @@
 #define X313_BUFFSUB(x,y) (((x)>=(y))? ((x)-(y)) : ((x)+ (CCAM_DMA_SIZE-(y))))
 #define X313_BUFFADD(x,y) ((((x) + (y))<=CCAM_DMA_SIZE)? ((x) + (y)) : ((x) - (CCAM_DMA_SIZE-(y))))
 
-#endif /* _X313_MACRO */
+#endif /* _X393_MACRO */

src/include/elphel/c313a.h

@@ -9,8 +9,8 @@
 //#define ELPHEL_DEBUG 0 //global debug on/off in multiple files
 //#define ELPHEL_DEBUG_STARTUP 000a4c00 ; 
 //#define ELPHEL_DEBUG_STARTUP 0 ; // removed - add write to fpga init script
-//#define ELPHEL_DEBUG 0 //global debug on/off in multiple files
-#define ELPHEL_DEBUG 1 //global debug on/off in multiple files
+#define ELPHEL_DEBUG 0 //global debug on/off in multiple files
+//#define ELPHEL_DEBUG 1 //global debug on/off in multiple files
 #define ELPHEL_DEBUG_DELAY 100000 //delay after some printk-s
 #define ELP_KERR(x) printk("%s:%d:%s: ERROR ",__FILE__,__LINE__,__FUNCTION__);x
 #define ELP_FERR(x) fprintf(stderr,"%s:%d:%s: ERROR ",__FILE__,__LINE__,__FUNCTION__);x