All of lore.kernel.org
 help / color / mirror / Atom feed
From: David Herrmann <dh.herrmann@googlemail.com>
To: linux-input@vger.kernel.org
Cc: padovan@profusion.mobi, jkosina@suse.cz, oliver@neukum.org,
	dh.herrmann@googlemail.com
Subject: [PATCH 06/12 v2] HID: wiimote: Add wiimote send function
Date: Fri, 17 Jun 2011 16:49:16 +0200	[thread overview]
Message-ID: <1308322162-13953-6-git-send-email-dh.herrmann@googlemail.com> (raw)
In-Reply-To: <1308322162-13953-1-git-send-email-dh.herrmann@googlemail.com>

The wiimote driver needs to send raw output reports to the wiimote
device. Otherwise we could not manage the peripherals of the wiimote
or perform memory operations on the wiimote.
We cannot use hidinput_input_event of the lowlevel hid driver, since
this does not accept raw input. Therefore, we need to use the same
function that hidraw uses to send output. Side effect is, the raw
output function is not buffered and can sleep.

Signed-off-by: David Herrmann <dh.herrmann@googlemail.com>
---
V2: Allocate new buffer for every packet that is sent out to comply to DMA
constraints.

 drivers/hid/hid-wiimote.c |   19 +++++++++++++++++++
 1 files changed, 19 insertions(+), 0 deletions(-)

diff --git a/drivers/hid/hid-wiimote.c b/drivers/hid/hid-wiimote.c
index afae0e9..b6ae2a1 100644
--- a/drivers/hid/hid-wiimote.c
+++ b/drivers/hid/hid-wiimote.c
@@ -26,6 +26,25 @@ struct wiimote_data {
 	struct input_dev *input;
 };
 
+static ssize_t wiimote_hid_send(struct hid_device *hdev, __u8 *buffer,
+								size_t count)
+{
+	__u8 *buf;
+	ssize_t ret;
+
+	if (!hdev->hid_output_raw_report)
+		return -ENODEV;
+
+	buf = kmemdup(buffer, count, GFP_KERNEL);
+	if (!buf)
+		return -ENOMEM;
+
+	ret = hdev->hid_output_raw_report(hdev, buf, count, HID_OUTPUT_REPORT);
+
+	kfree(buf);
+	return ret;
+}
+
 static int wiimote_input_event(struct input_dev *dev, unsigned int type,
 						unsigned int code, int value)
 {
-- 
1.7.5.2


  parent reply	other threads:[~2011-06-17 14:50 UTC|newest]

Thread overview: 14+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2011-06-17 14:49 [PATCH 01/12 v2] HID: wiimote: Add Nintendo Wii Remote driver stub David Herrmann
2011-06-17 14:49 ` [PATCH 02/12 v2] HID: wiimote: Register wiimote hid " David Herrmann
2011-06-17 14:49 ` [PATCH 03/12 v2] HID: wiimote: Add wiimote device structure David Herrmann
2011-06-17 14:49 ` [PATCH 04/12 v2] HID: wiimote: Register input device in wiimote hid driver David Herrmann
2011-06-17 14:49 ` [PATCH 05/12 v2] HID: wiimote: Synchronize wiimote input and hid event handling David Herrmann
2011-06-17 14:49 ` David Herrmann [this message]
2011-06-17 14:49 ` [PATCH 07/12 v2] HID: wiimote: Add output queue for wiimote driver David Herrmann
2011-06-17 14:49 ` [PATCH 08/12 v2] HID: wiimote: Add wiimote event handler David Herrmann
2011-06-17 14:49 ` [PATCH 09/12 v2] HID: wiimote: Add wiimote input button parser David Herrmann
2011-06-17 14:49 ` [PATCH 10/12 v2] HID: wiimote: Add wiimote led request David Herrmann
2011-06-17 14:49 ` [PATCH 11/12 v2] HID: wiimote: Cache wiimote led state David Herrmann
2011-06-17 14:49 ` [PATCH 12/12 v2] HID: wiimote: Add sysfs support to wiimote driver David Herrmann
2011-06-24 12:14 ` [PATCH 01/12 v2] HID: wiimote: Add Nintendo Wii Remote driver stub Jiri Kosina
2011-06-27 14:29   ` David Herrmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1308322162-13953-6-git-send-email-dh.herrmann@googlemail.com \
    --to=dh.herrmann@googlemail.com \
    --cc=jkosina@suse.cz \
    --cc=linux-input@vger.kernel.org \
    --cc=oliver@neukum.org \
    --cc=padovan@profusion.mobi \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.