Re: [PATCH 10/14] HID: wiimote: replace hid_output_raw_report with hid_hw_output_report for output requests

From: David Herrmann
Date: Wed Feb 12 2014 - 05:39:47 EST


Hi

On Mon, Feb 10, 2014 at 6:58 PM, Benjamin Tissoires
<benjamin.tissoires@xxxxxxxxxx> wrote:
> For BT transport layer,
> ret = hid_output_raw_report(A, B, C, HID_OUTPUT_REPORT);
> is equivalent to
> ret = hid_hw_output_report(A, B, C);
>
> So use the new API where available

Reviewed-by: David Herrmann <dh.herrmann@xxxxxxxxx>

Thanks
David

>
> Signed-off-by: Benjamin Tissoires <benjamin.tissoires@xxxxxxxxxx>
> ---
> drivers/hid/hid-wiimote-core.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/drivers/hid/hid-wiimote-core.c b/drivers/hid/hid-wiimote-core.c
> index d7dc6c5b..d003914 100644
> --- a/drivers/hid/hid-wiimote-core.c
> +++ b/drivers/hid/hid-wiimote-core.c
> @@ -28,14 +28,14 @@ static int wiimote_hid_send(struct hid_device *hdev, __u8 *buffer,
> __u8 *buf;
> int ret;
>
> - if (!hdev->hid_output_raw_report)
> + if (!hdev->ll_driver->output_report)
> return -ENODEV;
>
> buf = kmemdup(buffer, count, GFP_KERNEL);
> if (!buf)
> return -ENOMEM;
>
> - ret = hid_output_raw_report(hdev, buf, count, HID_OUTPUT_REPORT);
> + ret = hid_hw_output_report(hdev, buf, count);
>
> kfree(buf);
> return ret;
> --
> 1.8.3.1
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/