On Wed, 23 Dec 2009 00:57:43 +0100 (CET) Jiri Kosina <jkosina@xxxxxxx> wrote: > On Mon, 21 Dec 2009, akpm@xxxxxxxxxxxxxxxxxxxx wrote: > > > From: "Rick L. Vinyard Jr." <rvinyard@xxxxxxxxxxx> > > > > This is a driver for the Logitech G13 gamepad, and contains three key > > parts. In the USB reports the device identifies itself as a HID, and as a > > result this driver is under the HID framework. > > > > There are two primary sub-components to this driver; an input device and a > > framebuffer device. > > > > Although identified as a HID, the device does not support standard HID > > input messages. As a result, a sub-input device is allocated and > > registered separately in g13_probe(). The raw events are monitored and > > key presses/joystick activity is reported through the input device after > > referencing an indexed keymap. > > > > Additionally, this device contains a 160x43 monochrome LCD display. A > > registered framebuffer device manages this display. The design of this > > portion of the driver was based on the design of the hecubafb driver with > > deferred framebuffer I/O since there is no real memory to map. > > > > Signed-off-by: Rick L. Vinyard, Jr <rvinyard@xxxxxxxxxxx> > > Cc: Oliver Neukum <oliver@xxxxxxxxxx> > > Cc: Dmitry Torokhov <dtor@xxxxxxx> > > Cc: Jiri Kosina <jkosina@xxxxxxx> > > Signed-off-by: Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> > > The driver doesn't build for me. Seems like missing some > framebuffer-related include at a first glance. > > Rick, care to fix that up please? > > drivers/hid/hid-g13.c:121: error: field ___fb_defio___ has incomplete type > drivers/hid/hid-g13.c:1207: error: variable ___g13_fb_defio___ has initializerbut incomplete type > drivers/hid/hid-g13.c:1208: error: unknown field ___delay___ specified ininitializer > drivers/hid/hid-g13.c:1208: warning: excess elements in struct initializer > drivers/hid/hid-g13.c:1208: warning: (near initialization for ___g13_fb_defio___) > drivers/hid/hid-g13.c:1209: error: unknown field ___deferred_io___ specified in initializer > drivers/hid/hid-g13.c:1209: warning: excess elements in struct initializer > drivers/hid/hid-g13.c:1209: warning: (near initialization for ___g13_fb_defio___) > drivers/hid/hid-g13.c: In function ___g13_probe___:drivers/hid/hid-g13.c:1351: error: ___struct fb_info___ has no member named ___fbdefio___ > Missing CONFIG_FB_DEFERRED_IO, I assume? --- a/drivers/hid/Kconfig~hid-logitech-g13-driver-002-fix +++ a/drivers/hid/Kconfig @@ -187,6 +187,7 @@ config LOGITECH_G13 tristate "Logitech G13 gameboard support" depends on HID_LOGITECH depends on FB + depends on FB_DEFERRED_IO select FB_SYS_FILLRECT select FB_SYS_COPYAREA select FB_SYS_IMAGEBLIT _ -- To unsubscribe from this list: send the line "unsubscribe linux-input" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html