Don't typedef the low-level functions structure. Use struct* the two times

where necessary (this avoids confusing Doxygen as well).

server/drivers/glcd-low.h

@@ -42,7 +42,7 @@ typedef struct glcd_private_data {
 	int offbrightness;		/**< current brightness (for backlight off) */
 	int last_output_state;		/**< cache last value to output() */
 	int backlightstate;		/**< state of backlight currently used */
-	struct hwDependentFns *glcd_functions;	/**< pointers to low-level functions */
+	struct glcdHwFcns *glcd_functions;	/**< pointers to low-level functions */
 	void *ct_data;			/**< Connection type specific data */
 	/* Renderer settings */
 	void *render_config;		/**< Settings for the font renderer */
@@ -56,7 +56,7 @@ typedef struct glcd_private_data {
 } PrivateData;
 
 /** Structure holding pointers to display specific functions */
-typedef struct hwDependentFns {
+struct glcdHwFcns {
 	/* report and debug helper. Set by global glcd init */
 	void (*drv_report)(const int level, const char *format,... /* args */ );
 	void (*drv_debug)(const int level, const char *format,... /* args */ );
@@ -78,7 +78,7 @@ typedef struct hwDependentFns {
 
 	/* Close the interface on shutdown */
 	void (*close)(PrivateData *p);
-} GLCD_functions;
+};
 
 /* ================== Framebuffer functions and macros =================== */
 

server/drivers/glcd_drv.c

@@ -105,7 +105,7 @@ glcd_init(Driver *drvthis)
 	 * Set up low-level functions. These should be overwritten by the
 	 * connection type's init function.
 	 */
-	if ((p->glcd_functions = (GLCD_functions *) calloc(1, sizeof(GLCD_functions))) == NULL) {
+	if ((p->glcd_functions = (struct glcdHwFcns *) calloc(1, sizeof(struct glcdHwFcns))) == NULL) {
 		report(RPT_ERR, "%s: error mallocing", drvthis->name);
 		return -1;
 	}