public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Lars Op den Kamp <lars@opdenkamp.eu>
To: Hans Verkuil <hverkuil@xs4all.nl>, linux-media@vger.kernel.org
Subject: Re: [PATCH 0/5] Pulse-Eight USB CEC driver
Date: Mon, 11 Jul 2016 09:34:47 +0200	[thread overview]
Message-ID: <57834C17.7030100@opdenkamp.eu> (raw)
In-Reply-To: <1468156281-25731-1-git-send-email-hverkuil@xs4all.nl>

Hi Hans,

I'm subscribed to this mailing list, though haven't been participating 
in discussions here (no time). I work for Pulse-Eight and did most of 
the CEC software.

There's no difference between firmware v4 and v5 for the USB model of 
the adapter. v5 just adds support for the new Intel NUC internal CEC 
adapter.

You can snoop the bus by setting the ack mask to 0 and just read what 
comes in (MSGCODE_SET_ACK_MASK 0)
This is the same as "cec-client -m" does when using libCEC: 
https://github.com/Pulse-Eight/libcec/blob/master/src/libcec/adapter/Pulse-Eight/USBCECAdapterCommands.cpp#L584

Let me know if you need more help, or send an email to 
support@pulse-eight.com (which will likely end up in my inbox anyway 
eventually).

thanks, Lars

On 10-07-16 15:11, Hans Verkuil wrote:
> From: Hans Verkuil <hans.verkuil@cisco.com>
>
> This adds support for the Pulse-Eight USB CEC dongle. It has been tested
> with both v4 and v5 firmware.
>
> It is still in staging because 1) the CEC framework it depends on is still in
> staging, 2) the code needs to be refactored a bit and 3) it needs more testing.
>
> That said, it's in pretty decent shape. It's pretty neat, but I do wish it
> would support a CEC bus snooping mode: that would make it an ideal CEC bus
> sniffer. But I don't see any support for it, unfortunately.
>
> If anyone knows how this can be achieved then please let me know!
>
> Please note that this needs support from inputattach (part of linuxconsoletools),
> a patch is included in the TODO file.
>
> Regards,
>
> 	Hans
>
> Hans Verkuil (5):
>    cec: add check if adapter is unregistered.
>    serio.h: add new define for the Pulse-Eight USB-CEC Adapter
>    pulse8-cec: new driver for the Pulse-Eight USB-CEC Adapter
>    MAINTAINERS: add entry for the pulse8-cec driver
>    pulse8-cec: add TODO file
>
>   MAINTAINERS                                   |   7 +
>   drivers/staging/media/Kconfig                 |   2 +
>   drivers/staging/media/Makefile                |   1 +
>   drivers/staging/media/cec/cec-adap.c          |   5 +-
>   drivers/staging/media/pulse8-cec/Kconfig      |  10 +
>   drivers/staging/media/pulse8-cec/Makefile     |   1 +
>   drivers/staging/media/pulse8-cec/TODO         |  35 ++
>   drivers/staging/media/pulse8-cec/pulse8-cec.c | 502 ++++++++++++++++++++++++++
>   include/uapi/linux/serio.h                    |   1 +
>   9 files changed, 563 insertions(+), 1 deletion(-)
>   create mode 100644 drivers/staging/media/pulse8-cec/Kconfig
>   create mode 100644 drivers/staging/media/pulse8-cec/Makefile
>   create mode 100644 drivers/staging/media/pulse8-cec/TODO
>   create mode 100644 drivers/staging/media/pulse8-cec/pulse8-cec.c
>


  parent reply	other threads:[~2016-07-11  7:41 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-07-10 13:11 [PATCH 0/5] Pulse-Eight USB CEC driver Hans Verkuil
2016-07-10 13:11 ` [PATCH 1/5] cec: add check if adapter is unregistered Hans Verkuil
2016-07-10 13:11 ` [PATCH 2/5] serio.h: add new define for the Pulse-Eight USB-CEC Adapter Hans Verkuil
2016-07-10 13:20   ` Hans Verkuil
2016-07-11 17:00   ` Dmitry Torokhov
2016-07-10 13:11 ` [PATCH 3/5] pulse8-cec: new driver " Hans Verkuil
2016-07-10 13:11 ` [PATCH 4/5] MAINTAINERS: add entry for the pulse8-cec driver Hans Verkuil
2016-07-10 13:11 ` [PATCH 5/5] pulse8-cec: add TODO file Hans Verkuil
2016-07-11  7:34 ` Lars Op den Kamp [this message]
2016-07-11  8:57   ` [PATCH 0/5] Pulse-Eight USB CEC driver Hans Verkuil

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=57834C17.7030100@opdenkamp.eu \
    --to=lars@opdenkamp.eu \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox