On May 28 Mauro Carvalho Chehab wrote:
> Signed-off-by: Mauro Carvalho Chehab <mche...@redhat.com>
> ---
> drivers/media/dvb/firewire/firedtv-rc.c | 152
> ++-----------------------------
> drivers/media/dvb/firewire/firedtv.h | 2 +-
> 2 files changed, 11 insertions(+), 143 deletions(-)
Also in drivers/media/dvb/firewire/Kconfig, INPUT needs to be replaced by
RC_CORE, right?
> diff --git a/drivers/media/dvb/firewire/firedtv-rc.c
> b/drivers/media/dvb/firewire/firedtv-rc.c
> index f82d4a9..3c2c9b3 100644
> --- a/drivers/media/dvb/firewire/firedtv-rc.c
> +++ b/drivers/media/dvb/firewire/firedtv-rc.c
-#include <linux/input.h>
+#include <media/rc-core.h>
[...]
> int fdtv_register_rc(struct firedtv *fdtv, struct device *dev)
> {
> - struct input_dev *idev;
> + struct rc_dev *idev;
> int i, err;
>
> - idev = input_allocate_device();
> + idev = rc_allocate_device();
> if (!idev)
> return -ENOMEM;
>
> fdtv->remote_ctrl_dev = idev;
> idev->name = "FireDTV remote control";
> + idev->phys = "/ir0"; /* FIXME */
Something similar to drivers/media/dvb/dvb-usb/dvb-usb-remote.c::
usb_make_path(d->udev, d->rc_phys, sizeof(d->rc_phys));
strlcat(d->rc_phys, "/ir0", sizeof(d->rc_phys));
should be implemented for this, right?
Also, idev->name should be idev->input_name and idev->phys should be
idev->input_phys.
> --- a/drivers/media/dvb/firewire/firedtv.h
> +++ b/drivers/media/dvb/firewire/firedtv.h
-struct input_dev;
+struct struct rc_dev;
> @@ -91,7 +91,7 @@ struct firedtv {
> wait_queue_head_t avc_wait;
> bool avc_reply_received;
> struct work_struct remote_ctrl_work;
> - struct input_dev *remote_ctrl_dev;
> + struct rc_dev *remote_ctrl_dev;
>
> enum model_type type;
> char subunit;
(Do you want to reroll it or should I resubmit it myself after actually
trying it?)
--
Stefan Richter
-=====-===-- -=-= ===--
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
